>>438292

О, хорошо что ты написал. Я собирался посмотреть твой проект, но все откладывал на потом.

Вообще, уровень проекта довольно высокий, я вижу что ты разобрался и с асинхронной работой с сокетами, и с протоколами HTTP/Websocket, и с ООП.

Вот советы и замечания по коду.

Не вижу где в репозитории конфигурация для mongoDB. Например, где определяются индексы? Где создаются коллекции?

У тебя немного странно разложены файлы по папкам. Обычно делают одну папку для сущностей (вроде RoomEvent, RoomUser), другую для мапперов, и т.д. У тебя же как-то все получилось разбросано по разным папкам.

Не очень понятно зачем наследовать RatchetServer от Pimple. Обычно наследование используют чтобы расширить наследуемый класс, но RatchetServer это явно не расширение для DI контейнера. Возможно, ты копировал схему котору. использует Silex, но мне кажется это неудачная идея.

Далее, напрягает то, что у тебя там по факту 2 контейнера и приходится определять сервисы по 2 раза: в RatchetServer, и в Webapp. Гораздо лучше если бы и websocket сервер и приложение разделяли общий контейнер. Вполне можно использовать тот, что в Webapp.

Далее, про реализацию ODM (маппинга сущностей на объекты в MongoDB). Обычно в мапперах реализуют подход IdentityMap: http://design-pattern.ru/patterns/identity-map.html

При этом подходе одной сущности всегда соответствует ровно один объект. Если мы 2 раза пытаемся загрузить пользователя с одним и тем же id, то мы получаем тот же самый объект. При этом если ты хочешь реализовать IdentityMap сам, ты натолкнешься на утечку памяти: так как объекты только добавляются туда и никогда не удаляются, эта структура начинает неограниченно расти в памяти. При этом удалять объекты ты не можешь так как не знаешь, исользуются ли они сейчас где-то или нет. Решением этой проблемы является WeakMap — специальная структура, которая позволяет сборщику мусора собирать объекты из нее, если они больше нигде не используются: http://php.net/manual/ru/book.weakref.php

WeakMap давно уже есть в других языках, вроде Java, и скоро появится в браузерах: https://developer.mozilla.org/ru/docs/Web/JavaScript/Reference/Global_Objects/WeakMap

Если WeakMap недоступен, то придется использовать другие решения, вроде явного удаления сущностей из IdentityMap.

Как альтернатива, можно попробовать применить готовый ODM вроде https://github.com/doctrine/mongodb-odm

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

Класс RoomEvent наверно стоило сделать абстрактным.

Далее, не очень понятно по каким принципам распределен код по классам RatchetServer, Chat, RoomUserHandler, RoomEventHandler, такое ощущение что они примерно одним и тем же занимаются. Было бы логично оставить RatchetServer работу с соединениями, а всю логику поместить например в Chat. Зачем вообще нужны Handler-классы?

В приложениях обычно делают слой сервисов, который находится на уровень выше мапперов ( http://martinfowler.com/eaaCatalog/serviceLayer.html ), но в твоем случае получается 2 уровня: Handler и Chat, и один из уровней тут явно лишний. То есть лучше сделать 1 или несколько классов-сервисов, без Handler классов.

> public function leftUserFromRoom(User $user, $roomName)

> public function removeUserFromRoom(User $user, $roomName)

Названия очень неудачные, прямо следуют совету отсюда: http://learn.javascript.ru/write-unmain-code#хитрые-синонимы

Насчет проблемы с заходом пользователя с нескольких устройств или вкладок, тебе надо вместо пользователя ввести понятие «соединения». Это относится и к удалению пользователя из комнаты (надо удалять соединение) и к untrackUser в UserHeartbeat.

> https://github.com/d-beekeeper/ratchat/blob/master/app/User/UserRegister.php#L39

> public function __construct(Webapp $app,$username,$password,$checkPass)

Очень странно что в конструктор передаются имя и пароль. Наверно логичнее сделать класс-сервис для регистрации пользователей, а не класс для регистрации единственного пользователя. Тем более что в функцию registerUser(User $user) ты и так передаешь объект-пользователя.

> https://github.com/d-beekeeper/ratchat/blob/master/app/ADataMapper.php#L32

> protected function convertId($_id)

Это очень странная функция. Ее не должно быть и она выглядит как костыль. Потому что создается ощущение что ты сам не знаешь в каком формате у тебя хранится id. нужно сделать, чтобы он везде хранился в единообразном виде, а не как попало. Особенно вот этот код шедеврально выглядит:

> if (!\MongoId::isValid($_id)) {

> $_id = substr(md5($_id),0,24);

Это какой-то кривой костыль, явно.

То же самое и с датой. Ты можешь делать преобразование id и дат в маппере при сохранении/загрузке: маппер для этого и предназначен вообще-то.

> @param RoomEvent|RoomEvent[] $roomsEvents

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

> $roomUser->user = $this->cleanupUserInfo($roomUser->user);

Это (фильтрацию аттрибутов) надо делать на этапе экспорта из объектов в массив (для кодирования в JSON). Ты даже можешь сделать у сущности метод вроде getPublicData() или делать фильтрацию снаружи сущности.

Также, получается не очень хорошая ситуация: ты в BackendToFrontendMessage при преобразовании в JSON удаляешь часть полей и у пользователей, а потом восстанавливаешь из JSON. В итоге у нас в приложении ходит 2 набора сущностей: полноценные и с отрезанными полями. Это провоцирует баги.

> https://github.com/d-beekeeper/ratchat/blob/master/app/BackendToFrontendMessage.php#L56

> public function fromJson($jsonString)

Обычно такие функции делают статическими

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

> https://github.com/d-beekeeper/ratchat/blob/master/app/User/User.php#L73

> if (isset($this->username)) $attrs['username'] = $this->username;

Не очень понятно, почему ты прововеряешь равенство поля null через isset, и не очень понятно почему такие поля не попадают в массив. В чем удобство того, что мы имеем массив с неизвестным числом полей?

> public function receiveByAttributes(User $user)

Тоже странная функция. Как я понимаю, она используется для проверки уникальности имени, ну так и надо сделать функцию isUsernameUnique

> https://github.com/d-beekeeper/ratchat/blob/master/app/VkAuth/VkApi.php#L43

тут наверно returnUrl надо пропустить через urlencode

>>438292

Вместо curl в VkApi удобнее использовать нормальный ООП HTTP клиент, например Guzzle, это еще и сделает код гибким и тестируемым, позволив передать вместо Guzzle заглушку, и даст готовые объекты Request/Response (кстати на Request/Response наконец-то готовится стандарт: https://github.com/php-fig/fig-standards/blob/master/proposed/http-message.md ).

> $decoded = json_decode($string,true);

> if (is_null($decoded) || json_last_error() !== JSON_ERROR_NONE) {

При ошибке до if управление скорее всего не дойдет так как json_decode выдает Warning который некоторые фреймворки (не знаю как насчет Silex) превратят в ErrorEcxeption. Если ты хочешь обрабатывать ошибку сам придется поставить @. Это конечно бардак, система ошибок в PHP.

> https://github.com/d-beekeeper/ratchat/blob/master/app/VkAuth/VkAuthProvider.php#L40

> throw new AuthenticationException(...);

Здесь ты теряешь информацию из пойманного исключения. Чтобы этого не было, обычно исключения разрешают передать «предыдущее» исключение в конструктор. В доументации легко увидеть соответствующий параметр: http://api.symfony.com/2.1/Symfony/Component/Security/Core/Exception/AuthenticationException.html

> public function receiveByAttributes(User $user)

Тоже не очень удачно сделана функция, лучше явно передавать массив условий.

> TODO следующий блок нужно вынести в отдельный скрипт, исполняемый кроном раз в N секунд, где N < UserHeartbeater::MAX_USER_DEAD_TIME/2

Вообще, ты мог бы засунуть это в server.php, он же асинхронный и он может делать действие раз в N секунд. И запускать в отдельном процессе, чтобы он не блокировал обслуживание вебсокетов. Тем более что у крона минимальная периодичность запуска минута.

А сейчас у тебя если никто не обновит страницу чата, старые сообщения не удалятся.

> https://github.com/d-beekeeper/ratchat/blob/master/web/index.php#L63

> $app->match('/chat',function (Request $request) use($app) {

тут что-то многовато кода. Хоть бы по функциям разбил.

> https://github.com/d-beekeeper/ratchat/blob/master/views/chat.twig#L66

> <div id="chat-message-template" style="display: none;">

Шаблоны гораздо удобнее и семантичнее хранить в теге скрипт: http://learn.javascript.ru/templates#хранение-шаблона-в-документе

Более того, ты бы мог использовать стандартный шаблонизатор, тысячи их:

- JsRender http://habrahabr.ru/post/148573/
- handlebars
- другие, тысяи их: http://garann.github.io/template-chooser/

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

Более того, у тебя есть дублирование шаблонов:

https://github.com/d-beekeeper/ratchat/blob/master/views/_messages.twig#L3
https://github.com/d-beekeeper/ratchat/blob/master/views/chat.twig#L67

От него можно избавиться за счет макросов twig.

> <script src="http://autobahn.s3.amazonaws.com/js/autobahn.min.js">

> https://oss.maxcdn.com/html5shiv/3.7.2/html5shiv.min.js

Не советую так делать. Этим ты только повышаешь даунтайм приложения. Храни скрипты на своем сервере, не бери пример с js-хипстеров.

> {{ app.security.token.user.username }}

> app.session.getFlashBag.get('vk_error')

Стоит явно передавать это в шаблонизатор, а то шаблонизатор очень глубоко в недра приложения лезет.

> https://github.com/d-beekeeper/ratchat/blob/master/views/chat.twig#L21

Что за странный <a> без href?

> <meta name="viewport" content="width=device-width, initial-scale=1">

Это ты скопировал или у тебя правда адаптивный дизайн?

> options.chat.find('#chat-message-template')

Вместо того чтобы передавать словарь с одим параметром, лучше сразу передавать chat в конструктор. Вообще, options имеет смысл когда опций куча и большинство не обязательны, и это по моему не очень хороший подход. Тут например можно опечататься и никакого сообщения об ошибке не будет.

> options.chat.data('eventSequenceNum'),

Вообще тут наверно проще без data-аттрибутов обойтись и передать число прямо в конструктор или метод setSequenceNum.

> Если не удалось установить ws connection, или он поработал и разорвался - делаем fallback к ajax transport

Мне кажется, если вебсокет отвалился, но соединение возможно, лучше повторно соединяться через WS. Отваливающиеся соединения — обычное дело.

У тебя на клиенте есть какое-то ограничение на число отображаемых сообщений? Браузеры тормозят при большом числе DOM нод. Например, нет особого смысла отображать все сообщения, достаточно последних N, и в неактивных комнатах тоже, а добавлять новые можно при прокрутке. Посмотри, как например скайп это делает (он отображает сообщения внутри WebView, встроенного браузера, если я ничего не путаю).

> alert('Ошибка: ' + value);

alert очень плохой с точки зрения юзабилити.

> this.chatRooms[event.chatRoom].addMessages

Нет проверки что такая комната есть.

> this._$sendbtns.attr('disabled','disabled');

disabled надо ставить через prop(), так как это свойство. Атрибут лишь задает его начальное значение. Не путай свойства и атрибуты: http://stackoverflow.com/questions/5874652/prop-vs-attr

> Chat.prototype._enableControlBtns = function () {

> this._$roomAddBtn.click(function () {

удалять/добавлять события — признак плохой архитектуры. Используй disabled или переменную-флаг лучше.

> if (code==2 || code==3) {

Там конастант нет что ли? Что за магические числа?

> https://github.com/d-beekeeper/ratchat/blob/master/web/js/chat.js#L290

> if ($(this).queue().length == 0)

Это вообще где-то в документации описано? Почему ты в объект jQuery передаешь непонятно что? Если ты хочешь исплоьзовать очередь из jQuery лучше нормально создать отдельный объект через this.queue = $().queue( );

> $messageTemplate.find('a.msg-template-avatar-link').attr('href',avatar);

Тут явно стоит использовать шаблонизатор

В общем, такой вот набор советов по улучшению. Но в общем, код на очень хорошем уровне.

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

Большие компании делают такие тесты и иногда даже изобретают свои инструменты: http://habrahabr.ru/company/yandex/blog/202020/

Для простых ситуаций вполне может хватить простых инструментов вроде ab или siege. ab (Apache Benchmark) просто шлет указанное число раз в указанное число потоков один и тот же запрос. Он примитивный, но позволяет оценить скорость обработки запросов. siege сложнее, он может слать запросы на разные URL по заранее составленному списку, имитируя действия разных пользователей. Также, ты можешь отправлять запросы например своим скриптом на php c http клиентом вроде guzzle.

Есть еще относительно мощный Apache JMeter, но с ним я не работал: http://habrahabr.ru/post/140310/

Искать узкие места в скрипте можно с помощью профайлера, входящего в xdebug. Профайлинг можно запускать как для веб-приложения, так и для Cli скрипта при передаче определенных параметров. Файл с результатами можно просмотреть через программы вроде KCacheGrind или WinCacheGrind.

С помощью нагрузочного тестирования можно заметить проблемы при обработке большого числа соединений или сообщений: увеличение времени ответа, увеличение потребления памяти, падения приложения, рост числа ошибок.

Вот что, по моему, стоит протестировать:

- изменение времени добавления сообщения по мере их накопления. Просто сделай скрипт который отправляет сообщения в чат и меряй время обработки запроса. Например, время на отправку 1, 10, 100, 1000, 10000, 100000 запросов, а потом посмотри, нет ли деградации по мере накопления сообщений

- то же самое но с N подключенными слушателями

- проверку на максимальное чило клиентов (не уверен что это можно сделать стандартными средствами). Соединяешься с чатом с N клиентов, и смотришь как меняется время ответа с ростом числа клиентов и не упадет ли приложение

- то же самое, но каждый клиент отправляет 1 сообщение

- время загрузки страницы с чатом при наличии N сообщений в чате

- также, можно померять производительность ajax- и POST-транспорта, но это не так критично, так как мы предполагаем что использовать будет вебсокет.

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

Кстати, как у тебя с очисткой старых сообщений? Они когда-нибудь очищаются? Или производительность по мере их накопления деградирует?

Ну и вторая вещь, которой можно было бы заняться, это покрыть приложение тестами. Вот например мой обзорный урок про виды тестирования: https://gist.github.com/codedokode/a455bde7d0748c0a351a Почитай. может интересно станет с этим разобраться.

>>438319

Ты можешь выставлять заголовки (и ставить куки) пока не начал что-то выводить. Если начал то поздно так как то что ты выводишь идет в теле ответа, а заголовки идут перед телом.

>>438437

Вот ты бы лучше это безобразие исправил:

> ($name, $attackLow, $attackHigh, $health, $armorLow, $armorHigh, $block, $dodgeCrit, $crit, $charisma, $agility, $skill)

Запутаться же проще простого

> construct($heroFirst, $heroSecond)

Тут нужен тайп-хинт: http://php.net/manual/ru/language.oop5.typehinting.php

> global $phrasesDodge;

Глобальные переменные это плохо. Лучше сделать в Game метод getRandomPhrase(). Константы не нужны так как они нигде не используются, а выбрать случайный элемент можно и без них.

> можно ли делать так как я сделал в function fillNotFilledStrings()?

Вообще, плохой код. Зачем ты в классе Game снаружи меняешь свойтва героев. если можно просто сделать методы у героев и поместить код туда?

Этот код вообще абсолютно неправильный:

> $this->heroes[0]->doubleHit = $charAbs;

Шанс на двойной удар это не свойство героя так как оно зависит от соперника. Значит оно должно вычисляться в методе а не храниться в свойстве.

> $this->heroes[0]->hitChance = round(($this->heroes[0]->skill

надо сделать метод $hero->getHitChance вместо этого

> $this->heroes[0]->enemyBlock = $this->heroes[1]->block;

Это вообще бессмысленный код. Зачем копировать свойство если мы всегда можем взять его из второго героя?

Вообще, странная идея хранить в герое свойства другого героя. Бред же. А если драка будет 3 на 3, ты в герое будешь свойства каждого соперника хранить?

Функцию fillNotFilledStrings() надо удалить и лишние свойства тоже удалить и сделать методы в классе Hero вместо этого.

> private function makeOneHit($hero)

Почему передается только один герой? Он сам с собой дерется что ли? Надо сделать чтобы передавалось 2 героя, кто с кем дерется.

> if ($this->heroes[0]->health < $secondDamage) {

для этого лучше сделать метод. Более того, вот это вот heroes[0] смотрится плохо. Нельзя циклом обойти, раз уж у тебя массив?

> return 1;

Что это за магические числа? Непонятно что это значит? Кто победил? Тогда логичнее возвращать не число.а сам объект и null в случае ничьи.

> мощный рубящий удар, цель удара чудом избежала чудовищного удара.

Что-то тут у тебя слово «удар» 3 раза повторяется. Лучше бы избегать таких повторов, а то как-то не очень хорошо смотрится текст.

> if ($this->heroes[0]->health < $secondDamage) {

> $this->heroes[0]->health = 1;

> $this->heroes[0]->health -= $secondDamage;

Лучше сделать у героя методы takeDamage($damage) и isAlive() Ты же вместо того чтобы положить относящийся к герою код в его класс, разимазываешь его равномерно по всей программе. В ООП объект это набор данных + методы для управления ими. А ты не хочешь делать методы, а исплоьзуешь объект как массив, зря.

Вообще, чтобы ты лучше прочуствовал эту концепцию, сделай свойства закрытыми, то есть не public, а private и используй методы для доступа к ним и их изменения.

>>438676

path to tmplate тоже стоит передавать через аргументы.

>>438735

iconv

>>438830

Ладно, пусть так будет, хотя мне хак с font-size не особо нравится. Хорошие варианты по моему это вариант с комментарием и незакрытые теги. Главное чтобы ты знал про эту особенность CSS.

>>438846

А зачем? Пока юзер не нажал на кнопку он может все 10 раз поменять и отредактирвоать.

>>438852

Это плохая идея так как форма не будет отправляться по нажатию Enter. Надо исплоьзовать событие submit и предотвращать действие по умолчанию в обработчике (preventDefault/returnValue = false).

> если нужно знать результат отправки на .onreadystatechange указываешь колбек функцию

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

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

>>438861

А как ты поймешь что оно заполнено и что пользователь ничего не будет менять? по моему это неудачная идея.

А так, уход из поля это событие blur, если ты его не знаешь то изучи какие вообще события есть:

https://developer.mozilla.org/en-US/docs/Web/Events (англ, подробно)
http://javascript.ru/tutorial/events

>>438860

Код по моему с ошибкой. Ты после битвы не восстанавливаешь состояние героев, в частности здоровье. Правильным решением будет клонировать героев и драться каждый раз новыми клонами (мануал: http://php.net/manual/ru/language.oop5.cloning.php ).

>>438907

Не считается.

> как прочитать его с помощью js?

Тебе надо почитать учебник для начала: http://learn.javascript.ru/form-elements

>>438931

Можно настроить дропбокс папку на 1 уровень выше чем папка сервера наверно. Либо сделать git репозиторий в папке дропбокса и пушить в него периодически.

>>438987

На гитхаб. Чтобы им пользоватьяс, надо освоить гит, есть отличная книга: http://git-scm.com/book/ru/v2

Если что то непонятно задавай вопросы.

Если не работал с командной строкой, прочти мой урок: https://gist.github.com/codedokode/10539568

Пока не освоил гит, файлы можно выложить сюда gist.github.com, рабоатет перетаскивание файлов с рабочего стола.

Чтобы работало, тебе надо найти бесплатный хостинг и выложить свой php код туда. Вот список хостингов: http://www.google.ru/search?sourceid=chrome&ie=UTF-8&q=free+php+hosting

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

Например www.000webhost.com без рекламы.

>>438987

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

>>439003

> помощь мне нужна даже в простейших заданиях.

Наш тред для этого и сделан

Решение почти верное, но нет проверки на ничью. Ведь может выпасть одинаковое число баллов.

exit во втором if лишний, работать будет без него.

?> в конце можно и нужно не писать

>>439004

Почему, codepad вполне нормальный, дизайн у него лучше чем ideone, хотя там по моему чего-то не хватает из расширений.

>>439005

if принято писать с маленькой буквы (с большой пишут константы и имена классов, что это такое ты потом узнаешь). exit в строке 26 лишний, можно без него. Нет проверки на ничью.

>>439011

Команды в скрипте выполняются по очереди, сверху вниз, только один раз. Чтобы они закольцевались, надо сделать цикл, об этом ты узнаешь в следующих уроках.

>>439024

> что бы можно было ввести "iv 49"

> и две подстроки искались бы отдельно во всех столбцах?

Можно разбаваить фразу процентами:

'%iv%49%'

И искать в строке склеееной из всех колонок:

WHERE CONCAT(name, ' ', score) LIKE ?

Но это не найдет слова если они идут в другом порядке (49 iv). Чтобы искать в любом порядке, нужны более сложные подходы, например full text search :

http://dev.mysql.com/doc/refman/5.0/en/fulltext-search.html (англ)
http://www.mysql.ru/docs/man/Fulltext_Search.html (рус, кратко)

или внешние поисковые движки вроде sphinx: https://gist.github.com/codedokode/10539366

В этой задаче достаточно использовать что-нибудь на основе LIKE но если тыхочешь освоить еще и sphinx то я не против.

>>439049

> как правильно делать разлогинивание пользователя через сессии? Не просто же ансетать все переменные которые ты клал в этот массив во время логина?

Почему, unset вполне пойдет, а также там есть session_destroy для большей надежности.

> Вот при удачном логине я начинаю сессию и кладу в массив $_SESSION два элемента.

сессия удаляется при 20-30 минутах неактивности. Это неудобно и тебе придется дополнительно что-то класть в куки дял автоматического перезалогинвания. Сессия не постоянное, а временное хранилище.

> $login = mysqli_real_escape_string($link, $_POST['username']);

Научился бы уже плейсхолдерами пользоваться: http://php.net/manual/ru/mysqli.quickstart.prepared-statements.php

Ну и лучше исплоьзовать не процедурную, а ООП-версию mysqli, так проще будет PDO и другие библиотеки потом освоить.

>>439050

задавай вопросы. Это тред для начинающих.

>>439054

Все верно, только ?> в конце ставить не стоит.

>>439221

По идее начинать стоит с мануала, но в мануале php тема освещена плохо: http://php.net/manual/ru/book.session.php

> http://phpfaq.ru/sessions#use

тут вроде нормальное описание

> http://www.softtime.ru/bookphp/gl8_1.php

> Учебник по PHP 4

Ой-ой, какой старый, хотя про сессии информация в принципе не устарела

> http://habrahabr.ru/post/182352/

Это очень странная статья, там даже в комментах указывают. Этот код

> Контроль отсутствия активности пользователя встроенными средствами PHP

> ini_set('session.gc_maxlifetime', $sessionLifetime);

Скорее всего нерабочий, так как очистка сессий может выполняться крон-скриптом как это делается в дебиане и ini set никак на него не влияет.

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

>>439218

> И кто из вас ОП?

Оба поста мои

>>439220

> <!DOCTYPE html>

> session_start( );

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

К тому же смешивать в одном файле html и php код плохая идея: http://www.phpinfo.su/articles/practice/shablony_v_php.html

>>438085

ИТТ мы будем говнокодить на самом поганом и в то же время самом популярном языке для веб-разработки на стороне сервера. Юзаем LAMP или WAMP стек, по умолчанию.

Essential Links

Официальный сайт:
http://php.net

Вики-ресурсы о сабже:
http://en.wikipedia.org/wiki/Php
http://ru.wikipedia.org/wiki/Php
http://lurkmore.to/PHP

PHP – The Right Way (лучшие практики)
http://phptherightway.com/

The League of Extraordinary Packages is a group of developers who have banded together to build solid, well tested PHP packages using modern coding standards.
http://thephpleague.com/

Russian-language resources

http://php.net/manual/ru/ – руководство по PHP на русском. Включая FAQ.
http://php.ru – русскоязычное сообщество и форум.
http://phpclub.ru – еще одно быдлокодерское сообщество и форум.
http://softtime.ru/bookphp/gl8_1.php – учебник по PHP 4 – старый конечно, но как справочник функций с примерами может пригодиться.
http://ruseller.com/ – коллекция относительно качественных материалов для веб-мастеров.

Books

Уже порядком устаревшие, но тем не менее культовые обучающие книги по PHP для новичков:
1) PHP 5 для чайников – Джанет Валейд (2005) [Диалектика]
2) PHP5 в подлиннике (2-е издание) – Котеров, Костарев (2008) [bhv]
И сотни других.

Костыли для PHP

Разработка многозадачных приложений на PHP5 (2008)
http://ibm.com/developerworks/ru/library/os-php-multitask/

Evolving PHP without Waiting for New Versions (2013)
В статье автор приводит большой список возможностей, которых не хватает в PHP (26 пунктов). Для каждой такой возможности приводит ссылки на существующие реализации.
http://phpclasses.org/blog/post/203-26-Ways-to-Show-that-PHP-Can-Be-Better-Than-PHP.html

Libs, classes

Крупный агрегатор PHP-классов, библиотек и прочего добра.
http://phpclasses.org/browse/

Список полезных PHP классов и библиотек (2009)
http://habrahabr.ru/post/70808/

Awesome PHP – огромный список отличных библиотек и утилит, а также ресурсов по PHP на все случаи жизни. Есть также awesome-python и awesome-ruby.
https://github.com/ziadoz/awesome-php

Pear
Ныне мертвый стандартизованный набор библиотек PEAR (PHP Extension and Application Repository).
http://ru.wikipedia.org/wiki/PEAR

Composer
Dependency Manager for PHP.
https://getcomposer.org/

>>439474

Ты можешь сделать то что я писал выше то есть поставить var_dump($_FILES); в начале скрипта и показать нам что он выводит? Твое описание к сожалению не позволяет установить причину проблемы.

>>439470

> Остальное используется редко

дело даже не в том что редко, а в том, что ты можешь этого не знать. Ты используешь Doctrine 2 например но не знаешь что она использует внутри паттерны Data Mapper, Identity Map, lazy Loading и другие (хотя конечно это плохо если не знаешь).

>>439453

> А когда начинаешь пердолить бложек или ресурс с богатой админкой/лк, паттерны тоже не нужны?

Такие вещи никто не пишет с нуля а берут фреймворк или CMS на худой конец и делают как там принято и как описано в документации.

> И где всё это брать после начального этапа?

У нас есть неплохие практические задаки которые учат писать простые приложения с базой данных, с ООП/MVC. Они в ОП посте есть.

>>439429

А зачем ты используешь win 1251? Странная идея конечно.

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

SET NAMES делаешь и если делаешь то какой?

Это читал?

http://phpfaq.ru/charset
http://www.tmanager.ru/russian/charset.html
http://fstrange.ru/coder/mysql/kodirovka-krakozyably.html
http://gahcep.github.io/blog/2013/01/05/mysql-utf8/

>>439847

Ты делаешь неправильно. Вот посмотри на этот код: https://ideone.com/IRcAww

Ты вставляешь кусок html прямо после вызова фреймворка. Получается у тебя для любых страниц выводится один и тот же html? Все страницуы будут одинаковые?

Более того, плохо вообще в одном файле смешивать php код и html шаблон.

Фреймворк slim предоставляет микрошаблонизатор для решения этой проблемы. Фактически этот микрошаблонизатор просто подключает указанный php файл и передает в него переменные.

Почитай документацию (если ты не понимаешь английский хотя бы примеры кода посмотри и попробуй перевод через bing translate почитать: http://www.bing.com/translator/ - там часть текста понять можно).

Там есть раздел view: http://docs.slimframework.com/#View-Overview

Там есть пример кода где видно как используется метод $app->render('myTemplate.php' ....)

Ты должен использовать его если хочешь вывести страницу. Вызывается app->render обычно изнутри обработчика в $app->get().

А твой код в принципе нерабочий. Например вот это:

> header("Pragma: public");

не будет работать так как перед этим Слим выведет страницу 404.

Ну и с самими заголовками перебор, там большинство их них не нужны. Реально нужно только 2:

header("Content-Disposition: attachment");
header("Content-Type: image/jpeg"); // не jpg

Также в заголовках еще ошибки:

- в Content-Type надо указывать не расширение, а MIME-тип вроде image/jpeg. MIME типы регистриурет организация под названием IANA, так что это что-то вроде стандарта. Вот список MIME типов: https://ru.wikipedia.org/wiki/%D0%A1%D0%BF%D0%B8%D1%81%D0%BE%D0%BA_MIME-%D1%82%D0%B8%D0%BF%D0%BE%D0%B2

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

- не используй параметр filename в заголовке content-disposition. Для него не определена кодировка (точнее определено стандартом что нельзя использовать ничего кроме латиннциы) и разные бруаузеры воспринимают его по-разному.

Вот статьи про то, какие вообще заголовки бывают:

http://en.wikipedia.org/wiki/List_of_HTTP_header_fields (англ)
https://ru.wikipedia.org/wiki/%D0%A1%D0%BF%D0%B8%D1%81%D0%BE%D0%BA_%D0%B7%D0%B0%D0%B3%D0%BE%D0%BB%D0%BE%D0%B2%D0%BA%D0%BE%D0%B2_HTTP

Заголовки это часть протокола HTTP. Вот статья про сам протокол HTTP если ты его не очень знаешь:
http://scriptsite.ru/article/show/5/

Это был разбор кода. Теперь вернемся к твоему вопросу.

> Из $app->get вообще можно что то получить?

> Мне нужно в хтмл использовать $file, но как её достать?

Ты вызываешь app->render и передаешь туда все переменные, которые нужны.

Если есть еще какие-то вопросы по Слиму или вообще, уточняй.

Вообще, со Слимом твое приложение (точнее файл index.php) должен выглядеть примерно так:

подключение Слима;

// объявляем сервисы (синглтоны слима)
$app->container->singleton ....
$app->container->singleton ....

// задаем функции обработчики для разных видов запросов
$app->get ...
$app->get ...
$app->post ...

// запускаем приложение
$app->run( );

Если ты знаком с MVC то у нас фактически тут контроллеры пишутся внутри app->get и app->post. Напомню что app->get задает функию-обработчик для определенных GET запросов, а post для POST.

>>439878

А еще, ты делал задание про студентов? Или NotBadCode это как раз ты? Я постараюсь сегодня его все же проверить.

>>439887

Да, это не круто. HTML код должен быть в шаблонах.

>>439878

Это был лишь пример на практике используется app->render.

>>439877

ссылкой c id поста в ней например, вида /posts/edit/123456

>>439847

Еще нехороший момент: зачем ты пишешь присваивание внутри return?

> return $file=$mapper->showFilebyID($id);

надо писать в 2 команды как

$file=$mapper->showFilebyID($id);
return $file;

Но если подумать то какой смысл создавать переменную file? Не логичнее ли сразу написать

return $mapper->showFilebyID($id);

Ты случайно не думаешь что в return $file = ... имя переменной file на что-то влияет? return просто возвращает значение, неважно как названа переменная (и есть ли там переменная вообще). Он возвраащет число, строку или что ты там ему передашь, а имя переменной не важно.

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

>>439323

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

Советы и замечания по коду.

https://github.com/NotBadCode/Students

> `place` int(2) NOT NULL,

> `sex` int(1) NOT NULL,

Тут лучше подойдет ENUM. Я например смотрю на SQL код и не могу понять что значит 0 или 1. И другой человек, который захочет разобрать твой код, тоже не поймет без анализа кода.

В сложных случаях в SQL ты можешь добавлять комментарии к колонкам или всей таблице. Эти комментарии сохраняются в базе и отображаются в программах для работы с БД. Используй их: http://stackoverflow.com/a/200033

> `groupnum` int(5) NOT NULL,

Лучше номер группы сделать строкой. А то как ты будешь сохранять номер 00010?

> `year` int(4) NOT NULL,

Есть тип YEAR и догадайся-ка, для чего он придуман?

Я вижу ты плохо знаешь типы данных. Почитай (учить наизусть не надо, но ответ на вопрос «в чем хранить дату» или «в чем хранить номер телефона» или «в чем хранить сумму с копейками» знать надо):

http://phpclub.ru/mysql/doc/column-types.html
http://artemfedorov.ru/etc/mysql/field-types/

Также, если в таблице некоторые колонки не должны иметь повторы, добавь к ним уникальный ключ (UNIQUE KEY).

> https://github.com/NotBadCode/Students/blob/master/index.php#L3

После <?php не надо добавлять отступ (убрать оступ в большинстве редакторов можно выделив все и нажав Shift + Tab)

Также, я советую поставить в редакторе настройку «заменять таб на 4 пробела», а то смотри как у тебя код расползся на гитхабе некрасиво.

> $id=$_COOKIE['studentscookie']['id'];

Это ненадежно. Злоумышленник может подставить себе любой id (куки ведь хранятся в браузере) и редактировать чужую страницу. Чтобы он этого не мог сделать, надо вместо предсказуемого id использовать сложный случайный код, который генерируется при регистрации (у каждого студента он свой и злоумышленник его не знает). 32 или больше символа из диапазона [a-z0-9] вполне подойдут.

> if($mapper->emailUsed($_POST['email']))

Имя функции начинается с глагола, например isEmailUsed. Также, не забывай всегда ставить {} после if.

> https://github.com/NotBadCode/Students/blob/master/index.php#L13

> https://github.com/NotBadCode/Students/blob/master/index.php#L27

У тебя тут 2 практически одинаковых ветки кода (редактирование и добавление). Попробуй объединить их в одну.

> https://github.com/NotBadCode/Students/blob/master/index.php#L37

> header("Location: index.php");

После header надо делать die так как нет смысла продолжать выполнения скрипта и что-то выводить: браузер это все равно не отобразит.

> https://github.com/NotBadCode/Students/blob/master/list.php#L5

> if(!isset($_COOKIE['studentscookie']['id'])){

То же самое, после header надо делать die. Ну и по условиям задачи список можно смотреть без регистрации. Ну и ты не должен молча пересылать пользователя на другую страницу. Ты должен в такой ситуации выводить там сообщение вроде «Список могут просматривать только зарегистрированные пользователи. Зарегистрируйтесь, чтобы полуить к нему доступ.»

Кстати, после обновления данных/регистрации надо выводить соответствующее сообщение. А то если форма молча перезагружается то не понятно — все в порядке или это ошибка какая-то?

> if(isset($_POST['submitsearch'])){

Для поиска (и других действий не меняющих данные на сервере) используют GET. Это позволяет например переслать ссылку на результаты поиска. Посмотри например как выглядит ссылка если зайти на http://ya.ru и что-нибудь поискать.

Аналогично, сортировка и постраничная навигация (она у тебя есть?) делается GET-ссылками.

> https://github.com/NotBadCode/Students/blob/master/lib/login.php

Это принято называть config.php или settings.php.

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L9

> class DataMapper

В файле с классом не должно быть постороннего кода вроде «new PDO». Каждый класс должен быть назван с большой буквы, а имя файла соответсвовать имени класса: Class -> Class.php В одном файле может быть ровно один класс и ничего больше.

Насчет именования классов и файлов, есть стандарты:

http://www.php-fig.org/psr/psr-0/ru/
http://www.php-fig.org/psr/psr-4/ru/

То что я написал выше, примерно им соответствует (PSR-4 требует все классы класть в неймспейс, но в нашем приложении из 10 файлов это не имеет особого смысла).

Класс логично назвать не DataMapper, а ProfileMapper.

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L11

> public $DBH;

Тут явно должно быть private или protected. Класс не должен не с кем делиться объектом PDO. И вообще, старайся как можно больше полей и методов делать закрытыми.

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L21

> ORDER BY $sort $order");

Если ты подставляешь переменные в запрос, сделай тут же в функции проверку что они соответствуют разрешенному списку значений, иначе непонятно есть тут SQL инъекция или нет (SQL инъекция это когда злоумышленник вставляет свои данные в SQL запрос). Изучи теорию, проверь есть ли у тебя тут инъекция (я думаю есть) и добавь проверку:

https://ru.wikipedia.org/wiki/%D0%92%D0%BD%D0%B5%D0%B4%D1%80%D0%B5%D0%BD%D0%B8%D0%B5_SQL-%D0%BA%D0%BE%D0%B4%D0%B0
http://forum.antichat.ru/thread43966.html

SQL инъекция при умелом использовании позволяет получить все данные из базы, а в тяжелых случаях еще и менять их, читать файлы на сервере, в том числе с паролями, то есть полностью взломать сайт.

> WHERE name LIKE :request

Там требуется искать по любой колонке.

> $STH->fetchAll(PDO::FETCH_CLASS,

Имей в виду что такая конструкция создает объект без вызова конструктора (это пример непраивльного проектирования: ситуаций когда объект создается в обход конструктора быть не должно. Тот кто это делал явно не разбирается в ООП). Это неприятно, так как можно долго искать почему код в конструкторе не срабатывает. Добавь хотя бы флаг PDO::FETCH_PROPS_LATE (мануал http://php.net/manual/ru/pdo.constants.php )

Вот пример кода: http://www.electrictoolbox.com/php-pdo-fetch-class-gotcha/

Вот вопрос на SO с хорошим примером исплоьзования FETCH_CLASS когда параметры передаются через конструктор: http://stackoverflow.com/a/9134544 но он годится только когда их немного (так как делать конструктор с 9 аргументами неправильно).

>>439323

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L35

> public function emailUsed($email)

Для проверки есть ли email не надо выбирать объект. Хватило бы SELECT COUNT(×) + fetchColumn().

> https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L49

> $STH->setFetchMode(PDO::FETCH_CLASS, 'profile');

Не используй setFetchMode, а передавай параметры прямо в fetch. Почему? Потому что setFetchMode делает код непредсказуемым: мы не знаем что нам вернет $STH->fetch так как не знаем с какими параметрами вызывалась setFetchMode до этого. В итоге мы теперь должны каждый раз ставить setFetchMode. Это в общем тоже пример неправильного проектирования, таких функций быть не должно.

> public function addStudent(Profile $profile)

Обычно такая функция после вставки должна получить id из базы и записать его в $profile.

> $STH->bindparam(":name", $name);

> $name=$profile->showName( );

Зачем ты испльзуешь bindParam? Он предназачен для двухсторонней привязки (с получением данных из базы) и почти никогда не нужен. Используй bindValue, в который данные можно передать без промежуточной переменной, и сделай функцию в 2 раза короче.

> public function getLastID()

Такой функции лучше не делать, так как результат который она вернет зависит от того какие функции вызывались ранее. Лучше при вставке в объект прописывать его id

> $STH=$this->DBH->query("SELECT COUNT(...

Это неправильно. Во-первых, в базе могут быть пропуски (записи удалялись) и надо писать MAX(id). Но и MAX(id) использовать неправильно из-за того что при параллельной регистрации получается такая ситуация:

- первый студент делает INSERT
- второй студент делает INSERT
- первый студент делает SELECT MAX(id) и полуает id второго студента
- второй студент делает то же самое

Видишь, какие сложные баги нам это принесет? К счастью, решение есть. У PDO есть метод lastInsertId который возвращает id последней вставленной записи в текущем сеансе связи с базой.

https://github.com/NotBadCode/Students/blob/master/lib/pdo.php#L122

> $id=$result;

> return $id;

Что за ерунда? Может еще десяток переменных добавим?

> https://github.com/NotBadCode/Students/blob/master/lib/profile.php#L20

> if($id!="") $this->id=$data['id'];

Пиши в 3 строчки и через {}

> https://github.com/NotBadCode/Students/blob/master/lib/profile.php#L15

> public function setFields($data)

У тебя в одной функции объединено 2 вещи: простановка полей из массива и проверка на правильность. Надо разнести их в 2 отдельных функции, а то проверку нельзя вызвать без установки значений. Неудобно и нелогично.

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

> https://github.com/NotBadCode/Students/blob/master/lib/profile.php#L36

> $regExp="/^[a-zA-Z0-9_+.-]+@[a-z0-9.-]+(\.)[a-z]{2,}$/ui";

Почитай статью:

http://habrahabr.ru/post/55820/
http://habrahabr.ru/post/175375/

Я от себя посоветовал бы во второй регулярке поменять точку на \S. Это защитит от ошибок когда пользователи вместо @ пишут пробел. Хотя твоя регулярка неплохая и годится для задачи.

Ну и знай что в PHP есть функция для валидации email (хотя мне эта функция не нравится, так как она встроена в PHP и ты не можешь никак на нее повлиять):

http://php.net/manual/ru/filter.filters.validate.php
http://php.net/manual/ru/book.filter.php

> if($id!="") $this->id=$data['id'];

А зачем ты id ставишь из входящих данных? Чтобы можно было редактировать чужой профиль что ли?

> $regExp="/^[а-яa-zё-]+$/ui";

> $regExp="/^[0-9]+$/";

Тут надо разные переменные. Почитай: http://learn.javascript.ru/write-unmain-code#повторно-используйте-имена

> $data['year']<1990

Люди старше 25 не нужны?

> public function showID()

Это принято называть getId().

Сортировку я думаю лучше сделать не формой, а в виде кликабельных заголовков у таблицы со стрелочками. Так надо меньше кликов чтобы отсортировать таблицу. Вот как тут: https://ru.wikipedia.org/wiki/%D0%A1%D0%BF%D0%B8%D1%81%D0%BE%D0%BA_%D0%B3%D0%BE%D1%80%D0%BE%D0%B4%D0%BE%D0%B2_%D0%A0%D0%BE%D1%81%D1%81%D0%B8%D0%B8 только с перезагрузкой страницы.

> https://github.com/NotBadCode/Students/blob/master/templates/list.html#L33

Надо исплоьзовать <?php так как <? может быть отключено.

> https://github.com/NotBadCode/Students/blob/master/templates/list.html#L41

для вывода таблиц есть теги table, tr, th, td:

http://htmlbook.ru/samhtml/tablitsy
http://htmlbook.ru/html/table

Также, я вижу что ты исплоьзуешь br, то есть плохо знаешь верстку. Не хочешь как-нибудь порешать мои задачки на HTML/CSS?

> https://github.com/NotBadCode/Students/blob/master/templates/profile.html#L13

> <input type="text" name="name"

Имя надо выводить не рядом с полем, а в поле. Для этого там есть атрибут value:

http://htmlbook.ru/html/input/value

Для выпадающих списков и радиокнопок тоже можно задать значение по умолчнаию:

http://htmlbook.ru/html/input/checked
http://htmlbook.ru/html/option/selected

> <?if($error=="sex"):?> style="border:2px solid red;"

лучше не писать стили, а добавить CSS класс. Тогда мы легко можем менять вид проблемного инпута через CSS. Явно стоит тебе порешать мои задачки на верстку.

> https://github.com/NotBadCode/Students/blob/master/templates/profile.html#L50

> <? if($showlist==1){?>

Тут надо версию if с двоеточием. Ибо вот это <?php } ?> плохо заметно в HTML-коде, endif заметнее.

И еще. У тебя 2 шаблона и в обоих скопипащена шапка и подвал. Избавься от копипасты.

>>440189

заводить новые переменные со странными именами (что за sta?) не надо. можно написать

start = start || 0;

без var так как эта переменная и так уже создана.

А так, верно.

>>440033

Это приводит к ошибкам. Так как если ты видишь

if (...)
action;

и дописываешь еще одну команду то она не попадет в if. Проще всегда ставить скобки чем усложнять себе жизнь разбором что относится к нему, а что нет.

В особо запущенных случаях бывает 2 ифа или else или цикл с ифом и разобраться уже сложнее.

Вдобавок, рекомендация по оформлению кода PSR-2 ( http://www.php-fig.org/psr/psr-2/ru/ ) говорит:

> Тело каждой управляющей структуры НЕОБХОДИМО заключать в фигурные скобки. Это стандартизирует вид управляющих структур и уменьшает вероятность возникновения ошибок при добавлении новых строк в тело.

А если ты не ставишь скобки то скобки за тобой придется ставить кому-то другому при редактировании.

>>439161

О, ура, спасибо что посмотрел!

Такс:

>Не вижу где в репозитории конфигурация для mongoDB. Например, где определяются индексы? Где создаются коллекции?

Эм, индексы я вообще не создавал, отложил это на попозже, и так и не сделал. А коллецкии разве надо руками создавать? Мне показалось они сами создаются, если их не существует, но ты в неё пишешь.
Кстати где в коде это делать? Что, каждый раз при запуске PHP скрипта, проверять, есть ли нужные коллекции\индексы? Это же разовая процедура должна быть

>озможно, ты копировал схему котору. использует Silex

Ага, хотел чтоб было также как в веб-части.

>Гораздо лучше если бы и websocket сервер и приложение разделяли общий контейнер

Тогда в серверной части в контейнере лежало бы куча всякого, ему вообще ненужного

>Обычно в мапперах реализуют подход IdentityMap

>Зачем вообще нужны Handler-классы?

Вот handler классы я изначально как раз и делал, для того, чтоб они кешировали обьекты RoomUser и RoomEvent, чтоб не было повторных обращений к БД, когда мы теже самые обьекты пытаемся получить. Но, по ходу дела понял, что это очень плохая идея, т.к. у нас есть два "писателя" в БД, server и webapp, Соответственно, Webapp что то поменял в коллекции RoomUser - как RoomUserHandler в Server части об этом узнает? В общем фигня вышла, я этот код выкинул, а ненужные промежуточные классы остались.

>Насчет проблемы с заходом пользователя с нескольких устройств или вкладок, тебе надо вместо пользователя ввести понятие «соединения»

Да, я пришел точно к такому же выводу: https://github.com/d-beekeeper/ratchat/issues/1
Только жаль что поздно до этого додумался. Надо приличный кусок кода под это дело теперь переделывать.
Сначала думал, что можно будет обойтись "малой кровью" - простым счетчиком заходов юзера в отдельную комнату. Типа юзер зашел - увеличиваем. Вышел - уменьшаем. Эвент выхода генерим только когда счетчик==0. Но Heartbeater с таким решением не совместим в принципе. А соединения так просто не добавить, ибо там у каждого должен быть уникальный ID и т.д.
Кроме того, в серверно части эти самые соединения есть - на уровне Ratchet. И как раз именно с уникальными ID. А значит webapp-соединение должно быть совместимо в server-соединением - в общем тут надо подумать, как грамотно сделать. Возможно вообще реализовать аналог WAMP, но с транспортом в виде периодических ajax запросов вместо WebSockets.

> protected function convertId($_id)

>Это очень странная функция

>Это какой-то кривой костыль, явно.

Йап, ты как всегда прав. Надо переделать. А:

> if (!\MongoId::isValid($_id)) {

> $_id = substr(md5($_id),0,24);

это из за того, что если мы сделаем new MongoId('some_custom_string') - будет ошибка. Почему то в PHP Mongo-клиенте, если мы создаем MongoId - ему обязательно нужно передавать hex строку (типа fa3fcc2e...) длиной ровно 24 символа. При том, что сам mongo server прекрасно хавает любые произвольные строки в качестве _id. ХЗ почему так.

>Не очень понятно, почему ты прововеряешь равенство поля null через isset, и не очень понятно почему такие поля не попадают в массив.

Ну... я подумал что не имеет смысла сохранять в БД поля, которые null... зачем они там? Только для однообразности?

>>439162

>Что за странный <a> без href?

>Это ты скопировал или у тебя правда адаптивный дизайн?

Эм, ну это все копипаста из заготовленных шаблонов на на странице мануала по Bootstrap

>Мне кажется, если вебсокет отвалился, но соединение возможно, лучше повторно соединяться через WS.

Это все вшито в AutobahnJs client - он там сам пытается реконнектить вебсокет. см. ниже:
{
skipSubprotocolCheck:true,
maxRetries: 5,
retryDelay: 1000
}
5 попыток реконнета через 1000 мс. И только если не удалось - тогда вызвается onDisconnectCallback

>У тебя на клиенте есть какое-то ограничение на число отображаемых сообщений?

Неа, учту.

> alert('Ошибка: ' + value);

Да я это прекрасно понимаю, просто лень уже был писать модально-всплывающую фигню для отображения ошибок. Потом когда нить доделаю.

> if ($(this).queue().length == 0)

>Это вообще где-то в документации описано?

Нуу... там написано что очередь можно повесить на любой обьект. Ну я и повесил... надо же было её куда то приткнуть, мне показалось что так - норм вариант.

>Кстати, как у тебя с очисткой старых сообщений? Они когда-нибудь очищаются?

Неа. По хорошему надо отдельную коллекцию - архив эвентов сделать, и туда переносить из основной коллекции events старые.

>Ну и вторая вещь, которой можно было бы заняться, это покрыть приложение тестами.

Да, я это читал и писал с рассчетом на то, чтоб потом тесты можно было прикрутить относительно легко.

Займусь всем этим чуть позже. Щас пока не до чата. Во первых заколебался я его делать - это было совсем не просто без соотв. опыта, а во вторых щас немного не до этого: работаю джуном и ищу работу на позицию мидла, так что можешь скрестить за меня пальцы - твои уроки не проходят даром.