The OpenNET Project / Index page

[ новости /+++ | форум | теги | ]



"Выпуск каталогизатора домашней библиотеки MyLibrary 3.0"
Версия для распечатки Пред. тема | След. тема
Форум Разговоры, обсуждение новостей
Исходное сообщение [ Отслеживать ]
Есть идеи по улучшению форума и сайта ? Пишите.
. "Выпуск каталогизатора домашней библиотеки MyLibrary 3.0" +/
Сообщение от Аноним (56), 27-Фев-24, 15:20 
> На этом мои возможности заканчиваются.

Ну можно еще:
1. Обложить try-except-ом и аккуратно все ловить и обрабатывать
2. Если оно в core dump-ит, то вынести в отдельный процесс эти рарчики, и следить за ним снаружи

> Таки разберитесь, что это вообще такое, и всё станет сразу понятно.

Да, стало
1. bie указатель на _экземпляр_ класса BookInfoEntry, который возвращает get_book_info.
2. Вы в 277-й строке bie.reset() вызываете, и забываете, что там раньше было
3. В 280-й строке ваш get_book_info может raise MLException, правильно? Но это должно произойти _до_ того, как ваш bie получит новое значение, правильно?
4. Однако, в 285-й строке вы зачем-то снова делаете bie.reset. Значит, одно из двух - либо это копипаста (и вот как раз такие штуки ловят нормальные ревьюверы и иногда - ИИшечка), либо ваш bie может откуда-то получить значение параллельно всему этому, и вы его reset чтобы не разбираться и не заморачиваться с аккуратным параллельным программированием.

> Я стесняюсь спросить - а зачем они тут?

Плавали-знаем. Да, так технически можно разрабатывать хобби-проект и _технически_ - это ок.
Из опыта - с ростом кодовой базы и сложности - бегать по всему коду с дебаггером на каждый чих становится все более и более сложно. Стесняюсь спросить - а почему переписать с нуля это-вот-все оказалось проще, чем инкрементально развивать? Неужели сложность крышу проломила?

Выкладывать в опенсорс проект, в котором тестов нет никаких - очень _этически_ сомнительное действие. Потому что кроме прямого действия - "читалка библиотеки опенсорсная" - у проекта есть и косвенное действие, даже два:

1. Вы демонстрируете свой код другим разработчикам, и они впитывают (до какой-то степени, естественно) ваши паттерны поведения. И паттерн - "пройдусь дебаггером и так сойдет" - он вреден. Как и паттерн - "юзер сам виноват что мой код у него падает - это опенсорс, я сделаль, берите-что-дают-а-то-и-эти-кончатся"

2. Вы демонстрируете свой код потенциальным коллегам и работодателям (да, люди читают гитхабчики других людей!) И паттерны поведения, как технического так и личностного, могут ограничить ваши карьерные перспективы.

Кстати - пойти и зафайлить багу в библиотечке rar-чиков - это прям хорошо и правильно, keep doing!

Ответить | Правка | Наверх | Cообщить модератору

Оглавление
Выпуск каталогизатора домашней библиотеки MyLibrary 3.0, opennews, 22-Фев-24, 21:48  [смотреть все]
Форумы | Темы | Пред. тема | След. тема



Партнёры:
PostgresPro
Inferno Solutions
Hosting by Hoster.ru
Хостинг:

Закладки на сайте
Проследить за страницей
Created 1996-2024 by Maxim Chirkov
Добавить, Поддержать, Вебмастеру