Решение на Първа задача - температура и химични елементи от Добрин Цветков
Резултати
- 6 точки от тестове
- 0 бонус точки
- 6 точки общо
- 17 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
................. Finished in 0.00797 seconds 17 examples, 0 failures
История (8 версии и 9 коментара)
Добрин обнови решението на 13.10.2016 15:24 (преди около 9 години)
Добрин обнови решението на 13.10.2016 17:09 (преди около 9 години)
Добрин обнови решението на 13.10.2016 17:16 (преди около 9 години)
wanted_unit :/ Колко пари е наградата ако го намеря? Какво ще кажеш за from_units и to_units?
Също, това подреждане на аргументите е в разрез със стиловите правила - сложи ги на един ред със скобите.
Не се грижи за невалидни аргументи. Просто изпусни else-а. В условието пише, че няма да ви даваме невалидни данни. Освен това, да върнеш произволен стринг е лошо поведение в такъв случай. Просто го остави да гръмне или да върне nil.
Можеш ли да се сетиш как да разделиш сметките на две стъпки така, че да не се налага да имаш вложени if-ове?
Идеята ми е помощна функция convert_to_celsius, която винаги да се извиква в началото на convert_between_temperature_units и след това помощна функция convert_from_celsius. Или с 3 помощни функции convert_to_celsius, convert_to_fahrenheit и convert_to_kelvin. Заемам се веднага и след малко качвам нов вариант. Благодаря за забележките :)
Този празен ред не трябва да го има.
Тази таблица е добре да я изкараш в константа извън функцията - MELTING_POINTS
Тази скоба трябва да е на предния ред.
Защо ги подреждаш така? Имаш много място на реда.
Добрин обнови решението на 14.10.2016 13:32 (преди около 9 години)
Добрин обнови решението на 14.10.2016 16:11 (преди около 9 години)
Така е доста по-добре :) А какво ще кажеш за идеята да има само 2 функции - вместо convert_to_celsius, convert_to_fahrenheit и convert_to_kelvin да има само convert_to_celsius и convert_from_celsius :) Така ако искаш да добавиш нова мерна единица няма да се налага да добавяш и още функции.
