Радослав обнови решението на 14.10.2016 22:02 (преди около 8 години)
+def from_celsius(degrees, wanted_unit)
+ if wanted_unit == 'F'
+ (degrees * 9) / 5.0 + 32
Тук може вместо да умножаваш по 9 и да делиш на 5, да умножиш по 1.8 директно.
+ elsif wanted_unit == 'K'
+ degrees + 273.15
+ else
+ degrees
+ end
+end
+
+def to_celsius(degrees, current_unit)
+ if current_unit == 'F'
+ ((degrees - 32) * 5) / 9.0
А тук да разделиш на 1.8
+ elsif current_unit == 'K'
+ degrees - 273.15
+ else
+ degrees
+ end
+end
+
+def convert_between_temperature_units(degrees, current_unit, wanted_unit)
+ from_celsius(to_celsius(degrees, current_unit), wanted_unit)
Хитър подход, конвертирайки на 2 стъпки си спестяваш няколко проверки и формули. Супер идея :)
+end
+
+def melting_point_of_substance(substance, units)
+ substances = {'water' => 0, 'ethanol' => -114, 'gold' => 1064, 'silver' => 961.8, 'copper' => 1085}
Името на този hash би могло по-добре да описва какво съдържа. В него има точки на топене на различни елементи, следователно нещо от рода на melting_points
е по-описателно. Съответно и вземането на температурата по-долу би изглеждало като melting_points[substance]
, което веднага ти казва какво има в резултата.
Също, това е дреболия, но ще е малко по-прегледно да го дефинираш като константа в началото на файла, вместо да е вътре в метода. :)
+ from_celsius(substances[substance], units)
+end
+
+def boiling_point_of_substance(substance, units)
+ substances = {'water' => 100, 'ethanol' => 78.37, 'gold' => 2700, 'silver' => 2162, 'copper' => 2567}
И тук същото като за melting_points.
+ from_celsius(substances[substance], units)
+end
Хитър подход, конвертирайки на 2 стъпки си спестяваш няколко проверки и формули. Супер идея :)