Почему закрыли ветку с моим расширением?

Обсуждение сайта и форума yiiframework.ru
Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 15:28

andrei.obuhovski писал(а): Встречный вопрос: Почему принято разбивать классы по файлам, а файлы по папкам? Можно ведь все файлы в одной папке держать. И называть их frontend_module_my-module_views_default_index.php Или вы так и делаете? :)
Не доводите до абсурда, вот примеры названий классов у меня
BlogPost
PortfolioPicture
SitePage
и находятся они где положено.

Это простые и понятные имена, которые не вызывают никаких конфликтов, и с которыми просто работать как мне, так и любому другому программисту, к которому попадет мой код.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 15:29

denis909 писал(а):а меня все хотят убедить в том, что 10 лет так делали, а теперь вдруг так делать вдруг стало нельзя.
не нельзя, а не рекомендуется, поскольку вся разработка на php изменилась за последние 5 лет.
denis909 писал(а):Причем, убеждают в этом не создатели PHP, а создатели одного из многих фреймворков на нем
всех действующих фреймворков без исключения..
denis909 писал(а):Хотя, признаю, одного из лучших!
ну началось) тут целые ветки про недостатки фреймворка по сравнению с другими, а вы ему дифирамбы поете) хорош он только в одном - быстроте разработки простого приложения.
Последний раз редактировалось zelenin 2016.01.27, 15:31, всего редактировалось 1 раз.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 15:30

denis909 писал(а):
andrei.obuhovski писал(а): Встречный вопрос: Почему принято разбивать классы по файлам, а файлы по папкам? Можно ведь все файлы в одной папке держать. И называть их frontend_module_my-module_views_default_index.php Или вы так и делаете? :)
Не доводите до абсурда, вот примеры названий классов у меня
BlogPost
PortfolioPicture
SitePage
и находятся они где положено.

Это простые и понятные имена, которые не вызывают никаких конфликтов, и с которыми просто работать как мне, так и любому другому программисту, к которому попадет мой код.
а мои классы называются:
Blog\Post
Portfolio\Picture
итд. И никаких неудобств и конфликтов.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 15:36

>а мои классы называются:
>Blog\Post
>Portfolio\Picture
>итд. И никаких неудобств и конфликтов.

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

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 15:37

denis909 писал(а):>а мои классы называются:
>Blog\Post
>Portfolio\Picture
>итд. И никаких неудобств и конфликтов.

А я и не призывал никого бросать пространства имен и не использовать их.
Используйте на здоровье.
Но я, на данный момент, не хочу делать это везде.
И для тех, кто со мной согласен, сделал расширение, которое снимает существующие во второй версии ограничения.
И у людей теперь есть выбор, делать или не делать так как вы пишете, а без этого расширения выбора нет.
ок. Я просто написал, что ваша аргументация сомнительна, поскольку ни то ни другое неудобств не доставляет. Ваш единственный здравый аргумент - "мне так нравится". Поэтому все и ополчились, поскольку люди не понимают как может нравиться то, что им очевидно неудобно.
Последний раз редактировалось zelenin 2016.01.27, 15:39, всего редактировалось 1 раз.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 15:38

zelenin писал(а): ну началось) тут целые ветки про недостатки фреймворка по сравнению с другими, а вы ему дифирамбы поете) хорош он только в одном - быстроте разработки простого приложения.
Я не писал - самого лучшего, но то что Yii в топе, это не мнение, а факт.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 15:40

denis909 писал(а):Я не писал - самого лучшего, но то что Yii в топе, это не мнение, а факт.
не буду разочаровывать) у меня свои мнения о топе.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 15:42

zelenin писал(а):Я просто написал, что ваша аргументация сомнительна, поскольку ни то ни другое неудобств не доставляет. Ваш единственный здравый аргумент - "мне так нравится". Поэтому все и ополчились, поскольку люди не понимают как может нравиться то, что им очевидно неудобно.
То что это помогает перенести готовый проект с первой версии фреймворка на вторую, вызывает сомнения?

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 15:48

denis909 писал(а):
zelenin писал(а):Я просто написал, что ваша аргументация сомнительна, поскольку ни то ни другое неудобств не доставляет. Ваш единственный здравый аргумент - "мне так нравится". Поэтому все и ополчились, поскольку люди не понимают как может нравиться то, что им очевидно неудобно.
То что это помогает перенести готовый проект с первой версии фреймворка на вторую, вызывает сомнения?
ни в коем случае. Сомнительно, именование классов - это меньшая проблема при переносе проекта с версии на версию. Апи фреймворка сильно поменялось и о переносе большого проекта без частичного переписывания речи не может идти. А маленький проект на то и маленький, потому что кодовая база доступна для ручного редактирования в сжатые сроки.
Ну то есть при ремонте машины мне будет удобно, если будет молоток на всякий случай, но погоды мне это не сделает.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 16:00

Данное расширение позволяет практически без изменений использовать контроллеры, отображения, и половину компонентов.
А сложность переноса моделей зависит от их толщины.
Вы явно недооцениваете полезность расширения. Оно не сделает за вас всю работу, но серьезно сократит её количество.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 16:05

denis909 писал(а):Данное расширение позволяет практически без изменений использовать контроллеры, отображения, и половину компонентов.
А сложность переноса моделей зависит от их толщины.
Вы явно недооцениваете полезность расширения. Оно не сделает за вас всю работу, но серьезно сократит её количество.
кстати, запамятовал совсем, хотя именно сам в нашем проекте это делал:

Код: Выделить всё

  "autoload": {
    "psr-4": {
      "": [
        "site/protected/behaviors",
        "site/protected/bundles",
        "site/protected/commands",
        "site/protected/components",
        "site/protected/models",
        "site/protected/widgets"
      ]
    }
  }
это решает проблему с корневым неймспейсом без вашего расширения. В перечисленных директориях лежат именно классы в yii1-стиле, в корневом неймспейсе.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 16:44

В тикете этот вопрос обсуждался, что действительно можно делать через composer автозагрузку.
Но основная задача расширения это все-таки не автозагрузка в стиле 1.X, а работа в глобальном пространстве имен в приложении (контроллерах), и расширение выполняет в первую очередь эту задачу.
А автозагрузка 1.X бонусом, чтобы расширение было цельное, не тянуло зависимостей, в том числе от composer не зависело и работало и без него.
В случае переноса проектов из первой версии это актуально, можно просто обновить фреймворк, не подключая composer.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 16:55

denis909 писал(а):Но основная задача расширения это все-таки не автозагрузка в стиле 1.X, а работа в глобальном пространстве имен в приложении (контроллерах)
то есть чтобы контроллеры были доступны? это все? не узенькая задача?
denis909 писал(а):В случае переноса проектов из первой версии это актуально, можно просто обновить фреймворк, не подключая composer.
без композера о yii2 забудьте.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 17:04

без композера о yii2 забудьте.
Позвольте, мне самому это решать. Не спорю, что работать в Yii2 без composer жутко неудобно, но в некоторых случаях, когда разговор идет только об обновлении версии фреймворка в проекте, это может быть допустимо. Но если такой аргумент не нравиться, то можно считать что это сделано для того чтобы меньше править конфиги при переносе.

то есть чтобы контроллеры были доступны? это все? не узенькая задача?

Это даже забавно, вы больше всех обсуждаете расширение, но не понимаете что это такое и зачем нужно.
Или я вас не понимаю что такое "контроллеры доступны".

Разговор кончено не про сами контроллеры, а про код внутри них. Методы именования классов в приложении.

Простыми словами: чтобы не надо было во всех контроллерах менять Post на \Post или писать use Post а работало и так.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 17:10

denis909 писал(а):Это даже забавно, вы больше всех обсуждаете расширение, но не понимаете что это такое и зачем нужно.
Или я вас не понимаю что такое "контроллеры доступны".

Разговор кончено не про сами контроллеры, а про код внутри них. Методы именования классов в приложении.

Простыми словами: чтобы не надо было во всех контроллерах менять Post на \Post или писать use Post а работало и так.
ну в чем проблема: без композера контроллеры не видны автозагрузчику yii2, поскольку обязаны лежать в определенном неймспейсе. Насчет остального таких правил нет и проблема решается композером. Таким образом практически все решается композером, а имена контроллеров вы все равно писать нигде не будете, кроме как единожды в файле самого класса. И без композера использовать yii2 вы тоже не будете, поскольку это доставит намного больше гемороя и не может быть обусловлено ничем.

Аватара пользователя
denis909
Сообщения: 103
Зарегистрирован: 2009.04.02, 16:04
Откуда: Санкт-Петербург
Контактная информация:

Re: Почему закрыли ветку с моим расширением?

Сообщение denis909 » 2016.01.27, 17:24

Сomposer тут непричем, он замечательный, но вы похоже так и не поняли в чем дело.

Yii 2 в начале каждого класса контроллера обязывает писать так:

namespace app\controllers;

И если бы это было сделано только для того чтобы автозагрузчик увидел контроллер, то и черт бы с ним, но проблема в том, что написав так, все что внутри файла находится уже в локальном пространстве имен контроллеров, и ни один класс в таком контроллере нельзя использовать без его предварительно объявления, или без символа \ перед названием класса.

Расширение исправляет это, и позволяет работать с классами в контроллере без их предварительного объявления.

Много это, или мало, вам виднее, но для себя каждый решит сам.

zelenin
Сообщения: 10144
Зарегистрирован: 2013.04.20, 11:30

Re: Почему закрыли ветку с моим расширением?

Сообщение zelenin » 2016.01.27, 17:29

denis909 писал(а):Сomposer тут непричем, он замечательный, но вы похоже так и не поняли в чем дело.

Yii 2 в начале каждого класса контроллера обязывает писать так:

namespace app\controllers;

И если бы это было сделано только для того чтобы автозагрузчик увидел контроллер, то и черт бы с ним, но проблема в том, что написав так, все что внутри файла находится уже в локальном пространстве имен контроллеров, и ни один класс в таком контроллере нельзя использовать без его предварительно объявления, или без символа \ перед названием класса.

Расширение исправляет это, и позволяет работать с классами в контроллере без их предварительного объявления.

Много это, или мало, вам виднее, но для себя каждый решит сам.
ну теперь прояснилось. Вы сэкономили на \. Ок. Полезное расширение.

Ответить