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

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

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

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

>>631159

Это 100% баг. Я установил чистую систему + из нее гном + gdm3, никаких настроек не менял, ничего не устанавливал, версия дебиана №8 (в названиях я путаюсь так как в них нет никакой логики).

И я знаю причину бага.

Алсо линукс я знаю не так и плохо.

>>631167

Нет, нелогично. Хватит одной, обычная связь многие-ко-многим.

>>631173

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

>>631210

Работает в 15-й убунте, скайп 4 версии без рекламы, фейсбука и прочей дряни.

Кстати кто не умеет отключать фейсбук и рекламу в скайпе на винде: прописываете в хостс apps.skype.com на 127.0.0.1 и перезапускаете скайп.

>>631214

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

>>628558

Попробуй порешать наши задачи, показывать решения, исправлять замечания - я уверен, что будет прогресс. Тем более что ты не совсем начинающий, тебе легче чем другим. Вуз вузу рознь, если тебя интересуют информационные технологии, то надо искать такие программы: https://park.mail.ru/pages/index/ или например ШАД Яндекса. Ну и есть большая разница, сдавать зачеты или сидеть разбираться в заданиях.

>>628704

Названия неудачные и затрудняют чтение кода.

$totalAr -> $girls, $positions (и не пиши слово array в названии переменной, и тем более не сокращай его)
$e -> $number, $current
as $k => $v -> $key => $position

Заполнение массива проще сделать функцией range (или хотя бы циклом for).

В остальном верно.

>>628710

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

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

>>628770

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

>>628775

> w1: https://ideone.com/sgIZfu

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

> w2: https://ideone.com/DBu62z

Верно.

> w3: https://ideone.com/7hEeAu

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

> w4: https://ideone.com/rdAHwD

Верно, но тут

> $anonDice1 + $anonDice2) > ($compDice1 + $compDice2))

можно было использовать compSum/anonSum, и иф принято писать с маленькой букв.

>>628800

Молодец, верно.

>>629061

Файлы names/surnames лучше убрать в папку, например data.

> name text NOT NULL,

> surname text NOT NULL,

Тут надо varchar с разумным ограничением длины. Тебе надо прочитать в мануале главу про типы колонок (всю!) и выучить названия не менее 5 разных типов (INT и BIGINT считаются за один). Также, ответь на вопрос, в чем разница между REAL и NUMERIC? Где может пригодиться нумерик? Какой тип использовать для хранения телефонного номера? Номера паспорта?

> status text NOT NULL,

> gender text NOT NULL,

Тут нужен enum (который судя по документации в постгрессе даже лучше чем в майскул: http://www.postgresql.org/docs/9.1/static/datatype-enum.html , перевод http://postgresql.ru.net/manual/datatype-enum.html )

> sgroup text NOT NULL,

> email text NOT NULL,

варчар нужен

> byear text NOT NULL,

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

Также, в постргресе есть очень крутая штука, ограничение на значение поля CHECK: http://postgresql.ru.net/manual/ddl-constraints.html#AEN2554

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

Для уникальных полей надо поставить соответствующее ограничение.

> CREATE TABLE tokens (

А почему отдельной таблицей? И почему нет внешнего ключа? И почему ид текстом?

Также, я вижу ты не очень знаешь SQL, советую порешать задачи на него из ОП-поста.

> CREATE EXTENSION IF NOT EXISTS plpgsql WITH SCHEMA pg_catalog;

> REVOKE ALL ON SCHEMA public FROM PUBLIC;

> REVOKE ALL ON SCHEMA public FROM postgres;

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

https://github.com/foobar1643/student-list/blob/master/public/design/template/form.html
Это не должно быть в публичной папке, так как это код, не предназначеннный для вызова пользователем напрямую.

https://github.com/foobar1643/student-list/blob/master/app/AutoLoader.php
Не надо копипастить, копипаста - зло, сделай список путей в массиве, и их перебор в автозагрузчике. Более того, если указать эти папки в include_path то можно использовать встроенный в пхп автозагрузчик spl_autoload (детали в мануале).

А лучше конечно освой неймспейсы, урок https://github.com/codedokode/pasta/blob/master/php/autoload.md

> https://github.com/foobar1643/student-list/blob/master/app/AppCore.php#L11

> $this->view_settings = new stdClass();

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

> } catch(PDOException $e) {

> print("Database connection error. " . $e->getMessage()

Неправильная работа с исклчениями. ВОт паста.

--------

Как надо обрабатывать исключения:

- записать информацию в лог
- показать пользователю заглушку
- на заглушке выставить HTTP 503 код ответа для роботов
- на компьютере разработчика (при display_errors = 1) показать подробности и стектрейс

Как сделано у тебя:

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

Как будет если просто не ловить исключение:

- информация пишется в лог (ок)
- на компьютере разработчика выводятся подробности (ок)
- пользователь видит белую страницу (не ок)
- отдается код 200 (не ок)

Но даже так лучше чем твой вариант.

Почитай также урок https://gist.github.com/codedokode/65d43ca5ac95c762bc1a

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

Неправильно что ты создаешь PDO в контроллере. Заечм он там? Его надо создавать например в файле bootstrap.php одитн раз и использовать для создания классов работающих с БД. А ты в каждом контроллере создаешь свой ПДО.

Название странное AppCore. Что еще за коре? Это базовый класс контродлера, ControllerBase.

> protected $connection_error;

Зачем поле? Зачем хранить строку с ошибкой?

> $this->view_settings->page_data = new stdClass();

> $this->view_settings->error = array();

> $this->view_settings->page_data->student = array();

Не очень понятно зачем это. Например зачем в базовом классе создавать поле student - он нужен на всех страницах? И что значит page_data? Название ничего не говорит мне.

> https://github.com/foobar1643/student-list/blob/master/app/Controller/ControllerIndex.php

> private $pager;

> private $total_pages;

> private $offset;

> private $sorting_pattern;

Зачем их делать полями? Тебе надо эти значения сохранять между вызовами?

> process_get_request

У нас в пхп в рекомендации PSR принято функции и переменные именовать кемел кейсом: processGetRequest()

> if($get_data['page']

А если нет гет-параметра page то мы получаем ошибку доступа к несуществующему элементу массива? У тебя включено отображения ошибок? Ты видишь, что они есть? Если нет, выставь error_reporting = -1, display_errors = On в php.ini у себя локально.

> $index = array_search($get_data['sort'], $this->sorting_patterns);

> if($this->sorting_patterns[$index]) {

Тут нужен in_array()

Также, не вижу смысла разделять контроллер на process_get_request и run, так как непонятно по какому принципу код разделен на 2 части. Лучше сделать одну функцию.

https://github.com/foobar1643/student-list/blob/master/app/View/ViewIndex.php
Не очень понятен смысл этого класса, не проще ли этот код поместить в контроллер?

https://github.com/foobar1643/student-list/blob/master/app/AppConfig.php
Конфиг лучше сделать обычным php- или ini-файлом:

$config['x'] = 1;

или

[db]
host=localhost
username = 1

https://github.com/foobar1643/student-list/blob/master/public/design/template/form.html#L2

> Проверьте правильность вводимых данных и попробуйте еще раз.

А где текст ошибки?

https://github.com/foobar1643/student-list/blob/master/public/design/template/form.html#L23
Вместо print надо использовать <?= ... ?>. Также, нужна защита от XSS (htmlspecialchars). У меня есть урок. Также нужна клиентская HTML5 валидация.

https://github.com/foobar1643/student-list/blob/master/app/Controller/ControllerForm.php#L28

> $this->student->name = $this->form_validator->process_field($post_data["name_field"]);

htmlspecialchars надо делать только при выводе данных. Зачем ты в базу данных вставляешь данные в HTML виде? Надо хранить в базе исходыне неискаженные данные.

Работу с авторизацией и токенами надо вынести в отдельный класс.

https://github.com/foobar1643/student-list/blob/master/app/Controller/ControllerForm.php#L43

> header("Location: form.php?notify=success"); /

После этого не надо выводить страницу. Ее все равно никто не увидит.

>>629116

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

>>629158

Не, надо знать раза в 2-3 боьше чем ты перечислил, ветки еще, мерджи, конфликты, пул/пуш. Советую почитать git-book, есть на русском и там достаточно просто все.

>>629279

Для этого надо знать языки HTML, CSS, JS, PHP, работу с базой данных. Довольно много знаний надо.

> посоветовали бесплатное амазноновское хранилище.

Оно не бесплатное. Особенно с нынешним-то курсом.

>>629285

> private function chooseProfession($post) {

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

> public function countTotalSalary($qty, $rank, $post, $boss) {

Это не очень правльная функция, должно быть так:

public function countTotalSalary() { ... }

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

> abstract class Employee extends Department {

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

У тебя же неправильно: Сотрудник это не разновидност Департамента, значит наследование использовать так нельзя.

> class Manager extends Employee {

> protected function countSalary() {

Ты скопипастил тут код 4 раза.

>>629306

Будем считать этот косяк ОПа тестом на сообразительность.

>>629523

Тебе я написал советы и замечания выше

>>629589

Начни с гугления. Интеграция значит обмен данными между сайтом и системой 1С (например выгрузка данных о товарах из 1С в магазин, и выгрузка данных о заказах из магазина в 1С). Для этого 1С предлагает несколько вариантов, например:

- через ФТП, 1С будет выклаыдвать новые данные, например новые товары, на ФТП сервер и оттуда же забирать обновления с сайта, например заказы.
- через HTTP, 1с будет отправлять запросы с данными на какой-то URL

Подробнее лучше уточнить у ваших 1с-ников. Ведь вполне возможно что используются не стандартные возможности 1С, а что-то самописное.

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

>>629605

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

>>629631

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

>>629732

Неправильно считает если взять сумму кредита = 1000: http://ideone.com/B81a83

Должно быть всего выплачено 2030 (1000 + процент + комиссия)

Также, выражение повторяется 3 раза, избавься от повторов: $creditBalance x $percent + $servicePayment

>>629735

Босюь что никто не станет решать твою задачу, решай сам.

>>634316

Давай начнем с теории. Знаешь ли ты зачем придуманы исключения? Читал ли урок https://gist.github.com/codedokode/65d43ca5ac95c762bc1a

Если нет, прочитай.

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

try {
$app->run();
} catch (..) {
$app->showErrorPage();
}

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

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

Теперь прочти пасту про то что должно происходить при исключении:

------

Как надо обрабатывать исключения:

- записать информацию в лог
- показать пользователю заглушку
- на заглушке выставить HTTP 503 код ответа для роботов
- на компьютере разработчика (при display_errors = 1) показать подробности и стектрейс

Что получается при catch + echo:

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

Как будет если просто не ловить исключение:

- информация пишется в лог (ок)
- на компьютере разработчика выводятся подробности (ок)
- пользователь видит белую страницу (не ок)
- отдается код 200 (не ок)

--------

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