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

    +1

    1. 1
    2. 2
    3. 3
    4. 4
    5. 5
    6. 6
    7. 7
    <? $dir = $APPLICATION->GetCurDir();
$slashCount = substr_count($dir, "/");

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

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

    Запостил: liquor, 14 Июля 2016

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

    • ava guest 15.07.2016 09:47 # 0

      что не так?
      Ответить

    • ava bakagaijin 15.07.2016 10:20 # +1

      Что касается именно проверки количества слэшей в строке - всё ж правильно.
      Ответить

    • ava liquor 15.07.2016 10:20 # +1

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

    • ava myaut 15.07.2016 12:53 # +6

      http://govnokod.ru////////////20380////////////
      Ответить

      • ava 3_14dar 15.07.2016 19:35 # 0

        Ну кстати, не-пхп программы, которые роутят регекспами, вполне могут выдать 404. А вот на наличие или отсутствие завершающего слеша, имхо, материться некультурно.
        Ответить

        • ava bormand 15.07.2016 20:01 # +4

          > роутят регекспами
          Фублядь, фунахуй!
          Ответить

          • ava 3_14dar 15.07.2016 20:10 # −4

            Ебанутый? Так все фреймверки делают.
            Ответить

            • ava bormand 15.07.2016 20:11 # 0

              Можно пример любого из "всех фреймворков"?
              Ответить

              • ava 3_14dar 15.07.2016 20:21 # 0

                django. Перенаправляет с site/page на site/page/, на site/page// уже выдает 404.
                Ответить

                • ava bormand 15.07.2016 20:21 # 0

                  Я про регексы в роутинге.
                  Ответить

                  • ava 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')),
]
                    Ответить

                    • ava 3_14dar 15.07.2016 20:31 # 0

                      Во всех что я видел так было. Это только в цги скрипты берутся с файловой системы. С просвещением :)
                      Ответить

                      • ava bormand 15.07.2016 20:32 # 0

                        Ну мне попадались всякие @route("post/<id:int>"), а это явно не регулярки.
                        Ответить

                        • ava 3_14dar 15.07.2016 20:35 # 0

                          Ну по сути это и есть регулярки, post/(\d+). Или ты что-то другое хотел?
                          Ответить

                          • ava bormand 15.07.2016 20:45 # 0

                            > post/(\d+)
                            Фубля. Нетипизированная параша.

                            З.Ы. Сорри, я бухой.
                            Ответить

                            • ava 3_14dar 15.07.2016 20:47 # 0

                              >Сорри, я бухой.
                              Похоже.

                              > Нетипизированная параша.
                              >\d+
                              Што?
                              Хотя там может быть тупо попытка сделать int() от того что между слешами.

                              Чзх? http://govnokod.ru/19667#comment339758
                              Ответить

                              • ava bormand 15.07.2016 20:51 # +1

                                Ну вот я выше привёл пример из фласка: "post/<int:id>" - вот это норм. В процедуру прилетает именно int, всё чотко и понятно.

                                А \d+ это хуйня какая-то, которую ещё парсить. Не барское это дело.
                                Ответить

                                • ava 3_14dar 15.07.2016 20:57 # 0

                                  При желании, думаю, можно и регекспами сделать. Просто не комильфо.
                                  Ответить

                                  • ava bormand 15.07.2016 20:59 # 0

                                    Да просто проще выдавать урл в виде /100500/hui-pizda-opisanie чем /100500-hui-pizda-opisanie и парсить его регулярками... Юзеру всё равно похуй.
                                    Ответить

                                    • ava 3_14dar 15.07.2016 22:07 # 0

                                      Второе я видел гораздо чаще, но исходников парсинга не видел, сорри.
                                      Ответить

                                    • ava guest 26.07.2016 08:27 # 0

                                      Как бы - дело не в простоте. В первом случае у вас некая статья в каталоге(или группе статей) "100500", во втором - id статьи и название. Делается это для того, чтобы юзер мог найти статью в истории браузера, а также глядя на ссылку получил примерное представление о том, куда она ведет.

                                      Да и нечто вроде inval( subStr($link), 1 , strPos($link,'-') ) вряд ли можно назвать "парсингом регулярками"
                                      Ответить

                                      • ava guest 26.07.2016 08:28 # 0

                                        * inval( subStr($link, 1 , strPos($link,'-') ) )
                                        Ответить

                                        • ava guest 26.07.2016 08:28 # 0

                                          * intval( subStr($link, 1 , strPos($link,'-') ) )

                                          P.s. Сегодня явно не мой день)
                                          Ответить

                            • ava bakagaijin 15.07.2016 21:00 # 0

                              (highfive)
                              Я ещё вот думаю сходить за пивом, за мешком штукатурки и мелочами (балкон чиню), но я уже трохи того. Но схожу, и добавлю.
                              Ответить

                  • ava 3_14dar 15.07.2016 20:25 # 0

                    ^post/<post>

                    Чем не регексп?
                    Ответить

                  • ava guesto 16.07.2016 05:48 # 0

                    Зенд, джанго, спрингмвц
                    Ответить

            • ava guest 16.07.2016 21:33 # −3

              Почему бомбит, пидарахи?
              Ответить

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