Use fprintf instead of printf.

This commit is contained in:
Gerhard Hoffmann 2023-05-12 10:04:04 +02:00
parent 4bc343c5b2
commit 124eb64018

View File

@ -23,12 +23,12 @@ bool Configuration::ParseJson(Configuration* cfg, const char* json)
{ {
if (cfg == nullptr) if (cfg == nullptr)
{ {
printf("TariffConfiguration pointer not set\n"); fprintf(stderr, "TariffConfiguration pointer not set\n");
return false; return false;
} }
if (json == NULL) if (json == NULL)
{ {
printf("%s", "Input JSON string is NULL\n"); fprintf(stderr, "%s", "Input JSON string is NULL\n");
return false; return false;
} }
@ -40,17 +40,17 @@ bool Configuration::ParseJson(Configuration* cfg, const char* json)
ParseErrorCode err = document.GetParseError(); ParseErrorCode err = document.GetParseError();
if (err != 0) if (err != 0)
{ {
printf("%s %d (%s)\n", "Unable to parse JSON, error code:", err, GetParseError_En(err)); fprintf(stderr, "%s %d (%s)\n", "Unable to parse JSON, error code:", err, GetParseError_En(err));
return false; return false;
} }
// Validate JSON, check if it's a JSON object // Validate JSON, check if it's a JSON object
printf("%s", "JSON parsing has been successful\n"); fprintf(stderr, "%s", "JSON parsing has been successful\n");
if (!document.IsObject()) { if (!document.IsObject()) {
printf("%s", "Error: not a (valid) JSON object\n"); fprintf(stderr, "%s", "Error: not a (valid) JSON object\n");
return false; return false;
} }
printf("%s", "Valid JSON object identified\n"); fprintf(stderr, "%s", "Valid JSON object identified\n");
// Validate JSON, check configuration members // Validate JSON, check configuration members
if (!document.HasMember("Currency") if (!document.HasMember("Currency")
@ -65,7 +65,7 @@ bool Configuration::ParseJson(Configuration* cfg, const char* json)
printf("%s", "Error: not a valid configuration JSON\n"); printf("%s", "Error: not a valid configuration JSON\n");
return false; return false;
} }
printf("%s", "Valid JSON configuration identified\n"); fprintf(stderr, "%s", "Valid JSON configuration identified\n");
ATBCurrency Currency; ATBCurrency Currency;
ATBDuration Duration; ATBDuration Duration;
@ -115,7 +115,7 @@ bool Configuration::ParseJson(Configuration* cfg, const char* json)
const char* inner_obj_name = k->name.GetString(); const char* inner_obj_name = k->name.GetString();
if (inner_obj_name == NULL) if (inner_obj_name == NULL)
{ {
printf("Inner object name is NULL\n"); fprintf(stderr, "Inner object name is NULL\n");
continue; continue;
} }
@ -240,7 +240,7 @@ bool Configuration::ParseJson(Configuration* cfg, const char* json)
return true; return true;
} }
catch (...) { catch (...) {
printf("%s\n", "General exception has occurred while parsing JSON\n"); fprintf(stderr, "%s\n", "General exception has occurred while parsing JSON\n");
return false; return false;
} }
} }