>>419997

Есть языки работающие в браузере (JS + HTML/CSS), есть серверные (чтобы делать динамические сайты) вроде PHP, Ruby, Python, есть языки для десктопных приложений вроде C++, есть мало где использующиеся языки типа Си (некоторые на них микрконтроллеры программируют, но я слышал что С++ на них тоже давно уже запускается), есть языки для мобильных приложений (Java, Obj-C), есть языки из маковской экосистемы (Obj-C), языки для написания крупных и корпоративных приложений (C#/.NET, Java), и есть применяющиеся энтузиастами новые языки для обкатки новых подходов в программировании вроде Haskell, Scala, D, Rust, Go.

>>420009

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

>>420029

Ну ладно, посмотрим что получится.

>>420059

> есть Node

Какое отношение серверный язык имеет к верстке? Думаю, никакого.

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

Поддержки JS может не быть на сайте где проверяются решения

>>420079

Если ты не понимаешь ООП, у меня есть простой урок: http://archive-ipq-co.narod.ru/l1/pasta.html

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

Алсо ты по моему echo забыл

>>420100

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

для каждого животного которое есть на карте:
животное->сделатьХод()

>>419997
>>420143
Сука, бомбануло так, что решил угробить себе жизнь, зашкворившись об ваш клуб PHP-шников.

> есть серверные

> Ruby, Python

Нет. Это языки общего назначения. Можно использовать как замену PHP на сервере, так и на клиенте. Можно автоматизировать простенькие задачи, типа массового переименования файлов (собственно, Ruby для этого и проектировался изначально). У них только две беды: 1) они динамически типизированы; 2) под них есть только интерпретаторы. На самом деле, это гибридные интерпретаторы, так называемые виртуальные машины, но от этого скорости исполнения не прибавляется, и не исчезает необходимость ставить эту самую виртуальную машину.

> C++

> языки для десктопных приложений

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

> есть мало где использующиеся языки типа Си

Сука, вторично бомбануло. Анончик ниже правильно ссылку дал, C был, есть и будет самым распространённым языком в мире. Есть компиляторы в высокооптимизированный машинный код практически подо всё, что может включаться. Так что это самый кроссплатформенный язык на планете («write once, compile everywhere»). У него только две беды: 1) семантически он довольно близок к Ассемблеру; 2) у него хуёвые средства абстракции. Вместе эти две беды делают его непригодным ни для чего, кроме прошивок для микроконтроллеров, ядер операционок и низкоуровневых библиотек расширения других ЯП. Собственно, он и создавался Керниганом и Ритчи как язык для написания операционных систем. Нет, на нем можно, конечно, написать и гуёвину и даже серверное приложение (и запускать его через FCGI или unix domain sockets), но написание на нем очень быстро превращается в жонглирование указателями и segmentation fault-ами. Даже быстрее, чем на C++. Алсо, так как он семантически близок к Ассемблеру, скомпилированные программы получаются предельно производительными. Если уметь.

> Java

> для мобильных приложений

Анус твой — для мобильных приложений! Java выпускается в трех редакциях: 1) Java ME, которая действительно нужна только для мобилок; 2) Java SE, которая есть платформа общего назначения, но на которой в силу её недостатков можно писать только кровавый энтерпрайз с ебическими требованиями («86 ГБ памяти хватит всем!»); 3) Java EE, которая суть Java SE с дополнительными библиотеками для так называемых серверов приложений (application servers). Бед у неё только две: 1) очень тяжелые и тормозные виртуальные машины, принципиально неоптимизируемые из-за косяков в спецификации языка; 2) многословность, превращающая даже самую простую утилитку для раздербанивания списка покупок в громадную кучу AbstractSingletonFactoryProxyBean-ов. Алсо, её хвалёная кроссплатформенность — это миф, C++ и C имеют компиляторы под такие кофемолки, на которые ВМ Java в принципе не влезет.

> Obj-C

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

> C#/.NET

> для написания крупных и корпоративных приложений

И опять нет. C# и вообще весь .NET — это платформа для написания программ общего назначения. Была запилен как замена для Java (потому что у Java есть фатальный недостаток, Луркоморье тебе в помощь). Минусов два: 1) виртуальная машина (хоть и шустрая); 2) нормальная инфраструктура есть только под сперму. Впрочем, если не нужны шибко развесистые GUI и ограничиваться только десктопом, то имеющейся инфраструктуры хватит с головой.

А PHP — говно ссаное, да.

>>419992
>>419758

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

По базе данных:

> `sex` varchar(4) COLLATE utf8_unicode_ci DEFAULT NULL,

Погугли по слвоам MySQL ENUM

Также, почитай какие есть типы колонок в MySQL:

http://phpclub.ru/mysql/doc/column-types.html (рус, кратко)
http://dev.mysql.com/doc/refman/5.6/en/data-types.html (англ, подробно)

> `groupindex` int(11) DEFAULT NULL,

Номер группы не надо делать числом, вдруг будет группа 2106M или 212-20?

> `birthdate` int(11) DEFAULT NULL,

Тут надо использовать более подходящий тип

Обычно в базе с помощью NOT NULL помечают обязательные поля, а DEFAULT NULL - те, которые можно не заполнять. У тебя все почему-то помечены как необязательные.

По коду:

Каждый класс должен быть в своем файле, и в это файле не должно быть ничего постороннего. Классы пишутся с большой буквы, и файл должен называться как и класс: Student -> Student.php. Файлы классов можно положить в папку с названием app или lib.

Вот такого быть не должно:

> $isRegistered = "<a href='register.php' class=\"navigation\">Зарегистрироваться</a>";

Весь HTML-код должен быть помещен в шаблоны, как описано тут: http://www.phpinfo.su/articles/practice/shablony_v_php.html

Файлы шаблонов стоит называть с расширением .phtml, .php или .tpl и поместить в отдельную папку templates.

> if(isset($_COOKIE['studentscookie']['name'])){

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

$id = getUserId( );
if ($id) ...

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

$students = $mapper->find($text);

Для поиска студентов по строке текста. Или хотя бы так:

$students = findStudents($text);

Вот мой урок про способы работы с базой данных: https://gist.github.com/codedokode/c4cbc4d7dc8e45ea074a (он может быть сложным для начинающего, что поделать).

> WHERE name LIKE '%$search%'

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

> header('Location: http://www.students.ru/profile.php');

Лучше не вписывать имя хоста явно, а вынести в конфиг или брать из $_SERVER['HTTP_HOST']

> try{

> $DBH= new PDO("mysql:host=$host;dbname=$dbname", $user, $pass);

Лучше сделать функцию типа getPdo(). Также, try/catch тут не нужен, при ошибке текст исключения и так выведется если включено display_errors. Вот если что мой урок про исключения: https://gist.github.com/codedokode/65d43ca5ac95c762bc1a

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

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

>>420151

Это индекс по числу упоминаний в гугле: http://www.tiobe.com/index.php/content/paperinfo/tpci/programminglanguages_definition.html

Причем там много странностей:

Поиск в ютубе для С дает 682000, а PHP 640 000 результатов
Поиск в Гугле для С дает 93M результатов, для PHP 157M.

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

Ну и плюс:

> Yandex: SOURCES_NOT_PARSABLE

В общем, странный индекс.

>>420159

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

> Это языки общего назначения.

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

> под них есть только интерпретаторы

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

Кстати, для PHP есть JIT интерпретатор — HipHop VM. Понятно, что динамические языки компилировать бесползено, а вот JIT подход позволяет неплохо их ускорить — например Гугл ускорил JS за счет JIT в v8.

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

На практике использование более оптимального алгоритма дает больший выигрыш чем перестановка команд местами.

> Анус твой — для мобильных приложений!

Я упомянул ее еще в ентерпайзных приложениях. А под мобильными я имел в виду конечно не давно умерший ME, а Android.

> Бед у неё только две: 1) очень тяжелые и тормозные виртуальные машины

Тяжелые они только в том плане что медленно запускаются (что для серверных приложений нормально), а работают они нормально: https://www.techempower.com/benchmarks/ при том что в яве есть и JIT и умные сборщики мусора и инструменты для профайлинга и анализа.

(заметь там в топе С++, а не Си. Потому что что-то сложнее лабы1 написать на Си нереально. Вот вконтактовцы писали — и на это больно смотреть: https://github.com/vk-com/kphp-kdb )

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

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

> Минусов два: 1) виртуальная машина

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

>>420176

> Если я освою джаваскрипт, а потом захочу обычную джаву о

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

>>420206

Спасибо на добром слове. Ты можешь заходить решать какие-нибудь задачки, у нас тут много всего изучается: PHP, HTML/CSS, JS, SQL.

>>420229

> return $this->isWinner = 1;

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

> while ($player->isWinner == 0) {

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

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

Вот и PHP тебе говорит про ошибку внизу:

> PHP Notice: Undefined variable: player in /home/tqO7cS/prog.php on line 132

> return $player->name;

> break;

return выходит из функции (и из цикла соответственно) так что break не нужен.

> создал сам массив с клетками (может не надо было двумерный можно наверное было обойтись и одномерным) не пойму как влепить в него позиции игроков для отображения

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

для каждого игрока:
- определить его координаты и символ
- поставить в указанные координаты указанный символ

В этом случае правда символ игрока заменит цифру, которая в клеточке, это годится? Если нет то можно клеточку с игроком сделать строкой вида 45(#)

> foreach (range($x 10 + 1, $x 10 + 10) as $point) {

Ерунда какая-то. Почему нельзя сделать цикл по j от 0 до 9? И зачем ты кладешь цифры

> unset($line);

Лучше класть в line пустой массив

>>420229

А, я понял, зачем ты кладешь цифры в массив. Так подписаны клеточки. Все равно, заполнять лучше без range, а примерно так:

для i от 0 до 9 {
для j от 0 до 9 {
k = посчитать номер клеточки;
положить k в массив;
}
}

>>420254

Проиграй еще и с википедии: https://ru.wikipedia.org/wiki/%D0%9F%D0%B0%D1%81%D0%BA%D0%B0%D0%BB%D1%8C_(%D1%8F%D0%B7%D1%8B%D0%BA_%D0%BF%D1%80%D0%BE%D0%B3%D1%80%D0%B0%D0%BC%D0%BC%D0%B8%D1%80%D0%BE%D0%B2%D0%B0%D0%BD%D0%B8%D1%8F)

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

>>420294

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

Да. Есть книга по гиту: http://git-scm.com/book/ru/v1

А гитхаб это лишь публичный git репозиторий для проектов.

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

>>420317

> индекс по числу упоминанй в гугле

> Поиск в Гугле для С дает 93M результатов, для PHP 157M.

Это говорит о том, что пыходауны чаще остальных практикуют GDD (Google-driven development).

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

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

> PHP

> высоконагруженных

> масштабируемых

Вот тут я просрался по полной. Пиздуй читать, как «масштабировали» ВКонтакте, и как разрабатывается Google. А Википедия вообще высоконагруженным проектом не является.

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

И не удивляйся. Чем тупее и ленивее человек, тем выше вероятность, что он возьмет что попроще и спросит у кого-нибудь конкретный вопрос, вместо того чтобы читать доки самому. Я также не удивлюсь, если http://php.net/manual/en/ никто из этого треда в глаза не видел.

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

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

> Ты любитель переставлять байты [без «не»?]? Тогда ты изначально загоняешь себя в рамки требованием к похожести на Си.

Логика уровня PHP-треда. Нет, спасибо, я предпочитаю OCaml.

> Php не компилируется и это не мешает создавать на нем гостевухи.

Поправил. Алсо, как одно соотносится с другим?

> Кстати, для PHP есть JIT интерпретатор…

Я сравнивал ЯП с точностью до наличия JIT-ирующих VM и компиляторов. И да, HipHop VM положение не спасает (погугли, по-моему, им удалось ускорить выполнение всего лишь где-то в 2 раза, плюс HHVM JIT справляется только с каким-то подмножеством PHP, в остальном сваливается в обычную интерпретацию), потому что семантика PHP многократно уёбищнее семантики Java.

А YARV даже JIT-ить не пришлось.

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

…Если речь не идет о highload.

> Тяжелые они [VM Java]

…потому, что жрут много памяти. Пидорасы из Sun и Oracle предпочли послать нахуй менеджер памяти ядра, тупо отхавав у него максимум RAM, и перевелосипедить свой менеджер памяти со сборщиком мусора и шлюхами. Алсо, уже на старте в память грузится просто ебическое количество классов и ресурсов.

> Это есть, да, но она все же лаконичнее чем ее предшественник Си++.

Нет. У Java средства абстракции по гибкости примерно как у C. Просто для пидорения scalable robust yoba enterprise solutions этих средств зачастую достаточно (ибо enterprise solutions сами по себе есть примитивные CRUD-ы, просто очень большие). Но если вылезать за рамки use case-ов, предусмотренных Sun и Oracle, начинается ад, боль и унижение.

> ради более удобного синтаксиса там делаются языки поверх JVM вроде Скалы.

Получается CCAHNHA, которая сворачивает голову JIT и выдает перлы вроде выбрасывания исключения при выполнении return из лямбды. JVM, в отличие от .NET, заточена только и исключительно под Java, и любая попытка скомпилировать в байткод JVM что-то, отличное от «public static cherez static» или «AbstractSingletonFactoryProxyBean» приводит к длиннющей лапше опкодов, после JIT-а не укладывающихся ни в какой кеш и ссущих на голову branch predictor-у.

> C#

> Насколько я понимаю, с VM там [в C#] код получается <…> переносимее

В теории — да, как и у Oracle. На практике переносимость получилась еще хуже, чем у Oracle — только в пределах семейства ШИНДОШС. То есть, это даже не все десктопы.

>>420339

Добавляй по 3 поля с номерами чтобы их связать, в чем проблема?

input director[0] = 'Кубрик'
input name[0] = 'Одиссея'
input year[0] = '2004'
input director[1] = 'Лукас'
...

Алсо, ты же эти данные скорее всего вводишь в поля ввода. Что тебе мешает эти поля поместить в форму чтобы не надо было добавлять скрытые поля?

Алсо еще вариант закодировать все это в JSON и слать одним полем.

>>420392

При чем тут PHP? При использовании другого серверного языка вроде Си++ решение потребует меньше кода?

>>420398

> Это говорит о том, что пыходауны чаще остальных практикуют GDD

Ты же сам этот тест на основе гугла и привел в пример, а теперь отрицаешь этот подход. Ну ты смешной.

> больше всего кода до сих пор написано на COBOL.

Пруф? По моему это городская легенда.

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

Тогда веб языки будут в огромном перевесе. Ибо на 1 Си++ программиста приходится человек 50-100 из веба.

> Пидорасы из Sun и Oracle предпочли послать нахуй менеджер памяти ядра, тупо отхавав у него максимум RAM, и перевелосипедить свой менеджер памяти со сборщиком мусора и шлюхами

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

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

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

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

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

> У Java средства абстракции по гибкости примерно как у C.

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

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

Ну и еще там есть препроцессор для отстреливания себе ног.

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

> scalable robust yoba enterprise solutions

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

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

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

> на голову branch predictor-у.

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

Ну и еще раз напомню что тут https://www.techempower.com/benchmarks/ в тесте, имитирующем аспекты работы реального веб приложения, Ява занимает со 2 по 7 место (отставая от 1 места лишь на 15%). Вполне неплохо.

>>420408

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

Вывод надо бы покрасивее сделать. Тебе надо предусмотреть скажем что под одну клетку поля мы отводим ровно 5 символов и добивать пробелами до нужного числа. У тебя есть padRight но что-то она не работает как надо.

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

Если ты сделаешь вывод в несколько строк, то надо будет наверно сделать несколько функций вроде вывестиПервуюСтрочку, вывестиВторуюСтрочку.

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

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

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

$jumps = array(
array('from' => 10, 'to' => 16),
...
);

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

> $this->x = 2;

> $this->y = 4;

> $this->position = 42;

Как я понимаю, position вычисляется из x и y? Если так, незачем вообще хранить это свойство, надо сделать метод getPosition который будет ее считать. Запомни: меньше свойств — меньше геморроя с их обновлением.

Функция getMove слишком огромная. В ней находится огромный if, из-за чего ее тяжело читать. Надо либо сократить код внутри него либо вынести в отдельный метод. Переделай функцию чтобы она занимала не более 15 строк.

Ну вот пример, как эту функцию можно сделать короче:

move() {
step = случайное число;
сдвинутьсяНа(step);
}

сдвинутьсяНа(step) {
увеличить x;
если x переполнился, увеличить y;
если (карта->можноСделатьпереход) {
сделатьПереход;
}
}

> $zone[$i][$j] = $k++;

Это ненадежно. Лучше вычислять k из i и j.

Если что-то непонятно, справшивай. Если понятно, пиши код.

>>420526

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

> После нажатия кнопки ваш код будет протестирован на правильность.

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

> Ваш код должен состоять из функции с именем sequence. Именно она будет тестироваться.

Или может добавить ссылку «как проверяется код» раскрывающую блок с пояснениями.

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

И я бы сделал сигнализацию об ошибках более явной — например добавил галочки/крестик перед пунктом при правильном/неправильном прохождении теста (это можно сделать даже css через generated content). А то не очень ясно что значит результат.

Для исключений хорошо бы выводить информацию о нем, а не просто «произошла какая-то ошибка»:

> encountered a declaration exception

В цифры 1 2 3 4 трудно попасть мышью, я бы добавил к ним невидимый паддинг справа/слева который тоже кликабелен и делает площадь ссылки больше. Чтобы было видно что паддинг относится к ссылке можно подсвечивать его при наведении.

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

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

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

Но вообще, очень хорошо сделано.

>>420526

А, еще, «неправильно» пишется слитно.

>>420538

В объекте (или хеше, словаре) можно хранить что угодно, в том числе другой объект. То есть свойством может быть не только число или строка, но и другой объект, массив, функция. Посмотри на такой код:

// делаем один объект
var person = { name: 'Ivan', age: 20 };

// делаем второй объект и помещаем внутрь него первый
var smth = { man: person };

// Делаем массив и помещаем внутрь него person
var arr = [ person ];

// обращаемся к разным полям так:
person.name
person['name']
smth.man.name
smth.man.age
arr[0].name
arr[0]['name']

Ну и хочу тебе посоветовать параллельно читать сайт learn.javascript.ru гле вроде неплохо все объясняетя. И могу также предложить свои задачки на JS: https://gist.github.com/codedokode/ce30e7a036f18f416ae0

Проверить первые 4 можно тут: http://dkab.github.io/jasmine-tests

> Я вообще не понимат, что делает in в for.

for .. in это цикл по всем свойствам объекта. Он выполняется столько раз сколько есть свойств у объекта (вложенные не считаютя, только свойства первого уровня так сказать), и в переменную firstName каждый раз кладется имя свойства. ну например

for (k in person) {
console.log(k);
}

Выведет 2 строки (не обязательно в том же порядке, порядок может быть любой):

name
age

> И что делает функция list?

Выводит ключи объекта friend, то есть bill и steve

Если на кодеакадеми непонятно, читай параллельно learn.javascript.ru

>>420552

> \twhile ($amount >= $nominal) {

> $amount -= $nominal;

Повторяющееся вычитание заменяется делением (а сложение — умножением), забыл?

Избавься от внутреннего цикла while.

> после небольшого гугления начал делать простым жадным перебором:

Для простого вариант его хватит. Для сложного гугли «задача сдача банкомат» или «задача о ранце»

>>420557

Ну норм. Я вижу ты Concrete TI реализовал? ну ок, давай посмотрим что выйдет.

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

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

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

Например для погрешности можно прокомментировать что это погрешность и в чем она задается — абсолютно или в процентах.

> name` VARCHAR(6500)

Издеваешься? Давай более реалистичное значение

> `added_date` DATETIME NOT NULL DEFAULT CURRENT_TIMESTAMP,

По моему по дефолту проставляется только колонка типа TIMESTAMP так как TIMESTAMP привязан к часовому поясу а DATETIME нет. Изучи-ка различия между TIMESTAMP и DATETIME.

> `allow_errors` BOOLEAN,

нужно добавлять NOT NULL DEFAULT X а то у тебя получается 3 значения для поля

> `type` INT(11) UNSIGNED NOT NULL,

Используй ENUM. Что-то у меня ощущение что ты плохо знаешь типы MySQL, перечитай-ка:

http://dev.mysql.com/doc/refman/5.0/en/data-types.html (англ)
http://phpclub.ru/mysql/doc/column-types.html

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

>>418985

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

>

>------------

>

>$values = значения по умолчанию (пустые);

>$errors = пустой массив;

>

>Если (форма отправлена через POST) {

>Читаем значения полей в $values;

>

>Если (все заполнено верно) {

>Делаем требуемое действие (например вставляем запись в БД);

>Редиректим куда-нибудь;

>Выходим;

>}

>

>Кладем ошибки в $errors;

>}

>

>Выводим форму($values, $errors);

>

>-------------

Подскажи как здесь быть, конкретно вот здесь:

>Если (все заполнено верно) {

Делаю вот таким вот образом:

$name = preg_match('/^[\\w\\.]+$/', $_FILES["file"]["name"]) ? $_FILES["file"]["name"] : '';
$extension = $upload->getExtension($_FILES["file"]["name"]) ? $upload->getExtension($_FILES["file"]["name"]) : '';
$size = (($_FILES["file"]["size"] / 1024) < 20000000) ? $size : '';
$temp = isset($_FILES["file"]["tmp_name"]) ? $_FILES["file"]["tmp_name"] : '';

Дальше не знаю как сделать по лаконичней, либо проверять самостоятельно, складывая значения в массив а потом искать пустые значения, либо пилить новую функцию вида проверитьЗначения($name, $extension, $size, $temp) где выполняется тоже самое и возвращается массив $errors. Все это очень не красиво, на мой взгляд. Посоветуй как сделать красивей и понятней.

Аноны, мне сегодня некогда, давайте может завтра со всем разберемся. Решайте задачки пока.

>>420801

Плюс/минус и карму. Добавление картинок и видео с ютуба. Алсо, советую залить код на гитхаб и показать нам для проверки.

>>420695

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

Открой сайт с вакансиями вроде hh.ru, geekjob, brainstorage и посмотри какие требования предъявляются к разработчикам (также можешь в /wrk открыть одеск-тред и почитать мнения там).

>>420729

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

Да и зачем вдруг тебе понабобилось писать свою? Их же сотни уже написаны. И CMS это инструмент для непрограммистов. для тех кто сайты делает выбирая галочки в админке. Программисты делают сайты на фреймворках вроде Silex, Yii2, Symfony 2.

>>420670

> PHP Notice: Undefined variable: paymentTotal in /home/DKY5mW/prog.php on line 10

У тебя ошибка - обращение к несуществующей переменной. Надо ее исправить. В остальном вроде выглядит нормально.

>>420669

Это подробнее расписано на learn.javascript.ru. В переменную key на каждом шаге цикла помещается имя очередного ключа (свойства) объекта.

>>420607

> Подскажи как здесь быть, конкретно вот здесь:

> >Если (все заполнено верно) {

> Делаю вот таким вот образом:

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

if (validateForm(...))

или

if ($form->validate(...))

Проверка правильности файла наверно будет из нескольких шагов вроде:

Если (файл не загрузился) то ...
Если (файл слишком большой ) то ...
Если (файл не того типа) то ...

Сохраняем файл.

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

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

Анон просил проверить TestHub. Проверяю что успел.

Модели: вот тут вот есть такой код

https://github.com/sqghub/TestHub/blob/master/models/Question.php#L54

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

Вот тут https://github.com/yiisoft/yii2/blob/master/docs/guide/db-active-record.md#working-with-relationships описаны функции link/unlink для создания связей. Может они подойдут?

Также написано вроде возможность подключить транзакции уже есть: https://github.com/yiisoft/yii2/blob/master/docs/guide/db-active-record.md#transactional-operations (может это даже включено по умолчанию).

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

Далее, вот тут ты определяешь отношение в зависимости от типа вопроса:

https://github.com/sqghub/TestHub/blob/master/models/Question.php#L24

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

Еще, ты определяешь свойство

> private $answers = [ ];

Но как я понимаю из документации https://github.com/yiisoft/yii2/blob/master/docs/guide/db-active-record.md#working-with-relational-data там обращение к answers должно перехватываться Юи, и добавляя поле ты возможно ломаешь этот механизм.

В общем, опять же надо проверить, работает ли загрузка вопроса из БД, ответов, и их удаление и сохранение. Там документация скудная, так что если что сразу лезь в исходники Юи и смотри там.

> https://github.com/sqghub/TestHub/blob/master/controllers/TestController.php#L33

Я бы вынес questionTypes и questionTypeLabels в модель, то есть можно сделать метод getAllQuestionTypes и например getQuestionLabel($type). Тогда эти названия можно будет в нескольких местах использовать.

> return $this->render('createTest.twig', [

Это у тебя скопипастено 2 раза, лучше сделать без копипасты, например:

если (пост) {
обрабатываем пост данные;
редиректим если ок;
}

выводим форму;

https://github.com/sqghub/TestHub/blob/master/controllers/TestController.php#L60

> $question = new Question( );

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

> as $questionNumber => $formQuestion

Советую номер передавать отдельным скрытым или видимым инпутом — а то ты намучаешься при переупорядочивании вопросов переименовывать все эти поля.

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

По яваскрипту:

Тут есть другие варианты решения. Например, можно посмотреть в сторону knockout. Это шаблонизатор с дата биндингом, то есть при изменении например типа вопроса в объекте, он сам будет менять DOM и не нужен будет весь этот код с кучей getElementById.

Второй вариант использовать jQuery ради экономии времени на код работы с DOM — он будет чуть короче (но суть особо не поменяется).

Пока в принципе можно так оставить.

Ну или еще один вариант — вообще не делать объекты Question/Answer, а просто сделать несколько отдельных функций вроде поменятьТипВопроса, добавитьВопрос и писать в HTML разметке так:

<button onclick="addQuestion(...)" type="button">Добавить Вопрос</button>

То есть не делать на стороне JS ООП-модель. Если логика простая то это может сработать.

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

> document.getElementById('questions[' + this.number + '][number]')

сделать метод вроде getQuestionNode()

> if (source.id == ("questions[" + this.question.number + "][answers][" + this.number + "][delButton]")) {

Лучше ставить обработчик на саму кнопку:
onEvent(this.getDeleteButton(), 'click', function () { ... });

> function extend(Child, Parent) {

Тут стоит использовать более хороший Object.create если он есть в браузере, а если нет то хак с функцией.

> this.number = question.answers.length + 1;

Инкапсуляцию по моему нарушаешь, ты сделал выделение нового номера ответа не в классе Вопрос, а вынес наружу. Лучше сделать метод question.getNextAnswerNumber() или сделать чтобы при вызове question.addAnswer он сам проставлялся.

> var answerHtml = document.getElementById("questionType" + typeNumber).innerHTML;

> answerHtml = answerHtml.replace(/questionNumber/g, question.number);

Это надо вынести в функцию такого вида:

html = renderTemplate(tpl, { a: 1, b: 2 });

> this.node.onclick = function (event) {

> this.parent.clickFunction(event);

События всплывают, может не обязательно на каждом вопросе обработчик ставить? А хватит одного на весь тест?

Но вообще хорошо что ты ООП используешь, это да.

> document.getElementById('questions[' + this.number + '][number]').innerHTML = newNumber;

Тут лучше сделать по-другому: при создании вопроса находить нужные DOM элементы и сохранять в свойства.

> document.getElementById("questions").getAttribute("count");

Не надо передавать данные через аттрибуты. Вместо этого лучше непосредственно передавать данные в шаблоне, например в виде JSON:

var questions = <?= json_encode($questions) ?>;
initTest(questions);

Остальное я еще проверю, когда будет время.

Шаблоны:

https://github.com/sqghub/TestHub/blob/master/views/test/createTest.twig#L30
ЧТобы не копипастить шаблон вопроса 2 раза, вынеси его в макрос.

>>421092

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

Это да.

>Ну и еще, ты не удаляешь старые вопросы по моему.

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

>описаны функции link/unlink для создания связей

Я так понял, они нужны при добавлении связи для связанных уже по hasOne/hasMany классов. Т.е. классы уже связаны, а линк/анлинк нужен для уже объектов.

>Далее, вот тут ты определяешь отношение в зависимости от типа вопроса:

>Будет ли это работать, если мы загрузили вопрос из БД, поменяли его тип, поменяли ответы и сохраняем?

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

>В идеале, этот класс стоит покрыть тестами для таких случаев

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

>там обращение к answers должно перехватываться Юи, и добавляя поле ты возможно ломаешь этот механизм.

Я думаю оно перехватывается для имеющихся методов. К тому же можно всегда определить гет/сет и пользоваться ими даже внутри класса.

>Я бы вынес questionTypes и questionTypeLabels в модель

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

>если (пост) {

Дело в том, что в блоке "если пост" кода больше, чем в остальном. Это я так перевернул по странному.

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

Пока что это создание нового теста, а не редактирование уже имеющегося, т.е. у вопросов нет id. А в редактировании все равно new Question() делать придется, только в нем каждый раз будет $question -> findOne и затем save.

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

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

>По яваскрипту:

>Например, можно посмотреть в сторону knockout.

Это как-то очень круто. Или рано. Или и то, и другое.

>Ну или еще один вариант — вообще не делать объекты Question/Answer, а просто сделать несколько отдельных функций

Думал об этом, но в итоге пришел к выводу, что объектами проще.

>Ну и еще, надо выносить код работы с DOM в функции.

Даже для функций в одну строчку?

>Лучше ставить обработчик на саму кнопку:

>onEvent(this.getDeleteButton(), 'click', function () { ... });

ВОУ, А ТАК МОЖНО!?
Это же круто. Правда надо прерывать всплытие в каждой функции. Этим я займусь.

>Тут стоит использовать более хороший Object.create если он есть в браузере, а если нет то хак с функцией.

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

>Инкапсуляцию по моему нарушаешь, ты сделал выделение нового номера ответа не в классе Вопрос, а вынес наружу. Лучше сделать метод question.getNextAnswerNumber() или сделать чтобы при вызове question.addAnswer он сам проставлялся.

Унесу в addAnswer. Это, пожалуй, логичнее будет.

>События всплывают, может не обязательно на каждом вопросе обработчик ставить? А хватит одного на весь тест?

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

>Но вообще хорошо что ты ООП используешь, это да.

Да у меня по-другому не получилось как-то. Но иногда чувствуется, что ооп-то кастрированное.

>Остальное я еще проверю, когда будет время.

Есть еще такая идея. Может быть не разбираться со всей этой фигней, вроде нумерации вопросов/ответов в именах и иже с ними, а просто добавлять дивы. Меньше проблем будет потом, когда перетаскивание и все такое буду делать. А в пост отправлять не то, что есть в формах, а собирать данные скриптом в json-строку и получать на сервере готовый массив по сути. Раз уж страница с выключеным js не работает, то гулять на полную с ним, а?

>ЧТобы не копипастить шаблон вопроса 2 раза, вынеси его в макрос.

С ним можно впихнуть что-то в середине шаблона?

>>422154

$0 это даже не настоящая переменная, так как ты не можешь написать например

echo $0;

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

>>422107

Вот советы и замечания, напишу кратко.

> global $desk;

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

Также, вот этот код

> oreach ($desk->cases as $key => $value) {

> if ($this->position == $key) {

Стоит перенести в Desk и сделать методом такого вида:

$target = $desk->getJumpPosition($pos)

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

> if (100 - $lastPosition < $roll) {

Если записать условие в виду $lastposition + $roll > 100 то понять будет гораздо проще.

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

> $zone[] = $k++;

Есть функция range для этого

Функцию getShow можно сделать частью объекта-карты, то есть карта умеет создавать строку, изображающую ее.

> foreach ($zone as &$square) {

> foreach ($players as $player) {

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

> global $round;

> global $players;

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

> } while ($player->isWinner == 0);

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

>>421854

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

По SQL коду

> `sex` enum('М','Ж')

Принято для ENUM использовать латинницу, так как это что-то вроде константы. Например MALE/FEMALE или M/F

> sname

Непросто догадаться что это фамилия, потому стоит добавить комментарий к колонке:

`sname` varchar(20) COLLATE utf8_unicode_ci DEFAULT NULL COMMENT 'Фамилия студента' ....

> DEFAULT NULL,

Обычно NOT NULL значит что поле обязательно для заполнения, а NULL что необязательно. У тебя имя и фамилию необязатаельно указывать?

> ENGINE=MyISAM

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

http://itif.ru/otlichiya-myisam-innodb/

По коду:

> $student = new student;

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

Класс должен быть описан в файле, совпадающем с его именем. Например класс Student уместно положить в файл Student.php. В одном файле должен описываться ровно один класс. Класс StudentMapper должен быть в файле StudentMapper.php

В файле с классом не должно быть постороннего кода вроде такого: https://github.com/MindiMakridi/Students/blob/master/lib/DataMapper.php#L105

> https://github.com/MindiMakridi/Students/blob/master/lib/DataMapper.php#L22

> ORDER BY $sort");

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

> https://github.com/MindiMakridi/Students/blob/master/lib/DataMapper.php#L26

Это непраивльно с точки зрения архитектуры. Класс DataMapper умеет делать только одну вещь: загружать студентов из базы и сохранять ихз в базу. Ни при каких обстоятельствах он не должен ничего выводить на экран. Также, весь HTML код должен располагаться в шаблонах. За их пределами не должно быть ни одного echo.

Разумеется, Mapper также не должен обращаться к кукам.

> public function showStudents($sort, $class){

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

Также, ошибка, ты создал 2 класса, обозначающих одно и то же: student и profile. Они оба содержат информацию о студенте. Раз так, надо объединить их в один класс, соответствующий студенту, незачем делать 2 класса для одной и той же сущности.

> https://github.com/MindiMakridi/Students/blob/master/lib/profileclass.php#L3

Есди ты хочешь сделать константу, делай не глобальную константу, а константу внутри класса: http://php.net/manual/ru/language.oop5.constants.php

https://github.com/MindiMakridi/Students/blob/master/lib/profileclass.php#L16 — тут слишком длинная строка, запиши массив вертикально на несколько строк.

Также, в файле profileclass съехало форматирование, надо пропустить его через phpformatter.com ибо читать тяжело (и другие файлы где съехало форматирование тоже). Если ты используешь IDE, там есть горячие клавиши для форматирования: https://gist.github.com/codedokode/8759492

> https://github.com/MindiMakridi/Students/blob/master/index.php#L38

Я думаю, лушче тут оставить только один include, а первый файл подключить из main.html

> https://github.com/MindiMakridi/Students/blob/master/lib/functions.php#L4

Не надо тут писать return $x = $y; Надо писать либо так:

return $y;

либо так:

$x = $y;
return $x;

То есть не надо складывать 2 команды в одну строку.

> https://github.com/MindiMakridi/Students/blob/master/lib/DataMapper.php#L59

Здесь ты передаешь информацию о студенте в массиве. Но у нас же есть класс (точнее 2 класса) описывающих студента. Значит, незачем использовать массив, надо передавать объект класса Student.

> https://github.com/MindiMakridi/Students/blob/master/templates/main.html#L14

Шаблон предназначен для вывода переданных ему данных. Он не должен лезть в другие данные типа POST/GET и тем более не должен ничего загружать из базы — это не его задача. Это должно делаться в index.php

Это https://github.com/MindiMakridi/Students/blob/master/templates/template.html#L2 тоже надо вынести из шаблона куда-нибудь в index.php или аналогичный файл.

> <?php echo

Это лучше писать короче как <?= $x ?> (короткие теги используются только для вывода, а для остального исплоьзуй длинные <?php)

Шаблоны Students/templates/profile.html и Students/templates/register.html содержат копипасту. Надо избавиться от дублирования похожего кода, так как это очень плохая вещь, и объединить их в один файл без повторов.

Куки выставляются, как известно, отдачей HTTP-заголовка Set-Cookie. Заголовки в протоколе HTTP идут до тела ответа, потому если ты вывел хоть один пробел, заголовок (и куки) выставить уже нельзя. Потому вот эта строчка: https://github.com/MindiMakridi/Students/blob/master/profile.php#L46 работать не будет.

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

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

>>421538

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

>>421462

Насчет яваскрипта, я еще немного подумал, и все же не уверен, нужны ли нам там класссы Вопрос/Ответ. Я думаю так: у нас вся информация о вопросах/ответов фактически хранится в множестве инпутов. Стоит ли делать класс который ее будет дублировать, писать сложные методы для синхронизации этой информации в обе стороны(между классом и формой)? Не проще ли тут сделать несколько функций типа ПереключитьТипВопроса, ДобавитьВариантОтвета, без построения дерева всех вопросов и ответов?

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

Ну и насчет сложной работы с id, это надо как-то убирать в функции вроде такой:

var deletBtn = getQuestionDeleteButton(questionNode);

Ну и искать наверно все же удоюнее не по сложному id, а по классу. Для этого конечно придется написать функцию поиска в DOM элемента с определенным классом. То есть например все кнопки удаления у нас помечены классом

js-delete-button (js значит что это не для CSS а для JS класс)

И когда мы хотим получить кнопку удаления в определенном вопросе. то пишем так:

var deletBtn = findFirstByClass(question, 'js-delete-button');

В новых браузерах для поиска по классу есть querySelector и getElementsByClassName, в старых IE придется обходить все узлы-потомки и сравнивать класс.

В jQuery это пишется например так (аналогично примеру выше):

var deleteBtn = $('.js-delete-button', question);

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

> Эту функцию я планирую использовать только при создании. Хотя можно и при редактировании

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

> Я так понял, они нужны при добавлении связи для связанных уже по hasOne/hasMany классов. Т.е. классы уже связаны, а линк/анлинк нужен для уже объектов.

Я так понимаю, что link для связи один-ко-многим проставляет в одном объекте (или в табице БД?) id другого то есть как раз связывает их.

Глянь в исходники, может там можно понять?

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

Ну редактирвоать существующий удобнее:

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

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

Если ты создал свойство answers то при обращении к нему из класса перехват работать не будет. Он же на магических методах вроде _ _ get основан: http://php.net/manual/ru/language.oop5.overloading.php#object.get

> Даже для функций в одну строчку?

У тебя там раз 10 скопирован код склеивания id с номером вопроса. Это копипаста логики, и ее не должно быть — жолжна быть функция например вычисляющая id для элемента.

> Ну я это взял как универсальное решение прямо из учебника.

Ты либо не так понял, либо это старая статья. В новых браузерах Object.crеate лучше, так как он не создает лишних промежуточных объектов.

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

Да это е так сложно, по моему. делаешь скрытое или видимое поле «Номер вопроса» и при смене порядка вопросов перепроставляешь все эти поля заново по возрастанию. Соответственно в PHP скрипт для каждого вопроса приходит его порядковый номер и он может их отсортировать.

> а собирать данные скриптом в json-строку и получать на сервере готовый массив по сути

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

Ну и при ошибке в JS скрипте форма все равно отправится и данные не потеряются.

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

>>422684

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

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

https://gist.github.com/codedokode/d7e7f11449fc3bcb24b4

или такого

https://gist.github.com/codedokode/9424217

Это, согласись, уже ближе к rss лентам и корзинам. Также, мы даем дополнительные задания на изучение HTML/CSS, JS и SQL.

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

- учебник (основы PHP и ООП)
- задание на список студентов (учимся работать с формами, БД, выводить данные)
- задание на файлообменник (изучаем MVC и микрофреймворк Slim, учимся работать с файлами, может быть изучаем шаблонизатор Twig)
- какое-нибудь задание на большой фреймворк вроде Yii/Yii2/Symfony
- автоматизированное тестирование
- задания на HTML/CSS/JS/SQL (можно делать параллельно с другими) чтобы подтянуть их уровень

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

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

Наверно, но я бы назвал это «вот пример веб-приложения которое я сделал»

> я Вам на почту отписался

Ок, я там подробнее отвечу потом. Насчет заданий, что насчет 2 примеров заданий выше? Задание на список студентов выглядит просто, но даже на нем можно много разных интересных вещей изучить. А на файлообменник посложнее.