KC
я такой вывод делаю из всего обсуждения
Size: a a a
2020 November 12
OR
А я пишу что-то иное, вроде бы.
ST
по твоему выходит — лучше думать про соответствие кодстайлу а не про то насколько корректна программа?
лучше думать про то, про что лучше думать, исходя из контекста 😊
OR
Слушай, ты прямо взялся экстраполировать не на шутку, лучше предложи автору исправление и будешь молодцом.
ST
ну вот ты в репле пробуешь какое-нибудь выражение, в котором что-то может «бомбануть»
ты всегда доводишь это выражение до идеала, что бы обработать все возможные варианты, даже если ты получил ответ на свой вопрос в первую же минуту?
ты всегда доводишь это выражение до идеала, что бы обработать все возможные варианты, даже если ты получил ответ на свой вопрос в первую же минуту?
KC
нет конечно, но по крайней мере закрываю дырки которые я вижу прямо сейчас
KC
операция деления? ну проверь что знаменатель не ноль, это не сложно
ST
а так в общем случае тут надо бы не просто обработку ошибок, а написать тесты
ST
операция деления? ну проверь что знаменатель не ноль, это не сложно
там, наверное, не ноль, а nil?
KC
идею ты понял )
ST
идею ты понял )
в данном случае идея другая
проверить спекой входящий конфиг
что там нет такого инпута, который поломает деление
и тогда при делении ничего проверять не нужно
ты сам любое деление обкладываешь проверками на значение знаменателя, тип аргументов и прочее?
проверить спекой входящий конфиг
что там нет такого инпута, который поломает деление
и тогда при делении ничего проверять не нужно
ты сам любое деление обкладываешь проверками на значение знаменателя, тип аргументов и прочее?
KC
не любое, это зависит от контекста
ST
не любое, это зависит от контекста
🤦♂️ (сарказм 😊 )
KC
в данном случае идея другая
проверить спекой входящий конфиг
что там нет такого инпута, который поломает деление
и тогда при делении ничего проверять не нужно
ты сам любое деление обкладываешь проверками на значение знаменателя, тип аргументов и прочее?
проверить спекой входящий конфиг
что там нет такого инпута, который поломает деление
и тогда при делении ничего проверять не нужно
ты сам любое деление обкладываешь проверками на значение знаменателя, тип аргументов и прочее?
но тут явно нет никаких проверок вокруг и очевидно что ошибка вывалиться пользователю
зачем такое допускать?
зачем такое допускать?
KC
🤦♂️ (сарказм 😊 )
ну да ну да ) но я в это время писал продолжение )
ST
но тут явно нет никаких проверок вокруг и очевидно что ошибка вывалиться пользователю
зачем такое допускать?
зачем такое допускать?
но решение-то ты плохое предлагаешь
если проверку сделать в месте деления, то полезное сообщение пользователю не сформируешь
полезное сообщение может быть только тогда, когда мы говорим, что инпут кривой и что в нём нужно исправать
если проверку сделать в месте деления, то полезное сообщение пользователю не сформируешь
полезное сообщение может быть только тогда, когда мы говорим, что инпут кривой и что в нём нужно исправать
KC
но решение-то ты плохое предлагаешь
если проверку сделать в месте деления, то полезное сообщение пользователю не сформируешь
полезное сообщение может быть только тогда, когда мы говорим, что инпут кривой и что в нём нужно исправать
если проверку сделать в месте деления, то полезное сообщение пользователю не сформируешь
полезное сообщение может быть только тогда, когда мы говорим, что инпут кривой и что в нём нужно исправать
я не предлагал решения, только указал что тут есть проблема в делении
как это решать — на усмотрение автора
как это решать — на усмотрение автора
ST
но тут явно нет никаких проверок вокруг и очевидно что ошибка вывалиться пользователю
зачем такое допускать?
зачем такое допускать?
пока круг пользователей не выйдет за рамки автора скрипта, вопрос не имеет однозначного ответа 😊