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

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

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

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

<? $dir = $APPLICATION->GetCurDir();
$slashCount = substr_count($dir, "/");

//убираем со страницы отдельной новости
if($slashCount<6){ ?>
...
}?>

Какая наикрасивейшая проверка на количество слешей в урл.
Супер решение

Запостил:

liquor, 14 Июля 2016

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

guest 15.07.2016 09:47 # 0

что не так?
Ответить
bakagaijin 15.07.2016 10:20 # +1

Что касается именно проверки количества слэшей в строке - всё ж правильно.
Ответить
liquor 15.07.2016 10:20 # +1

Да. Тут скорей не говнокод, а говнологика: делать проверку на слеши, что в последствии влечёт невозможность простого смены url-a
Ответить
myaut 15.07.2016 12:53 # +6

http://govnokod.ru////////////20380////////////
Ответить
- 3_14dar 15.07.2016 19:35 # 0
  
  Ну кстати, не-пхп программы, которые роутят регекспами, вполне могут выдать 404. А вот на наличие или отсутствие завершающего слеша, имхо, материться некультурно.
  Ответить
  - bormand 15.07.2016 20:01 # +4
    
    > роутят регекспами
    Фублядь, фунахуй!
    Ответить
    - 3_14dar 15.07.2016 20:10 # −4
      
      Ебанутый? Так все фреймверки делают.
      Ответить
      - bormand 15.07.2016 20:11 # 0
        
        Можно пример любого из "всех фреймворков"?
        Ответить
        
        3_14dar 15.07.2016 20:21 # 0
        
        django. Перенаправляет с site/page на site/page/, на site/page// уже выдает 404.
        Ответить
        
        bormand 15.07.2016 20:21 # 0
        
        Я про регексы в роутинге.
        Ответить
        
        bormand 15.07.2016 20:23 # 0
        
        З.Ы. Oh shi... И правда пиздец из регулярок можно писать...
        urlpatterns = [ url(r'^forgot-password/$', ForgotPasswordFormView.as_view()), url(r'^api/', include(router.urls, namespace='api')), ]
        Ответить
        
        3_14dar 15.07.2016 20:31 # 0
        
        Во всех что я видел так было. Это только в цги скрипты берутся с файловой системы. С просвещением :)
        Ответить
        
        bormand 15.07.2016 20:32 # 0
        
        Ну мне попадались всякие @route("post/<id:int>"), а это явно не регулярки.
        Ответить
        
        3_14dar 15.07.2016 20:35 # 0
        
        Ну по сути это и есть регулярки, post/(\d+). Или ты что-то другое хотел?
        Ответить
        
        bormand 15.07.2016 20:45 # 0
        
        > post/(\d+)
        Фубля. Нетипизированная параша.
        
        З.Ы. Сорри, я бухой.
        Ответить
        
        3_14dar 15.07.2016 20:47 # 0
        
        >Сорри, я бухой.
        Похоже.
        
        > Нетипизированная параша.
        >\d+
        Што?
        Хотя там может быть тупо попытка сделать int() от того что между слешами.
        
        Чзх? http://govnokod.ru/19667#comment339758
        Ответить
        
        bormand 15.07.2016 20:51 # +1
        
        Ну вот я выше привёл пример из фласка: "post/<int:id>" - вот это норм. В процедуру прилетает именно int, всё чотко и понятно.
        
        А \d+ это хуйня какая-то, которую ещё парсить. Не барское это дело.
        Ответить
        
        3_14dar 15.07.2016 20:57 # 0
        
        При желании, думаю, можно и регекспами сделать. Просто не комильфо.
        Ответить
        
        bormand 15.07.2016 20:59 # 0
        
        Да просто проще выдавать урл в виде /100500/hui-pizda-opisanie чем /100500-hui-pizda-opisanie и парсить его регулярками... Юзеру всё равно похуй.
        Ответить
        
        3_14dar 15.07.2016 22:07 # 0
        
        Второе я видел гораздо чаще, но исходников парсинга не видел, сорри.
        Ответить
        
        guest 26.07.2016 08:27 # 0
        
        Как бы - дело не в простоте. В первом случае у вас некая статья в каталоге(или группе статей) "100500", во втором - id статьи и название. Делается это для того, чтобы юзер мог найти статью в истории браузера, а также глядя на ссылку получил примерное представление о том, куда она ведет.
        
        Да и нечто вроде inval( subStr($link), 1 , strPos($link,'-') ) вряд ли можно назвать "парсингом регулярками"
        Ответить
        
        guest 26.07.2016 08:28 # 0
        
        * inval( subStr($link, 1 , strPos($link,'-') ) )
        Ответить
        
        guest 26.07.2016 08:28 # 0
        
        * intval( subStr($link, 1 , strPos($link,'-') ) )
        
        P.s. Сегодня явно не мой день)
        Ответить
        
        bakagaijin 15.07.2016 21:00 # 0
        
        (highfive)
        Я ещё вот думаю сходить за пивом, за мешком штукатурки и мелочами (балкон чиню), но я уже трохи того. Но схожу, и добавлю.
        Ответить
        
        3_14dar 15.07.2016 20:25 # 0
        
        ^post/<post>
        
        Чем не регексп?
        Ответить
        
        guesto 16.07.2016 05:48 # 0
        
        Зенд, джанго, спрингмвц
        Ответить
        
        bormand 16.07.2016 05:59 # 0
        
        Ослик, суслик, паукан.
        Ответить
        
        guesto 16.07.2016 09:32 # 0
        
        Чай, кофе, потанцуем
        Ответить
        
        bormand 16.07.2016 09:35 # 0
        
        Ночь, улица, фонарь, аптека.
        Ответить
        
        guesto 16.07.2016 09:37 # 0
        
        Православие, са одержавие, соборность
        Ответить
        
        defecate-plusplus 16.07.2016 09:40 # +4
        
        Православие, самодержавие, соборность и мокренькая кисонька
        Ответить
        
        guesto 16.07.2016 09:59 # +1
        
        Секс, драгс и пхп
        Ответить
        
        inkanus-gray 16.07.2016 11:54 # 0
        
        Я знал, что рано или поздно мы перейдём и на эту дрянь.
        Ответить
      - guest 16.07.2016 21:33 # −3
        
        Почему бомбит, пидарахи?
        Ответить

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

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

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

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