1. PHP / Говнокод #625

    +152

    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
    31. 31
    public function step($step = 1)
	{
	
		switch ( $step )
		{
			case 1 :
				return $this->step1();
				break;
			/*
	         case 2 :
	           return $this->step2();
	         break;

	         case 3 :
	           return $this->step3();
	         break;

	         case 4 :
	           return $this->step4();
	         break;

	         case 11 :
	         	return $this->step11();
	         break;
                 */
			default :
				return $this->step1();
				break;
		
		}
	}

    ^%)

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

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

    • ava guest 27.02.2009 14:29 # 0

      а как по другому? =\
      Ответить

    • ava guest 27.02.2009 16:30 # 0

      страйкер:
      switch($step) {
    case 1:
    default:
        return $this->step1();
        break;
}
      Ответить

    • ava guest 27.02.2009 17:06 # 0

      Иван:
      Если учесть, что закомментаренный код не будет использоваться, то так
      public function step(){
	return $this->step1();
}
      Ответить

    • ava guest 27.02.2009 20:36 # 0

      страйкер, и каждый раз переписывать код? и так замечательно работает. а если каменты убрать, то работать небудет
      Ответить

    • ava guest 28.02.2009 19:14 # 0

      Keeper:
      Ужасная и унылая попытка написать конечный автомат. Автору:
      1. Заменить магические числа на осмысленные именованные константы.
      2. Дать осмысленные названия функциям, реализующим отдельные шаги.
      3. Написать нормальный диспетчер функций. Хотя бы использующий массив ссылок на эти функции.
      Ответить

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