Определение знака зодиака по дате рождения

Вопщем то я написал прогу, но препод говорит что она не эффективная...
Снизу прикрепил то, что я своял
Можете подсказать как сделать?

Требуется определить знак зодиака в зависимости от даты рождения. Пользователь вводит месяц и день своего рождения, а в ответ получает знак зодиака (либо сообщение, что введённой пользователем даты не существует). Алгоритм и программа должны корректно работать для всех дат от 1901 до 2099 года включительно (в случае, если введённый пользователем год меньше 1900 или больше 2100, программа должна сообщать что не может сосчитать дату).

Примечания:
Не забывайте учитывать, что если год високосный, то в феврале 29 дней, а если обычный, то 28.

Знаки зодиака:
Овен [21.03 - 20.04]
Телец [21.04 - 20.05]
Близнецы [21.05 - 21.06]
Рак [22.06 - 22.07]
Лев [23.07 - 23.08]
Дева [24.08 - 23.09]
Весы [24.09 - 23.10]
Скорпион [24.10 - 22.11]
Стрелец [23.11 - 21.12]
Козерог [22.12 - 20.01]
Водолей [21.01 - 20.02]
Рыбы [21.02 - 20.03]

Вложения

zodiac.rar (977 байт, 1311 просмотров)

Хм, забавно, но по-моему вполне может быть из-за того, что вы неэффективно используете память: объявили переменную 'a' типа string (256 байт памяти!!!) и ни разу не использовали ее в программе.
ЗЫ Больно придирчивый препод
ЗЗЫ А вообще конечно надо постараться избавиться от громоздкой конструкции оператора выбора case of. Как вариант: создать файл, в который записать все периоды знаков зодиаков и их названия, и уже после введение даты с клавиатуры, считывать значения периодов и названий из файла и проверять в какой из периодов входит введенная дата, возможно так преподу больше понравится...

0

а чем он аргументировал неэффективность? в принципе я бы определял сразу какой год - высокосный или нет, и какую-нить булевую переменную завел, проверок было бы поменьше

0

мне сказали что там както это все через два case делается

ЗЫ эт мне надо к завтрашнему дню... может кто помочь??? а то не поставят зачет

Еще из проверки на високосность надо убрать проверку на 100 и 400. С 1901 по 2099 год таких нет. По этой проверке 1900 год не високосный, так его нет, а 2000 год високосный.

0

Может быть стоит сразу считывать дату отключив проверку ввода-вывода ({-I}), а потом делать только один раз проверку всей даты (и года, и месяца, и числа) и спокойно писать case, безо всяких там еще
а то конструкция одна и таже много раз повторяется.

0