Решение на Първа задача от Даяна Маринова

Обратно към всички решения

Към профила на Даяна Маринова

Резултати

  • 6 точки от тестове
  • 0 бонус точки
  • 6 точки общо
  • 8 успешни тест(а)
  • 0 неуспешни тест(а)

Код

EXCHANGE_RATE = {usd: 1.7408, eur: 1.9557, gbp: 2.6415, bgn: 1}
def convert_to_bgn(price, currency)
(price * EXCHANGE_RATE[currency]).round(2)
end
def compare_prices(price_first, currency_first, price_second, currency_second)
calculate_first = price_first * EXCHANGE_RATE[currency_first]
calculate_second = price_second * EXCHANGE_RATE[currency_second]
calculate_first <=> calculate_second
end

Лог от изпълнението

........

Finished in 0.00678 seconds
8 examples, 0 failures

История (2 версии и 2 коментара)

Даяна обнови решението на 11.10.2015 21:57 (преди над 8 години)

+CURRENCY = { usd: 1.7408, eur: 1.9557, gbp: 2.6415, bgn: 1}
+
+def convert_to_bgn price, currency
+ (price * CURRENCY[currency]).round(2)
+end
+
+def compare_prices price_first, currency_first, price_sec, currency_sec
+ price_first * CURRENCY[currency_first] <=> price_sec * CURRENCY[currency_sec]
+end

Като цяло много добре, само няколко дребни забележки:

  • Не слагай space в началото на hash-oве
  • Слагай скоби около аргументите на метод когато дефинираш метод с аргументи
  • Не съкращавай така price_sec
  • Думата CURRENCY не дава усещане за някакъв вид mapping. В случая има точен термин обменен курс

Btw твоето решение е първото, което отговаря изцяло на спецификацията на задачата :)

Даяна обнови решението на 12.10.2015 09:46 (преди над 8 години)

-CURRENCY = { usd: 1.7408, eur: 1.9557, gbp: 2.6415, bgn: 1}
+EXCHANGE_RATE = {usd: 1.7408, eur: 1.9557, gbp: 2.6415, bgn: 1}
-def convert_to_bgn price, currency
- (price * CURRENCY[currency]).round(2)
+def convert_to_bgn(price, currency)
+ (price * EXCHANGE_RATE[currency]).round(2)
end
-def compare_prices price_first, currency_first, price_sec, currency_sec
- price_first * CURRENCY[currency_first] <=> price_sec * CURRENCY[currency_sec]
+def compare_prices(price_first, currency_first, price_second, currency_second)
+ calculate_first = price_first * EXCHANGE_RATE[currency_first]
+ calculate_second = price_second * EXCHANGE_RATE[currency_second]
+ calculate_first <=> calculate_second
end