(30 июл 2014, 22:17) (
0/
0)
[
0]
Речь идёт не о конкретном примере моего кода, а о принципе. Синтаксис языка позволяет пропустить фигурные скобки, если за условием следует лишь одно действие. Но так делать не рекомендуется! За условием должны следовать фигурные скобки, а действие(-я) со следующей строки. После действия закрывающая фигурная скобка.
Это считается хорошим тоном в дизайне кода. И не зря! Действительно удобно читать и легко понять.
(30 июл 2014, 21:24) (
0/
0)
[
0]
t,fkf;f,fufl.re,
Чем же интересно? Тем, что сделал его более удобным и простым для чтения и понимания?
(30 июл 2014, 18:10) (
0/
0)
[
0]
t,fkf;f,fufl.re,
Чувствуешь разницу между "фигурные скобки" и "табуляция кода"?
if (someVar == true) someVar2 = false; // НЕ ПРАВИЛЬНО!
Правильно будет так:
if (someVar == true) {
someVar2 = false;
}
Ну и естественно про табуляцию не забываем
(30 июл 2014, 14:05) (
0/
0)
[
0]
ebalazhabagadyuku, всё равно остались недочеты.
ЗЫ - всегда ставь фигурные скобки там, где они должны быть. Есть понятие как дизайн кода, который улучшает читабельность. Лучше соблюдать эти стандарты, если хочешь развиваться и возможно в будущем работать в команде.
(29 июл 2014, 13:48) (
0/
0)
[
0]
t,fkf;f,fufl.re,
Можно и так
Вообще, было бы хорошо встроить индикатор загрузки в механизм.
Т.е. где-то на странице мы имеем скрытый элемент типа такого:
<div id="ajax-loader" style="display:none"><img src="loader.gif"></div>
В js будет что-то вроде:
ajax({
loader_id: "loader",
...
});
А внутри функции перед отправкой запроса делаем видимым индикатор, по завершении display: none;
А callback'и пусть служат для обработки ответа или ошибки.
В таком случае можно выкинуть параметры response_container_id и error_container_id. Пусть всей обработкой информации и выводом занимаются callback'и.
Добавлено 29.07.14 в 17:28:40:
Как сделаешь новую версию выгружай сюда js файл. Я может че добавлю, и будет норм функция!
(29 июл 2014, 12:24) (
0/
0)
[
0]
ebalazhabagadyuku,
А что касается отдельного контейнера для ошибок, так это куда удобнее, чем в контейнер для контента выводить. Во-первых, ты каждый контейнер ты можешь навесить соответствующие стили (ошибки обычное делают красными, так чтобы они сразу бросались в глаза). Во-вторых, сообщение об ошибке может понадобиться организовать всплывающим диалогом или ещё как... вариантов может быть много.
Как вариант, я вижу возможность совместить оба варианта: в функции проверяешь, если контейнер для ошибки не задан, то используешь контейнер для содержимого.
(29 июл 2014, 12:09) (
0/
0)
[
0]
Hatsune Miku,
А твоя задача, как программиста, не выучивать всё подряд, а быть в курсе, знать, что есть и при необходимости применять. А насчёт мозга - уж поверь, он куда эффективнее сервера!
(29 июл 2014, 11:42) (
0/
0)
[
0]
ebalazhabagadyuku, да какая разница на что смахивает? Главное функционал и удобство. А передавать параметры объектом куда удобнее, чем руками формировать строку при каждом вызове функции. Формирование строки параметров тоже действие, которое придется повторять при каждом использовании функции! Но ведь смысл функции в том, чтобы избежать повторений кода.
Добавлено 29.07.14 в 12:07:58:
Ох и навели тут офтопа
Завязывайте, тема ещё не закрыта, а значит есть что обсудить ПО ТЕМЕ.
(28 июл 2014, 17:56) (
0/
0)
[
0]
На стороне клиента (js):
ajax({
url: 'some url',
method: 'POST',
resoinse_container_id: 'id1',
'error_container_id': 'id2',
vars: {
var1: 'value1',
var2: 'value2'
}
});
А на стороне сервера (php), я бы сделал по-другому...
(28 июл 2014, 17:04) (
0/
0)
[
0]
ebalazhabagadyuku, да не вопрос
Приятно поучаствовать в размышлениях других.
Добавлено 28.07.14 в 17:07:49:
Кстати, только сейчас обратил внимание, что параметры (args.vars) ты принимаешь строкой. Это не очень удобно, так как если в js например собирать данные из формы, то после их будет не удобно формировать в строку при вызове функции. Лучше будет принимать параметры объектом и уже в самой функции формировать из него строку.