Код нужно писать не как попало, а аккуратно и по правилам. Почему? Потому, что на неакуратно написанный код не хочется даже смотреть.

Если тебе лень выравнивать код руками, закачай его на http://beta.phpformatter.com/ и нажми «format». Робот исправит выравнивание и отступы в мгновение ока (да, прогресс не стоит на месте). Если ты используешь мощную IDE вроде PhpStorm, там тоже есть функция форматирования кода.

Горячие клавиши для форматирования кода в разных IDE: https://gist.github.com/codedokode/8759492

Вообще, в PHP долгое время не было единого стандарта оформления кода, все писали как попало и было много бардака, но сейчас дело лучше — есть стандарты PSR-1 и 2. Вот как надо оформлять код:

- переменные и функции пишутся с маленькой буквы, подчеркивание не используется, используется camelCase, пример: $x, $numberOfPeople, printResults()
- Название функции начинается с глагола, в стиле «сделайЧтоТо»
- не знаешь английский? Не беда, в 21 веке есть решение этой проблемы. Не пиши транслитом, открой лучше Гугл Транслейт или slovari.yandex.ru и найди название для переменной там
- в именах классов используется CamelCase, первая буква большая, «_» может использоваться
- мы предпочитаем подстановку переменных вместо конкатенации строк: "I am $age years old" — хорошо, 'I am ' . $age . ' years old' — плохо из-за обилия точек и кавычек
- мы используем для отступов 4 пробела (можно настроить редактор, чтобы при нажатии Tab он вставлял 4 пробела)

Вот ссылка на стандарты, где все это описано подробнее и даны примеры оформления:

PSR-1: https://github.com/samdark/fig-standards-ru/blob/master/accepted/ru/PSR-1-basic-coding-standard.md
PSR-2: https://github.com/samdark/fig-standards-ru/blob/master/accepted/ru/PSR-2-coding-style-guide.md

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

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

Давай удочку, а не рыбу

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

Будь доброжелателен

Не годится: «Ты мануал хоть раз в жизни открывал, обезьяна?»
Не годится: «В гугле забанили?»
Не годится: «Твой код плохой»
Хорошо: «Вот, как можно улучшить этот код: ...»
Хорошо: «Ты неправильно используешь функцию abc(). Вот ее описание: ссылка, и как видишь ей надо передать строку, а не массив»

Не придирайся к знанию английского или русского языка.

Объясняй

Не очень хорошо: «сделай как в этом коде»
Хорошо: «если ты вставляешь текст от пользователя в SQL запрос, то получается SQl-инъекция, которая позволяет взломать твой сервер (ссылки). Чтобы этого избежать, надо вставлять данные с помощью плейсхолдеров (ссылки)»
Хорошо: «Помни, что код пишется для людей. Если писать такие большие функции, то в них становится трудно разобраться...»

Не проповедуй

Мы учим использованию самых распространненных подходов, стандартов, библиотеки фреймворков. Если ты не любишь ООП, пробелы в коде, jQuery, сам PHP, то рассказать об этом стоит в каком-нибудь другом треде.

Не придирайся к знанию английского языка, анон пишет как умеет.

Ах да. Если тебе кажется, что что-то в учебнике или задачах можно сделать лучше — пиши, обратная связь всегда очень полезна.

>>1020292

> Нужно для каждого статуса для заданного диапазона дат получить кол-во транзакций с данным статусом,

Группировка не годится? Только таблицу повернуть, чтобы были такие колонки

date | ok | wait | cancel
------------------------

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

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

> Чо они вагрантом не пользуются

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

Там ведь разные сервисы, хранилища файлов, базы данных, и тд.

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

> То есть persist на каждом шагу и не нужен.

persist используется для передачи новых созданных объектов под управление entity manager. На каждом шагу он не нужен, да.

> В этом и состоит суть паттерна, что изменения в сущностях отслеживаются автоматически и сохраняются в базу при коммите?

Вроде примерно это (судя по https://martinfowler.com/eaaCatalog/unitOfWork.html ). Но там написано, что главное преимущество UOW это запись изменений атомарно, одной транзакцией, а не кучей мелких транзакций.

Доктрина содержит в UOW ссылки на все управляемый ей сущности (загруженные из БД, а также добавленные через persist), а также их исходное состояние и при flush() сравнивает их текущее состояние с исходным, находит различия, формирует пакет изменений и сбрасывает эти изменения в базу одной транзакцией.

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

> Ну то есть в классе UOW имеется маппер и репозиторий для каждой сущности,

Нету там ни мапперов, ни репозиториев.

Условно говоря, UOW это идея, что изменения надо накапливать и сбрасывать одной транзакцией, а не делать запрос в тот момент, когда изменяется сущность.

> Ну то есть в классе UOW имеется маппер и репозиторий для каждой сущности,

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

> при извлечении из бд в репозиторий

В репозитории ничего не хранится.

> сохраняется ссылка на модель с статусом типа "clean", когда мы меняем свойства модели через сеттер, статус меняется на "dirty"

Нет. По умолчанию при загрузке из БД Доктрина просто сохраняет значения полей и ссылку на объект, а при flush() сравнивает текущее состояние объекта с исходным. Но ты можешь использовать и другие опции, например сделать объекты, которые будут сами сообщать в доктрину об изменениях, чтобы сэкономить время на поиск изменений (но это усложняет сущности). То есть возможны варианты когда объекты сообщают об изменениях, и когда Доктрина сама их ищет, это настраивается.

Также, если тебе интересно, Доктрина не использует геттеры/сеттеры, а получает доступ к приватным свойствам с помощью reflection. Потому ты можешь не делать геттеры или сеттеры, если логика подразумевает что поле нельзя менять или читать снаружи. Или ты можешь например при изменении одного поля менять второе, то есть пишешь любую логику, какую хочешь.

> Я придумал новую интеллектуальную игру

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

Это снова https://github.com/tsubaku
Я поработал над указанными ошибками, но исправил не все.
Во-первых, что исправил:

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

Убрал из конфига, в PDO обращаюсь к таблицам по имени. Теперь их названия прибиты гвоздями. Не уверен, что это лучше, но сделал.

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

Убрал

>Тут код отформатирован ужасно: https://github.com/tsubaku/flights/blob/master/submit.php

Переформатировал. Теперь правильно? Перевёл в camelCase названия пхп-функций. Переменные не переводил, их слишком много.

>Если ты отдаешь JSON, то тип ответа должен быть application/json, не надо изобретать свои стандарты. И логично заголовки ставить не в начале скрипта, а перед отдачей самого JSON

Заменил text/plain на application/json везде, где возвращается JSON. Перенёс хидеры к echo json_encode. Нужны ли хидеры там, где только echo? А там, где вообще ничего нет, просто вызов функции?

> $pass = md5(md5(trim($pass)));

>Безопаснее использовать соленый хеш, как описано в моем уроке https://github.com/codedokode/pasta/blob/master/security/password-hashing.md

Заменил md5 на password_hash/password_verify

>При загрузке картинок нет проверки типа файла, человек загрузит .png или .txt, а ты его переименуешь в .jpg например.

Сделал проверку и корректное расширение для jpg, png, gif, остальным типам скрипт отказывает в загрузке.

> https://github.com/tsubaku/flights/blob/master/show_list_clients.php#L10

> while(list ($key, $val) = each ($_POST)){

>Это давно устарело, нужно использовать foreach тут

Переделал

>Название функции должно начинаться с глагола, то есть не protection, а redirectIfNoPrivileges(...)

Заменил на verifyAuthorization

> <a href='#' class='a_button_delete'

Ссылка должна куда-то вести. Для создания кнопки есть тег button.
Оформил в виде button. Немного поехала вёрстка кнопки, потом поправить.

> while ($i <= count($ru_rows_array)-1){

Тут нужно использовать foreach. Ты сишник наверно? Не надо тянуть плохие практики из Си в язык, где есть foreach.
Заменил в нескольких местах count на foreach

> if (is_null($_COOKIE['id'])

Это неправильно и вызовет ошибку, если в массиве нет ключа 'id'. У тебя наверно включено игнорирование предупреждений, или же ты не смотришь в логи ошибок, раз этого не видишь. Нужно использовать array_key_exists или isset здесь.
Хм, никаких ошибок в логах. Хотя, может быть я просто проверил не все случаи. Заменил is_null на empty.
https://github.com/tsubaku/flights/blob/master/functions.php#L187

> $info = getimagesize($path);

>Тут не проверяется вариант, когда функция вернет вместо массива false.

Добавил проверку

Что исправил с оговорками:

>В шаблонах у тебя вроде бы есть повторяющиеся куски - шапка страницы.

Шапки немного повторяются, но всё же есть различия и не думаю, что сливать общие части имеет смысл. Плюс, так их легче воспринимать. Я разделил скрипт core.js, вынеся в отдельные файлы скрипты менеджера и охранника, а в шаблонах убрал загрузку лишних скриптов.

>https://github.com/tsubaku/flights/blob/master/index.php#L10

> if ($user_level == 9) {

>Тут нужно ииспользовать константу с понятным названием вместо цифры

> $level = 'manager';

>И тот тоже желательно константу.

Немного переписал функцию проверки, теперь она возвращает ид, уровень доступа и фамилию. Проверку оставил по тому же условию.

> echo "<br />";

В HTML слеш в конце тега не ставится, в отличие от XML/XHTML.
Так у меня как раз XHTML
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

> https://github.com/tsubaku/flights/blob/master/functions.php#L292

> header("Location: status_codes.php?result=403");

Если ты хочешь отдать код ответа HTTP, то надо его отдавать сразу. Потому что редирект - это код вроде 302. То есть ты вместо отдачи кода 403 отдаешь код 302 в ответ на запрос. И вместо "доступ запрещен" говоришь "страница переехала на другой адрес".
Я так и не понял, как сделать так, чтобы браузер при ошибках показывал суть ошибки. Ну или хотя бы стандартные коды. Так что просто заменил все редиректы на header("Location: login.php");

Что не удалось:

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

Выделил папку core, в неё перенёс все НЕ публичные пхп, шаблоны и закачиваемые фотографии. Но как закрыть её, не понимаю. .httaccess полностью рубит доступ и скрипты просто не работают. Как сделать так, чтобы сервер обращаться к ним мог, а левый юзер нет, я не нашёл информации.

>Тут https://github.com/tsubaku/flights/blob/master/show_list_clients.php в одном файле смешана логика и вывод данных в HTML, почитай про шаблоны http://web.archive.org/web/20161119062218/www.phpinfo.su/articles/practice/shablony_v_php.html

Немного переписал и упростил вывод таблиц охранников и клиентов, но как их шаблонизировать, я не знаю (статью читал). Данные в ячейки подставляются в цикле, имена ячеек тоже. При шаблонизации для каждой ячейки придётся писать отдельный вызов, получится монструозно.

>https://github.com/tsubaku/flights/blob/master/functions.php#L5

>Тут слишком сложная функция с слишком большой глубиной отступов

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

Что скажете?

>>1021692

> add_departament

> get_Count_Workers

Что за нестандартный стиль названий функций? PSR рекомендует camelCase для методов:

- https://svyatoslav.biz/misc/psr_translation/#_PSR-1
- https://svyatoslav.biz/misc/psr_translation/#_PSR-2

> public function print_Info()

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

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

Немного наверно непонятное объяснение получилось, но суть в том, что код будет аккуратнее, если Компания будет хранить информацию о компании, а вывод ее будет делаться в другом месте кода.

> //Cократить в каждом департаменте 40% (округляя в большую сторону) инженеров,

> //преимущественно самого низкого ранга.

> public function Destruction_Workers

Тут та же проблема. Это сильно специфический метод, написанный под одну конкретную задачу (антикризисные меры), и его лучше вынести в отдельный класс. А так, ты по сути взял код, который должен быть в отдельном классе и размазал по Компании и Департаменту.

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

Гораздо аккуратнее будет если Компания будет заведовать списком департаментов, а специфичные вещи вроде "найти и уволить 40% рабочих" будут в отдельном классе для антикризисных мер.

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

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

> Destruction_Workers

Имена функций начинаются с глаголов, сделатьЧтоТо

> public function add_Workers($count, $rank, $proffesion, $is_Head)

Этот метод сам создает работников, и не позволяет добавить уже созданного где-то в другом месте работника. Лучше сделать добавление именно готового объекта, это будет универсальнее.

> public function get_Workers_And_Destruction($count_in_percent,$proffesion)

Тут функция просто создана методом копирования кода. Так не годится. Нужно убрать повторяющийся код.

Такой код тяжело читать, так как трудно понять, чем именно один блок кода отличается от другого.

Также, у тебя выводятся предупреждения, которые надо исправить:

> PHP Notice: Undefined variable: sum_salary in /home/UH8YEW/prog.php on line 139

> PHP Notice: Undefined variable: sum_pages in /home/UH8YEW/prog.php on line 155

> PHP Notice: Undefined variable: sum_salary in /home/UH8YEW/prog.php on line 139

> PHP Notice: Undefined variable: sum_coffee in /home/UH8YEW/prog.php on line 147

> PHP Notice: Undefined variable: sum_pages in /home/UH8YEW/prog.php on line 155

>>1021684

Не надо добавлять бесполезные посты в этот тред

>>1021671

да

>>1021505

Это не линукс, например, там нет неймспейсов тех же.

>>1021439

> Есть некий аналог open server? Чтобы можно было легко переключаться между версиями апача, пхп, майэскьюэль без дрочки в консоле?

не знаю, может и нет, я сам привык консолью пользоваться и GUI видеть не могу.

> Можно переустановить систему без потери софтины и ее настроек?

Ты можешь сделать у себя на жестком диске несколько разделов, в один поставить винду, в другой линукс, а на третьем хранить данные, доступные из обоих систем. При этом, если повозиться, то можно сделать переразбиение без уничтожения данных на диске (partition magic это умеет), лишь бы места было достаточно.

Настройки в линуксе обычно хранятся в файлах в /etc, которые легко забекапить и не беспокоиться про переустановку. Также, в линуксе обычно при удалении программы через менеджер пакетов ее настройки не удаляются.

Насчет разных версий PHP - в линуксе есть штука https://github.com/phpbrew/phpbrew которая умеет скачивать и компилировать нужные версии PHP, а потом переключать переменные окружения, чтобы их использовать. Разумеется, тебе придется изучить, как она работает. А так, большинство версий линукса не поддерживают стандартным способом установку 2 версий одной программы.

>>1021482

Только Debian лучше ставить Testing, а то в Stable софт стабильный, но старый.

>>1021432

В смысле, "как"? Не очень понятно, как ответить на твой вопрос. Проектируешь, как это должно работать, и реализуешь.

>>1021262

https://github.com/tsubaku

Сначала отвечу на пост:

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

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

Зачем оно вообще нужно? Такое (префиксы имен таблиц) делали много лет назад, чтобы разместить 2 проекта в одной базе, так как нищехостинги требовали отдельные деньги за вторую. Сейчас это по моему неактуально, берешь дешевый VPS и создаешь сколько угодно баз.

> Переформатировал. Теперь правильно?

Не очень:

https://github.com/tsubaku/flights/blob/master/login.php
Тут нужно убрать самый первый отступ (после <?php), то есть строка require_once('./core/php/functions.php'); идет без отступа. Для отступов лучше использовать 4 пробела, а не таб.

https://github.com/tsubaku/flights/blob/master/core/php/submit.php
Тут отступы вообще разные. Почему отступ в строке 22 - 8 пробелов? должно быть 4. И почему там так много пустых строк? Пока что плохо.

Рекомендации PSR можно почитать тут в переводе

- https://svyatoslav.biz/misc/psr_translation/#_PSR-1
- https://svyatoslav.biz/misc/psr_translation/#_PSR-2

Код можно форматировать автоматически, через сайт или IDE, вот ссылка https://gist.github.com/codedokode/8759492

Не понимаю, в чем проблема прогнать файл через сайт или нажать комбинацию клавиш в IDE.

> Нужны ли хидеры там, где только echo? А там, где вообще ничего нет, просто вызов функции?

header логично ставить радом с echo так как они относятся к одной задаче, и header должен срабатывать только когда выводится JSON. А когда ты их ставишь в разных местах, то это во-первых нелогично (мы сначала сообщаем, что будет выведен JSON, а только потом где-то в другом месте его выводим), во-вторых может вести к ошибкам, когда выводится header с application/json а потом выводится не JSON, а обычный текст или например сообщение об ошибке.

То есть нелогично функцию вывода JSON разделять на 2 части и разбрасывать их в разные места кода.

Кстати, тут тоже нелогично: https://github.com/tsubaku/flights/blob/master/core/php/show_list_clients.php
В заголовках указан тип text/plain, а затем идет вывод HTML кода.

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

> Сделал проверку и корректное расширение для jpg, png, gif, остальным типам скрипт отказывает в загрузке.

Пока не очень удачно. Тут https://github.com/tsubaku/flights/blob/master/core/php/submit.php#L25 ты проверяешь тип файла, который берется из тела POST запроса и указывается браузером. Но браузер может там указать что угодно. Надежднее проверить расширение.

>> if (is_null($_COOKIE['id'])

>> Это неправильно и вызовет ошибку, если в массиве нет ключа 'id'.

> Хм, никаких ошибок в логах.

Вот пример, показывающий ошибку: http://ideone.com/bMeVWL

Возможно у тебя в php.ini включено игнорирование ошибок с помощью настройки error_reporting (зачем так делать??) или отключено их логгирование. Или у тебя всегда есть эта кука.

> Шапки немного повторяются, но всё же есть различия и не думаю, что сливать общие части имеет смысл. Плюс, так их легче воспринимать.

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

> Я так и не понял, как сделать так, чтобы браузер при ошибках показывал суть ошибки. Ну или хотя бы стандартные коды. Так что просто заменил все редиректы на header("Location: login.php");

Ты выводишь HTTP заголовок, затем тело страницы. Что тут сложного?

header("HTTP/1.0 500 Internal Error");
echo "Everything is broken, sorry\n";
die();

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

> Выделил папку core, в неё перенёс все НЕ публичные пхп, шаблоны и закачиваемые фотографии. Но как закрыть её, не понимаю.

У сервера есть такая настройка, как "корневая папка" (в Апаче она называется document root, в нгинксе root). Ты создаешь у себя папку public и в настройках веб-севрера прописываешь путь к ней как document root, например в Апаче:

DocumentRoot /var/www/example.com/public/

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

> Немного переписал и упростил вывод таблиц охранников и клиентов, но как их шаблонизировать, я не знаю

Элементарно это делается. В скрипте-контроллере ты подготавливаешь данные:

$guards = getGuardsList();
// подключаем шаблон
require 'templates/guards.phtml';

В шаблоне формируешь и выводишь HTML страницу:

<?php require __DIR__ . '/header.php'; ?>

<h1>Охранники</h1>
<ul>
<?php foreach ($guards as $guard): ?>
<li><?= htmlspecialchars($guard['name']) ?></li>
<?php endforeach ?>
</ul>

<?php require __DIR__ . '/footer.php'; ?>

> https://github.com/tsubaku/flights/blob/master/core/php/functions.php#L224

> unset($row_content); // разорвать ссылку на последний элемент

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

Вот еще проблемное место:

> function showTable($year, $month)

> ...

> $stmt = $pdo->query('INSERT INTO flights () VALUES()'); //Добавляем пустую строку

Функция вывода таблицы не должна ее модифицировать.

> https://github.com/tsubaku/flights/blob/master/core/php/write_in_table.php

Тут нечитаемая стена кода, надо выносить из нее отдельные функции.

> Вопрос про функцию showTable()

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

Сначала нужно разделить получение данных и их вывод. Сначала получаем все данные из БД, только потом выводим.

При этом часть кода можно выносить в отдельные функции. Вот например вначале идет кусок лапши, получающий список рейсов между 2 датами. Его легко вынести в отдельную функцию:

$flights = getFlightsInRange($fromDate, $toDate);

Вот мы заменили 8 строк лапши на один вызов функции. И эту функцию мы сможем вызывать и в других местах, если нам понадобится список рейсов там.

Точно так же можно вынести например получение списка охранников (getGuardList()), получение названия месяца по номеру (getMonthName()), и тд.

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

HTML код выводить, как у тебя, с помощью echo, очень неудобно и неэффективно.

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

Пока ты просто пишешь длинную последовательность действий, а надо разбивать ее на шаги.

Также, тебе нужно как-то организовать файлы и функции. Сейчас я логики не вижу, файлы называются как попало, функции тоже по ним как-то произвольно раскиданы. Должна быть какая-то логика.

Соответственно, сейчас тебе надо работать над разделением:

- вынести весь HTML код в шаблоны
- вынести весь SQL-код в функции в отдельном файле или файлах, то есть сделать отдельный слой доступа к базе данных, а не размазывать это по всему коду

Я еще раз напомню, что у нас в ОП посте есть задача на список студентов. Хоть она и называется "задача", но там много комментариев и по сути это что-то вроде сборника полезных советов по написанию приложений с формами и таблицами. То есть приложение того же типа, что и у тебя. И тебе желательно найти время ее прочитать, так как у меня нет желания пересказывать тут все ее содержимое по второму разу.

>>1021978

>https://github.com/tsubaku/flights/blob/master/login.php

>Тут нужно убрать самый первый отступ (после <?php), то есть строка require_once('./core/php/functions.php'); идет без отступа. Для отступов лучше использовать 4 пробела, а не таб.

Блин, про manager.php, index.php, guard.php и login.php я забыл. Сделаю.

>https://github.com/tsubaku/flights/blob/master/core/php/submit.php

>Тут отступы вообще разные. Почему отступ в строке 22 - 8 пробелов? должно быть 4. И почему там так много пустых строк? Пока что плохо.

Хм, submit.php тоже забыл. Странно. Вроде я все файлы по очереди переформатировал.

>Код можно форматировать автоматически, через сайт или IDE, вот ссылка https://gist.github.com/codedokode/8759492

Через него и делал.

>То есть нелогично функцию вывода JSON разделять на 2 части и разбрасывать их в разные места кода.

Не понял. Там же только одна команда вывода json. В каком смысле "разбивать на 2 части"?

>Кстати, тут тоже нелогично:

>https://github.com/tsubaku/flights/blob/master/core/php/show_list_clients.php

>В заголовках указан тип text/plain, а затем идет вывод HTML кода.

А html - это не text/plain?

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

Хорошо, функцию сделаю. Но вот именно проблему копипасты не понимаю. Если что-то работает, и работает правильно, то как это переписывать?

>Но браузер может там указать что угодно. Надежднее проверить расширение.

Я гуглил на эту тему и когда кто-то в похожей ситуации делал проверку по расширению, ему так же указывали, что "клиент может послать что угодно, надёжнее проверять по типу". Олсо, не всё ли равно? Законопослушный юзер отправить jpg под видом jpg и всё будет нормально. А незаконопослушный отправит какую-нибудь НЁХ, но сервер переименует её в jpg и при обращении просто не откроет.
>>1021978

>Возможно у тебя в php.ini включено игнорирование ошибок с помощью настройки error_reporting (зачем так делать??) или отключено их логгирование. Или у тебя всегда есть эта кука.

Не включено. Я вручную удалял из браузера куки, ошибок не было, браузер корректно перенаправлял на страницу логина.

>Очевидно же, что у страниц в твоем проекте будут общая шапка, и может быть, подвал.

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

>Ты выводишь HTTP заголовок, затем тело страницы. Что тут сложного?

>Я вообще не понимаю, что сложного так сделать и зачем тут понадобился редирект.

А, так это пишется вручную. Теперь понятно. А редирект для удобства юзера.

>и в настройках веб-севрера прописываешь путь к ней как document root, например в Апаче:

У меня нет доступа к настройкам сервера. Вернее, на локалхосте есть, а вот на продакшене всё крутится у хостера (там всего-то статичный сайт нашей конторы, а этот сервис я делаю как бы его подсистемой).
Я посмотрю, как там это устроено.

>Элементарно это делается. В скрипте-контроллере ты подготавливаешь данные:

Я попробую.

> unset($row_content); // разорвать ссылку на последний элемент

>Эта строчка вообще смысла не несет

Блин, точно.

> function showTable($year, $month)

> ...

> $stmt = $pdo->query('INSERT INTO flights () VALUES()'); //Добавляем пустую строку

>Функция вывода таблицы не должна ее модифицировать.

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

> https://github.com/tsubaku/flights/blob/master/core/php/write_in_table.php

>Тут нечитаемая стена кода, надо выносить из нее отдельные функции.

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

Остальные комментарии тоже приму к сведению.

>Я еще раз напомню, что у нас в ОП посте есть задача на список студентов.

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

>>1021099

Работа с исключениями сделана абсолютно неправильно:

> try { }

> catch (\Exception $e) {

> Yii::warning($e->getMessage());

> return $this->redirect(Yii::$app->request->referrer);

Что за бред? В приложении происходит ошибка, и ты просто редиректишь на referer (а кстати, что если он пустой?). При ошибке надо вывести страницу ошибки, а не скрывать этот факт. Это ведь тебе самому потом усложнит поиск причин.

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

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

Вот простой пример, чем плох твой код. У тебя есть действие контроллера для рассылки сообщений actionMassSend(). Но что, если нам понадобится вызвать это действие где-то программно? У тебя нет функции вроде sendMassMessage($message, $users) и нам придется писать код заново. Ты не почему-то не додумался вынести это действие в отдельную функцию, а вписал в контроллер, перемешав с кодом получения данные из формы и кодом отдачи ответа пользователю.

>>1021061

Спасибо, сайтгайд хороший.

>>1021030

> Изменение номера - https://ideone.com/KJxmSg

Ок, верно

> Автозамена - https://ideone.com/LaOqC8

Ок, тоже верно

> Вывод email - https://ideone.com/NCD54g

Не найдется email i[|JvanANUS0sqi2PUNCTUMcX,#om

> Grammar Nazi - https://ideone.com/BR95EM

Эти однотипные строки надо объединить в одну регулярку:
$text = preg_replace('/[,](\\S)/ui', ', $1', $text);
$text = preg_replace('/[;](\\S)/ui', '; $1', $text);
$text = preg_replace('/[!](\\S)/ui', '! $1', $text);

И эти объединить:

> $text = preg_replace('/жы/ui','жи', $text);

> $text = preg_replace('/шы/ui','ши', $text);

> $text = preg_replace('/(\\S) а /', '$1, а ', $text);

Нужно еще правило для "но"

>>1021023

> Можешь объяснить, чем плохи чистые функции, зачем городить эти Utility service?

Я думаю, ради DI, если тебе нужна ссылка на другой сервис, то надо использовать DI. Если не нужна, можно наверно и обычные функции.

>>1020161

Копировать руками ничего не надо, надо автоматизировать деплой, используя скрипты с rsync, ansible или как вариант, деплой через гит.

>>1020741

Какая версия дистрибутива? И почему версия PHP какая-то левая, что еще за blackfire? ты каких-то странных патчей туда наустанавливал, возможно в этом проблема?

Что значит "все ломается"

Из твоего поста ничего понять нельзя.

>>1020506

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

Ну и она не на русском.

А ты видел это? https://github.com/codedokode/task-checker

Я пытаюсь сделать проверяльщик, который будет все писать подробно и на русском.

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

>>1020356

Нужно вынести файлы за пределы публичной папки (корня сервера).

>>1021991

> Не понял. Там же только одна команда вывода json. В каком смысле "разбивать на 2 части"?

Вывод JSON ответа состоит из 2 действий:

- добавить заголовок Content-Type с значением application/json
- вывести тело ответа (JSON код)

Вопрос был, какой смысл эти неразрывно связанные действия разносить отдельно.

> А html - это не text/plain?

Это text/html: https://ru.wikipedia.org/wiki/Список_MIME-типов

text/plain значит "обычный текст без тегов и форматрования". Браузер отобразит его просто черным текстом на белов фоне как есть.

> Но вот именно проблему копипасты не понимаю. Если что-то работает, и работает правильно, то как это переписывать?

Ты забываешь правило "код пишется для людей".

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

Также, в плане чтения кода строка

setNoCacheHeaders();

гораздо понятнее и читабельнее чем 2 строки с header.

> Я гуглил на эту тему и когда кто-то в похожей ситуации делал проверку по расширению, ему так же указывали, что "клиент может послать что угодно, надёжнее проверять по типу

Имя файла присылается всегда, а где гарантия, что браузер правильно определит и пришлет тип? У тебя есть информация, какие типы файлов какой браузер "знает", а какие нет? У меня нет такой информации например. Никто не запрещает прислать браузеру тип application/octet-stream например.

> Не включено. Я вручную удалял из браузера куки, ошибок не было, браузер корректно перенаправлял на страницу логина.

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

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

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

> У меня нет доступа к настройкам сервера. Вернее, на локалхосте есть, а вот на продакшене всё крутится у хостера (там всего-то статичный сайт нашей конторы, а этот сервис я делаю как бы его подсистемой).

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

Как костыль ты можешь попробовать загружать свой код на 1 папку выше корня сайта на сервере.

> Я тоже думал вынести скрипт добавления строки в отдельную функцию, а потом вызывать её, но вызывать-то всё равно придётся. Если в указанном месяце нет ни одного рейса, то должна втоматически создаваться пустая строка для нового рейса и после этого показываться таблица, состоящая из пустой строки.

Название функции должно ей соответствовать, если написано "вывести таблицу", то она должна заниматься именно выводом таблицы.

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

Просто там очень много информации в сжатой форме и за раз это не осилить. Прочти один пункт, разберись с ним, а потом как-нибудь переходи к другим.

>>1021998

видимо redis/rabbitmq

>>1021821

>Что за нестандартный стиль названий функций? PSR рекомендует camelCase для методов:

Думал что стиль названия для всего один. А оказывается для всего разные. fixed

>Вот это лучше бы вынести из Компании наружу, так как у каждого класса должна быть своя зона ответственности. Если у Компании зона отвественности - хранить список департаментов и считать по ним статистику, то вывод информации - это какая-то другая деятельность, которая явно должна делаться в другом месте.

А нормально что он будет в функции которая не в классе, а сама по себе?

>Гораздо аккуратнее будет если Компания будет заведовать списком департаментов, а специфичные вещи вроде "найти и уволить 40% рабочих" будут в отдельном классе для антикризисных мер.

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

>Имена функций начинаются с глаголов, сделатьЧтоТо

Заменил на destroy. А как я понимаю этот глагол пишется с маленькой буквы.

>> public function get_Workers_And_Destruction($count_in_percent,$proffesion)

>Тут функция просто создана методом копирования кода. Так не годится. Нужно убрать повторяющийся код.

Как я понял это про копирование foreach 3 раза. Сделал адекватно.

>Этот метод сам создает работников, и не позволяет добавить уже созданного где-то в другом месте работника. Лучше сделать добавление именно готового объекта, это будет универсальнее.

Добавил метод для поштучного добавления и для массива с кучей рабочих.

>Гораздо аккуратнее будет если Компания будет заведовать списком департаментов, а специфичные вещи вроде "найти и уволить 40% рабочих" будут в отдельном классе для антикризисных мер.

Я сделал отдельный класс. В нем функцию. и дальше если я пытаюсь эту фунецию на компанию использовать то ошибка. Не понимаю как сделать.

Есть еще вопрос насчет того что все переменные в классах закрыты и из за этого приходится делать кучу функций геттеров и сеттеров. Это нормально так делать? Всегда так надо делать? Почему бы не сделать все паблик и тогда код сократится в 2-3 раза, потому что не будет этих бесполезных функций getRank и тд.

Еще вопрос что плохого в классе монстре. Хочешь использовать функцию или сделать. Через поиск проверяешь нет ли такой и используешь/делаешь.

>>1025212
>>1025247

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

>>1024908

Ну, если у сайта один пользователь то особой разницы между fpm и built-in сервером и нет.

Так, на твой вопрос сразу ответить не получится. Так как может быть очень много причин. Хорошо бы (если есть время) попробовать найти, в чем именно дело. Для этого надо посмотреть, что происходит в процессе выполнения кода. Тут есть 2 способа:

- расширение xdebug и профайлинг с его помощью. Оно сгенерирует лог (cachegrind), который можно просмотреть специальной программой и увидеть, какие функции занимали сколько времени. Также, можно сгенерировать трейс, показывающий порядок вызова каждой функции, время выполнения и потребление памяти

Стоит помнить только, что включение отладки повлияет на код и он будет работать еще медленнее.

- наблюдение за программой через strace. Эта линукс-утилита показывает системные вызовы (запросы программы к операционной системе), которые делает твое приложение. Минус в том, что будут показаны только обращения к файлам, выделение памяти, то есть то, за чем нужно обращаться к ОС, а внутренняя работы программы будет не видна. С другой стороны, симфони делает десятки тысяч разных операций и лог strace, который меньше по объему, удобнее просматривать.

- наконец, есть еще poor man profiling, то есть ручная расстановка по коду команд вывода текущей функции и время, прошедшее от начала запроса. Это конечно может занять много времени.

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

Я тебе советую сделать какой-то тестовый контроллер, который просто выводит hello world, чтобы исключить проблемы в твоем коде.

Также, есть несколько вещей, которые можно попробовать сделать без профайлинга:

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

>>1024561

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

> попутно считая купюры и отнимая один от ключа до тех пор, пока он не станет равен нулю.

Многократное вычитание - это деление.

Так, твоя идея правильная и называется "жадный алгоритм". Жадный, так как он пытается взять как можно больше купюр каждого номинала.

>>1024904

> public function addDepartament($departament)

Тут нужен тайп-хинт. Вот короткая копипаста:

--------
### Тайп хинты

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

Мануал: http://php.net/manual/ru/language.oop5.typehinting.php

Обрати внимание, что php7 усовершенствовал систему тайп-хинтов - теперь можно в их качестве указывать примитивные типы вроде int/string, а в php7.1 стало можно указывать тайп-хинт для возвращаемого функцией значения: https://habrahabr.ru/post/267799/ , причем можно указать тип void, значащий, что функция ничего не возвращает (увы, возможность вернуть null пока не реализовали, так что ждем)
--------

Если ты используешь PHP7, то стоит добавить и скалярные (скалярные значения - это не составные значения, то есть не массивы или объекты) тайп-хинты.

> public function deleteWorker($number)

Вот это неудачно спректированная функция. Чтобы удалить работника, надо узнать его номер в приватном списке внутри департамента. Как его узнать? Надо вместо номера передавать самого работника, которого надо уволить. Объект - это сам по себе уникальный идентификатор.

То есть попробуй думать, что объект используется не только в этой задаче, а например в сложной системе учета работников. Удобно ли будет другим разработчикам использовать твой метод, когда им понадобится кого-то уволить?

> $is_Head,

Стиль названия не соответствует другим переменным

> public function getSalary($worker)

Зачем сюда передается worker? Он еще и не используется никак.

Далее, клаcc Analyst почему-то сделан не так, как другие профессии. Почему возможность менять зарплату предусмотрена только для аналитиков? Ну и это неправильно, что ты ради отдельной задачи про антикризисные меры добавляешь какие-то костыли в класс отдельной профессии. Если ты хочешь менять зарплату, то такая возможность должна быть предусмотрена для всех.

Одна из идей ООП - это разделение отвественности, когда у каждого класса есть своя задача, своя зона ответственности. Соответственно антикризисными мерами должен заниматься отдельный класс, а в другие классы помещать код для их реализации не надо. Там должны быть общие, универсальные методы, которые могут быть полезны вообще, а не только для антикризисных мер.

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

Также, ты сделал поле с зарплатой аналитика статическим. А это значит, что в принципе зарплату им нельзя менять индивидуально, только всем аналитикам во всех компаниях сразу.

> class Crisis

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

Отбор кандидатов на увольнение/повышени лучше сделать так:

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

Поиск работников нужно как-то унифицировать, а то у тебя там код поиска повторяется.

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

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

>>1007389

>>>Ну и подсчет высоты сообщений - это явно что-то неправильное. Это нужно для выравнивания сообщений вертикально? Неужели это нельзя было сделать средствами CSS? Задание размеров и расположения элементов через JS работает плохо, может влиять на производительность и работать с задержками.

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

>Это называется "вертикальное выравнивание". Я бы поискал варианты сделать это средствами CSS. Вот что приходит в голову:

>

>- flex, новая технология, которая не везде есть. С ее помощью можно выделить часть экрана под область сообщений, и задать выравнивание блока сообщений по нижнему краю. Ну и не забыть про overflow, чтобы при большом числе сообщений блок не растягивался, а появлялась бы прокрутка. Минус - работает в новых браузерах

>- display: table - работает начиная с ИЕ8 и древних фаерфоксов. Это как flex, только слабее по возможностям и со своими недостатками. Табличное отображение позволяет прижать содержимое ячейки (зоны сообщений) вниз. Тут есть проблемка - чтобы сделать прокрутку, блоку сообщений надо задать высоту, и задать ее в виде 100% (=высоте окна) наверно не получится, значит придется этот момент сделать через JS (определение доступной высоты блока и изменение ее при измнеении размера окна)

>- прижать блок сообщений вниз с помощью абс. поз. Работает тоже везде, и можно задать высоту как 100% от зоны для вывода сообщений (при условии что не используется display: table, так как абс. поз. не работает с элементами таблиц).

>

>Я бы советовал попробовать сделать отдельный пример (примеры) в jsfiddle или аналогичном сервисе, и потестировать его на маленьком и большом числе сообщений (когда должна появляться прокрутка).

>

>Я могу подсказать по CSS, но имея частично работающий код, сделать это проще.

С точки зрения дизайна интерфейса, критично ли будет то, что даже если все сообщения получены, то они всё равно будут прижиматься вниз, оставляя пустое пространство в верху? Я не вижу в этом ничего плохого, но кому-то пустое пространство может показаться не красивым.

https://jsfiddle.net/55khok9f/1/

https://github.com/someApprentice/chat/blob/master/public/js/conversation.js#L82
Что можно сделать после успешной отправки сообщения?

Можно обновить все сообщения, но сообщения и так обновляются каждые пол секунды. Как-то не хочется оставлять пустую функцию. Или её можно не писать?

https://arhivach.org/thread/261841/#1000396

>Ты не обрабатываешь ошибки. Видимо, уроки, которые ты читал, упускают такие важные вещи. Вот тебе тогда мои маленькие советы про использование аякса: https://github.com/codedokode/pasta/blob/master/js/ajax.md

https://github.com/someApprentice/chat/blob/master/public/js/chat.js#L59-L69
https://github.com/someApprentice/chat/blob/master/public/js/conversation.js#L85-L92

Стоит ли так выборочно обрабатывать ошибки? Для пользователя достаточно вывести ошибку соединения, а для разработчиков выбросить в консоль статус и текст ошибки.

Почему у меня не вбрасываются эти ошибки (ошибки не появляются в консоле)? Это из-за асинхронного кода?

https://arhivach.org/thread/266631/#1010744

>> Кстати, это нормально делать перевод на новую строку в цикле https://github.com/someApprentice/chat/blob/master/public/js/conversation.js#L280-L284 ?

>Это какие-то кривые костыли. Получается, логика отображения разбросана между шаблоном и JS кодом. Мы ведь в PHP не пытаемся так делать после генерации HTML из шаблона? И тут не надо. Наверняка там есть какие-нибудь хелперы для таких преобразований. В чем проблема сделать какой-нибудь хелпер для преобразования текстов сообщений?

К сожалению, у меня не получилось найти в ejs аналог функции nl2br и при этом чтобы экранировались тэги. В таком случае, будет приемлемым оставить этот костыль?
Или лучше, можете посоветовать какой-нибудь шаблонизатор который справиться с этой задачей?

И что если я захочу не экранировать определённые тэги? Должено быть какое-то решение чтобы справиться с этими задачами.

https://github.com/someApprentice/chat/blob/master/src/Controller/ConversationController.php#L78-L115
Это нормально, что у меня такая простыня с обработкой ошибок?

https://arhivach.org/thread/261841/#1000396

>Также, некоторые разработчики активно используют возможности HTTP (дополнительные заголовки, методы вроде PUT/DELETE, коды ответов), а некоторые не используют.

>Также, перед проектированием АПИ, советую почитать про REST и HATEOS (и далее, стандарт про шаблоны URL https://tools.ietf.org/html/rfc6570 ), но тут конечно есть разные мнения, ну к примеру в иделогии REST одним запросом нельзя получить несколько несвязанных массивов данных, а HATEOS раздувает размер ответа.

Я плохо понял что такое REST. Это просто архитектура которая позволяет обрабатывать запросы с методами вроде PUT\DELETE?

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

https://arhivach.org/thread/266631/#1010325

>Пересоздавать полный список сообщений неэффективно.

Я никак не могу придумать как выводить только новые сообщения. Определенно, нужен setTimeout\setInterval чтобы постоянно обновлять данные с сервера, но как при этом выводить только те сообщения которые ещё не выведены в шаблон?

https://github.com/someApprentice/chat/blob/master/src/Model/Database/MessageGateway.php#L25
Поскольку, я теперь вывожу сообщения за определённый период времени, то если за этот промежуток не было сообщений, при открытии диалога выведиться пустое окно сообщений из-за чего вытекает баг с автоматической подрузкой более старых сообщений, из-за того что отсутвует скролл блока сообщений. Это можно исправить увеличивая отрезок времени и повторном обращении к БД, до тех пор пока не получим нужное количество сообщений:

$count = $db->getMessagesCount();

$offset = 1;

if ($count > 10) {
while(count($messages) < 10) {
$offset++;
$messages = $db->getMessages(..., $offset);
}
}

Не будет ли это лишней нагрузкой на БД?

У меня есть идея как при отключенном js опускать окно диалога до последнего сообщения - использовать якори. Я уже делаю что-то подобное при получении более старых сообщениях (при выключенном js):

https://github.com/someApprentice/chat/blob/master/templates/chat.phtml#L24

><a ... href="conversation.php? ... #message-ID">Получить больше сообщений</a>

https://github.com/someApprentice/chat/blob/master/templates/chat.phtml#L32

><div id="message-ID ... class="message">

Точно так же можно сделать в ссылке контакта.

Но для этого нужно сделать несколько лишних запросов к БД:

$contacts = $db->getContacts();

$c = array();

foreach ($contacts as $contact) {
$lastMessage = $db->getLastMessage(...);

$c[] = array(
'contact' => $contact,
'lastMessage' => $lastMessage
);
}

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

Опять же, не создаст ли это большую нагрузку на БД? Учитывая, что с включенным js, при сортировке по последним сообщениям, обновление контактов будет происходить каждую секунду.

https://github.com/someApprentice/chat/blob/master/src/Controller/AuthController.php#L101-L103
https://github.com/someApprentice/chat/blob/master/public/js/conversation.js#L217
Не подвергаю ли я пользователей опасности, открывая кукисы для js?

Просто при открытии диалога нет возможности получить токен. Если только не спрятать его где-то на странице. Я сомневаюсь что это будет правильным решением, но с другой стороны не вижу никакой проблемы. Как лучше поступить?

У меня есть переменные, которые пишутся ни как camelCase или snake_case, например moremessages. Я знаю, что это плохой плохой вариант, но мне он кажется вполне читабельным. Лучше будет переимновать?

>>1027953

Чтобы разобраться в этой теме, придется почитать про двоичные числа. Любое число можно представить не только в обычной (10-ной) системе счисления, но и в двоичной. Например, 19 в десятичной = 10011 в двоичной.

https://ru.wikipedia.org/wiki/Двоичная_система_счисления

Двоичная система используется потому, что внутри компьютера все числа представлены как раз в двоичном виде. Например, если в ячейке памяти есть электрический заряд, то это обозначает 1, а если нет - то там хранится 0. Одно такое значение - 0 или 1 - называется бит. Используя несколько ячеек, мы можем с помощью этих битов кодировать произвольные числа. То есть двоичная система "родная" для компьютера и внутри числа представлены и обрабатываются в этой системе.

С числами в двоичной форме кроме обычных математических операций можно делать битовые операции вроде И или ИЛИ: https://ru.wikipedia.org/wiki/Битовые_операции

http://php.net/manual/ru/language.operators.bitwise.php

Если нам нужно как-то закодировать набор опций, каждая из которых может быть включена или выключена, мы можем использовать двоичные числа. То есть договориться, что например последний разряд в числе обозначает одну опцию (OPTION1 = 00001), предпоследний - другую (OPTION2 = 000010).

Тогда с помощью операции ИЛИ мы можем объединять эти опции: OPTION1 | OPTION2 = 0000011

В общем-то константы вроде PDO::FETCH_CLASS - это числа (можно проверить echo или var_dump), каждое из которых содержит одну единицу в двоичном виде, а с помощью операции ИЛИ их объединяют.

Такие опции еще называют "флаги" (по аналогии с флагами, используемыми для передачи сигналов визуально на кораблях). Вообще, эта система кодирования идет с древних времен, когда компьютеры были медленные и у них было мало памяти, и хотели ее использовать максимально эффективно (используя ровно 1 бит на одну опцию). Сейчас конечно это не так принципиально, но для флагов двоичные числа как раз хорошо подходят.

Чтобы проверить и закрепить знания, реши задачку:

- сделай набор из нескольких констант, каждая из которых будет представлять какую-то опцию, которая кодируется с помощью двоичных флагов (то есть константа хранит число, в двоичном виде содержащее ровно одну единицу)
- сделай функцию setBit, которая принимает на вход число и опцию, и возвращает число где установлен в единицу соответствующий опции бит
- сделай функцию clearBit, которая принимает на вход число и опцию, и возращает число, где соответствующий опции бит сброшен
- сделай функцию toggleBit, которая принимает на вход число и опцию и меняет соответствующий бит в числе на противоположный
- сделай функцию testBit, которая принимает на вход число и одну опцию и возвращает true/false в зависимости от того, включен в этом числе соответствующий флаг или нет
- сделай функцию countBits, которая принимает на вход число и возвращает число единиц в его двоичном представлении

Пример использования:

const OPTION_A= ...;
const OPTION_B = ...;
const OPTION_C = ...;

$flags = 0;
$flags = setBit($flags, OPTION_A);
$flags = setBit($flags, OPTION_B);

var_dump(testBit($flags, OPTION_A)); // true
var_dump(testBit($flags, OPTION_C)); // false

$flags = clearBit($flags, OPTION_A);
var_dump(testBit($flags, OPTION_A)); // false
var_dump(testBit($flags, OPTION_B)); // true

var_dump(countBits($flags)); // 1

Если что-то непонятно, уточняй.

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

https://arhivach.org/thread/261841/#1000397

>Насчет проектирования БД - там тоже много тонкостей есть, можно делать нормальзованную схему, но лучше денормализовывать ради повышения производительности. Ну например, возьмем то же сообщение. Можно хранить его в одном экземпляре, а можно в двух - в inbox получателя и в outbox отправителя. Это может облегчить выборку последних N сообщений, так как твой запрос с OR вряд ли хорошо ложится на индексы (вдобавок у тебя еще и сортировки там нет).

Безусловно, нужно хранить сообщения в нескольких экземплярах для отправителя и для получателя, чтобы в дальнейшем было удобно шифровать их для каждого пользователя соответственно. Но я не понимаю как хранить ID сообщений - получается что для каждого пользователя оно теперь будет своё. Из этого вытекает, что нельзя будет как-то преобразовывать эти сообщения т.е. редактировать или удалять. В принципе от этой идеи можно оказаться, как это сделано в популярных мессенджерах А я всё думал, почему же они это не сделали, просто странно иметь разные ID.
Хотя, можно сделать отдельную колонку, которая не имеет PRIMARY_KEY, и заполнять ID как-то программно.

Думаю лучше отложить пока идею с редактированием\удалением.

Такой же вопрос, и с inbox и outbox - в них те же самые сообщения будут иметь разные ID. Это неизбежно делать отдельную колонку и генерировать ID программно?

Или я не правильно понял структуру этих таблиц?

Никак не могу сформулировать структуру при реализации конференций... Попробую представить это здесь

Conferences
id
tete-a-tete //true or false (приватные переписки тоже будут представлены как конференции из двух человек для унификации кода)

Participants
id //Не уверен что от этой колонки будет толк, обычно всегда делают id
conferenceID
userID

Contacts
id
userID
conferenceID

Inbox
id
conferenceID
userID // ID пользователя которому принадлежит ящик
messageID
authorID
content
...

Outbox
id
conferenceID
userID // ID пользователя которому принадлежит ящик
messageID
authorID
content
...

Соответственно получение сообщений происходит одним запросом SELECT FROM Inbox WHERE conferenceID = :conferenceID AND userID = :userID UNION SELECT FROM Outbox WHERE conferenceID = :conferenceID AND userID = :userID

Вроде должно быть так?

----

Забыл ещё спросить в предыдущем посте

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

https://jsfiddle.net/jsqt57hz/

MVC style:
https://jsfiddle.net/hmgz7200/

Чтобы это пификсить нужно убрать с элемента обработчик функцией .off()

//Странно что не получилось написать $('body).off('submit', form, handleFormSubmit), как в примере мануала
https://jsfiddle.net/jsqt57hz/1/

Я всё правильно делаю?

>>1027651

>У меня есть идея как при отключенном js опускать окно диалога до последнего сообщения - использовать якори. Я уже делаю что-то подобное при получении более старых сообщениях (при выключенном js):

>Но для этого нужно сделать несколько лишних запросов к БД:

Это должно быть сделано с помощью функций JOIN и GROUP:

SELECT Contacts.userID, Messages.messageID as lastMessage
FROM Contacts
INNER JOIN Messages ON Contacts.userID= Messages.userID
GROUP BY Contacts.userID
ORDER BY Messages.date DESC

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