Калина обнови решението на 11.10.2016 21:09 (преди около 8 години)
Харесва ми, че си изкарала тези в константа :)
Не ми харесва, че съдържа думичката TEMP
. Не печелиш нищо от това, че си използвала съкращение. Защо не просто SUBSTANCES_TEMPERATURE_C
. И като сме почнали да разнищваме това име, кое ти звучи по-правилно - substances temperature
или substance temperatures
? :)
Последно, C
-то там е малко неясно. substance temperatures C
не ми звучи правилно. Може би една думичка in
ще оправи нещата? :)
EDIT: Като оправиш името и използваш хеш няма да има нужда от коментара - ще е очевидно от кода.
За температурите, вместо да имаш двойки може да използваш хешове - [0, 100]
=> {melting_point: 0, boiling_point: 100}
. Така долу във функциите няма да ти се наложи да използваш индексите, а [:melting_point]
Супер е, че си разделила двете функции от
и до
целзий. Тук можеш да го напишеш малко по-кратко като използваш inline варианта на when
:
case from
when 'F' then (temperature - 32) * (5.0 / 9)
when 'K' then temperature - 273.15
when 'C' then temperature
end