vitokz / loftschool-homework Goto Github PK

Перепутал слеши

include '../src/config.php';
include'../vendor/autoload.php';

Вынести в конфиг фаил, так же как настройки продключения к БД.

Код рабочий:

Предлагаю поменять порядок условий

if ($age > 0 && $age < 18) {
...
} elseif ($age >= 18 && $age <= 65) { 
...
} elseif ($age > 65) {

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

Нужно вынести все классы в отдельные файлы

Для подсчёта количества пользователей удобно воспользоваться функцией
https://www.php.net/manual/ru/function.array-fill-keys.php
Ты инициализируешь массив с нулевыми значениями для каждого юзера,
а затем в цикле для каждого юзера увеличишь счетчик.

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

Логику работы с файлами нельзя мешать с логикой работы с БД.

Все сохранение файлов нужно делать отдельным классом.

Чтобы избежать дублирования вынеси создание шаблонизатора в коструктор

(так же как было с phtml)

Ты можешь поключить сюда twig, и формировать тело письма из шаблона.

1. Список пользователей лучше вынести в отдельный фаил, например user_names.php
   Его содержание

return [
    'Sasha',
    'Geor',
    'Sos',
    'Vlad',
    'Diana',
];

В основном сценарии подключить этот фаил
$user_names = requre('user_names.php');

Т.к. в файле есть return, то при подключении его содержание можно сохранить в отдельную переменную.

2. Написать функцию generateRandomUsers() с двумя параметрами

$count - количество пользователей
$userNames - имена пользоателей

Вызвать ее в основном сценарии.
$users = generateRandomUsers(50, $user_names);

Сейчас ты делаешь ресайз в отдельном контроллере.

Предлагаю делать ресайз картинки при добавлении её в блог.

В функцию нужно добавить проверку того, что переданы числа (целые или вещественные)

Также нужно проверить, что деление происходит не на 0
(так как на 0 делить нельзя)

1. Создание массива
   

   loftschool-homework/1zadanie/Zadanie1-5.php

   Lines 2 to 6 in 079c906

   	$bmw = [];
   	$bmw['model'] = 'X5';
   	$bmw['speed'] = '120';
   	$bmw['doors'] = '5';
   	$bmw['year'] = '2015';

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

$bmw = [
    'model' =>  'X5',
    'speed' => 120,
    .....
];

Так же и с массивом $opel.

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

$machines = ['bmw' => $bmw, ...];

3. Вывод значений:
   Текущий вывод неоптимален, так как ты каждую машину выводишь в своем цикле. Если машин будет 1000, тебе придется писать 1000 циклов.

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

foreach($machines as $machineName => $machineData) {
    foreach($machineData as $propertyName => $propertyValue) {
    ....
    }
}

Рекомендую посмотреть описание функции тут:
https://www.php.net/manual/ru/control-structures.foreach.php

не зависит от файловой системы (application/json)