>> $this->sign ^= true;

> ^ это не булев, а целочисленный (побитовый) оператор. Он работает с числами а не true/false.

> Чему равен корень из 2? Это так называемая бесконечная дробь, которую нельзя представить в виде рационального числа.

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

>> return (string) floatval(bcdiv(...))

> floatval делает преобразование в float. float даже в 64-битном пхп сохраняет не более 14 значащих цифр

>>773648

>Далее, нет логики в наследовании хелперов. Вот у тебя есть Helper и есть LoginHelper, который его наследует. Непонятно, а зачем? Там все равно почти все методы статические. Точнее, часть статическая, а часть нет, и почему, непонятно.

>>773648

>И наконец, в некоторых местах, ты обращаешься к массивам вроде $_GET или $_COOKIE, не проверяя перед этим, есть ли в них такой элемент:

>Helper::redirect($_GET['go']);

>Но ведь он может и отсутствовать. Лучше использовать функцию, вроде $this->getQuery('go'), которая делает нужную проверку.

>>773648

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

> попробуй сдампить var_dump(true ^ false )

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

> А нельзя проверять что получилось целое число? Хотя, при работе с float гарантий что оно дейсивтельно целое, нету...

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

> Да и что делать с очень большими числами, которые pow не может обработать?

> Чувствую, что во мне говорит леность и слабость, но всё-таки: может PHP не самый удачный язык для настолько точных расчётов и больших чисел?

> Алсо, если вводить ограничение на длину вводимых чисел, как определять, что число слишком большое для данной битности ОС?

>>Некоторые пытаются возложить на объект Pager лишние функции, например чтение параметров поиска из $_GET или подсчет числа записей в базе.

> Значит этим должен заниматься контроллер?

> $pager = new Pager($records, $currentPage = $this->getQuery('page'), $recordsPerPage = 10);

> У меня getQuery($query) вбросит исключение если запроса не будет существовать

> Я не правильно написал метод getQuery и не понимаю когда должны вбрасываться исключения?

> public function move (World $world) {

> if ($this->dead == false && $world->field[$this->x][$this->y] != $this->icon) {

> if (isset($world->field[$this->x][$this->y - 1]) and ($world->field[$this->x][$this->y - 1] == $world->filler)) {

> $directions[] = array($this->x, $this->y - 1);

> }

> function inclineWord($number, $word1, $word2, $word5) {

> if ($number == 0)

> return "ноль";

> array_push($temp,$spelling[$hundreds]);

> $text = preg_replace("/ноль/u", "", $text);

> $text = preg_replace("/ +/u", " ", $text);

> array_push($temp," (".$number.") ".inclineWord($ones,"рубль","рубля","рублей"));

> } else {

> array_push($temp," (".$number.") рублей");

> 2. http://ideone.com/qFJOX5

> 3. http://ideone.com/SA4FYT

> 4. http://ideone.com/Of9mKI

> https://github.com/TheSidSpears/Students/blob/master/students.sql#L31

> `hash` text NOT NULL,

> $frontController=new FrontController($container);

> return JSONLoader::config();

> return JSONLoader::router();

> public $isAuthorized=false;

> https://github.com/TheSidSpears/Students/blob/master/router.json

> "controller": "Main",

> $this->viewData['students']=$this->c['table']->getStudents($this->sortBy,$this->orderBy,$this->limit,$this->offset,$this->find);

> if ($isAuthorized) {

> $user=$this->c['auth']->getUser();

> $this->user=$this->filterUserData($user);

> setcookie('hash',$student->hash,time()+360012365,'/',null,false,true);

> protected function fillStudent(Student $student){

> function __construct($container){

> $array=file_get_contents($filename,FILE_IGNORE_NEW_LINES);

> LIMIT :y OFFSET :x");

> $rows = $this->db->prepare("SELECT FROM `students` ORDER BY $sortBy $orderBy LIMIT :y OFFSET :x");

> $rows = $this->db->prepare("SELECT FROM `students` WHERE CONCAT(`name`,' ',`sname`,' ',`group_num`,' ',`points`,' ',`gender`,' ',`email`,' ',`b_year`,' ',`is_resident`) LIKE :search ORDER BY $sortBy $orderBy LIMIT :x,:y");

> https://github.com/TheSidSpears/Students/blob/master/app/classes/StudentValidator.php#L98

> foreach($mask['values'] as $value){

> if($s->$field=$value){

> 'regexp'=>"/^([а-яa-z][ ']*)+$/iu",

> function __construct(Student $s, $container, $id=NULL){

> https://ideone.com/t2U2f3

> https://ideone.com/KohVif

> https://ideone.com/0Ye4lY

> У меня идея такая:

> Затем мы делаем такой трюк: возводим $result в квадрат с помощью bcpow (без потерь точности). Сравниваем результат возведения (строку) с $a (строкой). Если они совпали ...

>> Там precision у bcdiv 20, то есть он всегда дробную часть добивает нулями. Тогда нули убирать регулярками/фунциями для манипуляций со строками?

> У тебя по идее знаменатель и числитель целые ведь. Зачем что-то добивать?

>Ну а НОК легко считается через НОД. Реализуй алгоритм Евклида

>У тебя парсинг оказался разделен на 2 части: проверка синтаксиса и сам парсинг.

>> str_pad("Выражение в инфиксной нотации", 61)

> Она не умеет считать русские буквы, имей в виду.

>Алсо, урок по работе с формами: https://github.com/codedokode/pasta/blob/master/forms.md

> >С другой стороны js\node могут привлекать существенно лучшим дизайном языка в мелочах, чем у того же php

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

> А в классе User хранить ссылку на него?

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

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

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

> . Но нужно где-то хранить отношения "поле_ввода=>формат_значения" и где-то объявлять константы.

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

>Почему для разных шаблонов используются разные функции require/include?

> function __construct($container){

> перечитай урок про DI. Это service locator и это плохая вещь.

> Сам класс авторизации странный, половины функций связанных с авторизацией, в нем нет, они в контроллере.

> В student->addInfo есть проблема. У тебя нет фильтрации по разрешенными полям и пользователь может менять любые свойства студента в том числе те, которых нет в форме. ну например что если мы добавим колонку is_admin - пользователь сможет передать $POST['is_admin'] = 1 при редактирвоании. И кстати об этом было написано в моем уроке.

> function __construct(Student $s, $container, $id=NULL){

> Почему ты пишешь код валидации в конструкторе? И почему передаешь контейнер? Почитай про DI.

> тут слишком много всего понамешано.

> Но как я понимаю в контроллеры всё равно контейнер придётся передавать целиком, так?

>> Сам класс авторизации странный, половины функций связанных с авторизацией, в нем нет, они в контроллере.

> Добавил setHash(). Еще что-то нужно? У меня нет идей

> Я подразумеваю, что всё что нельзя редактировать пользователю, то protected. А get_object_vars возвращает только public свойства. Разве не элегантно?

> потому что этот класс используется только так и никак иначе. Создать, проверить на ошибки, вернуть ошибки. Верно так делать? Это сокращает работу с классом до одной строки: $validErrors=new StudentValidator($student,$this->c['table'],$id);

>> тут слишком много всего понамешано.

> Понамешано? Я тебя не понял

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

> "/(^ )/u", //пробелы в начале строки

> $words = preg_split("/\b/ui",

> $regexp = '/\n/';

> $maxlength = mb_strlen($lines[0]);

> for ($column = 0; $column < $numberOfColumn; $column++) {

> $lines[$column] = mb_substr($lines[$column], 1);

> for ($participant = 0; $participant < $total; $participant++) {

>$seats[$participant] = $participant + 1;

> for ($participant = 0; $total >= $skip; $total--) {

> if ($participant <= $total - $skip) {

> $participant = $participant + $skip - 1;

> unset($seats[$participant]);

> sort($seats);

> for ($PlacesToEnd = $skip; $PlacesToEnd > 0; $PlacesToEnd--) {

> $placeInNewCircle++;