Говнокод #12290 — PHP — Говнокод.ру

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

PHP / Говнокод #12290

+57

// Просто удаляет элемент из массива
  function delElement($arr, $index)
  {
    $result = Array();
    for($i=0;$i<count($arr);$i++)
      if($index != $i) $result[] = $arr[$i];
    
    return $result;
  }

- Ваше мнение, коллега?
- Кал молодой особи пыхаписта, коллега. Очень ярко выражены типичные признаки.

Запостил:

clauclauclau, 14 Декабря 2012

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

bormand 14.12.2012 14:44 # +15
```
// Просто удаляет элемент из массива
function delElement($arr, $index) {
    foreach ($arr as $key => $value) {
        $db->exec("insert into tmp(key, value) values ('$key', '$value')");
    }
    $db->exec("delete from tmp where key='$index'");
    $result = Array();
    foreach ($db->query("select * from tmp") as $row) {
        $result[$row['key']] = $row['value'];
    }
    $db->exec("delete from tmp");
    return $result;
}
```
Ответить
- telnet 14.12.2012 15:04 # 0
  
  Notice: Undefined variable $db...
  Fatal Error: Call to a member function exec() on non-object...
  
  В остальном же зачотно =)
  Ответить
  - torbasow 14.12.2012 15:15 # +1
    
    Очевидно, это глобальная переменная.
    Ответить
    - wvxvw 14.12.2012 15:17 # 0
      
      Даже я знаю, что глобальные надо специально объявлять, типа как в том же Питоне.
      Ответить
      - torbasow 14.12.2012 15:29 # 0
        
        А, ну да. Я настолько редко их встречаю, что всё время об этом забываю.
        Ответить
  - bormand 14.12.2012 18:15 # 0
    
    Да, верно, забыл я этот global.
    
    P.S. Надо было вместо PDO поюзать mysql_*, ведь именно их юзает человек, впервые пришедший на данную платформу.
    Ответить
  - guest 14.12.2012 18:27 # +8
    
    ударим новыми подходами по возросшей вычислительной мощности компьютеров!
    Ответить
    - anonimb84a2f6fd141 14.12.2012 19:07 # +1
      
      Компьютер, так сказать, не роскошь, а средство...
      Ответить
- wvxvw 14.12.2012 15:20 # −4
  
  Кстати, вы будете смеятся, но... а что если в массиве элементов, больше чем можно запихать в память доступную рантайму?
  Ответить
  - psycho-coder 14.12.2012 17:35 # +2
    
    Кстати, вы будете смеяться, но... это невозможно
    Ответить
    - wvxvw 14.12.2012 17:42 # −1
      
      Кстати, вы будете смеятся, но это очень насущная проблема в системах связанных с ИИ. Т.е. построение и доступ к онтологиям которые не помещаются в память доступную рантайму. Но это немного через чур для ПХП, хотя, теоретически, можно везде, конечно.
      Ответить
      - psycho-coder 14.12.2012 17:53 # +1
        
        Пыха загнется от своих-же утечек, через пару часов работы в режиме ИИ.
        имхо, это для пыхе выше ее сил.
        Ответить
        
        wvxvw 14.12.2012 18:08 # 0
        
        У ПХП есть режим ИИ заменяющий пилота у штурвала на время самых ответственных задач? А было бы круто... например, Вася ПХПист написал die(mysql_error()); и тут штурвал перехватывает ИИ, удаляет васин код и пишет ошибку в лог.
        Ответить
        
        psycho-coder 14.12.2012 18:16 # 0
        
        Вы начали с ИИ, связав проблему и с ПХП, мол для него это слишком.
        Я вам ответил, что пыха загнется от таких извращениях в области ИИ.
        Вы опять приделываете ИИ к пыхе.
        
        А вопрос изначальный был в том, а что если в массиве элементов, больше чем можно запихать в память доступную рантайму?.
        Ответить
        
        bormand 14.12.2012 18:23 # +2
        
        > если в массиве элементов, больше чем можно запихать в память доступную рантайму
        то
        1) Надо поставить 64 битный рантайм, в котором ее скорее всего хватит
        2) Вынести массив в какую-нибудь базу, которая может
        3) Взять другой язык, у пыхи вполне определенная ниша, и ИИ туда не входит
        Ответить
        
        psycho-coder 14.12.2012 18:33 # 0
        
        Это не мне отвечать надо. Я хоть и не спец, но понимаю что пыха для этого не подходит.
        Ответить
        
        bormand 14.12.2012 18:50 # +1
        
        Сорри, промазал постом. Это было для wvxvw.
        Ответить
        
        wvxvw 14.12.2012 18:41 # 0
        
        Не, не хватит, даже для онтологий связаных с грамматикой (посторением фраз и предложений), физически на обычный ПК точно не влезет.
        
        Только смысл комментария был не в этом, но чет все с такой серьезностью бросились защищать ПХП от ИИ, что мне прям грустно стало.
        Ответить
        
        psycho-coder 14.12.2012 19:10 # +2
        
        ИИ не существует :-)
        Ответить
        
        wvxvw 14.12.2012 19:13 # +1
        
        Именно поэтому первые партии батареек для Скайнет будут сделаны из отборных ПХПшников.
        Ответить
        
        guest6 30.12.2023 01:42 # 0
        
        Какой штурвал?
        Ответить
  - bormand 14.12.2012 18:17 # +1
    
    Я не буду смеяться. Просто задачи в которых элементов больше, чем можно запихать в память доступную рантайму, они немного не для пыхи. У каждого языка есть своя ниша...
    Ответить
    - 3.14159265 14.12.2012 18:44 # 0
      
      I got your name
      I got your ass
      You will not laugh
      You will not lie
      You will learn by the number
      I will teach you
      Ответить
  - d3n4 15.12.2012 16:40 # 0
    
    Да мы смеемся, спасибо повеселил.
    Ответить
- anonimb84a2f6fd141 14.12.2012 18:01 # +1
  
  Сдаётся, что ОП-код не меньшая синтетика, чем ваш, @bormand
  Ответить
  - eth0 14.12.2012 19:56 # +1
    
    Более того, такое не грех и написать ради лулзов. Есть особо исключительные случаи, когда люди не могут в примитивные алгоритмы и не ищут библиотечные функции, но тут это выглядит слишком вычурно.
    Ответить

Говнокод: по колено в коде.

PHP / Говнокод #12290

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

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