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

    • По пацански, че.
      Ответить
    • кто-то в школе сравнения ниасилил
      Ответить
      • Какого чёрта айдишники из базы вообще в коде делают, вот в чём вопрос
        Ответить
        • может они и не из базы, это осталось за пределами сабжа.
          Ответить
      • А вдруг там бывают дробные или нечисловые значения?
        Ответить
    • in_array($car['make_id'],range(107,117));//мой ленивый кот написал бы так
      Ответить
      • а пачиму не $car['make_id']>106 && $car['make_id']<118 ?
        2 сравнения против ваших 10
        Ответить
        • т.е. вашего кота.
          Ответить
          • вероятно чтобы при добавлении еще нескольких марок не делать лепить плохочитаемую логику, а просто написать array_merge(range(107,117),array(125, 140, 141, 159));
            Ответить
            • Это уже говнокод какой-то.
              Простое сравнение - норм.
              Ответить
    • я бы через switch сделал.
      Ответить

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