View previous topic :: View next topic |
Author |
Message |
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Sun Dec 19, 2021 18:50 Post subject: |
|
|
little-brother wrote: | т.к. содержит комментарии, которые не предусмотрены форматом |
но т.к. таких файлов у меня нашлось под 3 сотни, и от разных производителей, то, имхо, стоит внедрить поддержку таких неформатных комментариев. Ибо проги эти файлы открывают без проблем. Значит какие-то библиотеки чтения/парсинга этого формата уже сделали такую поддержку. Вольно или невольно.. |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Sun Dec 19, 2021 19:40 Post subject: |
|
|
Quote: | Ибо проги эти файлы открывают без проблем |
Это проги походу этих самых рукожопых производителей, которые знают, что стандарт для слабых духом
Добавил в ToDo. Когда будет сделано, ничего не обещаю. |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Sun Dec 19, 2021 20:48 Post subject: |
|
|
Стандарт всего-лишь путеводная нить - и не более того!
На него надо смотреть ширше и глубже! И не обязательно его соблюдать буквально))))))) |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Mon Jan 31, 2022 23:59 Post subject: |
|
|
Как выяснилось библиотека, используемая для разбора json, умеет игнорировать комментарии, потому для добавления потребовалось пара строк.
Также, как просили на github, добавил возможность сохранять заполненные фильтры при переключении по узлам (Ctrl + Shift) и копирование результата как json. Вроде полезные фичи. |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Fri Feb 25, 2022 17:08 Post subject: |
|
|
Еще есть такая ошибка на вводе, как лишняя запятая в перечислениях:
{
"Controls": [
[1140, "Show copy text to clipboard icon"],
[1141, "Show replace selected text icon"],
],
"Id": 14,
"Caption": "Appearance"
}
на 4 строке после ] идет символ "," хотя после него нет никакого нового элемента в []
для узла "Controls". Т.е. этой запятой там быть не должно. И из-за "," ломается весь парсинг.
Хорошо бы уметь игнорировать такую "обычную" ошибку. Т.к. она оказалась весьма часто
повторяемой. Во многих файлах нашел такое... |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Fri Feb 25, 2022 17:35 Post subject: |
|
|
Отправил запрос на доработку разработчику библиотеки, используемой для парсинга json.
Остается только удивляться, где вы такой кривой софт берете |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Fri Feb 25, 2022 19:57 Post subject: |
|
|
little-brother wrote: | Отправил запрос на доработку разработчику библиотеки, используемой для парсинга json. |
Уххх, остается надеяться, что он/она не исчезли во тьме веков....
Иначе попадос полный... |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Fri Feb 25, 2022 20:16 Post subject: |
|
|
Последнее обновление репозитария - 21 час назад, так что все ок.
Однако автор может посчитать, что это ненужная вещь, тогда увы. |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Fri Feb 25, 2022 20:34 Post subject: |
|
|
little-brother wrote: | Однако автор может посчитать, что это ненужная вещь, тогда увы. |
Не просто увы, а полнейшая и лютейшая ж...
файлов-то много таких найдено.... |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Fri Feb 25, 2022 20:42 Post subject: |
|
|
Лично у меня с этим всё прекрасно - ни одного такого файла никогда в глаза не видел |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Sat Feb 26, 2022 13:28 Post subject: |
|
|
little-brother wrote: | Лично у меня с этим всё прекрасно - ни одного такого файла никогда в глаза не видел |
Блин, боюсь предположить - что вас еще и одни красотки окружают? Ну вааще свезло так свезло вам снимаю шляпу!
Добавлено спустя 54 минуты:
little-brother wrote: | Отправил запрос на доработку разработчику библиотеки, используемой для парсинга json. |
Кстати, а раз используется другая либа для парсинга - то там есть вообще вывод ошибок? Ибо эту запятую я искал... долго. пока не догадался прогнать файл через онлайн проверяльщик. И вот он уже с точностью до знака указал место ошибки. Вот бы было хорошо, если бы неудачный результат парсинга как-то выводился на экран в плагине - мол, да, хотели показать, но не смогли, т.к. ошибка вот такая-то..... |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Sat Feb 26, 2022 14:57 Post subject: |
|
|
Бегло глянул, не видно. Если ошибка есть, то просто либа вернет 0, а уже что там не так это уже не важно. |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Sat Feb 26, 2022 15:36 Post subject: |
|
|
little-brother wrote: | а уже что там не так это уже не важно. |
отнюдь.
Ведь это в моем софте эта ошибка может быть пропущена. А если я передам этот файл челу, у которого более строгая проверка будет - то она не пропустит этот файл. И чтоб такого не было - наличие вывода при парсинге в плагине очень даже важно. Можно не знать смысла ошибки, но указать последний символ, после которого появляется ошибка - это очень важно для последующего понимания "а почему что-то не работает, не открывается, не показывается". Как вот эта запятая. Фиг ее просто глазками найдешь. |
|
Back to top |
|
|
little-brother
Joined: 29 Sep 2021 Posts: 205
|
(Separately) Posted: Sat Feb 26, 2022 15:48 Post subject: |
|
|
Это я со стороны либы написал. В целом, если работа завязана на внешние файлы, то иногда делают этап проверки/очистки данных. |
|
Back to top |
|
|
Destiny
Joined: 15 Dec 2015 Posts: 212
|
(Separately) Posted: Sat Feb 26, 2022 22:41 Post subject: |
|
|
little-brother wrote: | Последнее обновление репозитария - 21 час назад, так что все ок. |
Ну пока что он даже хоть какого-то комментария там не написал, не факт, что даже прочитал и понял - о чем речь(((( |
|
Back to top |
|
|
|