Add INIReader::GetString to get nicer default_value behavior (#72)

* add INIReader::GetString Same as INIReader::Get, but returns default_value if the value is an empty string, which provides functionality similar to GetInteger and the others * clarify GetString description comment
benhoyt · Oct 12, 2018 · 2023872 · 2023872
1 parent 0ee2bf2
commit 2023872
Show file tree

Hide file tree

Showing 2 changed files with 11 additions and 0 deletions.
diff --git a/cpp/INIReader.cpp b/cpp/INIReader.cpp
@@ -30,6 +30,12 @@ string INIReader::Get(const string& section, const string& name, const string& d
     return _values.count(key) ? _values.find(key)->second : default_value;
 }
 
+string INIReader::GetString(const string& section, const string& name, const string& default_value) const
+{
+    const string str = Get(section, name, "");
+    return str.empty() ? default_value : str;
+}
+
 long INIReader::GetInteger(const string& section, const string& name, long default_value) const
 {
     string valstr = Get(section, name, "");

diff --git a/cpp/INIReader.h b/cpp/INIReader.h
@@ -27,6 +27,11 @@ class INIReader
     // Get a string value from INI file, returning default_value if not found.
     std::string Get(const std::string& section, const std::string& name,
                     const std::string& default_value) const;
+
+    // Get a string value from INI file, returning default_value if not found,
+    // empty, or contains only whitespace.
+    std::string GetString(const std::string& section, const std::string& name,
+                    const std::string& default_value) const;
 
     // Get an integer (long) value from INI file, returning default_value if
     // not found or not a valid integer (decimal "1234", "-1234", or hex "0x4d2").