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

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

+174
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
29. 29
30. 30
31. 31
32. 32
33. 33
34. 34
35. 35
36. 36
37. 37
38. 38
```
if(isset($_GET['aspx'])) include("parthers/aspx.php");  
elseif(isset($_GET['b7186'])) include("parthers/b7186.php");  
elseif(isset($_GET['dip'])) include("parthers/dip.php");  
elseif(isset($_GET['Maslyakov'])) include("parthers/Maslyakov.php");  
elseif(isset($_GET['Zhannetka'])) include("parthers/Zhannetka.php");  
elseif(isset($_GET['b7186rft'])) include("parthers/b7186rft.php");  
elseif(isset($_GET['mistral09'])) include("parthers/mistral09.php");  
elseif(isset($_GET['Simmka'])) include("parthers/Simmka.php");  
elseif(isset($_GET['polevnik'])) include("parthers/polevnik.php");  
elseif(isset($_GET['dmitriev83'])) include("parthers/dmitriev83.php");  
elseif(isset($_GET['orion'])) include("parthers/orion.php");  
elseif(isset($_GET['Krick'])) include("parthers/Krick.php");  
elseif(isset($_GET['ae63237'])) include("parthers/ae63237.php");  
elseif(isset($_GET['zen'])) include("parthers/zen.php");  
elseif(isset($_GET['antkuz'])) include("parthers/antkuz.php"); 
elseif(isset($_GET['psd'])) include("parthers/psd.php"); 
elseif(isset($_GET['122745105643'])) include("parthers/122745105643.php"); 
elseif(isset($_GET['amadey'])) include("parthers/amadey.php"); 
elseif(isset($_GET['prosto'])) include("parthers/prosto.php"); 
elseif(isset($_GET['Oushin'])) include("parthers/Oushin.php"); 
elseif(isset($_GET['alkir'])) include("parthers/alkir.php"); 
elseif(isset($_GET['werty2011'])) include("parthers/werty2011.php"); 
elseif(isset($_GET['valjek'])) include("parthers/valjek.php"); 
elseif(isset($_GET['32167911'])) include("parthers/32167911.php"); 
elseif(isset($_GET['biz'])) include("parthers/biz.php"); 
elseif(isset($_GET['hranitely'])) include("parthers/hranitely.php"); 
elseif(isset($_GET['stalker147'])) include("parthers/stalker147.php"); 
elseif(isset($_GET['boef8'])) include("parthers/boef8.php"); 
elseif(isset($_GET['Mastak'])) include("parthers/Mastak.php"); 
elseif(isset($_GET['kna888'])) include("parthers/kna888.php"); 
elseif(isset($_GET['index'])) include("parthers/index.php"); 
elseif(isset($_GET['abc'])) include("parthers/abc.php"); 
elseif(isset($_GET['ollgga69'])) include("parthers/ollgga69.php"); 
elseif(isset($_GET['Stalker007'])) include("parthers/Stalker007.php"); 
elseif(isset($_GET['maluw'])) include("parthers/maluw.php"); 
elseif(isset($_GET['diana2113327'])) include("parthers/diana2113327.php"); 
elseif(isset($_GET['staschel'])) include("parthers/staschel.php"); 
else include("parthers/file.php");
```
Набрёл в проектах на фри-ланс.ру. Описание проекта: "Есть вот такой код, нужно придумать как его оптимизировать"

Запостил: wxx, 31 Мая 2011

Tweet
Комментарии (65) RSS
- jQuery 31.05.2011 13:38 # 0
  
  сколько платят? :)
  Ответить
  - wxx 31.05.2011 13:42 # 0
    
    а фиг знает, я уже закрыл вкладку
    Ответить
    - wxx 31.05.2011 13:46 # 0
      
      Теперь самому интересно стало, заново нашёл: 300р, но ему в комментариях "оптимизировали", хотя сказали что неплохо бы вообще всё переписать.
      Ответить
      
      jQuery 31.05.2011 13:51 # −1
      
      порадуйте нас еще кусочками его кака-кода :)
      Ответить
- rO_ot 31.05.2011 13:45 # 0
  
  И найдется же тот, кто оптимизирует его как-то типа include($_GET['page']);, потом тот же заказчик будет на том же фрилан.сру просить защитить сайт:)
  Ответить
  - jQuery 31.05.2011 13:51 # +1
    
    $inc = $_GET['page'];
    $words = Array('aspx','b7186','dip');// и так далее
    if(!in_array($inc,$words)) {
    die;
    }
    include('parthers/'.$inc);
    
    я бы сделал как-то так
    Ответить
    - JeremyW 31.05.2011 13:55 # +1
      
      else include("parthers/file.php");
      Ответить
    - sayidandrtfm 31.05.2011 13:56 # 0
      
      Может в файл поместить, построчно?
      Ответить
      
      jQuery 31.05.2011 13:59 # −1
      
      да хоть в БД. хозяин - барин
      Ответить
    - daemon_master 31.05.2011 20:51 # −4
      
      как вы видите, там идёт $_GET['page'] и абсолютно не важно что там передаётся, просто тот факт, что 'page'
      
      $pages = array ('aspx', 'b7186', ...); $rc = 'default'; $cp = count ($pages); for ($i = 0; $i < $cp && 'default' == $rc; ++$i) if (isset ($_GET[$pages[$i]])) $rc = $pages[$i]; include_once ('parthers/'.$rc.'.php');
      
      почему циклом for, а не foreach, да просто чтобы не выходить через break, ибо его использование не столь гуммано
      Ответить
      
      7ion 31.05.2011 21:52 # +4
      
      Вы какую-то хуйню написали.
      Ответить
      
      daemon_master 31.05.2011 22:06 # −3
      
      а вы внимательней в тот код, который выше посмотрите как там идёт подключение страниц
      темболее, в моём решении, скорость выполнения данного алгоритма равна O(N) где N - размерность нашего массива страниц, причём, если же страница окажется вначале массива, то тем самым будет T(K), где K - позиция в массиве
      count ($pages) можно заменить вполне на константное числое, тем самым мы как можно более сокращаем количество операций в нашей оптимизации
      Ответить
      
      7ion 31.05.2011 22:10 # +1
      
      Я нихуя не понял и даже вникать не хочу.
      Вкратце и русским языком: чем ваше лучше этого:
      
      $inc = $_GET['page']; $words = array('aspx', 'b7186', 'dip'); if (in_array($inc, $words)) include 'parthers/' . $inc . '.php'; else include 'parthers/file.php';
      Ответить
      
      daemon_master 31.05.2011 22:15 # −3
      
      include 'parthers/'.'page'.'.'php';
      т.е. вы видите, на не нужно значение в массиве $_GET по ключу page, а нам нужен сам ключ page! т.е. на нужно перебрать ключи, а не значения по этим ключам найти! ссылка в виде index.php?page=1 и index.php?page=index должны открывать страницу parthers/page.php а не parthers/1.php и parthers/index.php
      всё же, прежде чем спорить, внимательней вы бы посмотрели на код повыше
      Ответить
      
      7ion 31.05.2011 22:22 # 0
      
      Раз на то пошло, то в оригинале мы должны исходить из строки запроса вида ?partner. Берем вместо $_GET['page'] переменную $_SERVER['query_string']. И все.
      Ответить
      
      daemon_master 31.05.2011 22:28 # −3
      
      тогда следующий случай, если в $_SERVER['query_string'] мы видим все наши текущие данные, входящие в $pages, то чем то, что напишете вы, по смыслу будет отличаться от того, что написал я?
      Ответить
      
      7ion 31.05.2011 22:31 # 0
      
      >если в $_SERVER['query_string'] мы видим все наши текущие данные, входящие в $pages
      Чо?
      Ответить
      
      daemon_master 31.05.2011 22:32 # −2
      
      проехали
      Ответить
      
      ReallyBugMeNot 01.06.2011 10:37 # 0
      
      * $_SERVER['QUERY_STRING']
      Ответить
      
      istem 01.06.2011 18:07 # +1
      
      Блеать... это же пиэйчпи
      
      $words = array('index','aspx', 'b7186', 'dip'); include 'partners/' . $words[ array_search( $_GET['page'] , $words) ] . '.php';
      Ответить
      
      7ion 01.06.2011 19:10 # 0
      
      А в случае неудачи заинклудится partners/false.php
      Ответить
      
      istem 01.06.2011 21:10 # +1
      
      в случае неудачи будет использовано первое значение массива... Это же php
      тьфу, ещё isset забыл...
      Ответить
      
      7ion 01.06.2011 21:44 # 0
      
      А, тьфу, точно.
      Это изящное решение, но не люблю я эти трюки с нестрогой типизацией, ведь функция false вернет, а то, что это 0 PHP уже сам додумает.
      Однако же, снимаю шляпу, я бы до такого не допер.
      И isset можно пренебречь, сферический пример же.
      Ответить
      
      istem 01.06.2011 21:55 # 0
      
      Надо отдать таки должное Расмусу... Он очень хотел сделать из говна - конфетку... (а из говнокодеров - человекообразных)...
      Его попытка разделить на ноль - чего-нибудь да заслуживает... (...etc... и тп )...
      Ответить
      
      7ion 02.06.2011 07:56 # −2
      
      Кто такой Расмус?
      Ответить
      
      istem 02.06.2011 08:07 # 0
      
      http://ru.wikipedia.org/wiki/Лердорф,_Расмус
      Ответить
      
      7ion 02.06.2011 08:47 # +2
      
      О Боги, это же главный соавтор многих цитат этого сайта!
      Ответить
      
      Lure Of Chaos 02.06.2011 11:57 # 0
      
      и что теперь, отслюнявливать ему отчисления?
      Ответить
      
      bugmenot 02.06.2011 13:07 # 0
      
      лиру дам, мне не жалко
      Ответить
      
      jQuery 31.05.2011 22:57 # −3
      
      мне мой код больше нравиться. правда его не много допилить еще надо
      Ответить
      
      bugmenot 01.06.2011 07:47 # 0
      
      > нравиться
      Ответить
      
      Uchkuma 01.06.2011 09:20 # +1
      
      > не много
      Ответить
      
      7ion 01.06.2011 12:12 # +1
      
      > die;
      Узнаю похапешную любовь к харакири из-за пустяков.
      Ответить
      
      jQuery 01.06.2011 14:27 # −3
      
      примера ради показал.
      Ответить
      
      Vindicar 01.06.2011 17:44 # −1
      
      ПХП писали эмо? О_о
      Ответить
    - wwwguru 10.06.2011 14:55 # 0
      
      А почему не "if (!isset($words[$inc])){"?
      Ответить
- Uchkuma 31.05.2011 13:48 # +1
  
  part hers
  Ответить
- wxx 31.05.2011 14:01 # 0
  
  Кстати, раз здесь такая живая обстановка, может подскажете где начинающий питонщик может небесплатно попрактиковаться (нет, не я должен платить).
  Ответить
  - jQuery 31.05.2011 14:05 # +1
    
    тренеруйся на кошках
    Ответить
    - wxx 31.05.2011 14:14 # +2
      
      программировать кошек?
      Ответить
  - rO_ot 31.05.2011 14:15 # 0
    
    %название_сайта_фриланса%.%домен_сайта_ф риланса%
    Ответить
    - wxx 31.05.2011 14:35 # 0
      
      Ну это понятно, на одном из них я и наблюдаю различные вариации того, что написано выше
      Ответить
    - wxx 31.05.2011 14:44 # 0
      
      нужен программист для стратапа, как портнер и сооавтор идеи, работа с базой данных. все вопросы на почту: [email protected] подробности при встречи
      Ответить
      
      sectus 31.05.2011 15:00 # +4
      
      Следуя традициям сайта не могу не процитировать:
      > портнер
      Ответить
      
      jQuery 31.05.2011 15:37 # 0
      
      жуть
      Ответить
      
      SmackMyBitchUp 31.05.2011 16:57 # +3
      
      стратапа? страпона мб?)))
      Ответить
      
      Govnocoder#0xFF 31.05.2011 20:37 # 0
      
      Это такой тонкий намёк на один ресурс....
      Ответить
      
      Govnocoder#0xFF 31.05.2011 20:37 # 0
      
      Стратапщик, блин.
      Ответить
      
      SmackMyBitchUp 31.05.2011 20:41 # 0
      
      Фигня, подробности, говорит - при встрече. Уж лучше он к нам :D
      Ответить
      
      Lure Of Chaos 31.05.2011 22:21 # 0
      
      страпёр
      Ответить
  - Uchkuma 31.05.2011 14:20 # +2
    
    > нет, не я должен платить
    я просто охуел от вашей наглости
    Ответить
    - wxx 31.05.2011 14:31 # 0
      
      Да я такой, а вообще, вдруг кто подумает что я ему хочу заплатить за возможность выполнить его задание выполнить.
      Ответить
    - rO_ot 31.05.2011 14:48 # −1
      
      Брать деньги за работу, это высшая степень наглости
      Ответить
      
      Uchkuma 01.06.2011 09:24 # +1
      
      я пластический хирург. можно, я на вас попрактикуюсь, а вы мне за это заплатите?
      Ответить
- Creologic 31.05.2011 16:14 # 0
  
  Однако.
  Ответить
- Int 31.05.2011 17:34 # 0
  
  Ну это ж классика, можно было даже не писать.
  Ответить
  - develx0 31.05.2011 21:15 # 0
    
    Точно, точно
    Ответить
    - Int 01.06.2011 19:16 # −2
      
      Плюсуй, чё.
      Ответить
- Lure Of Chaos 31.05.2011 22:22 # 0
  
  ммм, афигенный routing
  Ответить
- wxx 31.05.2011 23:27 # 0
  
  >Комментарии (40)
  Срань господня, что вы здесь устроили?
  Ответить
  - striker 01.06.2011 03:15 # +5
    
    А НАЧАЛ ВСЁ ТЫ! ОТВЕЧАТЬ ТЕБЕ!
    Ответить
  - 7ion 01.06.2011 12:16 # +4
    
    Думаем, как же это оптимизировать.
    Ответить
  - Lure Of Chaos 01.06.2011 15:19 # 0
    
    срань господню
    Ответить
- 3.14159265 08.06.2011 12:25 # 0
  
  помяну WebKilla настоящего
  Ответить
  - wxx 08.06.2011 13:43 # −1
    
    Это кто
    Ответить
Добавить комментарий
Ошибка компиляции комментария:

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

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

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

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

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

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

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

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

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

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

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

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

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

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

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