еще одна реализация роутинга

[ghost9]

Написал когда-то сабж для своего велосипеда, пытаясь портировать компонент из play framework . Потом немного поменял синтаксис, появились новые фичи. Потом потребовалось прикрутить роутинг к сайту-визитке, потом еще к одному, а потом родилась идея вынести все это в отдельный проект.

Как-то так:
https://github.com/paw34rus/PathFinder
https://packagist.org/packages/paw34rus/path-finder

Документация и тесты в процессе написания, но есть 2 примера использования(built-in и adapter) и пример конфига. Хотелось бы услышать объективную критику.

 

[hell0w0rd]

https://github.com/nikic/FastRoute

 

[ghost9]

https://github.com/nikic/FastRoute

Синтаксис шаблонов урл просто ужасен, дальше не смотрел.

{id:[0-9]+}

А если сложная регулярка? А если параметров много?
Мое решение на голову выше этой поделки.

 

[hell0w0rd]

ghost9, ага) по скорости сравни

 

[ghost9]

ghost9, ага) по скорости сравни

Почему ты так уверен, что мое решение медленное? Ты смотрел код? Или какие есть основания так полагать?

 

[Вурдалак]

ghost9, потому что ты хрен с горы в отличие от автора FastRoute'а. Покажи бенчмарки.

 

[hell0w0rd]

Дополню - не получается сравнить, потому что в примерах нет файла с конфигом для твоего роутера, по коду его гадать совсем не хочется.

 

[ghost9]

Дополню - не получается сравнить, потому что в примерах нет файла с конфигом для твоего роутера, по коду его гадать совсем не хочется.

Как это нету, когда есть, прямо на главной странице репозитария, в readme.md. Ты ссылку-то открывал вообще?

ghost9, потому что ты хрен с горы в отличие от автора FastRoute'а. Покажи бенчмарки.

Нормально разговаривать научись, тогда все тебе покажу, клоун.

 

[hell0w0rd]

ghost9, https://github.com/paw34rus/PathFinder/blob/master/examples/built-in.php#L14

 

[hell0w0rd]

А ну все, тут не о чем говорить.
https://github.com/paw34rus/PathFinder/blob/master/lib/PathFinder/Router.php#L45
http://nikic.github.io/2014/02/18/Fast-request-routing-using-regular-expressions.html - прочитай, а потом наезжай со своей поделкой

 

[Вурдалак]

ghost9, да ты не обижайся, кусок, тут же нет ничего личного. Ты написал кусок говна, если ты хочешь объективности. Ты даже модель роутера не смог делать нормальной, там откуда-то есть понятие контроллера, нет тестов, нарушение SRP, русский язык, etc. Этим никто пользоваться не будет, здесь тебя никто не оценит.

 

[c0dex]

Роутер клейна ктонить смотрел?

 

[hell0w0rd]

c0dex, ссылку?

 

[c0dex]

Я с мобилки, не найду щас

Ps https://github.com/chriso/klein.php

 

[ghost9]

ghost9, да ты не обижайся, кусок, тут же нет ничего личного. Ты написал кусок говна, если ты хочешь объективности. Ты даже модель роутера не смог делать нормальной, там откуда-то есть понятие контроллера, нет тестов, нарушение SRP, русский язык, etc. Этим никто пользоваться не будет, здесь тебя никто не оценит.

Ты выбирай выражения, когда что-то пишешь. Лично на тебя и твое мнение я клал [censored], ибо искать какой-то конструктив в потоке оскорблений нет ни малейшего желания. Иди на [censored].

А ну все, тут не о чем говорить.
https://github.com/paw34rus/PathFinder/blob/master/lib/PathFinder/Router.php#L45
http://nikic.github.io/2014/02/18/Fast-request-routing-using-regular-expressions.html - прочитай, а потом наезжай со своей поделкой

Прочитал, очень удивили результаты тестов. Я расматривал мысль склеить все марштуры в одну регулярку, но потом отбросил эту идею из-за опасения, что такая конструкция будет потреблять очень много оперативы, особенно если регулярки будут сложные. Идея с блоками довольно любопытна, спасибо за ссылку.