LibXL - на определённой ячейке таблицы ломается чтение из таблицы программа

@SkYMaaN · Регистрация: 05.04.2019

Author24 — интернет-сервис помощи студентам

Суть функции в чтении ячеек excel и помещении в JSON. Но на определённой ячейке таблицы ( на скрине выделил красным ) - программа ломается - выдаёт пустой текст в функцию конвертации Unicode to UTF - 8 (нужно для работы с JSON).
ОС: Win 10 Pro
Microsoft Visual Studio 2019
Библиотека Excel: https://www.libxl.com/home.html
Библиотека JSON: https://github.com/nlohmann/json
Функция чтения таблицы:

C++

void client::CreateJSONARR_Characteristics_of_soils_by_the_content_of_mobile()
{
    Book* book = xlCreateBook();
    if (book->load(path))
    {
        Sheet* sheet = book->getSheet(0);
        if (sheet)
        {
            json tmp =
            {
                {u8"Область", ""}, //0
                {u8"Обстежена_площа", ""}, //1
                {u8"Дуже_сильно_та_сильно_кислі", ""}, //2
                {u8"Дуже_сильно_та_сильно_кислі_%", ""}, //3
                {u8"Середньо_кислі", ""}, //4
                {u8"Середньо_кислі_%", ""}, //5
                {u8"Слабо_кислі", ""}, //6
                {u8"Слабо_кислі_%", ""}, //7
                {u8"Усього_кислих", ""}, //8
                {u8"Усього_кислих_%", ""}, //9
                {u8"Близькі_до_нейтральних", ""}, //10
                {u8"Близькі_до_нейтральних_%", ""}, //11
                {u8"Нейтральні", ""}, //12
                {u8"Нейтральні_%", ""}, //13
                {u8"Слабо_лужні", ""}, //14
                {u8"Слабо_лужні_%", ""}, //15
                {u8"Середньо_лужні", ""}, //16
                {u8"Середньо_лужні_%", ""},//17
                {u8"Дуже_сильно_та_сильно_лужні", ""}, //18
                {u8"Дуже_сильно_та_сильно_лужні_%", ""}, //19
                {u8"Середньозважений_показник", ""}, //20
            };
            for (size_t row = sheet->firstRow(); row < sheet->lastRow(); row++)
            {
                for (size_t col = sheet->firstCol(); col < sheet->lastCol(); col++)
                {
                    CellType cellType = sheet->cellType(row, col);
                    switch (cellType)
                    {
                    case CELLTYPE_STRING:
                    {
                        if (col == 0)
                        {
                            const wchar_t* s = sheet->readStr(row, col);
 
                            const auto key = tools::utf8::convert(L"Область");
                            const auto val = tools::utf8::convert(s);
                            tmp[key] = val;
                        }
                        
                        break;
                    }
                    case CELLTYPE_NUMBER:
                    {
                       ....
                        if (col == 12) //Нейтральні       // тут ошибка
                        {
                            const wchar_t* s = sheet->readStr(row, col);
                            wcout << s << endl;
                            const auto key = tools::utf8::convert(L"Нейтральні");
                            const auto val = tools::utf8::convert(s);
                            tmp[key] = val;
                        }
                        break;
                    }
                    }
                }     //for (size_t col = sheet->firstCol(); col < sheet->lastCol(); col++)
                const auto key = tools::utf8::convert(L"elements");
                const auto key1 = tools::utf8::convert(L"element");
                const auto key2 = tools::utf8::convert(L"date");
                jarr[key][key1].push_back(tmp);
                tmp.clear();
            }     //for (size_t row = 3; row < sheet->lastRow(); row++) 
            book->release();
        }
    }
}

Функция конвертирования:

C++

  ::std::string convert(const wchar_t* text)
        {
 
                assert(text); // тут ошибка
                if (text[0] == 0)
                    return ::std::string();
                const size_t len = ::str_length(text);
                return ::tools::utf8::convert(text, len);
        }

Подскажите пожалуйста, в чём ошибка?

@SkYMaaN · 10.12.2020, 17:39 **[ТС]**

.xls

@oleg-m1973 · 10.12.2020, 17:51

Сообщение от SkYMaaN

Подскажите пожалуйста, в чём ошибка?

Наверное в том, что когда sheet->readStr(row, col) возвращает nullptr, ты не проверяешь и вызываешь convert, в котором срабатывает assert

@SkYMaaN · 10.12.2020, 18:02 **[ТС]**

oleg-m1973, я писал проверки, даже внутри convert:

C++

 try
                            {
                                const wchar_t* s = sheet->readStr(row, col);
                                const auto key = tools::utf8::convert(L"Нейтральні");
                                const auto val = tools::utf8::convert(s);
                                tmp[key] = val;
                            }
                            catch (const exception ex)
                            {
                                const auto key = tools::utf8::convert(L"Нейтральні");
                                tmp[key] = "";
                            }

C++

const wchar_t* s = sheet->readStr(row, col);
                                if (s != nullptr)
                                {
                                    const auto key = tools::utf8::convert(L"Нейтральні");
                                    const auto val = tools::utf8::convert(s);
                                    tmp[key] = val;
                                }
                                else
                                {
                                    const auto key = tools::utf8::convert(L"Нейтральні");
                                    tmp[key] = "";
 
                                }

но результат тот же

@oleg-m1973 · 10.12.2020, 18:05

Сообщение от SkYMaaN

oleg-m1973, я писал проверки, даже внутри convert:

C++

 ::std::string convert(const wchar_t* text)
        {
 if (!text)
       return {};
 
                assert(text); // тут ошибка
                if (text[0] == 0)
                    return ::std::string();

Добавлено через 49 секунд
Не везде, значит писал

@SkYMaaN · 10.12.2020, 18:07 **[ТС]**

Благодарю, получилось.
Могу поинтересоваться почему проверки которые я писал в сообщении выше не сработали на пару с:

C++

::std::string convert(const wchar_t* text)
        {
          
            try
            {
                if (text[0] == 0)
                    return ::std::string();
                const size_t len = ::str_length(text);
                return ::tools::utf8::convert(text, len);
            }
            catch(const std::exception ex)
            {
                text = L"";
                if (text[0] == 0)
                    return ::std::string();
                const size_t len = ::str_length(text);
                return ::tools::utf8::convert(text, len);
            }
        }

И почему возвращается nullptr если ячейка в таблице заполнена?

@oleg-m1973 · 10.12.2020, 18:12

Сообщение от SkYMaaN

И почему возвращается nullptr если ячейка в таблице заполнена?

Returns NULL if the specified cell doesn't contain a string or error occurs.

https://www.libxl.com/spreadsheet.html#readStr

Добавлено через 37 секунд

Сообщение от SkYMaaN

Могу поинтересоваться почему проверки которые я писал в сообщении выше не сработали на пару с:

Где у тебя здесь проверка на nullptr?

@SkYMaaN · 10.12.2020, 20:26 **[ТС]**

Сообщение от oleg-m1973

Где у тебя здесь проверка на nullptr?

Вот

C++

const wchar_t* s = sheet->readStr(row, col);
                                if (s != nullptr)
                                {
                                    const auto key = tools::utf8::convert(L"Нейтральні");
                                    const auto val = tools::utf8::convert(s);
                                    tmp[key] = val;
                                }
                                else
                                {
                                    const auto key = tools::utf8::convert(L"Нейтральні");
                                    tmp[key] = "";
 
                                }

@oleg-m1973 · 10.12.2020, 20:28

Сообщение от SkYMaaN

Вот

В том коде, который ты показал вначале, этого нет. И у тебя там несколько вызовов sheet->readStr

@SkYMaaN · 10.12.2020, 20:49 **[ТС]**

Сообщение от oleg-m1973

В том коде, который ты показал вначале, этого нет.

Да, нету. Но я пробовал именно с таким фрагментом кода :

C++

const wchar_t* s = sheet->readStr(row, col);
                                if (s != nullptr)
                                {
                                    const auto key = tools::utf8::convert(L"Нейтральні");
                                    const auto val = tools::utf8::convert(s);
                                    tmp[key] = val;
                                }
                                else
                                {
                                    const auto key = tools::utf8::convert(L"Нейтральні");
                                    tmp[key] = "";
 
                                }

и выбивает туже ошибку

@hoggy · 10.12.2020, 21:43

Сообщение от SkYMaaN

с таким фрагментом кода :

с таким фрагментом кода всё в порядке.

Сообщение от SkYMaaN

и выбивает туже ошибку

если с таким фрагментом кода у тебя всё равно срабатывает ассерт,
значит дело не в этом фрагменте кода.
значит ассерт срабатывает где-то в другом месте.
значит ты где-то ещё накосячил с указателем.

@DrOffset · 10.12.2020, 22:00

Сообщение от SkYMaaN

выбивает туже ошибку

Значит convert где-то еще вызывается с nullptr помимо этой вашей проверки.

Добавлено через 15 минут
SkYMaaN, вообще говоря ассерт в такой ситуации показывает где вы накосячили в логике программы. Тем, что вы заглушили его явной if-проверкой

C++

1 2	if (!text) return {};

делу не помогает. Теперь вы не знаете где именно у вас появляется nullptr, и, соответственно, ничего с этим не делаете. Может быть оно так и надо, но учитывая ваши последующие вопросы, вы понятия не имеете что у вас происходит в программе, но уже применили решение, которое еще больше добавляет неизвестности.

Разберитесь сначала где именно точно должны быть nullptr, ассерт вам в этом поможет. И только потом при необходимости глушите их проверкой.

@hoggy · 10.12.2020, 22:09

Сообщение от oleg-m1973

if (!text)
return {};

Сообщение от oleg-m1973

Не везде, значит писал

конкретно в функции convert такая проверка не нужна, и только вредит.

@SkYMaaN 25 / 19 / 8 Регистрация: 05.04.2019 Сообщений: 338
	10.12.2020, 17:39 [ТС]	2
	.xls Миниатюры 0

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	10.12.2020, 17:51	3
	Сообщение от SkYMaaN Подскажите пожалуйста, в чём ошибка? Наверное в том, что когда sheet->readStr(row, col) возвращает nullptr, ты не проверяешь и вызываешь convert, в котором срабатывает assert 0

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	10.12.2020, 18:12	7
	Сообщение от SkYMaaN И почему возвращается nullptr если ячейка в таблице заполнена? Returns NULL if the specified cell doesn't contain a string or error occurs. https://www.libxl.com/spreadsheet.html#readStr Добавлено через 37 секунд Сообщение от SkYMaaN Могу поинтересоваться почему проверки которые я писал в сообщении выше не сработали на пару с: Где у тебя здесь проверка на nullptr? 0

@oleg-m1973 6579 / 4564 / 1843 Регистрация: 07.05.2019 Сообщений: 13,726
	10.12.2020, 20:28	9
	Сообщение от SkYMaaN Вот В том коде, который ты показал вначале, этого нет. И у тебя там несколько вызовов sheet->readStr 0

@hoggy 8739 / 4317 / 960 Регистрация: 15.11.2014 Сообщений: 9,760
	10.12.2020, 21:43	11
	Сообщение от SkYMaaN с таким фрагментом кода : с таким фрагментом кода всё в порядке. Сообщение от SkYMaaN и выбивает туже ошибку если с таким фрагментом кода у тебя всё равно срабатывает ассерт, значит дело не в этом фрагменте кода. значит ассерт срабатывает где-то в другом месте. значит ты где-то ещё накосячил с указателем. 1

@hoggy 8739 / 4317 / 960 Регистрация: 15.11.2014 Сообщений: 9,760
	10.12.2020, 22:09	13
	Сообщение от oleg-m1973 if (!text) return {}; Сообщение от oleg-m1973 Не везде, значит писал конкретно в функции convert такая проверка не нужна, и только вредит. 0