#176-Test CppStringT::translate() with char and wchar_t

Completed. Validated.

Author: schmouk

cpp-strings-tests/cpp-strings-tests.cpp

@@ -3886,5 +3886,19 @@ namespace cppstringstests
 			Assert::AreEqual(wexpected.c_str(), wres.c_str());
 
 		}
+
+		TEST_METHOD(translate)
+		{
+			CppString::TransTable trans_table("oizeaslbgOIZEASLG", "012345789012345789");
+			CppString text("This is a big 'Oiseau' that can be seen in 'Le Zoo'");
+			CppString expected("Th15 15 4 819 '01534u' th4t c4n 83 533n 1n '73 200'");
+			Assert::AreEqual(expected.c_str(), text.translate(trans_table).c_str());
+
+			CppWString::TransTable wtrans_table(L"oizeaslbgOIZEASLG", L"012345789012345789");
+			CppWString wtext(L"This is a big 'Oiseau' that can be seen in 'Le Zoo'");
+			CppWString wexpected(L"Th15 15 4 819 '01534u' th4t c4n 83 533n 1n '73 200'");
+			Assert::AreEqual(wexpected.c_str(), wtext.translate(wtrans_table).c_str());
+		}
+
 	};
 }

cpp-strings/cppstrings.h

@@ -348,14 +348,15 @@ namespace pcs // i.e. "pythonic c++ strings"
             }
 
             /** \brief Indexing operator. */
-            inline CppStringT operator[] (const key_type ch) noexcept
+            inline value_type operator[] (const key_type ch) noexcept
+            //inline CppStringT operator[] (const key_type ch) noexcept
             {
                 auto it = m_table.find(ch);
                 if (it != m_table.end()) {
                     return it->second;
                 }
                 else {
-                    return CppStringT(ch);
+                    return ch;  // CppStringT(ch);
                 }
             }
 
@@ -366,7 +367,8 @@ namespace pcs // i.e. "pythonic c++ strings"
 
         private:
             std::map<typename key_type, typename value_type> m_table{};  // the internal storage of the translation table. Access it via the indexing operator.
-        };
+        
+};
 
 
         //===   Constructors / Destructor   =======================
@@ -1774,24 +1776,14 @@ namespace pcs // i.e. "pythonic c++ strings"
         * to  be  translated  is not available as an entry in the tranlation
         * table, it is set as is in the resulting string.
         */
-        CppStringT translate(const TransTable& table) const noexcept
+        CppStringT translate(TransTable& table) noexcept
         {
-            /*
             CppStringT res{};
             for (auto ch : *this) {
                 try { res += table[ch]; }
                 catch (...) { res += ch; }
             }
             return res;
-            */
-
-            CppStringT res{};
-            auto _translate = [&](auto const ch) {
-                try { return table[ch]; }
-                catch (...) { return ch; }
-            };
-            std::ranges::copy(std::views::transform(*this, _translate), std::back_inserter(res));
-            return res;
         }