>>512549

>Пикрил КОНФИГ, а не код.

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

def some_db_logic():
db.query("select for replace a from b")
db.query("replace b set a = c where hui")

и у тебя таких функций десятки по коду
и ты хотел бы каждую завернуть в транзакцию. тебе чтоб не писать begin перед и commit/rollback после каждой - ты делаешь декоратор

def transactional(f):
def inner():
db.begin()
try:
res = f()
db.commit()
except:
db.rollback()
return res
return inner

и затем пишешь просто

@transactional
def some_db_logic():
...

или как это делается в ruby, просто на блоках кода

ActiveRecord::Base.transaction do
david.withdrawal(100)
mary.deposit(100)
end

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

То что ты говоришь про включение/выключение логирования делается через фича-флаги. https://www.martinfowler.com/articles/feature-toggles.html
то есть просто

if feature.notify_manager:
log.info('хуё-моё')

декораторы тут нахуй не нужны.

> не отличаешь декорацию ОБЪЕКТА от ФУНКЦИИ

так декоратор и есть функция. по определению.
если есть функция f(args: T) -> result: U
то декоратор - это такая функция, которая принимает в себя f и возвращает функцию g с такой же сигнатурой что и f
decorator(f) -> (g(args: T) -> result: U)

в коде используешь так:
допустим у тебя есть функция
sum = fn(a, b) -> a + b
и где-то в коде ты пишешь
value = sum(1, 2)

когда ты хочешь задекорировать её - ты просто добавляешь
sum = logger(sum)

и в итоге получается
sum = fn(a, b) -> a + b
sum = logger(sum) // в этой строчке вся мякотка
value = sum(1, 2)

Смотрю https://youtu.be/jSmfRyk-Jms

На 5:10 звучит фраза: "Можем выполнить за неделю, а можем за два дня. Если за два дня - то мы разрешаем себе говнокодить и говнокодим."

Этой фразы в общем достаточно чтобы понять уровень ДАфанасьева - мидл не выше среднего.

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

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

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

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

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

Вот прекрасная иллюстрация https://youtu.be/E8_dcn1k9aA на 9:20. Соискатель не отрефачил код и обосновал это фразой "мне что, весь проект отрефакторить?", что говорит о том, что проект находился в уёбищном состоянии, и все с этим жили. Ну живите дальше, чо.

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

>>515604

Мне кажется, проблема в том, что ты просто не понял, что такое функция. Давай начнем с основ.

Значение — это вещь, которую может обрабатывать программа. Например, числа и строки — это значения, и программа может как-то с ними работать, например, складывать числа, или разрезать строки на части. Вот пример кода:

$a = 4;
$b = 'x';

Здесь мы записываем в переменную $a значение 4, а в $b значение 'x' (строку). А вот пример, как можно преобразовать одно значение в другое:

$a = 4;
$b = $a + 1;

Здесь во второй строчке мы берем значение 4, прибавляем к нему 1 и получаем значение 5, которое мы записываем в $b.

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

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

mb_substr("hello", 2);

В данном примере кода мы вызываем функцию mb_substr и передаем в нее два значения: "hello" и 2. Эта функция возвращает строку "llo" (то есть, часть строки hello, начиная со 2-го символа), но мы никуда не сохраняем это значение и оно просто выкидывается.

Вот как мы можем сохранить возвращенное функцией значение в переменную:

$x = mb_substr("hello", 2);

Здесь функция mb_susbtr возвращает строку "llo" и она записывается в переменную $x. То есть, код выше примерно эквивалентен такому коду:

$x = "llo";

Разумеется, возвращаемое функцией значение можно записать не только в переменную, но и передать в другую функцию. Вот пример, где мы отрезаем часть строки функцией mb_substr, и передаем эту часть в функцию mb_strlen (которая считает длину строки):

$x = mb_strlen(mb_substr("hello", 2));

Здесь мы сначала вызвыаем функцию mb_substr, она возвращает значение "llo", и затем мы вызываем функцию mb_strlen, передавая в нее эту строку. Функци mb_strlen вернет значение 3, которое мы запишем в переменную $x.

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

>>516029

Посмотрел код. Действительно, в будущем лучше освоить гитхаб и заливать туда.

> $order = $sort_list[$sort];

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

Вместо этого надо передавать в функцию тип сортировки и пусть функция Student::getAllStudents сама построит нужный SQL-код. Кстати, типы сортировки можно сделать константами.

> function sort_link($title, $a, $b) {

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

> return '<a class="active" href="?sort=' . $b . '">' . $title . ' <i>▲</i></a>';

Здесь данные подставляются в HTML без экранирования.

> $page = isset(($_GET['page'])) ? html(intval($_GET['page'])) : 1;

> $search = html($_GET['search']);

Вызов html() здесь не нужен. Экранирование надо делать в шаблоне при выводе данных, а не при получении их из $_GET. Потому что при подстановке строки в HTML, в URL, в SQL-запрос используются разные правила экранирования и заранее ты не угадаешь, какой способ надо использовать.

> $s_string = "Показаны только абитуриенты, найденные по запросу \"$search\".";

Эту строку надо формировать в шаблоне, а не здесь в контроллере. Ты просто передаешь в шаблон переменную $search, и в нем ее проверяешь и выводишь текст при необходимости.

> class Student{

> private $name;

> private $surname;

> if ($page == 1) $offset = 0;

Код оформлен не по PSR. Скобка после имени класса идет на новой строке. If оформляется в несколько строчек. Если ты отправишь такой код на какую-то вакансию, то произведешь плохое впечатление. Советую почитать второй пост треда про оформление кода.

> $stmt->execute([$this->name, $this->surname, $this->sex,

> $this->group, $this->place, $this->email,

> $this->byear, $this->points,$this->pass]);

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

> private $name;

> function __construct($name, $surname, $group,

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

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

> $res = $pdo->query("SELECT pass FROM STUDENTS WHERE email = '$this->email'");

> $string = "SELECT * FROM STUDENTS ORDER BY $order LIMIT $st_per_page OFFSET $offset";

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

> $hash = $res->fetch(PDO::FETCH_NUM);

> $this->pass = $hash[0];

Используй fetchColumn, и он сразу тебе вернет нужное значение, а не массив.

> while($row = $result->fetch(PDO::FETCH_ASSOC)){

>$arr[] = $row;

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

> WHERE `name` LIKE ? OR

> surname LIKE ? OR `group` LIKE ? OR points LIKE ?

> $stmt->execute([$search, $search, $search, $search]);

Здесь можно было использовать именованные плейсхолдеры вроде `name` LIKE :search, чтобы не передавать один параметр 4 раза.

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

> spl_autoload_register(function ($class) {

> else die( $path);

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

> $errorN = $errorSn = $errorSe = $errorGr = $errorM = $errorBy

В такой ситуации не нужно делать 10 однотипных переменных, можно просто сделать массив или объект.

> else {

> $student['email'] = $email;

> $student['name'] = $name;

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

> class ConnectionDB{

> private static $host = 'testdomain'; // Change as necessary

Настройки надо вынести в конфиг, чтобы не надо было править код. Конфиг может быть в разных форматах: JSON, ini, или даже в виде PHP-файла с переменными.

> private static $chrs = 'utf8mb4';

Не надо это делать настройкой, так как скорее всего твой код не заработает с другими кодировками.

Также, у тебя в коде не определена константа PASSWORD_DEFAULT.

> `name` varchar(40),

Для полей, которые обязательны к заполнению, надо указывать NOT NULL

> sex varchar(2),

Здесь надо использовать тип ENUM, иначе можно вставить все, что угодно.

> email varchar(40),

Здесь надо добавить ограничение уникальности, чтобы нельзя было в БД вставить 2 одинаковых email.

> byear integer,

Здесь было бы хорошо добавить ограничение CHECK, которое бы запретило вставлять слишком маленькие или слишком большие значения.

> private static $opts =

> [PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,

Надо оформлять так:

private static $opts = [
(4 пробела) PDO::ATTR_ERRMODE => PDO::ERRMODE_EXCEPTION,
(4 пробела) ...
];

> preg_match('/^[A-Яa-яЁё]{1,20}$/ui',

Я бы разрешил в имени еще дефисы и апострофы, так как бывают фамилии вроде Римский-Корсаков или д'Артаньян.

> ('/^(м|ж){1}$/ui', $val);

Пол лучше сделать константами вроде Student::GENDER_MALE = 'm', а еще лучше освоить PHP8 и сделать enum (перечисление). А в БД вставлять m/f. Аналогично для поля местный/приезжий.

> public function validMail($val){

> global $pdo;

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

> if(password_verify($val, $_COOKIE['logged'])) return "";

Тут две ошибки. Во-первых, валидатор не должен сам лезть в куки, ему должны их передать снаружи. Во-вторых, ты тут в одной функции смешал проверку правильности email и аутентификацию пользователя. Это должны делать две разных функции. Одна проверяет правильность и уникальность email, вторая проверяет хеш.

> if($val>0 && $val<300) return "";

> else return "Введите число баллов от 0 до 300";

Но твоя функция не разрешает ввести 0 или 300. Тогда надо писать "до 299". Также, нет проверки, что введено целое число.

Год рождения надо проверять, что он не менее 1880 и не более текущего.

Шаблоны обычно называют index.phtml или index.html.php. Их логично поместить в папку templates или views.

> <? if($s_string) echo $s_string?>

В шаблоне используют версию if с двоеточием, а echo не используют.

> <input type="submit" value="Найти" />

В HTML не пишут слеш в конце одиночного тега. Так делают только в XML.

Заголовок шаблона лучше вынести в отдельный файл, чтобы не копипастить.

> Электронная почта: <input type="text"

Тут надо использовать type="email". На мобильных устройствах это меняет вид клавиатуры для более удобного ввода. Я советую почитать MDN https://developer.mozilla.org/ru/docs/Web/HTML/Element/Input , там перечислено, какие бывают типы полей ввода.

> Пол (м/ж): <input type="text"

Здесь надо использовать радиокнопки.

> Год рождения: <input type="text"

Здесь лучше тип number, опять же. У него можно указать параметры min/max, и он показывают цифровую клавиатуру на моб. устройствах.

Аноны, вам не кажется, что с образованием что-то не так? Почему работодатели на вакансию клепателя форм требуют высшее образование? Ну бред же.

На мой взгляд, для рядового программиста не требуется высшее образование. Он должен получить среднее профессиональное образование, например, в колледже или техникуме, где его обучат компьютерной грамотности, алгоритмам, ООП, паре языков вроде Питон/Java, ассемблеру, HTML, типографике, сетям, основам юзабилити, SQL. Года 3 для этого хватит. Мне кажется, этого достаточно, чтобы стать веб-разработчиком.

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

А высшие учебные заведения должны заниматься больше computer science. Их выпускники не должны писать микросервисы для перекладывания JSON из сокета в базу (разве что на 1-м курсе в рамках изучения основ). Они должны разрабатывать новые языки программирования, проектировать интегральные схемы, методы оптимизации, делать сложные вычисления, новые архитектуры нейросетей, может быть архитектурой распределенных высоконагруженных систем заниматься и т.д. В общем, делать то, что ближе к науке.

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

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

Даже в ИТМО преподают историю, физкультуру, ОБЖ и философию. Зачем за это платить? А если взять какие-то менее известные университеты, то там все еще печальнее.

>>523499

> Вот есть функция mt_rand();

> Там нет команды return

Функции бывают пользовательские и встроенные. Встроенные — это те, что уже включены в PHP и ты можешь сразу их использовать. А пользовательские — это те, которые ты написал сам.

mt_rand это встроенная в PHP функция (к тому же, она написана не на PHP, а на Си). И в ее исходном коде, естественно, есть return. Почему ты решил, что его там нету?

> Только для этого её надо в переменную сохранить?

В переменную мы сохраняем не функцию, а значение, которое она вернет. В данном случае, число:

$x = mt_rand();

> Я вот ещё не понял. Я прочитал, что любая функция возвращает значение.

> (С Хекслета - В этом смысле функции идентичны операциям — они всегда возвращают результат своей работы.)

Да, это верно. Даже если ты не поставишь return, функция вернет специальное значение null, которое обозначает отсутствие значения. То есть

function x() {}
$a = x();

Здесь в $a запишется null. Обрати внимание, что это именно значение null, а не строка "null". Поэтому можно сказать, что любая функция возвращает какое-то значение.

> и откуда вообще 1.

Потому, что функция print_r возвращает булево значение true ( смотри мануал https://www.php.net/manual/ru/function.print-r.php ):

> ... В противном случае возвращаемое значение будет равно true ...

Обрати внимание, что это именно булево значение true, а не строка из 4 букв "true". Также обрати внимание, что "выводит на экран" и "возвращает" это разные вещи. Функция print_r выводит на экран одно значение, а возвращает другое. Здесь она выводит 34, а возвращает true.

Когда ты пытаешься склеить строку 'xt' и значение true, true превращается в единицу и получается xt1. Это преобразование описано в мануале тут: https://www.php.net/manual/ru/language.types.string.php#language.types.string.casting

> ... Значение bool true преобразуется в строку "1" ...

> Почему 34 вылезло вперед

Давай разберем построчно код.

$rk = 'xt';

Записать в переменную $rk строку "xt"

$dd = $rk . print_r(34);

Здесь мы вызываем функцию print_r, которая выводит число 34, и возвращает true. При склеивании "xt" и true получается "xt1" и эту строку мы записываем в $dd

print_r($dd);

Выводим строку, хранящуюся в $dd, то есть "xt1".

> $sk = print_r();

> var_dump($sk);

> Почему мне не выводит значение этой переменной null ?

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

PHP Warning: print_r() expects at least 1 parameter, 0 given in /home/runner/9wx7sl1vsy/main.php on line 3
bool(false)

Это переводится так: в функцию print_r обязательно надо что-то передать, а ты ничего не передал. У тебя не показываются ошибки? Тогда напиши мне, какая у тебя ОС, как ты устанавливал PHP (скачал сам PHP или какую-то сборку вроде Wamp) и как ты запускаешь код (в консоли, в браузере), и я напишу, как включить показ ошибок.