Total Commander Forum Index Total Commander
Форум поддержки пользователей Total Commander
Сайты: Все о Total Commander | Totalcmd.net | Ghisler.com | RU.TCKB
 
 RulesRules   SearchSearch   FAQFAQ   MemberlistMemberlist   UsergroupsUsergroups   RegisterRegister 
 ProfileProfile   Log in to check your private messagesLog in to check your private messages   Log inLog in 

[wlx] jsontab - просмотр json файлов
Goto page Previous  1, 2, 3  Next
 
Post new topic   Reply to topic    Total Commander Forum Index -> Плагины Total Commander printer-friendly view
View previous topic :: View next topic  
Author Message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Sun Dec 19, 2021 18:50    Post subject: Reply with quote

little-brother wrote:
т.к. содержит комментарии, которые не предусмотрены форматом

но т.к. таких файлов у меня нашлось под 3 сотни, и от разных производителей, то, имхо, стоит внедрить поддержку таких неформатных комментариев. Ибо проги эти файлы открывают без проблем. Значит какие-то библиотеки чтения/парсинга этого формата уже сделали такую поддержку. Вольно или невольно..
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Sun Dec 19, 2021 19:40    Post subject: Reply with quote

Quote:
Ибо проги эти файлы открывают без проблем

Это проги походу этих самых рукожопых производителей, которые знают, что стандарт для слабых духом Very Happy

Добавил в ToDo. Когда будет сделано, ничего не обещаю.
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Sun Dec 19, 2021 20:48    Post subject: Reply with quote

Стандарт всего-лишь путеводная нить - и не более того!
На него надо смотреть ширше и глубже! И не обязательно его соблюдать буквально)))))))
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Mon Jan 31, 2022 23:59    Post subject: Reply with quote

Как выяснилось библиотека, используемая для разбора json, умеет игнорировать комментарии, потому для добавления потребовалось пара строк.

Также, как просили на github, добавил возможность сохранять заполненные фильтры при переключении по узлам (Ctrl + Shift) и копирование результата как json. Вроде полезные фичи.
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Fri Feb 25, 2022 17:08    Post subject: Reply with quote

Еще есть такая ошибка на вводе, как лишняя запятая в перечислениях:
{
"Controls": [
[1140, "Show copy text to clipboard icon"],
[1141, "Show replace selected text icon"],
],
"Id": 14,
"Caption": "Appearance"
}
на 4 строке после ] идет символ "," хотя после него нет никакого нового элемента в []
для узла "Controls". Т.е. этой запятой там быть не должно. И из-за "," ломается весь парсинг.
Хорошо бы уметь игнорировать такую "обычную" ошибку. Т.к. она оказалась весьма часто
повторяемой. Во многих файлах нашел такое...
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Fri Feb 25, 2022 17:35    Post subject: Reply with quote

Отправил запрос на доработку разработчику библиотеки, используемой для парсинга json.

Остается только удивляться, где вы такой кривой софт берете Very Happy
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Fri Feb 25, 2022 19:57    Post subject: Reply with quote

little-brother wrote:
Отправил запрос на доработку разработчику библиотеки, используемой для парсинга json.

Уххх, остается надеяться, что он/она не исчезли во тьме веков....
Иначе попадос полный...
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Fri Feb 25, 2022 20:16    Post subject: Reply with quote

Последнее обновление репозитария - 21 час назад, так что все ок.
Однако автор может посчитать, что это ненужная вещь, тогда увы.
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Fri Feb 25, 2022 20:34    Post subject: Reply with quote

little-brother wrote:
Однако автор может посчитать, что это ненужная вещь, тогда увы.

Не просто увы, а полнейшая и лютейшая ж...
файлов-то много таких найдено....
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Fri Feb 25, 2022 20:42    Post subject: Reply with quote

Лично у меня с этим всё прекрасно - ни одного такого файла никогда в глаза не видел Very Happy
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Sat Feb 26, 2022 13:28    Post subject: Reply with quote

little-brother wrote:
Лично у меня с этим всё прекрасно - ни одного такого файла никогда в глаза не видел

Блин, боюсь предположить - что вас еще и одни красотки окружают? Ну вааще свезло так свезло вам Wink снимаю шляпу! Laughing

Добавлено спустя 54 минуты:

little-brother wrote:
Отправил запрос на доработку разработчику библиотеки, используемой для парсинга json.

Кстати, а раз используется другая либа для парсинга - то там есть вообще вывод ошибок? Ибо эту запятую я искал... долго. пока не догадался прогнать файл через онлайн проверяльщик. И вот он уже с точностью до знака указал место ошибки. Вот бы было хорошо, если бы неудачный результат парсинга как-то выводился на экран в плагине - мол, да, хотели показать, но не смогли, т.к. ошибка вот такая-то.....
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Sat Feb 26, 2022 14:57    Post subject: Reply with quote

Бегло глянул, не видно. Если ошибка есть, то просто либа вернет 0, а уже что там не так это уже не важно.
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Sat Feb 26, 2022 15:36    Post subject: Reply with quote

little-brother wrote:
а уже что там не так это уже не важно.

отнюдь.
Ведь это в моем софте эта ошибка может быть пропущена. А если я передам этот файл челу, у которого более строгая проверка будет - то она не пропустит этот файл. И чтоб такого не было - наличие вывода при парсинге в плагине очень даже важно. Можно не знать смысла ошибки, но указать последний символ, после которого появляется ошибка - это очень важно для последующего понимания "а почему что-то не работает, не открывается, не показывается". Как вот эта запятая. Фиг ее просто глазками найдешь.
Back to top
View user's profile Send private message
little-brother



Joined: 29 Sep 2021
Posts: 205

Post (Separately) Posted: Sat Feb 26, 2022 15:48    Post subject: Reply with quote

Это я со стороны либы написал. В целом, если работа завязана на внешние файлы, то иногда делают этап проверки/очистки данных.
Back to top
View user's profile Send private message
Destiny



Joined: 15 Dec 2015
Posts: 212

Post (Separately) Posted: Sat Feb 26, 2022 22:41    Post subject: Reply with quote

little-brother wrote:
Последнее обновление репозитария - 21 час назад, так что все ок.

Ну пока что он даже хоть какого-то комментария там не написал, не факт, что даже прочитал и понял - о чем речь((((
Back to top
View user's profile Send private message
Display posts from previous:   
Post new topic   Reply to topic    Total Commander Forum Index -> Плагины Total Commander All times are GMT + 4 Hours
Goto page Previous  1, 2, 3  Next
Page 2 of 3

 
Jump to:  
You cannot post new topics in this forum
You cannot reply to topics in this forum
You cannot edit your posts in this forum
You cannot delete your posts in this forum
You cannot vote in polls in this forum


Powered by phpBB © 2001, 2005 phpBB Group