Решение на Първа задача - температура и химични елементи от Добрин Цветков
Резултати
- 6 точки от тестове
- 0 бонус точки
- 6 точки общо
- 17 успешни тест(а)
- 0 неуспешни тест(а)
Код
Лог от изпълнението
................. Finished in 0.00797 seconds 17 examples, 0 failures
История (8 версии и 9 коментара)
Добрин обнови решението на 13.10.2016 15:24 (преди около 8 години)
Добрин обнови решението на 13.10.2016 17:09 (преди около 8 години)
Добрин обнови решението на 13.10.2016 17:16 (преди около 8 години)
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 (преди около 8 години)
Добрин обнови решението на 14.10.2016 16:11 (преди около 8 години)
Така е доста по-добре :) А какво ще кажеш за идеята да има само 2 функции - вместо convert_to_celsius
, convert_to_fahrenheit
и convert_to_kelvin
да има само convert_to_celsius
и convert_from_celsius
:) Така ако искаш да добавиш нова мерна единица няма да се налага да добавяш и още функции.