Merge branch 'cuongtranba-ReviewStringBuilder'

Author: psoholt

VerbalExpressions/VerbalExpressions.cs

@@ -36,9 +36,9 @@ public static VerbalExpressions DefaultExpression
 
         #region Private Members
 
-        private string _prefixes = "";
-        private string _source = "";
-        private string _suffixes = "";
+        private StringBuilder _prefixes = new StringBuilder();
+        private StringBuilder _source = new StringBuilder();
+        private StringBuilder _suffixes = new StringBuilder();
         private RegexOptions _modifiers = RegexOptions.Multiline;
 
         #endregion Private Members
@@ -47,7 +47,7 @@ public static VerbalExpressions DefaultExpression
 
         private string RegexString
         {
-            get { return _prefixes + _source + _suffixes; }
+            get { return new StringBuilder().Append(_prefixes).Append(_source).Append(_suffixes).ToString();}
         }
 
         private Regex PatternRegex
@@ -108,7 +108,7 @@ public string Capture(string toTest, string groupName)
             if (!Test(toTest))
                 return null;
 
-            var match=PatternRegex.Match(toTest);
+            var match = PatternRegex.Match(toTest);
             return match.Groups[groupName].Value;
         }
 
@@ -137,19 +137,19 @@ public VerbalExpressions Add(string value, bool sanitize = true)
                 throw new ArgumentNullException("value must be provided");
 
             value = sanitize ? Sanitize(value) : value;
-            _source += value;
+            _source.Append(value);
             return this;
         }
 
         public VerbalExpressions StartOfLine(bool enable = true)
         {
-            _prefixes = enable ? "^" : string.Empty;
+            _prefixes.Append(enable ? "^" : String.Empty);
             return this;
         }
 
         public VerbalExpressions EndOfLine(bool enable = true)
         {
-            _suffixes = enable ? "$" : string.Empty;
+            _suffixes.Append(enable ? "$" : String.Empty);
             return this;
         }
 
@@ -341,11 +341,9 @@ public VerbalExpressions Or(CommonRegex commonRegex)
 
         public VerbalExpressions Or(string value, bool sanitize = true)
         {
-            _prefixes += "(";
-            _suffixes = ")" + _suffixes;
-
-            _source += ")|(";
-
+            _prefixes.Append("(");
+            _suffixes.Insert(0, ")");
+            _source.Append(")|(");
             return Add(value, sanitize);
         }
 
@@ -356,7 +354,7 @@ public VerbalExpressions BeginCapture()
 
         public VerbalExpressions BeginCapture(string groupName)
         {
-            return Add("(?<", false).Add(groupName,true).Add(">",false);
+            return Add("(?<", false).Add(groupName, true).Add(">", false);
         }
 
         public VerbalExpressions EndCapture()
@@ -457,6 +455,6 @@ public VerbalExpressions WithOptions(RegexOptions options)
 
         #endregion Public Methods
 
-      
+
     }
 }