Is there some way to reduce this code?

CString j;
j.Format("%.2f", cardsum);
rep_templ.Replace("%cardsum%", j);

rep_templ.Replace("%cardsum%", CString().Format("%.2f", cardsum)) is not an option, because .Format is void.

Is there some way to reduce this code?

CString j;
j.Format("%.2f", cardsum);
rep_templ.Replace("%cardsum%", j);

rep_templ.Replace("%cardsum%", CString().Format("%.2f", cardsum)) is not an option, because .Format is void.

• visual-c++
• mfc

• Using CString this cannot be done any more concisely. But you can avoid CString and use std::format instead (starting with C++20). This allows chaining function calls, e.g., format( ... ).replace( ... );. – IInspectable Commented Jan 30 at 17:14
• 1 Write a helper function, based on FormatV, that returns CString. Then you can write something like rep_templ.Replace("%cardsum%", FormatCString("%.2f", cardsum)) – Igor Tandetnik Commented Jan 30 at 23:20
• 1 Is there any specific reason why you are rolling your own text template engine? – Igor Levicki Commented Jan 31 at 1:29
• @IgorLevicki which text template engine would you suggest? – Jabberwocky Commented Jan 31 at 7:55
• 1 @Jabberwocky This one seems very capable github/pantor/inja, and I am sure you can Google for others. – Igor Levicki Commented Feb 3 at 12:04

2 Answers 2

Best way to reduce that code is to use one of the available text template engines such as Inja or Mustache (both results of Google Search, I am not affiliated with either and don't recommend one or the other, just the principle of not rolling your own C++ text template engine).

Here is an implementation of a function that returns a formatted CString. I use this in actual production code.

CString FormatString(LPCTSTR pszFormat, ...)
{
  ATLASSERT(AtlIsValidString(pszFormat));
  CString thestring;
  va_list argList;
  va_start(argList, pszFormat);
  thestring.FormatV(pszFormat, argList);
  va_end(argList);
  return thestring;
}

You can call it like this:

rep_templ.Replace("%cardsum%", FormatString("%.2f", cardsum));