CQ - Define a constant instead of using literal multiple times

Author: abhinavminhas

QueryDB/DBContext.cs

@@ -153,7 +153,7 @@ public List<DataDictionary> FetchData(string selectSql, bool upperCaseKeys = fal
         /// <returns>A string representing the result of the query. If the result is DBNull, an empty string is returned.</returns>
         public string ExecuteScalar(string sqlStatement)
         {
-            if (!Regex.IsMatch(sqlStatement, @"^\s*SELECT\s+.*", RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)))
+            if (!Regex.IsMatch(sqlStatement, Utils.SelectQueryPattern, RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)))
                 throw new QueryDBException(QueryDBExceptions.ErrorMessage.UnsupportedExecuteScalarCommand,
                     QueryDBExceptions.ErrorType.UnsupportedCommand, QueryDBExceptions.AdditionalInfo.UnsupportedExecuteScalarCommand);
             var value = string.Empty;
@@ -200,7 +200,7 @@ public string ExecuteScalar(string sqlStatement)
         /// <returns>The result of the query, converted to the specified type. If the result is DBNull, the default value for the type is returned.</returns>
         public T ExecuteScalar<T>(string sqlStatement)
         {
-            if (!Regex.IsMatch(sqlStatement, @"^\s*SELECT\s+.*", RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)))
+            if (!Regex.IsMatch(sqlStatement, Utils.SelectQueryPattern, RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)))
                 throw new QueryDBException(QueryDBExceptions.ErrorMessage.UnsupportedExecuteScalarCommand,
                     QueryDBExceptions.ErrorType.UnsupportedCommand, QueryDBExceptions.AdditionalInfo.UnsupportedExecuteScalarCommand);
             var value = default(T);
@@ -246,7 +246,7 @@ public T ExecuteScalar<T>(string sqlStatement)
         /// <returns>The number of rows affected.</returns>
         public int ExecuteCommand(string sqlStatement)
         {
-            if (Regex.IsMatch(sqlStatement, "^\\s*SELECT\\s+.*", RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)))
+            if (Regex.IsMatch(sqlStatement, Utils.SelectQueryPattern, RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)))
                 throw new QueryDBException(QueryDBExceptions.ErrorMessage.UnsupportedSelectExecuteCommand,
                     QueryDBExceptions.ErrorType.UnsupportedCommand, QueryDBExceptions.AdditionalInfo.UnsupportedSelectExecuteCommand);
             if (Database.Equals(DB.MSSQL))
@@ -294,7 +294,7 @@ public int ExecuteCommand(string sqlStatement)
         /// </returns>
         public bool ExecuteTransaction(List<string> sqlStatements)
         {
-            var selectExists = sqlStatements.Any(sqlStatement => Regex.IsMatch(sqlStatement, "^\\s*SELECT\\s+.*", RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)));
+            var selectExists = sqlStatements.Any(sqlStatement => Regex.IsMatch(sqlStatement, Utils.SelectQueryPattern, RegexOptions.IgnoreCase | RegexOptions.Singleline, TimeSpan.FromSeconds(5)));
             if (selectExists)
                 throw new QueryDBException(QueryDBExceptions.ErrorMessage.UnsupportedSelectExecuteTransaction,
                     QueryDBExceptions.ErrorType.UnsupportedCommand, QueryDBExceptions.AdditionalInfo.UnsupportedSelectExecuteTransaction);

QueryDB/Resources/Utils.cs

@@ -9,6 +9,11 @@ namespace QueryDB.Resources
     /// </summary>
     internal static class Utils
     {
+        /// <summary>
+        /// Regex pattern to validate that a SQL query starts with 'SELECT'.
+        /// </summary>
+        internal static readonly string SelectQueryPattern = @"^\s*SELECT\s+.*";
+
         /// <summary>
         /// Checks if a specified column exists in the given data reader.
         /// </summary>