Refactoring Тема 8: Особые случаи → Урок 3/5

[meloman-team]

1. Вынес названия колонок в константы, проще поправить в одном месте чем в 10.
2. Избавился от использования обезличенных индексов в пользу поиска по названию колонки, так сложнее ошибиться, плюс если метод говорит в комментариях про универсальность то нужно ее обеспечить полностью а не только для 2 вариантов. Например если запрос будет создан с изменением порядка столбцов логика сломается.
3. Не понял для чего избавляться от null и записывать 0 или "" если это делать, то я бы делал еще на этапе записи в БД, ну да ладно, оставил так как было
4. Добавил проверки на null в столбцах с указанием NOT NULL. Для меня логичнее вообще не делать эту проверку но как написал в пункте 3 оставил как было, но если уж так случится и не получим throw new IllegalArgumentException("unknown column '" + name + "'"); то хотелось бы получить внятную ошибку о том что произошло, на случай если изменится название колонки и достанем не то значение.

В общем сделал безобразно но единообразно :D понимаю что это учебный пример, но мимо пройти не смог, хотите принимайте, хотите нет, буду рад комментариям и критике, может узнаю что то новое и полезное.