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

Если тебе лень выравнивать код руками, закачай его на 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/php-fig/fig-standards/blob/master/accepted/ru/PSR-1-basic-coding-standard.md
PSR-2: https://github.com/php-fig/fig-standards/blob/master/accepted/ru/PSR-2-coding-style-guide.md

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

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

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

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

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

Объясняй

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

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

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

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

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

>>829307

Урок читал? https://github.com/codedokode/pasta/blob/master/arch/mvc.md

Что именно в нем непонятно?

>>830049

Что именно не работает? Какую ошибку пишет?

>>829343

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

>>827711

Такой класс обычно называют Front Controller. Он анализирует запрос и вызывает нужный контроллер для его обработки.

>>829264

Архитектура там сомнительного качества.

>>828823

Начинать надо с проектирования БД. И я подозреваю что будет не очень простая система.

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

>>830156

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

Ну то есть для тебя обучение должно быть чем-то вроде работы, на которой ты трудишься 8 часов в сутки несмотря на то что неохота, лень итд.

Придется наверно самостоятельно гуглить и искать информацию. Можно в соответствующем треде вопросы задать (не факт конечно что ответят).

У нас тут пхп тред, но по у нас есть задачки по HTML и по SQL (в ОП посте), которые вроде как пересекаются с тем, что тебе нужно. Ну например, если ты прорешаешь все наши 13 задач по HTML то у тебя будет достаточный минимальный уровень знаний этого языка.

Если ты действительно изучаешь Яву (а я что-то сомневаюсь что за 3 месяца реально изучить все что нужно для работы, ведь там надо кроме языка учить еще ООП, паттерны, библиотеки, фреймворки) то даже тут наш тред может немного тебе помочь. У нас есть в ОП посте задача про студентов - любой начинающий Ява-программист должен уметь ее решить. К ней идет большое число пояснений, и хотя часть из них относится только к PHP, но часть универсальна и не зависит от языка.

>>830159

За 4 месяца вряд ли, если только он не будет ежедневно по хардкору учить.

>>830161

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

>>830258

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

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

Если что, в ОП посте есть инструкция по установке пхп вручную.

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

>>830459

Работа со Слимом делается так: создаем public/index.php, в нем пишем:

- подключение и настройка Слима
- подключение и настройка твоих классов, загрузка конфига, если он есть
- определяем маршруты и функции-обработчики для них
- запускаем Слим через $app->run();

Собственно функции-обработчики и играют роль контроллеров из MVC. Разумеется, они должны быть небольшими по объему, чтобы index.php не распухал.

А остальное ты делаешь как обычно, ну то есть делаешь классы для работы с базой, для валидации, делаешь шаблоны twig для вывода страниц. И вызываешь все это из функций-обработчиков.

В принципе, Слим это по большому счету роутер (штука которая анализирует УРЛ и выбирает обработчик) + middleware + обработчик ошибок + DI контейнер + полезные классы для представления Запроса, Ответа. А что касается работы с базой, валидации - это уже не задача Слима, он тут ничего не делает.

>>830579

В этой статье есть бесполезные вещи, например бессмысленный класс Connect . Зачем он нужен когда есть PDO? Автору этого показалось мало, он еще и Фабрику там сделал. Такое ощущение что человек только открыл для себя паттерны и решил использовать каждый паттерн из книги.

Автор на ровном месте сделал 2 класса вместо того чтобы написать

$pdo = new PDO(...);

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

>>830813

MVC ничего не говорит про папки и файлы. Ты наверно ищешь PSR-4 про который у меня есть урок где-то тут: https://github.com/codedokode/pasta/blob/master/php/autoload.md

>>830829

В фреймворке как раз самих моделей и контроллеров обычно нет.

>>830893

В случае с твигом проще всего использовать наследование шаблонов. В случае не с твигом проще всего в начале шаблона поставить <?php require шапка.phtml ?>

>>830905

Наверно эта система layout в Blade где-то описана и есть документация ведь?

>>830957

Нет. Тебе надо вручную собирать выражение вида field = :field и при этом гарантировать безопасность.

>>830991

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

>>831049

1) старье
2) переусложненный синтаксис, накопившийся за долгое время
3) по моему нет автоэкранирования -> будут XSS

>>831163

Может где-то в скрипте бесконечный цикл? Лог ошибок не смотрел? Для начала попробуй поставить max_exucution_time на 1 секунду и посмотреть в каком месте буде ошибка что время истекло.

Еще есть вероятность что верия пхп какая-то кривая или криво собрана.

>>831314

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

>>833458

>И я не понимаю, зачем вообще нужен max-width. Ведь блочный элемент, если у него не заданы width и max-width, по умолчанию растягивается (с учетом бордеров и паддингов) на всю ширину родителя (урок https://github.com/codedokode/pasta/blob/master/html/shrink-to-fit.md ). Значит, тут надо просто вместе с display block поставить для width/max-width значения auto и все. Точнее, только для max-width, так как width нигде не задан.

>И я не понимаю, зачем вообще нужен max-width.

Чтобы вкладки с большим названием не растягивались на всё занимаемое им пространство.

Почему-то свойство max-width не переопределяться и вкладки сжимаются до 200px: https://jsfiddle.net/obdgvxfk/ . Если его не использовать то всё работает отлично https://jsfiddle.net/obdgvxfk/1/ .

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

Добавил поля: https://jsfiddle.net/tb7k9dw9/8/

>Насчет удаления атрибутов с id, я придумал такой вариант:

>Можно попробовать для связи табов и вкладок использовать их номера. CSS3 позволяет указать в стилях что-то вроде "N-й по счету элемент типа E" - и соответственно можно попробовать привязать первый инпут и первый label к первому телу вкладки, второй ко второму итд.:

Сделал: https://jsfiddle.net/1ax6k6hw/

>Тогда data-tab нам не нужен и нужен только id. Как от него избавиться, я увы, не знаю. Если положить input внутрь label то не получится использовать селектор ~.

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

>>833459

>Я имел в виду что можно написать например так:

>

>.content {

>width: 100%; / старые браузеры второе правило проигнорируют и возьмут первое /

>width: calc(...); / новые прочитают оба правила, но возьмут второе /

>}

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

>.content {

>width: 100%; / старые браузеры второе правило проигнорируют и возьмут первое /

>width: calc(...); / новые прочитают оба правила, но возьмут второе /

>}

>

>> Пусть пока будет так:https://jsfiddle.net/0u3x5rsw/3/

>> <div class="wrapper">

>Враппер не нужен - минимальную ширину можно задать на элементе html или body.

>Также, если присмотреться, левая граница меню не совпадает с левой границей заголовка - меню сдвинуто на 5-10px вправо, некрасиво.

Исправил: https://jsfiddle.net/0u3x5rsw/6/ https://jsfiddle.net/xx82knpf/2/

>>833459

>Задача про Короля Лир

>

>> Исправил: https://jsfiddle.net/tsuh4gjw/8/

>Что-то ничего не исправилось. Заголовок по прежнему вырван влево. Тут надо просто поставить на body поля слева и все.

Исправил: https://jsfiddle.net/tsuh4gjw/9/

>>833459

>> Неисчезающие инпуты

>> До сих пор не понял в чём смысл: https://jsfiddle.net/gpysbfar/4/

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

>О, еще одна хорошая головоломка, но я нашел ответ. label по умолчанию имеет display: inline, а как известно, у inline-элементов нет собственных размеров - их размер и форма совпадает с содержимым. Потому инлайн-элемент не имеет форму прямоугольника, а может например растягиваться на несколько строк в вслед за содержащимся в нем текстом.

А почему лейбел не растягивается на ширину сместившихся влево инпута?

>> Applies to: block containers and boxes that establish a formatting context

>> Применяется к: блокам-контейнерам (?) и боксам, задающим контекст форматирования

Если что, я умею читать английский.

>>830052

>Задача про радиокнопки

>https://jsfiddle.net/fp8o0exq/5/

>> .inputbox > input {

>> position: absolute;

>> left: -10000px;

>Вот у этого метода конечно есть подвох - что если завтра кто-то поставит наш элемент на гигантскую страницу которая шире этих 10 000 px? Может лучше сделать родителя c overflow: hidden и тогда не придется так далеко загонять? Или все же не стоит? Я в принципе не настаиваю.

Исправил: https://jsfiddle.net/fp8o0exq/6/

>>833459

>> Вроде получилось: https://jsfiddle.net/fp8o0exq/7/

>тут есть проблема: если двигать по кнопкам синюю рамку, то ширина кнопки при появлении рамки меняется и правая кнопка сдвигается на 1-2 пикслея вправо. Это некрасиво. Надо либо компенсировать лишний пиксель в бордере отрицательным маргином, либо сделать у всех кнопок border 1px и отрицательный маргин, чтобы они слегка наезжали друг на друга.

Исправлено: https://jsfiddle.net/fp8o0exq/9/

>>833459

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

Я тоже увидел это, но не уделил этому нужного внимания. Исправил: https://jsfiddle.net/fp8o0exq/8/

Еще я подумал, что можно было задать несколько теней с помощью box-sizing, но у них тогда бы был общий радиус теней https://jsfiddle.net/L69xmtwh/

>Или же можно попробовать сделать рамку с помощью свойства outline: http://htmlbook.ru/css/outline

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

>>834348

У max-width есть специальное значение, обозначающее "отсутствие ограничений": https://developer.mozilla.org/ru/docs/Web/CSS/max-width#Values

(если сложно читать англ то можно найти это свойство в htmlbook и посмотреть там).

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

> А почему лейбел не растягивается на ширину сместившихся влево инпута?

Понятия не имею. Видимо этот момент в спецификациях четко не описан и потому производители браузеров его реализуют как хотят. Но факт в том что у инлайн-элементов нет своих размеров. И overflow нельяз применять к ним.

> Если что, я умею читать английский.

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

ну вот например эти разделы

https://www.w3.org/TR/REC-CSS2/visuren.html
https://www.w3.org/TR/REC-CSS2/visudet.html

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

>>834510

>>Сюда надо бы добавить подробности ошибки разбора JSON

> А как?

Почитать внимательно офиц мануал по json_decode http://php.net/manual/ru/function.json-decode.php

https://github.com/someApprentice/Students

>>819083

>> Следует ли её переименовать в связи тем что она теперь может получать не всех студентов?

>Можно просто назвать getStudents или FindStudents.

>

>>>$limit = 2147483647

>> В мануале по mysql сказано о том что если нужно получить все записи, то нужно указать какое-то большое число

>А ты его по памяти написать сможешь когда тебе будут нужны все студенты? лучше сделать условие что если $limit = 0 то конструкция LIMIT не вставляется. Правда, это не даст нам указывать OFFSET без LIMIT, но это почти никогда и не нужно.

>

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

>Это неправилньо. Вот я смотрю на код функции getAllStudents() https://github.com/someApprentice/Students/blob/master/app/Model/Gateway/StudentGateway.php#L10 и хочу проверить, безопасна ли она.

Исправлена функция getAllStudents: https://github.com/someApprentice/Students/commit/363d244b1ded5a4801b03053e0e3538192a094e3 полное описание коммита внутри

>>819083

>> extract($this->getSortQuery());

>Вот это плохая идея, так как глядя на код непонятно какие переменные создаются. Где гарантия что эта строчка не перезапишет одну из переменных, которая определена выше? ну например я правлю твой код и хочу добавить переменную, как мне выбрать ее имя чтобы она не конфликтовала с теми, что создаются в этой строчке? А что если завтра кто-то добавит в возвращаемый getSortQuery() массив новое поле и оно по имени будет совпадать с одной из переменных? Получается мы можем сломать контроллер, добавив новое поле в массив совсем в другом месте.

Исправлено: https://github.com/someApprentice/Students/commit/c6f3e8282b0a190e97f86b460cf60485eac81f45

>>819083

>Также, у тебя в контроллере стоит 4 вызова render. Ты выносишь знание о структуре страницы в контроллер, но удобнее просто подключить один шаблон, а он уже пусть подключает что ему нужно.

Исправлено: https://github.com/someApprentice/Students/commit/54c05e525d22b9136ab3cf4d2e8df52574359d2c

Ответ на >>823225 / http://arhivach.org/thread/193343/#823225

> Ответ на любой вопрос

> http://ideone.com/dGLHaC

Верно

> Генератор имён готовый

> http://ideone.com/j0wiZB

Правильно.

> В тылу врага - шифровка

> http://ideone.com/0ddCh5

Тут тоже верно

> 1337 - шифровка

> http://ideone.com/gMlhjC

Все правильно.

> На словах ты Лев Толстой

> http://ideone.com/CL7D0o

Верно.

> Разбиение строки на символы

> http://ideone.com/mtlMmY

А тут вроде ничего не разбивается. Забыл сохранить?

> Палиндром

> http://ideone.com/fGPWl9

> $textLow = mb_strtolower($text); //переводим всё в нижний регистр

> $textLowTogether = str_replace(' ','', $textLow);

Тут незачем заводить новые переменные, логично класть результат назад в $text

Увы, программа не всегда определяет результат правильно: http://ideone.com/HBN7W4 - она считает слово палиндромом, если последние сравниваемые символы (в центре) совпадают.

> Айпад в кредит

> http://ideone.com/zsbANa

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

http://arhivach.org/thread/193343/#823666

>>823666

> Сначала не совсем понимал как сделать ответы к вопросом

Вопросы бывают разных типов и содержат разные наборы полей - это типичный случай наследования таблиц. У Фаулера описаны 3 паттерна, 3 способа реализации наследовния: Single Table Inheritance, Class Ti, Concrete TI. Советую погуглить и изучить. Сравнить преимущества и недостатки, выбрать более удобный.

Также надо убедиться что выбранный тобой фреймворк или библиотека поддерживает схему.

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

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

> Таблица attemts сделана для того чтобы зарегистрированные пользователи не могли бесконечно проходить тест, каждому тесту можно задать максимальное количество раз пересдачи. Тут я не уверен, нужен ли такой функционал вообще?

Нужен так как вроде там где-то в статистике выводится информация по числу пересдач

> И как быть с незарегистрированными пользователями?

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

> Для поиска по тестам хочу Sphinx взять, он получше и быстрее обычного поиска через LIKE.

Можно попробовать, почему бы и нет. У меня был урок по sphinx: https://gist.github.com/codedokode/10539366

> Про индексы знаю, тут они нужны будут в основном для внешних ключей?

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

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

> password character varying(500) NOT NULL,

зачем 500 символов? И ты надеюсь сам пароль не собираешься хранить?

> FOREIGN KEY (author_id) REFERENCES users (id)

А там не надо писать ON UPDATE/ON DELETE?

> test_id integer NOT NULL,

Тут внешнего ключа не хватает

> CREATE TABLE answers (

Тут нет внешних ключей

>>837543

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

>>824820 http://arhivach.org/thread/193343/#824820

мануал: json_encode, json_decode, погугли про сам формат

>>825267 http://arhivach.org/thread/193343/#825267

Код на PDO сам по себе получается довольно большой:

$s = $pdo->prepare(...);
$s->bind(....);
$s->execute(...);
$result = $s->fetchAll(...);

Если библиотека позволяет это записать например в виде

$result = $lib->fetchAll("SELECT ???", [$a, $b, $c]);

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

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

Из известных есть Doctrine DBAL, вот пример, как в ней делается запрос: http://docs.doctrine-project.org/projects/doctrine-dbal/en/latest/reference/data-retrieval-and-manipulation.html#fetcharray

Также ты можешь унаследовать ее и добавить свои функции.

Если тебе интересны другие библиотеки, можно поискать их тут

http://phptrends.com/
http://pronskiy.com/php-digest/

Слова для поиска: db, dbal, database, pdo

>>825645 http://arhivach.org/thread/193343/#825645

> Причем в нижнем варианте, при отправке запроса не появляется ошибка, просто не выполняется сортировка

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

>>826340 http://arhivach.org/thread/193343/#826340

Чтобы не городить много ифов, можно как-то попробовать разбивать код на функции или переставить команды местами.

>>826385 http://arhivach.org/thread/193343/#826385

> когда подгружаем список студентов, нужно еще отслеживать сортировку

Получается да, надо

> А еще получается, что нужно проверять авторизацию пользователя в обоих контроллерах index.php и register.php.

Возможно, что да.

> И каждый раз, как обновляется страница, например при сортировке. Значить дублируется код, и + запрос в базу каждое обновление страницы.

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

> Нужно ли действительно проверять авторизацию пользователя в каждом контроллере?

Если тебе это нужно для чего-то, то да.

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

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

> Как обойти трехуровневую вложенность if?

Попробовать переставить код или вынести что-то в функцию. Если не получитя, то оставить вложенность.

>>835551 http://arhivach.org/thread/193343/#835551

> $2 и $4 заключены в квадратные скобки, но значения в них могут быть пустыми и в этом случае в тексте будут появлятся лишние "[]". Поэтому, я решил после этой строки вставить чистку пустых квадратных скобок, но мне кажется, есть, наверное, более красивое решение этого.

Может быть можно было бы как-то через "или" сделать, по типу preg_replace("/([a-z])([а-я])|([а-я])([a-z])/", '$1$2[$3$4]', ...) - тогда пустых скобок бы не было.

>>837922

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

>>837921

Ты имеешь в виду класс со страницы 336? class Cookie { private $created.... ?

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

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

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

Работать с куками вполне нормально. Это пример инкапсуляции, когда мы скрываем внутри класса конкретный механизм авторизации, а наружу выставляем только методы вроде login/logout(). мы даже можем поменять механизм, не трогая остальной код.

Ну и по коду там конечно мне не все нравится. Подчеркивания - плохо. Публичные статические поля - нехорошо.

И еще мне не очень нравится реализация, когда в конструктор передается id пользователя. Я бы сделал конструктор без аргументов, а также методы loginAs($userId)/logout()/getUserId(). То есть сделал бы многоразовый класс, который умеет логиниться под разными логинами, а не заточен на работу с одним заданным в конструкторе id.

Еще мне не нравится использование исключений, я бы просто сделал чтобы метод getUserId() возвращал null если авторизация не подтверждена.

Еще, мне кажется, тут многовато полей. И название не очень удачное, это ведь не класс представляющий куку. Правильнее назвать AuthroizationHelper или CookieAuthorizationHelper, как-то так.

Сама схема с использованием шифрования неплохая, позволяет проверять авторизацию без обращения к БД. Правда есть и небольшой недостаток - при смене пользователем пароля куки не теряют свою силу. Лучше бы чтобы теряли, чтобы при похищении кук они бы переставали работать при смене пароля.

>>837904
>>837187

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

Советую подучить CSS, у нас в Оп посте как раз хорошие задачки по теме.

flex не везде работает, более того по моему есть разные его версии.

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

>>837893

У тебя когда долг становится меньше 5000, на него не начисляется процент и комиссия. Потому и меньше выходит наверно.

>>837866

getBody() возвращает объект и на нем сразу же вызывается метод write() без сохранения объекта в переменную. Так же как можно писать $x = func() + 2 , можно писать и func()->something().

Иногда еще делают чтобы каждый метод возвращал $this - это цепочечные вызовы. Мне не очень нравится.

>>836666

Просто люди любят всему перечить. Ну например, известно что ООП популярная технология, используемая около 30 лет в коммерческой разработке. Но тут приходят функциональщики и начинают свое "ООП не единственный путь, есть и другие подходы". Но попроси их объяснить как сделать дерево DOM, дерево AST, графический интерфейс с виджетами, представление данных из БД, на их функицональщине или хотя бы организовать код в большом проекте - и им нечего ответить. ООП идеально подходит для таких ситуаций, его и придумали в ходе решения подобных задач.

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

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

Или вот есть Го, где нет ООП потому что он не нужен. А что же у них есть? Синтаксис, который позволяет на функциях имитировать объекты и писать что-то вроде:

user := users.NewUser()
user.setName('Ivan')

Ну совсем на ООП непохоже! То есть видимо им ООП использовать хочется, но признавать что это ООП не хочется.

> А PHP Расмус Лердорф изначально создавал как набор инструментов, написанных на С, позволяющих легко и быстро разрабатывать динамический HTML. PHP таким задумывался и таким остался, он сам и есть фреймворк.

Может это и так, но чистый php - это довольно плохой и негибкий фреймворк. Ну к примеру в нем есть шаблоны, но нет автоэкранирования данных для защиты от XSS. В нем есть нефатальные (!) ошибки. В нем есть суперглобальные переменные вроде $_POST что побуждает людей использовать их в любом месте кода. В нем нельзя повлиять на цикл обработки запроса (например сделать что-то еще до прихода HTTP запроса или после его обработки).

Да, я помню, Лердорф даже привел пример приложения без фреймворка: https://toys.lerdorf.com/archives/38-The-no-framework-PHP-MVC-framework.html

Предлагаю вам самим изучить этот код и посмотреть.

> В мире Python и Ruby создание веб-сайтов с нуля — занятие довольно утомительное, поскольку эти языки для создания веб-сайтов не предназначались.

Бред же. В руби по моему HTTP сервер одной строчкой создается, если кому-то хочется писать на "чистом" руби. То есть создание сайтов на числом php и ruby одинково неудобно на мой взгляд.

> Чётко уясните: строк кода в любом проекте должно быть как можно меньше, чтобы код стал как можно чище и читабельнее!

Ну без фреймворка их придется написать больше.

> Но дело в том, что так называемое объектно ориентированное программирование очень часто бывает излишне сложным!

А бывает и нет.

> одно из главных преимуществ PHP — поддержка императивной, функциональной,

справедливости ради функциональный код на нем пиать довольно громоздко и неудобно. Ну-ка скажите мне как кратко записать функцию, получающую на вход объект и возвращающую его свойство name? function ($o) { return $o->name; }. Чтобы PHP поддерживал функциональный подход, это должно писаться в 4 раза короче.

> Так что объект, по сути, — это просто набор функций и переменных, которые теперь называют методами и атрибутами.

Нет. Принципиальная разница что в случае обектов мы можем создать несколько объектов с разными значениями полей, а в случае с функциями + переменными - не можем. Переменные есть только в одном экземпляре. Ну и в ООП есть наследование, инкапсуляция.

> Если вы решили следовать стандартам PHP-FIG, то вы должны понимать, что некоторые из них — например, стандарты автозагрузчика PSR-0 и PSR-4 и ряд других — напрямую влияют на то, как вы пишете код ПО.

А аворы намекают что надо не следовать PSR-4, а придумывать в каждом проекте свой способ называт файлы? Спасибо, не надо.

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

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