SQ improvement and naming convention context

Author: antoineatstariongroup

CLAUDE.md

@@ -117,3 +117,4 @@ Auto-generated DTOs use structured namespaces reflecting the KerML/SysML package
 - Prefer 'string.IsNullOrWhiteSpace' over 'string.IsNullOrEmpty' when checking the non-nullable value of a string
 - Prefer switch expressions/statements over if-else chains when applicable
 - Prefer indexer syntax (e.g., 'list[^1]') and range syntax (e.g., 'array[1..^1]') over LINQ methods (e.g., 'list.Last()', 'list.Skip(1).Take(n)') when applicable
+- Use meaningful loop variable names instead of single-letter names (e.g., 'charIndex' instead of 'i', 'currentChar' instead of 'c')

SysML2.NET.Tests/Extend/NamespaceExtensionsTestFixture.cs

@@ -148,11 +148,11 @@ public void VerifyComputeNamesOfOperation()
             var membership = new Membership { MemberName = "elementName", MemberShortName = "en", MemberElement = element };
             namespaceElement.AssignOwnership(membership, element);
 
-            Assert.That(namespaceElement.ComputeNamesOfOperation(element), Is.EquivalentTo(new[] { "en", "elementName" }));
+            Assert.That(namespaceElement.ComputeNamesOfOperation(element), Is.EquivalentTo(["en", "elementName"]));
 
             membership.MemberShortName = null;
 
-            Assert.That(namespaceElement.ComputeNamesOfOperation(element), Is.EquivalentTo(new[] { "elementName" }));
+            Assert.That(namespaceElement.ComputeNamesOfOperation(element), Is.EquivalentTo(["elementName"]));
         }
 
         [Test]

SysML2.NET/Extensions/StringExtensions.cs

@@ -92,24 +92,24 @@ public static int FindLastQualifiedNameSeparatorIndex(this string qualifiedName)
             var lastSeparatorIndex = -1;
             var inQuote = false;
 
-            for (var i = 0; i < qualifiedName.Length; i++)
+            for (var charIndex = 0; charIndex < qualifiedName.Length; charIndex++)
             {
-                var c = qualifiedName[i];
+                var currentChar = qualifiedName[charIndex];
 
-                switch (c)
+                switch (currentChar)
                 {
-                    case '\\' when inQuote && i + 1 < qualifiedName.Length:
-                        i++;
+                    case '\\' when inQuote && charIndex + 1 < qualifiedName.Length:
+                        charIndex++;
                         continue;
                     case '\'':
                         inQuote = !inQuote;
                         continue;
                 }
 
-                if (!inQuote && c == ':' && i + 1 < qualifiedName.Length && qualifiedName[i + 1] == ':')
+                if (!inQuote && currentChar == ':' && charIndex + 1 < qualifiedName.Length && qualifiedName[charIndex + 1] == ':')
                 {
-                    lastSeparatorIndex = i;
-                    i++;
+                    lastSeparatorIndex = charIndex;
+                    charIndex++;
                 }
             }