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

    +78

    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
    private function isRussianCar($car){

                           # Определялка российских машин.

                            if ($car['make_id'] == '107') return true;
                            if ($car['make_id'] == '108') return true;
		if ($car['make_id'] == '109') return true;
		if ($car['make_id'] == '110') return true;
		if ($car['make_id'] == '111') return true;
		if ($car['make_id'] == '112') return true;
		if ($car['make_id'] == '113') return true;
		if ($car['make_id'] == '114') return true;
		if ($car['make_id'] == '115') return true;
		if ($car['make_id'] == '116') return true;
		if ($car['make_id'] == '117') return true;
		return false;
	
    }

    Нашёл в одном из проектов

    Запостил: daemon_master, 15 Мая 2012

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

    • ava guest 15.05.2012 22:01 # 0

      По пацански, че.
      Ответить

    • ava Lure Of Chaos 15.05.2012 23:51 # +3

      кто-то в школе сравнения ниасилил
      Ответить

      • ava roman-kashitsyn 15.05.2012 23:55 # +3

        Какого чёрта айдишники из базы вообще в коде делают, вот в чём вопрос
        Ответить

        • ava Lure Of Chaos 16.05.2012 00:10 # 0

          может они и не из базы, это осталось за пределами сабжа.
          Ответить

      • ava inkanus-gray 16.05.2012 11:59 # 0

        А вдруг там бывают дробные или нечисловые значения?
        Ответить

    • ava pavel 15.05.2012 23:52 # +3 

      in_array($car['make_id'],range(107,117));//мой ленивый кот написал бы так
      Ответить

      • ava Lure Of Chaos 16.05.2012 00:09 # 0

        а пачиму не $car['make_id']>106 && $car['make_id']<118 ?
        2 сравнения против ваших 10
        Ответить

        • ava Lure Of Chaos 16.05.2012 00:09 # +1

          т.е. вашего кота.
          Ответить

          • ava pavel 16.05.2012 00:54 # −1

            вероятно чтобы при добавлении еще нескольких марок не делать лепить плохочитаемую логику, а просто написать array_merge(range(107,117),array(125, 140, 141, 159));
            Ответить

            • ava 7ion 16.05.2012 12:03 # 0

              Это уже говнокод какой-то.
              Простое сравнение - норм.
              Ответить

    • ava psycho-coder 16.05.2012 12:44 # −2

      я бы через switch сделал.
      Ответить

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