AS
Тут уже готовая крипта. Я бы хотел абстрактную крипту сделать, в познавательных целях
Size: a a a
2020 December 13
v.
Поделитесь мнением по решению тестового задания.
https://pastebin.com/20qbJ3ct
Звучит как: Представлен класс Patient. Укажите ошибки.
Я воспринял эту формулировку так, что нужно редактировать именно класс, не касаясь обращений к классу.
Поэтому хоть мне и кажется, что красивее все переделать на keyword arguments, не стал этого делать.
https://pastebin.com/20qbJ3ct
Звучит как: Представлен класс Patient. Укажите ошибки.
Я воспринял эту формулировку так, что нужно редактировать именно класс, не касаясь обращений к классу.
Поэтому хоть мне и кажется, что красивее все переделать на keyword arguments, не стал этого делать.
v.
И стоит ли выеживаться и менять camelCase на kebab_case ?
v.
Попробуй запустить код и увидишь все ошибки, на вскидку насчитал 4 штуки
Речь идет про финальный код? Его запускал, выдает то что я ожидаю..
RI
опиши, что видишь некорректным (и почему - это важно отметить). ровно это в задании отмечено
RI
то есть: ты увидел именованый аттрибут посреди кваргов. ты видишь это некорректным. и говоришь об этом, описывая почему некорреткно.
или, допустим, идентация или нейминг. если переименовал в снейк-кейс - то почему? или видишь, что идентация не мешает коду выполняться, но считаешь, что это не ок - то тоже почему? опиши. это важно
или, допустим, идентация или нейминг. если переименовал в снейк-кейс - то почему? или видишь, что идентация не мешает коду выполняться, но считаешь, что это не ок - то тоже почему? опиши. это важно
RI
ты хочешь переделать красивее - тоже обьясни. ты видишь в исходном коде недочеты в "красоте" - опиши их (именно к чему ведет изначальный вид - а к чему ведет твой новый. это важно отоазить). тут много факторов вытекает из того, как написан первичный код. они все важны :) люди хотя это увидеть) прощупать тебя :)
v.
Спасибо за советы, описал так:
# If only class can be modified than mistakes which should be changed:
# - keyword arguments should be after positional arguments
# - last_name always called as positional argument, so it should be converted to it
# - patient? method called on the instance of the Patient class so it should be instance method
# - full_name should have space between words
# - full_name should have conditional statement
# - (optional) Ruby style guides use kebab_case instead of camelCase
#
# Patient class will look like:
<code>
# However if changes to the method's calls is allowed then I would better change all initialize method arguments to keyword arguments.
# If only class can be modified than mistakes which should be changed:
# - keyword arguments should be after positional arguments
# - last_name always called as positional argument, so it should be converted to it
# - patient? method called on the instance of the Patient class so it should be instance method
# - full_name should have space between words
# - full_name should have conditional statement
# - (optional) Ruby style guides use kebab_case instead of camelCase
#
# Patient class will look like:
<code>
# However if changes to the method's calls is allowed then I would better change all initialize method arguments to keyword arguments.
E
Спасибо за советы, описал так:
# If only class can be modified than mistakes which should be changed:
# - keyword arguments should be after positional arguments
# - last_name always called as positional argument, so it should be converted to it
# - patient? method called on the instance of the Patient class so it should be instance method
# - full_name should have space between words
# - full_name should have conditional statement
# - (optional) Ruby style guides use kebab_case instead of camelCase
#
# Patient class will look like:
<code>
# However if changes to the method's calls is allowed then I would better change all initialize method arguments to keyword arguments.
# If only class can be modified than mistakes which should be changed:
# - keyword arguments should be after positional arguments
# - last_name always called as positional argument, so it should be converted to it
# - patient? method called on the instance of the Patient class so it should be instance method
# - full_name should have space between words
# - full_name should have conditional statement
# - (optional) Ruby style guides use kebab_case instead of camelCase
#
# Patient class will look like:
<code>
# However if changes to the method's calls is allowed then I would better change all initialize method arguments to keyword arguments.
В дополнение:
- отступы стоят как попало
- patient?(...) не понятно что делает - должен проверять id или имя? или и то, и то?
- отступы стоят как попало
- patient?(...) не понятно что делает - должен проверять id или имя? или и то, и то?
EE
метод patient? а в нем проверяется что НЕ пэйшинт
EE
а вот эту фразу я не понимаю
EE
# If only class can be modified than mistakes which should be changed:
EE
вы лучше пишите прямыми предложениями: подлежащее сказуемое
v.
Эту оговорку использую, т.к. указано что нужно указать именно ошибки в классе. Так то красивее изменить вызовы, чтоб все через kwargs шли