>>813508

из ОП поста все нахуй выкинь, эти ребусы тебе не помогут ни на одном собеседовании

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

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

1) Выбирай платформу (CMS или framework, но лучше что-то сложнее сайта на Вордпрессе, Joomla тоже уже никто не юзает)
2) Выбирай дешевый VPS хостинг
3) Все устанавливай
4) Ставь целью - сделать контентный сайт со всей тривиальщиной (создание сущностей (таблиц через DBAL движок), добавление к ним полей разного типа - текст, ссылки на другие сущности, множественные поля, загрузка файлов).
5) любой вопрос, который у тебя возникает - гугли

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

Собеседование:

Собеседование это не проверка твоих знаний, это

а) фейсконтроль. большинство начальников разраб-отделов просто ждут что ты скажешь им нужные слова, даже если они не совсем точные или не используемые в жизни

поэтому советую просто погуглить

"Вопросы на собеседовании для PHP разработчика"

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

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

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

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

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

"Information Expert"
"Low coupling"
"High cohesive"

поможет понять, как структурировать объектный код

все это поможет тебе в разговоре с собеседующим

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

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

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

>>808962

Не понял вопрос. Тут 2 разных массива. return это команда возврата из функции, а не название массива.

>>807577

ОП не разбирается в жс галереях.

>>808390

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

И кстати пишется это кроче:

private static $counter = 1;
...
self::$couner ++;

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

>>809564

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

Зато распространение легально и санкционированно.

>>809011

Если бы были студенты, файлообменник и testhub + знание SQL-нормализации -транзакций + большинство задач на HTML и JS решено - думаю, с руками оторвут.

>>809603

Погугли тестовые задания на пхп например.

>>809605

Странно что игнорят. может им с опытом человек нужен? Может с товим резюме что-то не так?

>>809752

Когда видишь что можешь еще что-то сделать, кроме своей работы, можешь попробовать об этом сказать.

>>810853

пересидим в нелегитимном один тред.

>>810168

Что-то переусложнено и цвета совсем не такие.Откуда например серый взялся ?

Надо упростить. Без box-sizing, попроще.

И что еще за spacer? Ты из 2000 вылез? Маргин же есть.

Контейнеры тоже убери.

>>810539

В классе могут быть одновременно и такие и такие методы и поля. Почитай теорию.

>>810853

Да, у него вообще не Синглтон. Как можно синглтон 2 раза создать?

>>810988

В программировании нет натуральных чисел. В PHP например есть int и float и они определяются способом хранения в памяти компьюетра.

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

То есть математик на такие "числа" даже смотреть не станет.

Вот тебе интересная статья по теме https://habrahabr.ru/post/112953/

>>811042

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

>>811139

ОП пришел, давай задачи.

>>811690

Не знаю, расскажи потом.

>>811781

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

Убрать пробел проще через str_replace

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

>>812083

В денвере древний пхп, может дело в этом?

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

Вообще лучше установить php и прописать в PATH. для начала прочти из ОП поста пасту про командную строку и про установку пхп. А денвер лучше вообще снести от греха подальше.

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

>>815520

Это описано в документации: https://getcomposer.org/doc/articles/scripts.md

Аргументы передавать нельзя. Но из функции-обработчика можно получить объект Composer и из него конфиг. Таким образом, нужные параметры можно закодировать в самом composer.json в ключе extra. Так поступают плагины к композеру, расширяющие его возможности.

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

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

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

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

Если бы мы могли как-то это описать в конфиге, чтобы оно само скачивалось и компилировалось, было бы неплохо. Но это конечно усложнит установку, так как потребует как минимум Node.JS ставить и пакеты к ней.

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

Вот ссылка на Assetic: https://github.com/kriswallsmith/assetic Он немного сложный, но умеет делать то, что нам требуется.

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

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

Так мы можем прийти к тому, что перенесем в композер половину bower и gulp (или половину assetic). правильно ли это?

Я тут погуглил и для начала, нашел такую штуку: https://github.com/fxpio/composer-asset-plugin

Это по сути аналог bower, который умеет читать данные из репозиториев bower и устанавливать его пакеты (если ты не в курсе - то bower это пакетный менеджер для фронтенда, т есть аналог композера для js и css библиотек).

У него есть свои проблемы:

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

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

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

https://github.com/someApprentice/Students

>>815058

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

Но если мы не меняем тип, то остальной код тоже не придётся трогать. Не могу понять как инкапсуляция тут помогает.

>>815058

>> public function setProperty($property, $value) {

>Это вообще не очень хорошая вещь. Во-первых, она не проверяет имя поля и позволяет создать поле с любым именем. Во-вторых, она по большому счету нарушает инкапсуляцию. Потому что, если ты добавишь в сеттер setName() какую-то проверку, то она не будет вызвана в setProperty().

У меня есть похожие методы на этот которые обращаются к полям не через сеттеры. Их тоже следует исправить?

>>815058

>Ну иногда такие классы называют Сервис, иногда Хелпер.

Тогда я бы предпочел иметь Сервисы (классы отвечающие за внутреннюю логику) и Хелперы (классы содержащие вспомогательные методы) в разных директориях. Это не критично если я буду пользоваться таким подходом?
По хорошему, мне, наверно, следует перенести логику из контролеров в сервисы.

https://github.com/someApprentice/Students/blob/master/app/Model/Gateway/StudentGateway.php#L10
Я решил не раздувать код и сделать сортировку или получение определенного количество записей прям в функции getAllStudents(), потому что у неё одинаковый запрос. Следует ли её переименовать в связи тем что она теперь может получать не всех студентов?

>$limit = 2147483647

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

https://dev.mysql.com/doc/refman/5.7/en/select.html

>To retrieve all rows from a certain offset up to the end of the result set, you can use some large number for the second parameter. This statement retrieves all rows from the 96th row to the last:

>SELECT * FROM tbl LIMIT 95,18446744073709551615;

Это действительно так? Бывают ли на практике такие ситуации что нужно получить количество записей большее чем позволяет максимальное значение int?

https://github.com/someApprentice/Students/blob/master/app/Controller/Controller.php#L22
Для борьбы с sql инъекциями, я решил что удобней будет делать проверку на разрешенные колонки в методе получения запроса, чем в методе выполнения этого запроса. Так ведь тоже можно?

https://github.com/codedokode/pasta/blob/master/security/sql-injection.md#Методы-борьбы

>if (!in_array($sort, $allowed)) {

> $sort = $allowed[0];

>}

А почему $allowed[0] возвращает найденное значение? Я в мануале не нашел чтобы описывался такой синтаксис, и у меня не получилось сделать так же в моем условии https://github.com/someApprentice/Students/blob/master/app/Controller/Controller.php#L35

https://github.com/someApprentice/Students/blob/master/app/Controller/IndexAction.php#L38-L53
https://github.com/someApprentice/Students/blob/master/app/Controller/SearchAction.php#L29-L44
Это ведь тоже копипаста, верно? Я только что подумал что это код можно вынести в отдельный методов родителя контроллеров. Скоро исправлю.

https://github.com/someApprentice/Students/blob/master/app/Model/Helper/LoginHelper.php#L55
Я помню как ты говорил, что методы можно делать статическими если всегда будут возвращать одно и то же значение и не имеют никаких зависимостей, но в этот метод передается и меняется класс Student. Не является ли это зависимостью?

Если мы инкапсулируем методы то не следует нам их сделать закрытыми?

https://github.com/someApprentice/Students/blob/master/app/Model/Helper/LoginHelper.php#L61
https://github.com/someApprentice/Students/blob/master/app/Model/Helper/LoginHelper.php#L68
Как закрыть эти методы? Если я пытаюсь обратиться к ним в статическом методе через $this, то я получаю ошибку о том что обращаюсь не в контексте объекта.

Вышел русский перевод офиц. док. по PostgreSQL: https://postgrespro.ru/docs/postgresql/9.5/index.html

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

>>818984

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

>>818927

Файл ini для настроек которые будет менять конечный пользователь (который разворачивает твой проект на своем сайте). зачем ему менять названиее папки с классами?

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

Тебе надо разобраться с автозагрузкой, есть урок на гитхабе https://github.com/codedokode/pasta/blob/master/php/autoload.md

>>818923

> Когда пользователь обращается к нашему приложению localhost/path/studentslist/ запускается скрипт index.php, который запрашивает куки пользователя и сверяет хэш, если хэш не задан (или такого нет в базе), то он запускает скрипт register.php?

Вообще, это не обязательно. Я бы на главной выводил список, а переход на страницу регистрации/редактирования сделал по ссылке.

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

> Каким образом запускается register.php, через require? Или редиректом?

Переход на форму авторизациии (при доступе к закрытой странице) и возврат лучше делать редиректами.

> А лист студентов (результат посика), просто подгружает шаблон, но адрес остается тот же "...index.php", то есть не перенаправляет пользователя на страницу "...list.php..."?

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

>>818881

А урок ОПа по регуляркам разве не объясняет?

Алсо есть мануал http://php.net/manual/ru/pcre.pattern.php

>>818792

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

Алсо есть такая статья (очень сложная) https://habrahabr.ru/post/171667/

>>818756

Все сразу знать не требуется.

>>818671

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

Перечитай внимательно учебник. Почитай официальный мануал, раздел про классы и объекты.

$rank это не обычная переменная, а поле и обращаться к ней в методе надо как $this->rank.

> public $salary = getSalary;

Там вообще писать нельзя. Справа при объявлении поля указывается его значение по умолчанию. Что такое getSalary? В данном случае пхп воспринимает это как константу (так как знака доллара или кавычек вокруг нее нет), которая нигде не объявлена, что и вызывает ошибку.

>>818657

При объявлении поля справа можно писать только примитивное значение, например, число, строку или массив. Нельзя писать выражение или использовать $this.

>>818623

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

>>818520

> abstract class Creep

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

> $salary += $salary * 0.25;

Можно использовать ＊=

> class Manager extends Creep

> {

>protected $salary = 500;

>protected $thirst = 20;

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

> $numberOfCreeps = count($this->listOfCreeps);

> return $numberOfCreeps

лучше сразу return count(...)

> public function fillListOfCreeps($creep){

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

> class Procurement extends Department

>{

>protected $name = 'Закупок';

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

> for($i = 0; $i < 9; $i++){

>$procurement->fillListOfCreeps($me1);

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

заполнение также стоит чуть-чуть автоматизировать. Например сделать массив вида

['3me1', '2en1']

или

[
[3, Manager::class, 1],
[3, Engineer::class, 1],
...
]

и создавать работников в соответствии с закодированным там списком

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

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

>>818435

В цикле пишется то, что надо сделать несколько раз подряд. Но 4-е и 5-е слово надо сгенерировать только один раз, а не 2 раза, значит они должны быть не в цикле.

>>818335

Вместо 3 репозиториев наверно удобнее было сделать один с 3 папками.

> https://github.com/enotocode/anothercredit/blob/master/index.php#L20

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

Да и как вообще можно "очистить" строку? Термин какой-то религиозный, а не из области программирования.

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

В твоем случае данные это числа, потому достаточно сделать трим и преобразовать их в число.

Также, ты используешь массив $values, но тут всего 3 переменных и проще наверно хранить их в отдельных переменных.

> foreach($_GET as $key => $value) {

Ты проверяешь и валидируешь все элементы массива GET. Но надо проверять только нужные тебе 3 значения. Другие значения может быть нужны будут где-то в другом месте в будущем и не надо их трогать.

> return "Выплатить кредит невозможно так как ежемесячный прирост <strong>{$monthlyGrowth}</strong> р.

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

https://github.com/enotocode/anothercredit/blob/master/template.html#L12
Для ввода чисел есть тип type=number, он же позволяет указать минимальное и макс. значение. Не забудь в этом случае с помощью атрибута title указать подсказку, что можно вводить.

Процент кстати может быть дробный.

> <?php if (isset($result)): ?>

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

> Экранирование https://github.com/enotocode/screening

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

Вместо br в HTML принято заключать текстарею и кнопку в отдельные блоки.

https://github.com/enotocode/screening/blob/master/template.html#L19
Знак & надо кодировать особым образом в HTML коде.

Вообще, для этих 5 символов коды надо выучить наизусть: < > & ' "

> Я тебя помню https://github.com/enotocode/anothercredit

Ок, все верно.

>>818176

При создании таблицы или базы данных ты указал DEFAULT CHARSET=utf8? Может у тебя база данных для латиницы сделана.

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

>>818119

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

>>818106

Давай вопросы. У нас целый тред для этого сделан.

>>818074

Да, придется.

>>818065

Если по-простому, то можно делать так:

$whereComment = '';

if (...) {
$whereComment = '....';
}

$sql = "SELECT ... $whereComment ... ";

+ условие для bindValue.

Если проект большой и серьезный, то обычно используют паттерн Query Builder - построитель запросов (читай мой урок по паттернам работы с БД). Там это делается так:

if (...) {
$query->addWhere('commentId = ?', $commentId);
}

Есть готовые библиотеки с Query Builder.

>>817865

> Но если мы не меняем тип, то остальной код тоже не придётся трогать. Не могу понять как инкапсуляция тут помогает.

Допустим у нас есть класс:

class A
{
public $x;
public function setX($x)
{
$this->x = $x;
}
}

Тут как видишь полной инкапсуляции нет. Поле $x не защищено от прямого доступа снаружи. Посторонний код может обращаться к полю $x напрямую, а не через метод setX.

Это приводит к таким проблемам:

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

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

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

Вот представь что ты автор класса, а пользоваться будут им другие люди. Как ты без инкапсуляции гарантируешь, что в поле $x будут числа от 1 до 5, если они видят что поле публичное и могут записать туда любое число?

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

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

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

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

> Но если мы не меняем тип, то остальной код тоже не придётся трогать. Не могу понять как инкапсуляция тут помогает.

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

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

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

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

> Тогда я бы предпочел иметь Сервисы (классы отвечающие за внутреннюю логику) и Хелперы (классы содержащие вспомогательные методы) в разных директориях.

Логично так и сделать.

> По хорошему, мне, наверно, следует перенести логику из контролеров в сервисы.

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

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

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

>>$limit = 2147483647

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

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

> Это действительно так? Бывают ли на практике такие ситуации что нужно получить количество записей большее чем позволяет максимальное значение int?

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

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

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

Я вижу что она подставляет переменную $order прямо в запрос и следовательно она небезопасна. Здесь уязвимость.

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

Нужно использовать метод, который гарантирует отсутствие уязвимости, то есть проверять надо в самой функции. В твоем варианте гарантий никаких нет. Вот откуда человек должен узнать что он обязан фильтровать значение $order? даже в комментарии к функции это не написано, и даже если было бы написано, не факт что его прочтут.

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

> А почему $allowed[0] возвращает найденное значение?

Так это просто берется первый элемент из массива разрешенных полей. Твой вариант тоже годится.

> Это ведь тоже копипаста, верно? Я только что подумал что это код можно вынести в отдельный методов родителя контроллеров. > Скоро исправлю.

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

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

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

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

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

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

Нет. Зависимость это объект который нужен всему классу для работы и которая как правило передается при создании объекта. Студент тут это аргумент над которым делаются операции.

> Если мы инкапсулируем методы то не следует нам их сделать закрытыми?

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

> Как закрыть эти методы? Если я пытаюсь обратиться к ним в статическом методе через $this, то я получаю ошибку о том что обращаюсь не в контексте объекта.

В статических методах нельзя использовать $this так как они вызываются на классе, а не на конкретном объекте и $this не имеет смысла. Увы, PHP вместо того чтобы запрещать такое, просто берет значение $this из вышележащего вызова метода. Это недоработка в PHP.

Также, у меня есть урок по MVC, если надо: https://github.com/codedokode/pasta/blob/master/arch/mvc.md