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

    +161

    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
    // getter для поля в конфиге. Очень костылёвый гавноалгоритм. Потом обязательно переделать!
        public function read($section, $key){
           if($section == true or $this->prefix == true){
             if(isset($this->arr[$this->prefix.$section][$key])){
                return $this->arr[$this->prefix.$section][$key];
             } else {
                return false;
             }
           } else {
             if(isset($this->arr[$this->prefix.$section][$key])){
                return $this->arr[$this->prefix.$section][$key];
            } else {
                return false;
            }
           }
        }

    Все правильно сделал.

    Запостил: djumpen, 15 Сентября 2011

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

    • $this->arr['truetrue']?? мда
      Ответить
      • 11 тоже подойдёт. Или 10 или 01 или А0, равенство не строгое.
        Ответить
        • я вас удивлю

          var_dump('false'==true); //boolean true
          Ответить
          • var_dump('false' === true)
            я о строгом равенстве речь веду
            Ответить
    • Тваюмать. Извиняюсь за корявоглазость. Конечно же:

      // getter для поля в конфиге. Очень костылёвый гавноалгоритм. Потом обязательно переделать!
      public function read($section, $key){
      if($section == true or $this->prefix == true){
      if(isset($this->arr[$this->prefix.$section][$key])){
      return $this->arr[$this->prefix.$section][$key];
      } else {
      return false;
      }
      } else {
      if(isset($this->arr[$key])){
      return $this->arr[$key];

      } else {
      return false;
      }
      }
      }
      Ответить
      • arr оч хорошее название
        Ответить
        • array, че))
          Ответить
          • то, что это массив, понятно по способу обращения с ним. а для чего он, хз )
            Ответить
            • Это же индусская нотация в пику венгерской.
              Ответить
            • Если интересно и вкратце, то это массив, полученный через parse_ini_file.
              Было желание сделать ее универсальной для конфигов с секциями и без них. Тоесть когда секцию не указываем - работаем с одномерным массивом. Если указали секцию - с двумерным.
              Самое интересное, этот код скорее всего пойдет в продакшн и будет поставляться как конфигурялка недешевых железок.
              Ответить
            • Он временный. Только явисты могут назвать его temporaryArrayForBigImportantVariable
              Ответить
              • явисты могут... а джависты назвали бы data или как-то там
                Ответить
              • Вы так говорите, как будто это что-то плохое.
                Ответить
              • Нет ничего более постоянного, чем временное.
                Ответить
    • показать все, что скрытоvanished
      Ответить

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