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

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

+150
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
```
$up = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".$x."_".($y-1)."_'"));
$down = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".$x."_".($y+1)."_'"));
$right = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".($x+1)."_".($y)."_'"));
$left = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".($x-1)."_".($y)."_'"));
$lup = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".($x-1)."_".($y-1)."_'"));
$rup = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".($x+1)."_".($y-1)."_'"));
$rdown = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".($x+1)."_".($y+1)."_'"));
$ldown = mysql_fetch_array (mysql_query("SELECT name FROM locations WHERE id='p_".($x-1)."_".($y+1)."_'"));
if (@$up["name"]) $up = 1; else $up=0;
if (@$down["name"]) $down = 1;  else $down=0;
if (@$right["name"]) $right = 1;  else $right=0;
if (@$left["name"]) $left = 1; else $left=0;
if (@$lup["name"]) $lup = 1; else $lup=0;
if (@$rup["name"]) $rup = 1; else $rup=0;
if (@$rdown["name"]) $rdown = 1;  else $rdown=0;
if (@$ldown["name"]) $ldown = 1;  else $ldown=0;
if ($_GET["go_nature"]=='up' and $up==1) $y-=1;
if ($_GET["go_nature"]=='down' and $down==1) $y+=1;
if ($_GET["go_nature"]=='left' and $left==1) $x-=1;
if ($_GET["go_nature"]=='right' and $right==1) $x+=1;
if ($_GET["go_nature"]=='lup' and $lup==1) $x-=1;
if ($_GET["go_nature"]=='rup' and $rup==1) $x+=1;
if ($_GET["go_nature"]=='lup' and $lup==1) $y-=1;
if ($_GET["go_nature"]=='rup' and $rup==1) $y-=1;
if ($_GET["go_nature"]=='rdown' and $rdown==1) $x+=1;
if ($_GET["go_nature"]=='ldown' and $ldown==1) $x-=1;
if ($_GET["go_nature"]=='rdown' and $rdown==1) $y+=1;
if ($_GET["go_nature"]=='ldown' and $ldown==1) $y+=1;
```
вксняшка

Запостил: SuperChel, 16 Января 2014

Tweet
Комментарии (35) RSS
- Lowezar 16.01.2014 13:28 # +1
  
  >вксняшка
  У вас вкус испортился.
  Ответить
- tirinox 16.01.2014 14:45 # +5
  
  Это что? Жизнь на PHP + MySQL?
  Ответить
  - Vasiliy 16.01.2014 14:47 # +1
    
    тонко
    Ответить
- SuperChel 16.01.2014 15:08 # −2
  
  мне интересно за что же минусы? Господа минусующие считают это хорошим кодом?
  Ответить
  - roman-kashitsyn 16.01.2014 15:12 # +5
    
    Видимо, просто уныло. Типичный школокод, такого без труда можно мегабайты накопать.
    Ответить
    - SuperChel 16.01.2014 15:15 # 0
      
      тогда понятно, согласен
      Ответить
  - Vasiliy 16.01.2014 15:12 # +3
    
    просто уныло.
    Ответить
  - anonimb84a2f6fd141 17.01.2014 20:09 # 0
    
    Однояйцевый близнец лабы.
    Ответить
- Lure Of Chaos 17.01.2014 03:12 # 0
  
  а собачки зачем?
  Ответить
  - Anderson 17.01.2014 03:42 # 0
    
    Если вдруг говноквери возвратят NULL
    Ответить
- krypt 17.01.2014 11:58 # 0
  Ммм.... Прелесть.
  Это как current = ((current+shift+size) % size) на 20 строк расписывать, когда shift равен только 1 или -1
  А последний блок условий так вообще доставляет.
  
  Для минусующих:
  
  for (int y = max(0, y-1); y < min(y+1, height); y++) for (int x = max(0, x-1); x < min(x+1, width); x++) if (x != 0 || y != 0) { <обрабатываем направление> }
  Ответить
  - com1 17.01.2014 12:55 # +2
    
    выложите как отдельный пост, плюсану адназначна! )))
    Ответить
    - krypt 17.01.2014 13:53 # 0
      
      Обоснуйте, почему приведённый в моём комментарии код - гк?
      Ответить
      
      defecate-plusplus 17.01.2014 13:56 # +3
      
      функции в условии цикла, выполняющиеся на каждой итерации, когда нет на это причин
      Ответить
      
      krypt 17.01.2014 14:07 # 0
      
      Чаще всего min и max это не функции, а таки дефайны. И, скорее всего, min и max будут вынесены из цикла компилятором при оптимизации.
      А вот преждевременная оптимизация - зло. +4 строки к коду.
      
      А вообще да, я хуйню написал в инициализации и условиях )
      Там должны быть координаты текущей точки.
      Ответить
      
      roman-kashitsyn 17.01.2014 14:09 # +3
      
      > +4 строки к коду
      tell us moar
      Ответить
      
      defecate-plusplus 17.01.2014 14:15 # +4
      
      +0 строк к коду
      
      for (int i = foo(), imax = bar(); i < imax; ++i)
      Ответить
      
      krypt 17.01.2014 14:36 # 0
      
      Плять. Увольняться и идти на завод на конвейер ><
      Ответить
      
      TarasB 17.01.2014 15:04 # +2
      
      У меня этот код выглядел бы так:
      
      Field.Window(x-1,y-1,3,3).Fill(MyCoolFunctor());
      Ответить
      
      bormand 17.01.2014 15:09 # +2
      
      > Fill(MyCoolFunctor())
      Уже в функциональщину подался? :)
      Ответить
      
      roman-kashitsyn 17.01.2014 15:15 # +2
      
      вслед за кармаком
      Ответить
      
      tirinox 17.01.2014 17:38 # 0
      
      Правда?
      Ответить
      
      roman-kashitsyn 17.01.2014 17:42 # +1
      
      http://functionaltalks.org/2013/08/26/john-carmack-thoughts-on-haskell/
      
      http://www.altdevblogaday.com/2012/04/26/functional-programming-in-c/
      Ответить
      
      TarasB 17.01.2014 16:11 # 0
      
      Нет, это просто конструктор хуйни с перегруженнм оператором ()
      Ответить
      
      bormand 17.01.2014 16:33 # +1
      
      На основе этой, кхм, хуйни со скобками и строится вся крестофункциональщина.
      Ответить
      
      defecate-plusplus 17.01.2014 16:44 # 0
      
      был бы оператор пробел, можно было бы даже существующими в с++03 средствами уже делать
      auto a = huskyll f g x;
      что означало бы f(g(x));
      Ответить
      
      TarasB 17.01.2014 17:05 # +1
      
      А если бы был оператор "отсутствующий пробел", то можно было бы вместо x*y писать xy, как и положено в математике
      Ответить
      
      bormand 17.01.2014 17:07 # 0
      
      Тебе в maple.
      Ответить
      
      defecate-plusplus 17.01.2014 17:12 # +2
      
      не перебарщивай
      в математике потому то и используются и маленькие, и большие, и греческие, и курсивные, и верхние-нижние индексы, и штрихи-крышечки-стрелочки-подчеркивания, потому что доска всё стерпит
      попробывали бы они это сделать в мичети крестах!
      Ответить
      
      Abbath 19.01.2014 23:28 # 0
      
      А когда буквы кончатся?
      Ответить
      
      krypt 20.01.2014 10:18 # +3
      
      А когда вышеперечисленные буквы кончаются, используются готические (/me содрогнулся от ужаса)
      Ответить
      
      tirinox 17.01.2014 17:40 # 0
      
      MyCoolSphincter()
      Ответить
      
      com1 18.01.2014 10:55 # 0
      
      дополню: старт цикла с "0", когда "0" не обрабатывается. итерация целочисленная, можно смело стартовать с "1" и избавиться от лишнего условия.
      Ответить
    - krypt 17.01.2014 14:08 # 0
      
      Гм, вижу, вы таки правы :) Сферический пример в вакууме оказался слишком сферичен.
      Ответить
- anonimb84a2f6fd141 17.01.2014 20:09 # +2
  
  Уже не лаба, но еще не прога.
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

Гости могут высказаться только в понедельник, среду, четверг или воскресение
Семь раз отмерь — один отрежь, guest!

А не использовать ли нам bbcode?

[b]жирный[/b] — жирный

[i]курсив[/i] — курсив

[u]подчеркнутый[/u] — подчеркнутый

[s]перечеркнутый[/s] — перечеркнутый

[blink]мигающий[/blink] — мигающий

[color=red]цвет[/color] — цвет (подробнее)

[size=20]размер[/size] — размер (подробнее)

[code=<language>]some code[/code] (подробнее)

Проверочный код: *

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

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

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

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