Обновлена используемая версия simstr.

Author: orefkov

CMakeLists.txt

@@ -5,7 +5,7 @@ include(FetchContent)
 
 project(
     simjson
-    VERSION 1.2.2
+    VERSION 1.2.3
     DESCRIPTION "Very simple json library"
     HOMEPAGE_URL "https://github.com/orefkov/simjson"
     LANGUAGES CXX
@@ -67,7 +67,7 @@ function(add_simstr)
         simstr
         GIT_REPOSITORY https://github.com/orefkov/simstr.git
         GIT_SHALLOW TRUE
-        GIT_TAG 458007e
+        GIT_TAG tags/rel1.2.6
         FIND_PACKAGE_ARGS NAMES simstr
     )
     FetchContent_MakeAvailable(simstr)

include/simjson/json.h

@@ -582,32 +582,36 @@ class JsonValueTempl : public Json {
         return at(std::forward<T>(key));
     }
     /*!
-     * @ru @brief Обращение к свойству константного объекта по набору ключей.
-     * @details Функция последовательно переходит от значения к значению по заданным ключам.
-     *  Как только указанный ключ не будет найден, перебор останавливается.
-     *  Например: config("a", "b", "c") для объекта {"a": {"b": {"c": 10}}} вернёт ссылку на 10.
+     * @ru @brief Обращение к свойству константного объекта по набору ключей/индексов.
+     * @details Функция последовательно переходит от значения к значению по заданным ключам/индексам.
+     *  Как только указанный ключ/индекс не будет найден, перебор останавливается.
+     *  Например: config("a", "b", 1, "c") для объекта {"a": {"b":[{"c": 2}, {"c": 10}]}} вернёт ссылку на 10.
      * @tparam T - типа ключа.
-     * @param key - ключ.
+     * @param key - ключ / индекс.
      * @param ...args - остальные ключи.
-     * @return const json_value& - ссылка на значение ключа.
+     * @return const json_value& - ссылка на найденное значение.
      *   Если значение не объект, или указанного ключа нет, не создаёт новую пару, а возвращает ссылку на UNDEFINED.
-     * @en @brief Access to a property of a constant object by a set of keys.
-     * @details The function moves sequentially from value to value based on the given keys.
-     * As soon as the specified key is not found, the search stops.
-     * For example: config("a", "b", "c") for object {"a": {"b": {"c": 10}}} will return a reference to 10.
+     * @en @brief Access to a property of a constant object by a set of keys/indexes.
+     * @details The function moves sequentially from value to value based on the given keys/indexes.
+     * As soon as the specified key/index is not found, the search stops.
+     * For example: config("a", "b", 1, "c") for object {"a": {"b": [{"c": 2}, {"c": 10}]}} will return a reference to 10.
      * @tparam T - key type.
-     * @param key - key.
+     * @param key - key / index.
      * @param ...args - other keys.
      * @return const json_value& - reference to the key value.
      * If the value is not an object, or the specified key does not exist, does not create a new pair, but returns a reference to UNDEFINED.
      */
-    template<jt::JsonKeyType<K> T, typename...Args>
+    template<typename T, typename...Args> requires (jt::JsonKeyType<T, K> || std::convertible_to<T, size_t>)
     const json_value& operator()(T&& key, Args&&...args) const {
         const json_value& res = at(std::forward<T>(key));
         if constexpr (sizeof...(Args) == 0) {
             return res;
         } else {
-            return res.is_undefined() ? res : res(std::forward<Args>(args)...);
+            if constexpr (jt::JsonKeyType<std::tuple_element_t<0, std::tuple<Args...>>, K>) {
+                return res.is_object() ? res(std::forward<Args>(args)...) : UNDEFINED;
+            } else {
+                return res.is_array() ? res(std::forward<Args>(args)...) : UNDEFINED;
+            }
         }
     }
     /*!

readme.md

@@ -3,14 +3,14 @@
 
 Designed to work with JSON when using the [simstr](https://github.com/orefkov/simstr) library.
 
-Version 1.2.2.
+Version 1.2.3.
 
 <span class="obfuscator"><a href="readme_ru.md">On Russian | По-русски</a></span>
 
 This library contains a simple implementation of a simple JsonValue object for working with JSON using string objects
 of the *simstr* library, since other libraries mainly work with `std::string` or raw `const char*`.
 The task was not to somehow compete in performance or optimality with other libraries, I mainly use it
-for working with small config files - read, modify, write.
+for working with small config files - read, modify, write. However, it also copes quite well with large files.
 
 For json objects, `std::unordered_map` is used, in the form of `hashStrMap<K, JsonValueTemp<K>>`,
 for arrays - `std::vector<JsonValueTemp<K>>`, strings are stored in `sstring<K>`.
@@ -50,7 +50,7 @@ downloads automatically).
     // Simple json, equal to 1.
     JsonValue json = 1; // int64
     stringa text = json.store(); // "1"
-    
+
     // Initialization of the object, ""_h - slightly optimizes the key, calculating the hash during compilation
     JsonValue obj = {
         {"Key1"_h, 1},
@@ -67,7 +67,7 @@ downloads automatically).
     // Saving keys, immediately several levels
     test["a"_h]["b"_h]["c"_h] = 10;
     text = json.store(); // {"a":{"b":{"c":10}}}
-    
+
     // Converts a json object to an array
     test[0] = "value";
     test[-1] = true;  // When using -1 - the value is added to the end of the array
@@ -114,9 +114,9 @@ void read_config_from_file(ssa folder, ssa file_name) {
         stringa config;
         if (is_file_exist(fullPath)) {
             config = get_file_content(fullPath);
-            
+
             auto [readed, error, line, col] = JsonValue::parse(config);
-            
+
             if (error != JsonParseResult::Success) {
                 std::cerr << "Error in parse config file " << fullPath <<
                   " at line " << line << ", col " << col << std::endl;

readme_ru.md

@@ -3,14 +3,14 @@
 
 Предназначена для работы с JSON при использовании библиотеки [simstr](https://github.com/orefkov/simstr).
 
-Версия 1.2.2.
+Версия 1.2.3.
 
 <span class="obfuscator"><a href="readme.md">On English | По-английски</a></span>
 
 В этой библиотеке содержится простая реализация простого объекта JsonValue для работы с JSON с использованием строковых объектов
 библиотеки *simstr*, так как другие библиотеки работают в основном с `std::string` или сырыми `const char*`.
-Задача как-то соревноваться по производительности или оптимальности с другими библиотеками не ставилась, я её применяю в-основном 
-для работы с небольшими конфиг-файлами - прочитать, изменить, записать.
+Задача как-то соревноваться по производительности или оптимальности с другими библиотеками не ставилась, я её применяю в-основном
+для работы с небольшими конфиг-файлами - прочитать, изменить, записать, однако и с большими файлами она вполне успешно справляется.
 
 Для json-объектов используется `std::unordered_map`, в лице `hashStrMap<K, JsonValueTemp<K>>`,
 для массивов - `std::vector<JsonValueTemp<K>>`, строки хранятся в `sstring<K>`.
@@ -50,7 +50,7 @@
     // Простой json, равный 1.
     JsonValue json = 1; // int64
     stringa text = json.store(); // "1"
-    
+
     // Инициализация объекта, ""_h - несколько оптимизирует ключ, вычисляя хэш при компиляции
     JsonValue obj = {
         {"Key1"_h, 1},
@@ -67,7 +67,7 @@
     // Сохраняем ключи, сразу несколько уровней
     test["a"_h]["b"_h]["c"_h] = 10;
     text = json.store(); // {"a":{"b":{"c":10}}}
-    
+
     // Преобразует json-объект в массив
     test[0] = "value";
     test[-1] = true;  // При использовании -1 - значение добавляется в конец массива
@@ -114,9 +114,9 @@ void read_config_from_file(ssa folder, ssa file_name) {
         stringa config;
         if (is_file_exist(fullPath)) {
             config = get_file_content(fullPath);
-            
+
             auto [readed, error, line, col] = JsonValue::parse(config);
-            
+
             if (error != JsonParseResult::Success) {
                 std::cerr << "Error in parse config file " << fullPath <<
                   " at line " << line << ", col " << col << std::endl;

src/json.cpp

@@ -243,7 +243,6 @@ inline static bool is_double_int64(double dbl) {
     return
         !std::isnan(dbl) &&
         !std::isinf(dbl) &&
-        // std::numeric_limits<int64_t>::max() при конвертировании в double меняет 9223372036854775807 на 9223372036854775808
         dbl <= (uint64_t(1) << 53) &&
         dbl >= -(int64_t)(uint64_t(1) << 53);
 }
@@ -254,20 +253,15 @@ SIMJSON_API std::optional<int64_t> JsonValueTempl<K>::to_integer() const {
     case Boolean:
         return val_.boolean ? 1 : 0;
     case Text: {
-        auto txt = as_text().template trimmed_right<ssType>();
-        auto [res, err, processed] = txt.template to_int<int64_t>();
+        auto txt = as_text().trimmed();
+        auto [res, err, processed] = txt.template to_int<int64_t, true, 0, false>();
         if (err == IntConvertResult::Success) {
             return res;
         }
         if (processed > 0 && err == IntConvertResult::BadSymbolAtTail &&
-            (txt[processed] == '.' || txt[processed] == 'e' || txt[processed] == 'E')) {
+            (txt[processed] == '.' || (txt[processed] | 0x20) == 'e')) {
             // Считаем, что в тексте double
-            auto dbl = std::nan("");
-            if constexpr (is_one_of_std_char_v<K>) {
-                dbl = txt.to_double();
-            } else {
-                dbl = lstringa<100>{txt}.to_double();
-            }
+            auto dbl = txt.to_double().value_or(std::nan("0"));
             if (is_double_int64(dbl)) {
                 return static_cast<int64_t>(dbl);
             }
@@ -301,11 +295,7 @@ SIMJSON_API double JsonValueTempl<K>::to_real() const {
     case Json::Boolean:
         return val_.boolean ? 1.0 : 0.0;
     case Json::Text:
-        if constexpr (is_one_of_std_char_v<K>) {
-            return val_.text.to_double();
-        } else {
-            return lstringa<100>{val_.text}.to_double();
-        }
+        return val_.text.to_double().value_or(std::nan("0"));
     case Json::Integer:
         return static_cast<double>(val_.integer);
     case Json::Real:
@@ -485,7 +475,10 @@ struct json_store {
                     }
                 }
             }
-            buffer += e_if(prettify && printed, uni_string(K, "\n") + e_c(indent - indent_count, indent_symb)) + uni_string(K, "}");
+            if (prettify && printed) {
+                buffer += uni_string(K, "\n") + e_c(indent - indent_count, indent_symb);
+            }
+            buffer += uni_string(K, "}");
             break;
         case Json::Array:
             buffer += uni_string(K, "[");
@@ -494,7 +487,10 @@ struct json_store {
                 store(it, indent + indent_count);
                 printed = true;
             }
-            buffer += e_if(prettify && printed, uni_string(K, "\n") + e_c(indent - indent_count, indent_symb)) + uni_string(K, "]");
+            if (prettify && printed) {
+                buffer += uni_string(K, "\n") + e_c(indent - indent_count, indent_symb);
+            }
+            buffer += uni_string(K, "]");
             break;
         }
     }
@@ -1120,11 +1116,7 @@ JsonValueTempl<K>* StreamedJsonParser<K>::addNumber(JsonValueTempl<K>* current)
     }
 
     if (!asInt || jsonValue.is_undefined()) {
-        if constexpr (is_one_of_std_char_v<K>) {
-            jsonValue = ssValue.to_double();
-        } else {
-            jsonValue = lstringa<100>(ssValue).to_double();
-        }
+        jsonValue = ssValue.to_double().value_or(std::nan("0"));
     }
 
     if constexpr (!All) {

tests/test_json.cpp

@@ -501,6 +501,22 @@ TEST(SimJson, JsonStdInitMap) {
     EXPECT_EQ(json2.store(false, true), "{\"four\":4,\"one\":1,\"three\":3,\"two\":2}");
 }
 
+TEST(SimJson, ManyKeys) {
+    JsonValueU json = {
+        {u"a"_h, {
+            {u"b"_h, {
+                {{u"c"_h, 1}},
+                {{u"c"_h, 10}}
+            }}
+        }},
+    };
+    EXPECT_EQ(json.store(), uR"=({"a":{"b":[{"c":1},{"c":10}]}})=");
+
+    const auto v = json(u"a"_h, u"b"_h, 1, u"c"_h);
+    ASSERT_TRUE(v.is_integer());
+    EXPECT_EQ(v.as_integer(), 10);
+}
+
 #if 0
 TEST(SimJson, JsonParseBig) {
     stringa content1 = get_file_content("citm_catalog.json");