VPF::Моя статья о создании ajax чата на php.

MoLeX

Дата 9.10.2009, 10:40 (ссылка)

(нет голосов)

Загрузка ...

Местный пингвин

Профиль
Группа: Модератор
Сообщений: 4076
Регистрация: 17.5.2007

Репутация: 46
Всего: 140

Цитата(nerezus @ 9.10.2009, 09:25

)

Код совершенно не понравился.
Комментировать код не умеем: додумался комментировать даже синтаксис, типа циклов.
много лишнего кода, например считывание из базы в массив.
Незнание синтаксиса SQL и обход незнания через array_reverse.

+

1. если у тебя и твой чат так построен то за что тебе деньги платят (за такой код мне лично жалко платить)?!
2. Если размещаешь статью, то размещай её на форуме

--------------------

Amazing

Elfet

Дата 12.10.2009, 17:10 (ссылка)

(голосов:2)

Загрузка ...

Белый и Пушистый

Профиль
Группа: Awaiting Authorisation
Сообщений: 3776
Регистрация: 2.4.2003

Репутация: нет
Всего: 16

Цитата(nerezus @ 9.10.2009, 10:25

)

много лишнего кода, например считывание из базы в массив.

Нет, совсем не лишний.

Цитата(nerezus @ 9.10.2009, 10:25

)

Незнание синтаксиса SQL и обход незнания через array_reverse.

Нет, синтаксис построен так как мне удобно было бы объяснить работу кода.

Цитата(nerezus @ 9.10.2009, 10:25

)

Не понравилась спичечная "типа оптимизация"(прирост ибо 0%) конкатенации строки в ухудшение читаемости.

"типа" я не понял про что вы?

Цитата(MoLeX @ 9.10.2009, 11:40

)

1. если у тебя и твой чат так построен то за что тебе деньги платят (за такой код мне лично жалко платить)?!

Мне платят за работу которую я проделал при проектировании, программировании и кодировании чата.

Цитата(MoLeX @ 9.10.2009, 11:40

)

2. Если размещаешь статью, то размещай её на форуме

MoLeX, скопировал.

--------------------

PHP Чат Скрипт

nerezus

Дата 12.10.2009, 18:27 (ссылка)

(голосов:2)

Загрузка ...

Вселенский отказник

Профиль
Группа: Участник
Сообщений: 3330
Регистрация: 15.6.2005

Репутация: 12
Всего: 43

Цитата

Нет, синтаксис построен так как мне удобно было бы объяснить работу кода.

"Ну и что, что код через задницу, так его объяснять удобнее?"
Неужели?

--------------------

Сообщество художников Artsociety.ru

Elfet

Дата 12.10.2009, 18:42 (ссылка)

(нет голосов)

Загрузка ...

Белый и Пушистый

Профиль
Группа: Awaiting Authorisation
Сообщений: 3776
Регистрация: 2.4.2003

Репутация: нет
Всего: 16

nerezus, я не вижу причин тому что бы называть мой код "через задницу".

--------------------

PHP Чат Скрипт

nerezus

Дата 12.10.2009, 19:15 (ссылка)

(голосов:2)

Загрузка ...

Вселенский отказник

Профиль
Группа: Участник
Сообщений: 3330
Регистрация: 15.6.2005

Репутация: 12
Всего: 43

google://ORDER BY

--------------------

Сообщество художников Artsociety.ru

bars80080

Дата 12.10.2009, 19:32 (ссылка)

(нет голосов)

Загрузка ...

прапор творюет

Награды: 1

Профиль
Группа: Завсегдатай
Сообщений: 12022
Регистрация: 5.12.2007
Где: Königsberg

Репутация: 71
Всего: 315

Цитата(nerezus @ 12.10.2009, 19:15

)

google://ORDER BY

наверно, что-то умное

Elfet

Дата 12.10.2009, 20:06 (ссылка)

(голосов:4)

Загрузка ...

Белый и Пушистый

Профиль
Группа: Awaiting Authorisation
Сообщений: 3776
Регистрация: 2.4.2003

Репутация: нет
Всего: 16

Вы что, неужели думаете что я не знаю что такое ASC и DESC?

По тому что вы увидели у меня в коде только DESC|array_reverse и не поняли зачем это, я могу судить о том что вы ещё мало разбираетесь в программировании вообще и в кодинге на php.

--------------------

PHP Чат Скрипт

solenko

Дата 12.10.2009, 20:12 (ссылка)

(голосов:4)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1473
Регистрация: 15.1.2006
Где: Украина

Репутация: 34
Всего: 67

Цитата(Elfet @ 12.10.2009, 19:06

)

По тому что вы увидели у меня в коде только DESC|array_reverse и не поняли зачем это

Я как бы тоже не понял. Поделитесь таинством?

Добавлено через 4 минуты и 10 секунд

Цитата(Elfet @ 7.10.2009, 15:38

)

Код


foreach ( $messages as $value )
                {
                        // продолжаем формировать скрипт для отправки пользователю
                        $js .= 'chat.append("<span>' . $value['name'] . '&raquo; ' . $value['text'] . '</span>");'; // добавить сообщние (<span>Имя &raquo; текст сообщения</span>) в наш div
                }

А зечем много раз дергать DOM дерево? Почему не собрать одну строку и не аппендить ее один раз?

--------------------

Ла-ла-ла-ла
Заметьте, нет официального подтверждения, что это не просто четыре слога.

Elfet

Дата 12.10.2009, 21:20 (ссылка)

(голосов:4)

Загрузка ...

Белый и Пушистый

Профиль
Группа: Awaiting Authorisation
Сообщений: 3776
Регистрация: 2.4.2003

Репутация: нет
Всего: 16

Цитата(solenko @ 12.10.2009, 21:12

)

А зечем много раз дергать DOM дерево? Почему не собрать одну строку и не аппендить ее один раз?

Тут согласен. Можно сделать и так, возможно это даже лучше когда у нас 10000 сообщений за раз приходит, однако больше 10 у меня за раз прийти не может, так что я думаю тут это не критично.

Цитата(solenko @ 12.10.2009, 21:12

)

Я как бы тоже не понял. Поделитесь таинством?

Дело в том что иначе усложняется SQL запрос для выбора последних N сообщений.

--------------------

PHP Чат Скрипт

solenko

Дата 13.10.2009, 00:53 (ссылка)

(голосов:5)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1473
Регистрация: 15.1.2006
Где: Украина

Репутация: 34
Всего: 67

Код


SELECT * FROM messages WHERE ( id > $last_message_id ) ORDER BY id DESC LIMIT 10

А что будет если пользователи будут писать более 10 ссобщений в 2 секунды? Пользователь никогда не увилит актуальное состояние чата? Зачем вообще нужел limit?

Цитата(Elfet @ 12.10.2009, 20:20

)

Дело в том что иначе усложняется SQL запрос для выбора последних N сообщений.

Я сразу и внимания то не обратил. Вопросы по этому поводу выше.

--------------------

Ла-ла-ла-ла
Заметьте, нет официального подтверждения, что это не просто четыре слога.

MoLeX

Дата 13.10.2009, 08:53 (ссылка)

(нет голосов)

Загрузка ...

Местный пингвин

Профиль
Группа: Модератор
Сообщений: 4076
Регистрация: 17.5.2007

Репутация: 46
Всего: 140

Цитата(solenko @ 13.10.2009, 00:53

)

точно. я даже и не обратил внимание.

--------------------

Amazing

Elfet

Дата 13.10.2009, 15:55 (ссылка)

(голосов:4)

Загрузка ...

Белый и Пушистый

Профиль
Группа: Awaiting Authorisation
Сообщений: 3776
Регистрация: 2.4.2003

Репутация: нет
Всего: 16

solenko, более 10 сообщений в 2 секунды - это уже спам, это не нужно добавлять в базу уже.

А LIMIT 10 нужен затем что бы при первой загрузке не показывать пользователю всю базу целиком, а только последние 10 сообщений.

--------------------

PHP Чат Скрипт

nerezus

Дата 13.10.2009, 20:55 (ссылка)

(голосов:1)

Загрузка ...

Вселенский отказник

Профиль
Группа: Участник
Сообщений: 3330
Регистрация: 15.6.2005

Репутация: 12
Всего: 43

Elfet,

Цитата

Дело в том что иначе усложняется SQL запрос для выбора последних N сообщений.

нисколько не усложняется.

Цитата

А LIMIT 10 нужен затем что бы при первой загрузке не показывать пользователю всю базу целиком, а только последние 10 сообщений.

Первый раз можно установить эту переменную в необходимое значение.

--------------------

Сообщество художников Artsociety.ru

Elfet

Дата 13.10.2009, 21:25 (ссылка)

(голосов:3)

Загрузка ...

Белый и Пушистый

Профиль
Группа: Awaiting Authorisation
Сообщений: 3776
Регистрация: 2.4.2003

Репутация: нет
Всего: 16

Цитата(nerezus @ 13.10.2009, 21:55

)

Первый раз можно установить эту переменную в необходимое значение.

В своей реализации я решил сделать именно так, так как мне кажется для объяснения принципов лучше, а нагружать код всякими излишними наворотами не нужно.

Добавлено через 2 минуты и 48 секунд
Используя DESC и LIMIT я убил двух зайцев - начальную инициализацию (загрузку последних 10 сообщений, получение id последнего сообщения) и подгрузку новых сообщений.

--------------------

PHP Чат Скрипт

solenko

Дата 14.10.2009, 09:39 (ссылка)

(голосов:2)

Загрузка ...

Эксперт

Профиль
Группа: Завсегдатай
Сообщений: 1473
Регистрация: 15.1.2006
Где: Украина

Репутация: 34
Всего: 67

Цитата(Elfet @ 13.10.2009, 14:55

)

solenko, более 10 сообщений в 2 секунды - это уже спам, это не нужно добавлять в базу уже.

1. А цифры вы инженерно-потолочным способом определяли?
2. Даже если это спам, то данные все равно будут вставлятся в базу, а вот полезное сообщение, полученное после него, пользователь не увидит
3. Вы или не застал эпоху чатов, либо просто никогда не бывали на популярном чате. Такое количество сообщений -- мелочи

Добавлено через 2 минуты и 48 секунд

Цитата(Elfet @ 13.10.2009, 20:25

)

В своей реализации я решил сделать именно так, так как мне кажется для объяснения принципов лучше, а нагружать код всякими излишними наворотами не нужно.

Как раз таки для учебного процесса лучше объяснить, что инициализация чата и получение сообщений -- разные подзадачи

Добавлено через 3 минуты и 58 секунд

Цитата(Elfet @ 13.10.2009, 20:25

)

Используя DESC и LIMIT я убил двух зайцев - начальную инициализацию (загрузку последних 10 сообщений, получение id последнего сообщения) и подгрузку новых сообщений.

нет, вы убили других двух зайцев: внесли ненужные ограничения на функционал и сделали код нелогичным

--------------------

Ла-ла-ла-ла
Заметьте, нет официального подтверждения, что это не просто четыре слога.

0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| PHP: Общие вопросы \| Следующая тема »