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

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

+164
1. 1
2. 2
3. 3
4. 4
5. 5
6. 6
```
if($_SESSION['use_test_xml'] && false) {
		if(!responseTestXml('chekin_'.$_GET['userid'].'_'.( ($_GET['placeid']) ? $_GET['placeid'] : $_GET['eventid'] ).'.xml')) {
			responseTestXml('err.xml',array('message'=>'Unxpected situation. Please use these users: 12345,54321 and places|events: 1-9.'));
		}
	} else {
...
```
плеать...

Запостил: realsugar, 24 Августа 2011

Tweet
Комментарии (27) RSS
- roman-kashitsyn 24.08.2011 11:43 # 0
  $_SESSION['use_test_xml'] && false
  доставило
  Ответить
  - realsugar 24.08.2011 11:47 # 0
    
    а уж мне как доставило((( я не говорю про сто строчек ветки if внизу...
    Ответить
  - ling 24.08.2011 15:40 # 0
    
    Это равнозначно комменту "не трогай/не работает".
    Ответить
- Lure Of Chaos 24.08.2011 13:34 # +1
  
  настоящие пиххиписты юниттестов не пишут, они пишут вот такие хуки прямо в коде!
  Ответить
  - SmackMyBitchUp 24.08.2011 18:21 # +1
    
    Эт не хук. Эт хуяк какой-то
    Ответить
  - Vasiliy 25.08.2011 08:05 # 0
    
    Чаще всего. Времени на написания юнит тестов просто нет, вся отладка это в лушем случае syslog на чаще echo / die
    Ответить
- jokz 24.08.2011 20:34 # −2
  
  ну и я довольно часто так пишу. только if( false && условие), чтобы если в условие функция, чтобы она не исполнялась.
  
  а в чем тут говнокод? говнокод на второй строчке. а это издержки отладки.
  Ответить
  - Lure Of Chaos 24.08.2011 21:43 # 0
    
    > ну и я довольно часто так пишу. только if( false && условие), чтобы если в условие функция, чтобы она не исполнялась.
    ну вот я и говорю, что это гребаные хуки
    Ответить
    - jokz 24.08.2011 23:06 # 0
      
      >Эт не хук. Эт хуяк какой-то
      
      я вот на это. помоему вполне нормально. одно слово в коде и никакого гемора ни с чем. помоему так гораздо проще.
      Ответить
      
      Lure Of Chaos 24.08.2011 23:09 # +2
      
      я про то, что тестовая функциональность не должна встраиваться внутрь рабочего кода. Надо делать отдельные тесты. И вообще, BDD наше всё
      Ответить
      
      jokz 24.08.2011 23:16 # 0
      
      не, ну естественно, что это не про продакшн речь идет. на стадии разработки я считаю мозга ебать лишь заглушить одно условие это чересчур. и, да, конечно речь не идет о крупногабаритных проектах, где такие хуки могут просто напросто затеряться и не о рабочих проектах естесна.
      Ответить
      
      Lure Of Chaos 24.08.2011 23:20 # +2
      
      нуу, лучше не делать различия между мелкими и крупными проектами. Любая программа стремится в своем развитии становиться крупнее и захватить наибольшее количество ресурсов
      если делать все грамотно даже в "домашних" проектах, то это не только опыт и привычка, но и возможность выводить их "в люди", а также использовать повторно в серьезных проектах
      Ответить
      
      realsugar 25.08.2011 09:17 # −4
      
      коллега, чем меньше наши "коллеги" думают головой тем больше мы зарабатываем в итоге, а люди, несущие просвещение, во все времена подвергались гонениям и щемлениям (с). не старайся донести истину до тех, до кого она не дойдет ;)
      Ответить
      
      jokz 25.08.2011 18:06 # 0
      
      пока мы, "не думая головой", делаем быстро и качественно проекты, вы устанавливаете behat и извращаетесь. вообще не поддерживаю TDD (так помоему называется), когда в голове укладывается вся структура проекта и хорошая память - это все ни к чему.
      Ответить
      
      realsugar 25.08.2011 18:59 # +1
      
      уважаемый, а вы работали в проекте, который делают ТРИ распределенные КОМАНДЫ?
      Ответить
      
      CPPGovno 25.08.2011 19:31 # 0
      
      Я работал над проектом распределенной системы, которую делал Я один.
      Ответить
      
      roman-kashitsyn 25.08.2011 20:10 # +2
      
      Я работал в проекте, в котором около 200 модулей (одним из них была распределённая операционная система; в модуле, с которым работал я, было примерно полмиллиона строк кода) с людьми из России, Швеции, Китая, Хорватии, Индии, Турции ...
      Ответить
      
      Vasiliy 26.08.2011 12:20 # −3
      
      Пока ты над ним работал, я всех баб перетрахал. На-ка выкуси....
      Ответить
      
      roman-kashitsyn 26.08.2011 12:22 # +4
      
      Фи, а ты потом проверялся у венеролога?
      Ответить
      
      realsugar 25.08.2011 23:39 # 0
      
      в общем, попахивает холиваром... предлагаю закончить и заняться ДЕЛОМ))
      Ответить
      
      jokz 26.08.2011 18:44 # 0
      
      нет, работаю исключительно сам. Но еще не было такого, что я не взял, например, проект изза того, что он слишком большой.
      
      да, уважаемый, читайте внимательно
      
      ------------------------------------------------------------
      >"и, да, конечно речь не идет о >крупногабаритных проектах, где такие >хуки могут просто напросто затеряться и >не о рабочих проектах естесна."
      Ответить
      
      jokz 24.08.2011 23:27 # 0
      
      не, вот про различия между крупными проектами и мелкими я бы даже поспорил.
      
      зачем делать каркас замка для уличного туалета?
      Ответить
      
      realsugar 24.08.2011 23:55 # 0
      
      от себя добавлю, что проект ОЧЕНЬ большой, и это говно извлечено из продакшна... тестами, к сожалению, даже и не пахнет... в наследство достался(
      Ответить
      
      Lure Of Chaos 24.08.2011 23:59 # 0
      
      аналогия неверна, уличный туалет не эволюционирует до пятизвездочного отеля
      Ответить
      
      jokz 25.08.2011 03:02 # −1
      
      ну какбы я про этоже и говорю. какбы ты расчитываешь и нагрузку и защиту основываясь на размере проекта.
      Ответить
      
      jokz 25.08.2011 03:24 # −3
      
      ну, ипать, говно вопрос, давайте чтобы заглушить одно условие создавать сценарии bdd и тестить.
      Ответить
- guest8 09.04.2019 11:22 # −999
  
  показать все, что скрытоvanished
  Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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