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

Если тебе лень выравнивать код руками, закачай его на 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, то рассказать об этом стоит в каком-нибудь другом треде.

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

Ариииииииигато, php-сенпааааааай ^________^

>>536831

>Заодно пусть этот метод передает животному ссылку на $world, и ее не надо будет передавать в каждый метод. Ее проще хранить в поле у животного.

>поле

В смысле в свойстве?

>https://github.com/someApprentice/Cat-and-Mouse/blob/master/Classes/Animal.php#L28

>public function overviewWorld(World $world) {

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

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

>https://github.com/someApprentice/Cat-and-Mouse/blob/master/Classes/Animal.php#L44

>public function searchTheAnimal(World $world, Animal $animal) {

>Назначение метода непонятно. Зачем искать животное в массиве если его координаты хранятся в свойствах x и y?

Этот метод предназначен чтобы найти чтобы найти других животных в обозримой видимости.

>protected $map = array();

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

Мне просто сложно представить этот мир. В принципе, я с тобой полностью согласен. Я подумаю над этим.

>https://github.com/someApprentice/Cat-and-Mouse/blob/master/Classes/World.php#L35

>public function moveAnimal(array $from, array $to) {

>Зачем тут массивы? Почему не $x1, $y1,$x2, $y2?

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

>Также, чем getCoordinate() удобнее чем getX() и getY()? По моему ты тут вводишь лишний массив без которого все только проще.

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

>https://github.com/someApprentice/Cat-and-Mouse/blob/master/Classes/World.php#L43

>$to['y'] = $to['y'] - ($to['y'] - $this->m);

>Что это значит? По моему, если сократить то получается $to['y'] = $this->m;

>Но дело даже не в этом. Смотри, пользователь тебя просит поставить животное на конкретную клетку. Почему ты пытаешься что-то добавлять от себя и поставить его на другую клетку? Где тут логика? Если переданы неверные координаты, надо выкидывать исключение чтобы программист разобрался и исправил программу, а не молча подменять числа в надежде что это сработает. А если та клетка занята? А если той клетки тоже нет? Твоя программа вместо того чтобы сразу сказать что в ней ошибка, пытается это скрыть. ты ведь этим только себе жизнь усложняешь, это тебе же дольше эту ошибку искать придется.

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

>https://github.com/someApprentice/Cat-and-Mouse/blob/master/Classes/Cat.php#L21

>Вот тут 8 почти одинаковых скопированных условий. Копипаста это верный путь к ошибкам и опечаткам. Где гарантия что там их нет? Надо заменить копипасту на цикл. Можно сделать либо 2 вложенных цикла (по x и по y), можно сделать массив направлений и сделать цикл по нему.

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

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

>>537362

break написано с ошибкой

>>537338

Узнают твой id в соцсетях или поставят от твоего имени лайк. Суть кликджекинга в том что на страницу сайта подключается виджет +1 от соцсети, только он делается прозрачным и двигается за мышью. Когда ты кликнешь, ты кликнешь в виджет и если ты залогинен в соцсети то поставшь лайк по которому тебя (зная время и к какому посту поставлен лайк) легко вычислить тебя.

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

Есть расширение для файерфокса, которое ловит попытки клинкуть по прозрачному ифрейму, по моему NoScript.

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

>>537323

Компьютер так не взломать.

>>537151

Узнать можно с помощью кликджекинга или найдя уязвимость в соцсети.

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

>>537010

Никак. Используй кодировку utf8 без ВOM, а с BOM PHP нормально не работает.

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

>>537446

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

>>537452

Тогда наверно проще Апач, но если хочешь можешь и nginx + fpm настроить, Апач там лишний.

>>535405

Задача про правописание, http://ideone.com/BRxyii

> "/\S+[^,]\s(а|но)/u"

Эта регулярка сработает на «длинный нос» так как нос начинается с «но»

> [,.!?:;]\S+

Эта на многоточие

> foreach($regexps as $i => $regexp)

Где фигурная скобка { после foreach? Как догадываться где начинается и заканчивается цикл? Надо ставить фигурные скобки.

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

Задача на номера телефонов: http://ideone.com/HyfBsZ

Задачу про номера телефонов надо проверить на большом числе телефонов, чтобы убедиться что твой код правильный. Но руками подставлять номера — долго и скучно. Пусть работает робот, а не человек!

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

Вот список номеров:

Правильные: array('84951234567', '+74951234567', '8-495-1-234-567', ' 8 (8122) 56-56-56', '8-911-1234567', '8 (911) 12 345 67', '8-911 12 345 67', '8 (911) - 123 - 45 - 67', '+ 7 999 123 4567', '8 ( 999 ) 1234567', '8 999 123 4567');

Неправильные: array('02', '84951234567 позвать люсю', '849512345', '849512345678',
'8 (409) 123-123-123', '7900123467', '5005005001', '8888-8888-88',
'84951a234567', '8495123456a',
'+1 234 5678901', // неверный код страны
'+8 234 5678901', // либо 8 либо +7
'7 234 5678901' // нет +
);

> function checkAndModifyNumber($numberRegexp,$number,$charRegexp){

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

Автозамена http://ideone.com/pVKdWf

Все верно сделано.

Поиск email http://ideone.com/MfScps

Не ищет email с цифрами например 001@001.example.ru

> [a-zA-Z_+-]+(.[a-zA-Z_+-])

> [a-z]+(.[a-z-])

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

Чтобы искать символ точки, его надо экранировать: \.

>>537503

>Статические методы используются когда методу не нужен $this.

Ок, принимаю за аксиому.

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

>LoginManager надо сделать нормальным, не статическим классом

http://pastebin.com/11PAUhyc
Переписал класс, но некоторые методы, которые не используют $this, по-моему вполне можно было оставить статическими. Например logout и authorizeUser
https://github.com/nsdvw/file-sharing/blob/master/app/Auth/LoginManager.php
(смотри git diff 23cb558..2024969 app/Auth , если ты скачивал мой репо, я же не знаю как ты проверяешь, прямо с гитхаба или клонируешь себе)
Ваши аргументы?

----------------------------------------
Какой логикой руководствоваться при выборе имени для переменной? Может есть какой-нибудь мануал на эту тему, типа алгоритм подбора грамотного имени.
Или придет с опытом? (статью с learn.javascript, пронизанную попоболью какого-то тестировщика, не предлагать: там говорится как не нужно делать, но не говорится как нужно)

Например мне необходимо преобразовать объект в массив, а затем упаковать в json. Допустим, метод назову $obj->toJson(), а как назвать промежуточный массив, который используется как локальная переменная этого метода?
$x = array();
foreach($this as $key=>$value) {
$x[$key] = $value;
}
return json_encode($x);

Как именовать X?

----------------------------------------
Почему многие сайты люто тормозят? Это связано с тем, что они сделаны на несвежих смс?
Там картинки что ли не поджимаются, или нет кеширования?
Как вообще это проанализировать? Ну типа последовательность действий, что нужно сделать чтобы понять с клиента, почему тормозит сайт? Наверное, что-то в инспекторе во вкладке network, я там еще не лазил, не знаю на что смотреть и в какой последовательности.
Вот например крупный украинский сайт с объявлениями https://auto.ria.com/
У меня секунд по 15 загружаются страницы.

>>537520

> Почему многие сайты люто тормозят?

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

Освой инспектор в Хроме или ФФ (Ctrl + SHift + I):

https://developer.chrome.com/devtools (англ подробно)
http://habrahabr.ru/post/143767/ (рус кратко)

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

Открыв эту вкладку, перезагрузи сайт с очисткой кеша (Ctrl + F5), для верности можешь еще поставить галочку «disable cache» в инспекторе.

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

Помни несколько вещей:

- браузер ограничивает число параллельных соединений (и скачиваемых файлов) на 1 домен, это то ли 6 то ли 8 соединений. Это делается для защиты от перегрузки сервера. Не знаю, действует ли лимит на одну вкладку или на весь браузер. Лимит рекомендован каким-то стандартом.

- возможно также есть лимит соединений на 1 вкладку в сумме, он не менее 20-30

- некоторые файлы блокируют загрузку других файлов. К примеру, синхронные (по умолчанию они такие) скрипты блокирют разбор HTML кода ниже (а следовательно загрузку упомянутых там файлов) до тех пор пока они не будут загружены. Если такой скрипт через document.write вставляет еще один скрипт то разбор HTML тоже блокируется

- загрузка CSS файлов и веб шрифтов не блокирует разбор HTML или загрузку файлов, но блокирует отображение страницы: CSS целиком, шрифты — текст написанный ими

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

статьи:

http://habrahabr.ru/post/182310/
http://webo.in/articles/habrahabr/56-non-blocking-javascript/
http://webo.in/articles/habrahabr/15-yahoo-best-practices/ (очень хороший сайт, хотя некоторые статьи староваты, рекомендую к изучению)
http://speedupyourwebsite.ru/ (хорошая книга)

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

Алсо по моему опыту тормозят чаще всего внешние скрипты: яндекс-карты, реклама и виджеты соцсетей.

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

>>537520

> class Captcha extends Form

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

Можно было бы писать class FormWithCpatcha extends Form.

Надо сделать капчу элементом формы, а класс Captcha сделать независимым от формы.

> public function fromForm(CommentForm $form)

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

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

> foreach ($this as $propertyName => $propertyValue) {

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

> noise.jpg

Почитай-ка про форматы картинок и их особенности. А то получается анекдот из 1 слова.

> $title = 'FileSharing — page not found';

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

> $app->loginManager->logout();

> $app->loginManager->loggedUser = null;

У тебя пробелы в понимании ООП. Объект должен сам менять свои свойства в этом случае, ты не должен это делать снаружи. Рекомендую вообще закрыть свойство loggedUser от доступа (это называется инкапсуляция, объект прячет свое внутреннее состояние и предоставляет только методы для работы с ним. Также как автомат по продаже кофе дает тебе кнопки и щель для денег но не дает доступа к внутреннему механизму).

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

> if ($_SERVER['REQUEST_METHOD'] === 'GET') {

Исплоьзуй app->request

> if ($loginForm->validate()) {

> if (!$user = $app->userMapper->findByEmail($loginForm->email)) {

Может проверку email тоже в форму засунуть?

> array(

+ 'loginError'=>$loginError,
+ 'loginEmail'=>$loginEmail,
+ 'loginPassword'=>$loginPassword,

Передавай объект формы а не 3 переменных.

> exit( ) ;

Неправильно, надо делать return или app->stop() иначе некоторые модули Слима не вызовутся. У него же луковичная архитектура.

> if $loginManager->loggedUser === null}

Лучше бы метод isLoggedIn() для читабельности

>>536884

Нашел время проверить твой макет.

В меню вверху (HOME PORTFOLIO) у тебя слишком жирный шрифт. У тебя там стоит вес 900, а должен быть более легкий. Посмотри, какой стоит в макете. Это видно невооруженным глазом.

То же самое с кнопкой «See Portfolio», у тебя там стоит 900.

В ИЕ10 вокруг ссылок синие рамки: https://www.browserstack.com/screenshots/14c5de63bad958d3b9d1c48d6d8dcce78bc11626

Я вроде бы в прошлый раз говорил, что в ИЕ в браузере по умолчанию стоит правило типа a img { border: ... } которое лечится прописыванием аналогичного в CSS, только со значением none. ИЕ кстати это делает с благой целью: точно так же как ссылки подчеркивают, чтобы показать, что по ним можно кликнуть. Увы, это в наш макет не вписывается.

> Наш соцсетьнейм

Там дриббл и пинтерест :) С дриббблом рекомендую ознакомиться, там удобно например искать по ключевым словам варианты как может выглядеть та или иная штука в интерфейсе: https://dribbble.com/

Также, у тебя разброд и шатание в плане использования классов и идентификаторов, нет никакой единой системы. В большом проекте это 100% приведет к бардаку. Почитай хотя бы про БЭМ:

http://frontender.info/MindBEMding/
https://ru.bem.info/method/definitions/

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

Соответственно полезно следовать какому-то единому подходу в выборе наименований.

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

Сам БЭМ целиком у них сложный, он включает не только подход к именованию классов, они пытаются вообще сделать полностью независимые компоненты (то есть компонент который содержит HTML, CSS и JS для своей работы). Сейчас разрабатывается стандарт Web Components http://habrahabr.ru/post/210058/ который будет делать примерно то же, только немного по-другому. Но общая идея та же: мы один раз делаем например, виджет для отображения таблицы с сортировкой или для выбора цвета и получаем универсальный компонент который можно подключить на любую страницу.

> @media all and (max-width: 720px) {

> .portfolio-text-buttons-and-works input:nth-of-type(1):checked ~ #wrapper_portfolio_works > .graphic,

> width: 49% !important;}

Почему такой сложный селектор на 6 строк? Тут что-то не так. Ты ведь всего лишь меняешь ширину картинки, почему нельзя написать что-то вроде .portfolio-images > a { width: 49%; } (названия выдуманы)? Я думаю, тут какая-то ошибка.

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

> .android-with-text, .pc-with-text, .phone-with-text, .clock-with-text {width: 50% !important;}

Здесь тоже вместо перечисления конкретных блоков можно было сделать как-то через родителя выборку, например .advantages-block > .advantage { ... }

> #header-words a {font-size: 16px !important;}

> h1 {font-size: 30pt !important;

Когда задаешь font-size задвавай и line-height

> <h1>We are <span class="bold900">Webpaint</span></h1>

Как-то не очень красиво. Что это за стили в имени класса? Красивее сделать так <h1>We are <strong>Webpaint</strong></h1> так как тег strong обозначает смысловое выделение текста.

Я также просил тебя объединить однотипные маленькие картинки (например андроид, pc, часы) в CSS-спрайт чтобы познакомиться с этой техникой.

> .android-with-text {

> .pc-with-text {

> .phone-with-text {

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

Также у тебя много копипасты и в других местах. Например, Get In Touch и Our Featured Works это h2 и можно было бы не повторять CSS код для них, а сделать один стиль для всех тегов h2. Также можно было сделать один класс для идущих за ними подзаголовоков тонким шрифтом.

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

> .portfolio-text-buttons-and-works input:nth-of-type(1):checked ~ #wrapper_portfolio_works > .graphic,

Тут надо вместо nth-of-type использовать идентификатор кнопки с названием. Привязываться к номеру элемента нанадежно.

> .portfolio-text-buttons-and-works input:nth-of-type(1):checked + label > span,

> .portfolio-text-buttons-and-works input:nth-of-type(2):checked + label > span,

> .portfolio-text-buttons-and-works input:nth-of-type(3):checked + label > span,

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

Ну и весит страница в общем 1.2Мб, многовато. Я не помню, говорил ли я тебе что картинки портфолио надо делать в PNG? Просто мне сейчас пришла мысль что если их делать в JPEG с высоким качеством то весить они будут меньше, а выглядеть может быть будут примерно так же (это конечно надо проверять, может и не будут). Тогда мы бы могли сэкономить на весе страницы, так как картинки портфолио там чуть ли не половину этого мегабайта весят.

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

Ну и что касается форматирования, ты писал что долго исправлять, я думаю это неверно, так как есть автоматизированные средства форматирования, бери любой: https://www.google.ru/search?q=css+beautify&oq=css+beautify&aqs=chrome..69i57j0l5.3720j0j7&sourceid=chrome&es_sm=0&ie=UTF-8

Дальше, думаю, тебе стоит поизучать JS и/или PHP.

>>536945

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

Это конечно твое дело, как и в каком порядке что изучать, но я думаю, стоит уделить внимание замечаниям что я написал. Как минимум осознать что там написано.

> И на NULL в parent_id не реагирует, я думал что внешний ключ обязан указывать на уникальное значение.

NULL не нарущает уникальность так как NULL это не конкретное значение, а «неизвестно». В уникальной колонке может быть сколько угодно NULL.

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

> Наверное, это какой-нибудь паттерн, когда внешний ключ указывает на колонку в той же таблице?

Нет, просто связь таблицы с самой собой.

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

Анон выше вроде написал название этой системы обозначений.

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

Да. Миграции это способ зафиксировать изменения структуры базы данных в репозитории. Чтобы можно было вернуться к любой ревизии и воссоздать базу данных в нужном состоянии.

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

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

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

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

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

> Для примера не будем брать PDO, «свелосипедим» свой класс на основе MySQL:

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

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

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

>>536835
Закончил с исправлениями, относящимися к php. (Там еще сто строк замечаний по js, не успел с ними разобраться)
Есть вопросы.

>так как логаут изменяет состояние системы, его надо делать через POST

Я не знаю, как сделать запрос постом, могу только из сабмита формы. А у меня там ссылка. Как при переходе по ссылке сделать запрос через POST, а не GET?

>$file->size = ViewHelper::formatSize($file->size);

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

Да как раз наоборот: в базе размер файла естественно у меня хранится в байтах, но при выводе в представление нужно преобразовать это число в human-readable, подобно тому что делает опция -h у команды ls в юниксах.
Наверное, тебе не понравилось, что я перезаписываю свойство объекта, который не должен меняться. А что делать? Создавать ради этой мелочи новый класс? Типа унаследовать File, принять экземпляр File в конструктор и пропустить через функцию форматирования?

>$user->fromForm($registerForm);

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

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

>https://github.com/nsdvw/file-sharing/blob/master/public_html/index.php#L255

> if ($file->isImage()) {

> тут же всего лишь выбор шаблона для отображения? Тогда это логичнее прямо в шаблоне и делать.

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

> https://github.com/nsdvw/file-sharing/blob/master/views/video_player.tpl

> https://github.com/nsdvw/file-sharing/blob/master/views/audio_player.tpl

> Эти файлы явно сделаны методом копипасты. от нее надо избавиться.

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

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

>>537886

>я думаю, стоит уделить внимание замечаниям что я написал. Как минимум осознать что там написано.

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

>>536866 (OP)
Привет, аноны. Делаю задачу с php narod.
Суть - программка, которая исправляет текст, делает буквы заглавными и делает знаки препинания норм. Но у меня он не хочет исправлять элементы массива при его переборе. Почему это так. Как исправить каждый элемент массива.
<?php

error_reporting(-1);
mb_internal_encoding('utf-8');

$text = "ну что. не смотрел еще black mesa.я собирался скачать ,но все как-то некогда было.";
// Для тестов
// $text = 'roses are red,and violets are blue.whatever you do i'll keep it for you.';
// $text = 'привет.есть 2 функции,preg_split и explode ,не понимаю,в чем между ними разница.';

/ Делает первую букву в строке заглавной /

function makeFirstLetterUppercase($text) {
\t
$text1 = preg_split('/[.]/', $text, -1);

foreach ($text1 as $number => $string )
{
$firstChar = mb_substr($string, 0, 1);
$firstChar = mb_strtoupper($firstChar);
$elseChars = mb_substr($string, 1);
\t$string = $firstChar.$elseChars.". ";

}
$text2 = implode(" ", $text1);

return $text2;
}

>>537940

Ты понял, да не все. Попробуй сделать запросы:

SELECT name, COLLATION(name) FROM students;
SELECT birth, COLLATION(birth) FROM students;
SELECT points, COLLATION(points) FROM students;
SELECT COLLATION(CONCAT(name, points)) FROM students;

Вот тут мануал если понимаешь английский:

https://dev.mysql.com/doc/refman/5.7/en/charset-general.html
https://dev.mysql.com/doc/refman/5.7/en/charset-collation-expressions.html
https://dev.mysql.com/doc/refman/5.7/en/charset-collate.html
https://dev.mysql.com/doc/refman/5.0/en/cast-functions.html

Там внизу написано:

> For implicit conversion of a numeric or temporal value to a string, such as occurs for the argument 1 in the expression CONCAT(1, 'abc'), the result is a character (nonbinary) string that has a character set and collation determined by the character_set_connection and collation_connection system variables.

Может быть еще можно поиграться с переменной collation_connection.

Если не понимаешь то суть в том что у нетекстовых колонок стоит что-то вроде binary collation (сравнивать как есть) и когда ты их используешь с CONCAT то это collation переносится на результат CONCAT.

Решения:

- попробовать явно указать CONCAT(...) COLLATE ... LIKE ...
- явно преобразовывать дату в строку с указанием collation перед тем как склеивать
- не искать по дате/баллам
- поменять collation_connection при соединении с БД, предварительно разобравшись за что она отвечает и не будет ли побочных эффектов. Дописать комментарий в коде об этом.
- сделать нечувствительность к регистру вручную:

WHERE LOWER(CONCAT(...)) LIKE LOWER(...)

В общем воспринимай эту проблему как возможность получше изучить особенности работы со строками в MySQL.

Я решил потестировать явное преобразование под linux:

SELECT COLLATION(NOW()); // binary
SELECT COLLATION(CONCAT('a', NOW())); // utf8_general_ci
SELECT COLLATION(CAST(NOW() AS CHAR)); // utf8_general_ci
SHOW VARIABLES LIKE 'collation_connection'; // utf8_general_ci
SELECT 'a' = 'A'; // 1

Похоже все же дело в этой переменной.

Тут https://dev.mysql.com/doc/refman/5.1/en/charset-connection.html пишут:

> collation_connection is important for comparisons of literal strings. For comparisons of strings with column values, collation_connection does not matter because columns have their own collation, which has a higher collation precedence.

>>537926

Числа и даты в MySQL преобразуется в строку (перед склеиванием) и им присваивается collation из collation_connection. у меня на линуксе она регистронезависимая у тебя очевидно регистрозависимая.

Задать ее можно через SET NAMES COLLATE

Если понимаешь английский, почитай мануал по ссылкам и разберись.

>>537928

Я могу ошибаться, но посмотри какой шрифт на жирной надписи Webpaint. Мне кажется 900 это на ней, а в меню меньше.

Если открыть архив с веб шрифтами то там много степеней веса:

Lato-Hairline.ttf
Lato-Thin.ttf
Lato-Light.ttf
Lato-Regular.ttf (regular это 400)
Lato-Medium.ttf
Lato-Semibold.ttf
Lato-Bold.ttf (это 700)
Lato-Heavy.ttf
Lato-Black.ttf (это очевидно 900)

Попробуй поставить 700 вместо 900 в меню и на кнопках и сравнить на глаз. Мне кажется что там должен быть менее жирный шрифт, у меня в Хроме он такой жирный что буквы вытекают из своих границ.

Я не могу посмотреть в фотошопе, так как у меня lato не установлен и фотошоп заменяет его при попытке кликнуть.

Хотя может там и 900, я попробовал поставить 700 и надписи стали тоньше чем в фотошопе. В общем, посмотри на глаз.

> Еще может быть вес кажется таким что и сами размеры в принципе возможно неправильны?

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

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

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

http://habrahabr.ru/post/166291/

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

Вот еще интересная статья про алгоритмы рендеринга, и то как можно повысить качество рендеринга шрифтов (не в браузере а на уровне ОС): http://www.antigrain.com/research/font_rasterization/index.html

Увы, она на английском, но зато очень много интересной информации содержит.

В CSS есть свойства влияющие на алгоритм рендеринга:

https://developer.mozilla.org/en-US/docs/Web/CSS/text-rendering
http://habrahabr.ru/post/122269/

Не используй их.

Вывод: шрифты рендерятся по-разному. Не рассчиытвай на попиксельное соответствие и совпадение размеров и формы букв на разных ОС.

>>537928

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

>>537986

> Я не знаю, как сделать запрос постом, могу только из сабмита формы

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

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

Ну или JS, но мне кажется тут чистым CSS все решается.

> Да как раз наоборот: в базе размер файла естественно у меня хранится в байтах, но при выводе в представление нужно преобразовать это число в human-readable, подобно тому что делает опция -h у команды ls в юниксах.

делай это в шаблоне:

<?= ViewHelper::formatSize($file->size) ?>

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

Это же относится и к паролю и хешу пароля. Они не должны смешиваться так как это разные вещи.

> Типа унаследовать File, принять экземпляр File в конструктор и пропустить через функцию форматирования?

Нет, просто преобразовывать данные только в момент вывода а не заранее. Либо сделать метод $file->getHumanReadableSize() но по моему это лишнее.

> Так удобно ведь, как иначе мне создавать объект User?

$user = $form->getUser( ) ;
$user = $form->user;

Причем у тебя по моему идет дублирование полей: у тебя есть User со свойствами типа name/email и RegisterForm c такими же свойствами. Зачем? Сделай чтобы внутри RegisterForm был объект User:

class RegisterForm
{
public $user;
public $password;

...

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

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

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

> В общем, я не понял, что ты хочешь.

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

> Разве? Какие-то условные конструкции, бизнес-логика, все это перенести в представление? Мне не очень нравится такая идея, пусть лучше я определю название подключаемого подшаблона в контроллере, и передам эту строку на рендер.

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

> Не понимаю, там совершенно разный код.

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

>>537928

Вот вдогонку еще ссылка про рендеринг шрифтов: http://www.joelonsoftware.com/items/2007/06/12.html

>>538282

Можно наверно.

>>538305

> $Bar->foo и $Foo - это один и тот же объект или два разных?

Сделай var_dump и посмотри на идентификаторы объектов и станет ясно.

Я могу сказать что в такой ситуации:

$a = new A;
$b = $a;

И $b и $a содержат ссылку на один и тот же объект. Чтобы при копировании создвался новый объект надо явно использовать оператор clone:

$b = clone $a;

Также, метод getFoo должен называться setFoo

Также, переменные пишут с маленькой буквы.

ОП(давно не слышно, живой?), помоги с проектированием базы.

Я хотел склепать на yii для пробы простой сайт с объявлениями. Но оказалось, что база у такого проектика не очень простая.
Во-первых, древовидные категории товаров. Прочитал твою статью, выбрал естественно самый простой способ (Adjacency List).
В принципе подойдет, удаление или перемещение ветки относительно несложное, глубина дерева максимум 3 уровня.

Но я не знаю как поступить с уникальными свойствами для каждой конкретной категории объявления. Например у автомобиля есть свойство объем двигателя, год выпуска, а у квартиры количество комнат и площадь, у одежды размер, а у вакансии зарплата и тип занятости (полная, неполная).
Первый вопрос, как хранить подобные свойства? О паттернах для таблиц читал, но поскольку это голая теория я не могу понять как применить на практике.
Если я выберу Single Table Inharitance, то получается все эти уникальные свойства придется делать ее колонками, и для большинства записей они будут выставлены в null.
Мало того, непонятно что тогда делать с моделями. Создавать одну модель? Но тогда будет сложно создавать новую запись, ведь непонятно какие из этих десятков столбцов дать юзеру на заполнение в форме. Будет странно, если пользователь, продающий щенков бульдога, увидит поля "коробка передач", "площадь" и т.д.
Нет, это определенно не годится.

Class Table Inharitance, если я правильно понял, подразумевает создание одной обобщенной таблицы, где хранятся общие для всех объявлений свойства (заголовок, описание, прикрепленные фото, контактные данные и т.д.), и отдельной таблицы для каждой комбинации уникальных свойств. То есть одна запись будет разбита на две таблицы. Меня смущает необходимость джойнить и инсертить сразу две таблицы соответственно при каждой вставке и выборке.

Concrete Table Inheritance отличается от Class TI тем, что не создается отдельная таблица для общих свойств, но для комбинаций уникальных по прежнему создается по таблице.
Вроде бы из этих трех вариантов самый гуманный. Хотя я по прежнему не могу прикинуть, какое же количество таблиц у меня получится в итоге. И какие названия им давать? По именам категорий не получится, потому что у некоторых категорий вообще нет уникальных свойств, а у некоторых комбинации этих свойств могут совпасть (например у меня есть категории "аренда квартир" и "продажа квартир", у них идентичный набор свойств "площадь", "кол-во комнат", "этаж").
Да, пожалуй остановлюсь на этом варианте, а имена таблицам буду давать, конкатенируя через подчеркивание их уникальные характеристики. Надеюсь, у меня не получится полсотни таблиц. Или это нормально? На некоторых cms я видел и по 150 таблиц, но не думаю, что там удачные решения.

Стоп, но опять-таки, что будет при создании новой записи? По задумке сначала пользователь должен увидеть форму, содержащую только общие свойства объявления. А когда он выберет из списка категорию, тогда должны аяксом подгрузиться все остальные необходимые поля. Но какой таблицы? Как связать категорию с таблицей типа Concrete TI? Как я уже говорил, для некоторых категорий набор характеристик совпадает, и я не могу давать таблицам имена по категориям.
Тогда снова кажется более симпатичным вариант с Class TI. Ведь он как раз содержит нужные мне таблицы: одну с общими данными, а вторую с уникальными для каждой конкретной категории. Тогда после выбора пользователем категории из списка должны подгрузиться поля нужной таблицы.
Но мы снова упираемся в количество. У меня 9 или 10 категорий первого уровня, около 120 второго, и черт знает сколько третьего. Там только пород кошек и собак около сотни (мне кстати пришлось писать парсер, не буду же я руками это все вбивать в базу?).
Уникальные свойства впрочем касаются именно категорий второго уровня. Например, я хочу чтобы пользователь выбрал из списка "Недвижимость"->"Аренда квартир", и тогда появится форма для поиска, где в выпадающем списке будут категории третьего уровня (квартиры посуточно, с почасовой оплатой, долгосрочно), а также поля для площади, кол-ва комнат и этажа.
Так что уникальные поля относятся к категориям именно второго уровня.

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

Так что выбрать-то? Class TI требует создания >100 таблиц и моделей, но легкий action create. Concrete TI сэкономит раз в пять кол-во таблиц и моделей, но я понятия не имею, что делать с формой создания нового объявления, как указать нужные для заполнения поля.

>>540043
Ты неясно выразился, поэтому я не сразу понял, тем более в три ночи.

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

У меня получилось так
property (таблица, где хранятся значения уникальных полей; плохо то что нельзя четко указать тип, придется использовать варчар)
-- id
-- value

category_property (сводная таблица, в которой перечислены возможные типы уникальных полей для каждой категории)
-- cat_id
-- prop_type_id

property_type (названия уникальных полей; пожалуй можно пожертвовать нормализацией и перенести их в предыдущую таблицу, избыточность на 100-1000 записей не страшна)
-- id
-- name
(например 31.Кузов;32.Тип занятости)

property_list (Список вариантов значений для каждого типа поля; некоторые поля могут принимать только значение из приведенного списка; прочие, которые не указаны, должны вбиваться пользователем в текстовое поле)
-- prop_type_id
-- variant
(31.Седан;31.Хэтчбек;31.Минивен;...32.На полную ставку;32.Неполная занятость.)

ad_property (собственно таблица с перечислением уникальных свойств для объявления)
-- ad_id
-- prop_id

Таким образом, логика отображения формы создания нового объявления будет такой:
1. Пользователю отображаются поля из таблицы Ad (то есть общие для всех типов
объявлений) и список категорий.
2. Пользователь выбирает категорию из списка.
3. По айди категории выбираем все записи из таблицы category_property.
4. Проверяем для каждого типа свойства, есть ли для него список значений в таблице
property_list
5. Если есть, то собираем все значения для данного типа свойства и выводим в представлении
select. Например кузов автомобиля должен выбираться из списка.
6. Если для данного типа свойства не найдено значений в таблице porperty_list,
выводим в представлении для данного свойства текстовое поле. Обычно это поля типа цена,
размер (одежды), площадь (квартиры) и т.д.

Недостаток в такой схеме в том, что уникальные поля хранятся в одном столбце, и нет возможности указать тип. Одним словом, будет сложность с валидацией формы.
И невозможен ORM, придется использовать DAO. Сложность выборки и вставки.
Но пока действительно трудно придумать что-то лучшее.

>>540068

>ты так по незнанию обозвал выше приведенный паттерн

Я тоже не знал, пошел гуглить.