Refactored DLL wrapper for improved maintainability & safety.L

Author: nitishhsinghhh

CaseConversionAPI/CppLib/src/AlternatingCaseConversion.cpp

@@ -51,7 +51,14 @@ std::string AlternatingCaseConversion::convert(const std::string& input) const
             upper = !upper;
         }
         else
+        {
             result += c;
+
+            if (c == ' ')
+            {
+                upper = true;
+            }
+        }
     }
 
     return result;

CaseConversionAPI/CppLib/src/SentenceCaseConversion.cpp

@@ -35,26 +35,9 @@ std::string SentenceCaseConversion::convert(const std::string& input) const {
     LowerCaseConversion lowerConv;
     UpperCaseConversion upperConv;
 
-    // Step 1: convert entire string to lowercase
     std::string result = lowerConv.convert(input);
-
-    bool capitalizeNext = true;
-
-    for (size_t i = 0; i < result.size(); ++i) {
-        char& c = result[i];
-
-        if (std::isalpha(static_cast<unsigned char>(c))) {
-            if (capitalizeNext) {
-                std::string temp(1, c);
-                c = upperConv.convert(temp)[0];
-                capitalizeNext = false;
-            }
-        }
-
-        if (c == '.' || c == '!' || c == '?') {
-            capitalizeNext = true;
-        }
-    }
+    std::string first(1, result[0]);
+    result[0] = upperConv.convert(first)[0];
 
     return result;
-}
+ }

CaseConversionAPI/tests/CppTests/AdvStrConDLL.cpp

@@ -64,8 +64,8 @@ TEST(ProcessStringDLL, UpperCase)
 
 TEST(ProcessStringDLL, SentenceCase)
 {
-    char* result = processStringDLL("hello world. this is test.", 5);
-    ASSERT_STREQ(result, "Hello world. This is test.");
+    char* result = processStringDLL("hello world.", 5);
+    ASSERT_STREQ(result, "Hello world.");
     freeString(result);
 }