Правильно ли отправлять данные всегда по одному адресу AJAX?

Question

[Erik2121]

Добрый день!
Смотрите у меня достаточно много всяких кнопок которые отправляют ajax запрос, и все они у меня летят по одному адресу:

$.ajax({
     url: '?action=main',
     type: 'post',
     data: {...}
});

Ну т.е это нормально если все идут по одному адресу?
Заранее спасибо.

Comments

[Антон]

Как я понял, у все все идет в один экшен в каком-то контроллере. Это странно. Боюсь представить код этого экшена.
Ну и вообще все относительно. Смотря какая у вас задача.
Мало информации.

[Erik2121]

Антон: Я просто хочу понять вообще как бы стоит бросать данные всегда по одному адресу?
Или всё же лучше создавать несколько обработчиков?

[Антон]

Erik2121: почитайте про MVC в целом и про "C" в особенности.
Вы используете какой-то фреймворк?

[Erik2121]

Антон: Ну как бы просто свой движок и некоторые идеи взял из фреймворка просто играюсь)) про MVC я знаю, вы вопрос то поняли?

[Антон]

Erik2121: ну если вы знаете про MVC, то вы знаете про контроллеры и его экшены. Вы знаете про маршруты и прочее. Вы знаете что это все облегчает жизнь и работу.

А у вас по вашим же словам абсолютно все идет в один экшен. По-моему вы сами способны ответить на свой вопрос.

[Erik2121]

Антон: Я вроде бы как бы знаю ответ, но сомневаюсь поэтому и спрашиваю, интересно Ваше мнение.

[Антон]

Erik2121: плюс я не знаю что такое экшен в вашем самопальном движке.

[Антон]

Erik2121: придерживайтесь MVC паттерна. Можете почитать про другие - я не ограничиваю.

У вас много кнопок. Что это за кнопки?

Они находятся в разных разделах? Если да, то один раздел = одному контроллеру. Делите на контроллеры.

Если эти кнопки находятся в разных разделах, но выполняют одну и туже роль, то создавайте общий контроллер (например, Ajax) с экшеном create (или update, если обновляете что-то) - также придерживайтесь CRUD.

[Антон]

Erik2121: если на простом примере create форм, то каждая форма имеет свой обработчик (экшен)к. Этот обработчик (экшен) лежит в своем разделе (контроллере).

[Erik2121]

3 кнопки:
1) отправляет сообщение
2) обновляет ленту сообщений
3) выводит все сообщения

[Антон]

Erik2121:
1. У кнопки есть форма? Если да, то это однозначно отдельный контроллер (PrivateMessagesController, как пример) и экшен create.
2. Лента сообщений - это все тот же контроллер PrivateMessagesController, экшен index.
3. Немного не понял чем отличается от второго пункта, но вроде суть одна и та же.

[Erik2121]

Антон: формы нет)

[Антон]

Erik2121: Не важно. Если сообщение планируете генерировать, то генерируйте его в экшене create.

[Erik2121]

Антон: спасибо за внимание и ответ, попробую.

[Антон]

Erik2121: у вас идет действие по созданию чего-либо - create. Если обновляете что-то, то это уже экшен update. Удаляете - destroy. Список - index.

Answer 1

[JaxxDexx]

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

Comments

[Erik2121]

Немного не понял, что значит одинаково?

[JaxxDexx]

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

И то я тут утрировал, так как парсить или проверять можно тоже по разному. В общем исходите из простых сущностей.

Answer 2

[ThunderCat]

у меня достаточно много всяких кнопок которые отправляют ajax запрос, и все они у меня летят по одному адресу:

если они все выполняют общую функцию(допустим добавляют разные товары в корзину) то все ок.
Если это принципиально разные действия, то и пути(route) у них должны отвечать действию, это
а) Не создает путаницы в коде js, четко определяя что делает запрос(например /ajax/basket/add/66)
б) четко разделяет на сервере логику каждого действия, не вбахивая тупо в одно место весь код и не плодя god objects
в) Это красиво, модно и пахнет елкой