Ⓢ
И нельзя ли избавиться от nullable fields?
Size: a a a
2019 August 20
N
я когда вижу Int? у меня вообще глаз дергаться начинает
F
я когда вижу Int? у меня вообще глаз дергаться начинает
А как вы работаете с базами данных и логикой есть значение/нет значения?
N
Foobar
А как вы работаете с базами данных и логикой есть значение/нет значения?
если у вас есть база где значение целочисленного типа может быть null то конечно, как правило в БД там 0 просто будет стоять)
F
Как вы отличите введённый пользователем 0 от дефолтного?
N
Foobar
Как вы отличите введённый пользователем 0 от дефолтного?
а зачем его отличать? еще раз, я допускаю что где то может быть такая логика, там это уместно, как и Optional в яве, но мне пока именно с целыми типами не встречалось такое
N
просто в приведенном выше коде внимательно на типы посмотрите и скажите нужен ли там Int?
MB
В этом коде много чего не нужно и делать так не надо. На то это и пример говнокода.
MB
Там должно было быть просто:
data class CategoryAvailableRequest(
val groupId: Int,
val partnerInfo: PartnerInfo
): MonoRequest
Ⓢ
👍
MB
С нуллабельностью там вообще отдельная история. Человек вероятно вообще не понял сути и решил, что ему привычнее как в java всегда считать, что везде может быть null.
И
я когда вижу Int? у меня вообще глаз дергаться начинает
Почему?
N
Почему?
потому что в 80% случаев это значит что человек просто не думал
N
остальные 20% оставляю на специфичную логику и missing parameters
AN
Да, говнецо, зачем же создавать обьекты внутри extension а
Не понял, в чем проблема
И
Простите, там уже писалось, но как быть если можно вставить нулл, идеально всю систему не построить, да и в старых проектах, не всегда разделяют классы данных. на дто и модель и прочее
AN
я когда вижу Int? у меня вообще глаз дергаться начинает
Почему? Это только если перфоманс ботлнек плохо.
AN
если у вас есть база где значение целочисленного типа может быть null то конечно, как правило в БД там 0 просто будет стоять)
вот так не надо делать. Лучше null, чем произвольное значение
AN
Там должно было быть просто:
data class CategoryAvailableRequest(
val groupId: Int,
val partnerInfo: PartnerInfo
): MonoRequest
ну это просто клон с джавы
N
вот так не надо делать. Лучше null, чем произвольное значение
точно не скажу за науку, но в веб приложениях я как то не встречал полезных null в int полях