Slight code optimizations

mdesalvo · Nov 30, 2024 · affae6c · affae6c
1 parent 30893ef
commit affae6c
Show file tree

Hide file tree

Showing 3 changed files with 16 additions and 16 deletions.
diff --git a/RDFSharp/Query/Mirella/Algebra/RDFAggregator.cs b/RDFSharp/Query/Mirella/Algebra/RDFAggregator.cs
@@ -136,7 +136,7 @@ internal string PrintHavingClause(List<RDFNamespace> prefixes)
             StringBuilder result = new StringBuilder();
             if (HavingClause.Item1)
             {
-                result.Append("(");
+                result.Append('(');
                 result.Append(ToString().Substring(1, ToString().LastIndexOf(" AS ?")));
                 switch (HavingClause.Item2)
                 {
@@ -160,7 +160,7 @@ internal string PrintHavingClause(List<RDFNamespace> prefixes)
                         break;
                 }
                 result.Append(RDFQueryPrinter.PrintPatternMember(HavingClause.Item3, prefixes));
-                result.Append(")");
+                result.Append(')');
             }
             return result.ToString();
         }

diff --git a/RDFSharp/Query/Mirella/RDFOperationPrinter.cs b/RDFSharp/Query/Mirella/RDFOperationPrinter.cs
@@ -45,7 +45,7 @@ internal static string PrintInsertDataOperation(RDFInsertDataOperation insertDat
                 #region TEMPLATES
                 sb.AppendLine("INSERT DATA {");
                 insertDataOperation.InsertTemplates.ForEach(tp => sb.Append(PrintPattern(prefixes, tp)));
-                sb.Append("}");
+                sb.Append('}');
                 #endregion
             }
 
@@ -88,7 +88,7 @@ internal static string PrintDeleteDataOperation(RDFDeleteDataOperation deleteDat
 
                 sb.AppendLine("DELETE DATA {");
                 deleteDataOperation.DeleteTemplates.ForEach(tp => sb.Append(PrintPattern(prefixes, tp)));
-                sb.Append("}");
+                sb.Append('}');
             }
 
             return sb.ToString();

diff --git a/RDFSharp/Query/Mirella/RDFQueryPrinter.cs b/RDFSharp/Query/Mirella/RDFQueryPrinter.cs
@@ -83,9 +83,9 @@ int subqueryUnionSpacesFunc(bool union)
                          .ToList()
                          .ForEach(gm =>
                          {
-                             sb.Append(" ");
+                             sb.Append(' ');
                              sb.Append(string.Join(" ", gm.PartitionVariables));
-                             sb.Append(" ");
+                             sb.Append(' ');
                              sb.Append(string.Join(" ", gm.Aggregators.Where(ag => !(ag is RDFPartitionAggregator))));
                          });
             }
@@ -562,14 +562,14 @@ internal static string PrintPropertyPath(RDFPropertyPath propertyPath, List<RDFN
         {
             StringBuilder result = new StringBuilder();
             result.Append(PrintPatternMember(propertyPath.Start, prefixes));
-            result.Append(" ");
+            result.Append(' ');
 
             #region Single Property
             if (propertyPath.Steps.Count == 1)
             {
                 //InversePath (will swap start/end)
                 if (propertyPath.Steps[0].IsInverseStep)
-                    result.Append("^");
+                    result.Append('^');
 
                 RDFResource propPath = propertyPath.Steps[0].StepProperty;
                 result.Append(PrintPatternMember(propPath, prefixes));
@@ -591,12 +591,12 @@ internal static string PrintPropertyPath(RDFPropertyPath propertyPath, List<RDFN
                         if (!openedParenthesis)
                         {
                             openedParenthesis = true;
-                            result.Append("(");
+                            result.Append('(');
                         }
 
                         //InversePath (will swap start/end)
                         if (propertyPath.Steps[i].IsInverseStep)
-                            result.Append("^");
+                            result.Append('^');
 
                         var propPath = propertyPath.Steps[i].StepProperty;
                         if (i < propertyPath.Steps.Count - 1)
@@ -607,7 +607,7 @@ internal static string PrintPropertyPath(RDFPropertyPath propertyPath, List<RDFN
                         else
                         {
                             result.Append(PrintPatternMember(propPath, prefixes));
-                            result.Append(")");
+                            result.Append(')');
                         }
                     }
 
@@ -623,7 +623,7 @@ internal static string PrintPropertyPath(RDFPropertyPath propertyPath, List<RDFN
 
                         //InversePath (will swap start/end)
                         if (propertyPath.Steps[i].IsInverseStep)
-                            result.Append("^");
+                            result.Append('^');
 
                         var propPath = propertyPath.Steps[i].StepProperty;
                         if (i < propertyPath.Steps.Count - 1)
@@ -638,7 +638,7 @@ internal static string PrintPropertyPath(RDFPropertyPath propertyPath, List<RDFN
             }
             #endregion
 
-            result.Append(" ");
+            result.Append(' ');
             result.Append(PrintPatternMember(propertyPath.End, prefixes));
             return result.ToString();
         }
@@ -661,9 +661,9 @@ internal static string PrintValues(RDFValues values, List<RDFNamespace> prefixes
                         result.Append("UNDEF");
                     else
                         result.Append(RDFQueryPrinter.PrintPatternMember(binding, prefixes));
-                    result.Append(" ");
+                    result.Append(' ');
                 }
-                result.Append("}");
+                result.Append('}');
             }
 
             //Extended representation
@@ -681,7 +681,7 @@ internal static string PrintValues(RDFValues values, List<RDFNamespace> prefixes
                             result.Append("UNDEF");
                         else
                             result.Append(RDFQueryPrinter.PrintPatternMember(bindingValue, prefixes));
-                        result.Append(" ");
+                        result.Append(' ');
                     });
                     result.AppendLine(")");
                 }