Решение на Астрологични забави от Красимир Атанасов

Резултати

10 точки от тестове
0 бонус точки
10 точки общо

6 успешни тест(а)
0 неуспешни тест(а)

Код

def interpret_chinese_sign(year):

    year_signs = {

        1900: "rat",

        1901: "ox",

        1902: "tiger",

        1903: "rabbit",

        1904: "dragon",

        1905: "snake",

        1906: "horse",

        1907: "sheep",

        1908: "monkey",

        1909: "rooster",

        1910: "dog",

        1911: "pig"

    while not(year in range(1900, 1912)):

        if year < 1900:

            year += 12

        else:

            year -= 12

    return year_signs[year]

def interpret_western_sign(day, month):

    month_tuples = [

        (20, "capricorn", "aquarius"),

        (19, "aquarius", "pisces"),

        (20, "pisces", "aries"),

        (20, "aries", "taurus"),

        (20, "taurus", "gemini"),

        (20, "gemini", "cancer"),

        (22, "cancer", "leo"),

        (22, "leo", "virgo"),

        (22, "virgo", "libra"),

        (22, "libra", "scorpio"),

        (21, "scorpio", "sagittarius"),

        (21, "sagittarius", "capricorn")

    border = 0

    first_sign = 1

    second_sign = 2

    border_signs = month_tuples[month - 1]

    if day <= border_signs[border]:

        return border_signs[first_sign]

    else:

        return border_signs[second_sign]

def interpret_both_signs(day, month, year):

    western_sign = interpret_western_sign(day, month)

    chinese_sign = interpret_chinese_sign(year)

    return western_sign, chinese_sign

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

▸ Покажи лога

......
----------------------------------------------------------------------
Ran 6 tests in 0.006s

OK

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

Красимир обнови решението на 11.03.2015 11:33 (преди около 10 години)

▸ Покажи разликите

С това решение лесно можеш да изпуснеш някой краен случай. Не ти ли се струва излишно да правиш отделен запис за всеки ден от годината?

Публикувано преди около 10 години

Струва ми се по - добре от 12 if-a, или искаш да кажеш че има и по елегантно репение?

Публикувано преди около 10 години

Малко по-добре е от 12 if-а, има по-добро решение. В момента големината на списъка ти зависи от броя на дните в година, а не от броя на зодиите (12 елемента е далеч по-добре от 365, ако може да се постигне :))

Публикувано преди около 10 години

Красимир обнови решението на 11.03.2015 15:25 (преди около 10 години)

▸ Покажи разликите

 def interpret_chinese_sign(year):

     year_signs = {

         1900: "rat",

         1901: "ox",

         1902: "tiger",

         1903: "rabbit",

         1904: "dragon",

         1905: "snake",

         1906: "horse",

         1907: "sheep",

         1908: "monkey",

         1909: "rooster",

         1910: "dog",

         1911: "pig"

     while not(year in range(1900, 1912)):

         if year < 1900:

             year += 12

         else:

             year -= 12

     return year_signs[year]

 def interpret_western_sign(day, month):

-    months = [

-        ["capricorn"] * 20 + ["aquarius"] * 11,  # January

-        ["aquarius"] * 19 + ["pisces"] * 8,  # February

-        ["pisces"] * 20 + ["aries"] * 11,  # March

-        ["aries"] * 20 + ["taurus"] * 10,  # April

-        ["taurus"] * 20 + ["gemini"] * 11,  # May

-        ["gemini"] * 20 + ["cancer"] * 9,  # June

-        ["cancer"] * 22 + ["leo"] * 9,  # July

-        ["leo"] * 22 + ["virgo"] * 9,  # August

-        ["virgo"] * 22 + ["libra"] * 8,  # September

-        ["libra"] * 22 + ["scorpio"] * 9,  # October

-        ["scorpio"] * 21 + ["sagittarius"] * 9,  # November

-        ["sagittarius"] * 21 + ["capricorn"] * 9  # December

+    month_tuples = [

+        (20, "capricorn", "aquarius"),

+        (19, "aquarius", "pisces"),

+        (20, "pisces", "aries"),

+        (20, "aries", "taurus"),

+        (20, "taurus", "gemini"),

+        (20, "gemini", "cancer"),

+        (22, "cancer", "leo"),

+        (22, "leo", "virgo"),

+        (22, "virgo", "libra"),

+        (22, "libra", "scorpio"),

+        (21, "scorpio", "sagittarius"),

+        (21, "sagittarius", "capricorn")

-    return months[month - 1][day - 1]

+    border = 0

+    first_sign = 1

+    second_sign = 2

+    border_signs = month_tuples[month - 1]

+    if day <= border_signs[border]:

+        return border_signs[first_sign]

+    else:

+        return border_signs[second_sign]

 def interpret_both_signs(day, month, year):

     western_sign = interpret_western_sign(day, month)

     chinese_sign = interpret_chinese_sign(year)

     tup = (western_sign, chinese_sign)

     return tup