1. Java / Говнокод #586

    +139.8

    1. 01
    2. 02
    3. 03
    4. 04
    5. 05
    6. 06
    7. 07
    8. 08
    9. 09
    10. 10
    11. 11
    12. 12
    13. 13
    14. 14
    15. 15
    16. 16
    17. 17
    18. 18
    19. 19
    20. 20
    21. 21
    22. 22
    23. 23
    24. 24
    25. 25
    26. 26
    27. 27
    28. 28
    29. 29
    30. 30
    public enum TimePeriod {
    	
    	NONE,
    	DAILY,
    	WEEKLY,
    	MONTHLY,
    	YEARLY;
    	
    	public static TimePeriod getTimePeriod(String periodStr) {
    		if (null == periodStr) {
    			return null;
    		}
    		if (0 == periodStr.length()) {
    			return NONE;
    		}
    		if (0 == periodStr.compareToIgnoreCase("daily")) {
    			return DAILY;
    		}
    		if (0 == periodStr.compareToIgnoreCase("weekly")) {
    			return WEEKLY;
    		}
    		if (0 == periodStr.compareToIgnoreCase("monthly")) {
    			return MONTHLY;
    		}
    		if (0 == periodStr.compareToIgnoreCase("yearly")) {
    			return YEARLY;
    		}
    		return null;
    	}
    }

    а правильно будет так
    public static TimePeriod parse(String type) {
    return valueOf(type.toUpperCase());
    }

    Запостил: guest, 20 Февраля 2009

    Комментарии (5) RSS

    • а если type == null, что произойдет?
      Ответить
    • guest:
      а еще добавь, что нужно ограничить значения до указанных в перечислении.
      Ответить
    • а еще (не всегда, но во многих случаях) лучше передавать эту строку в конструкторе TimePeriod. Тогда впоследствии ее можно будет легко поменять в одном месте. Причем по-умолчанию можно передавать null и проверять, что если null, то используем name(). Это будет еще более гибко и удобно.
      Ответить
    • http://sale-flowers.org/ - цветы в питере с доставкой

      Праздники и дни рождения, которые мы отмечаем, чаще всего не обходятся без цветов. Они оставляют впечатляющие воспоминания о каждой дате. У каждого человека имеется цветок, ему он отдаёт предпочтение из общего разнообразия. У нас в большом цветочном ассортименте вы найдете цветы на любой вкус.

      http://sale-flowers.org/ - цветы с доставкой в офис доставка букетов

      Если вы не уверены в цветочных предпочтениях того, кому предназначаются цветы, можете остановиться на красивых букетах. Наши букеты сделаны опытными флористами. Букет из алых роз, нежных орхидей, прекрасных хризантем и других, удивляющих особой красотой цветов, будет хорошим подарком, как даме, так и джентльмену. Если вы хотите доставить радость девушке, то купите к букетук примеру мягкую игрушку. Данный сюрприз станет по душе каждой представительнице женского пола.

      http://sale-flowers.org/ - заказать розы спб

      Розы считаются самыми популярными представителями флоры. Даря эти цветы, вы конечно же угодите каждому человеку. Эти красивые цветы имеют уникальный аромат, который сможет радовать длительное время. У нас на складе в наличии большой выбор сортов роз разнообразной длины и цветовой гаммы.

      http://sale-flowers.org/ - купить розы спб доставка

      На вопросы касающиеся подбора букета или создания его по индивидуальному заказу могут ответить наши специалисты.

      КУПОН СКИДКИ: FORUM
      Ответить

    Добавить комментарий