Правильно ли осуществляется преобразование и вывод элементов массива?

Question

[Человек]

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

Но у меня возникло внутреннее ощущение, что я неграмотно (костыльно) создал код для вывода результатов.
Это место я пометил комментом, пожалуйста дайте свое экспертное мнение по этому поводу.
Да и посмотрите на весь мой код, не говнокод ли это?

$massive = [1.5, 2.6, 3, 4, 5];

function conversion($mass){

$result = [];

foreach($mass as $items){
settype($items, "integer");
$result[] = $items;
}

return $result;
}

foreach(conversion($massive) as $items){   // это грамотный вывод результатов типов элемента массива?
echo gettype($items).', ';
}

Answer 1

[Rsa97]

function allToInt(array $arr): array
{
    return array_map(
        static fn(mixed $val): int => (int)$val,
        $arr,
    );
}

$result = allToInt([1.5, 2.6, 3, 4, 5]);
echo implode(
    ', ',
    array_map(
        static fn(mixed $val): string => gettype($val),
        $result,
    ),
);

Comments

[Человек]

settype делает же то же самое, что и преобразование (int)$val верно?

[Человек]

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

[Дмитрий]

User 4550, это читается как функция с аргументом
fn(mixed $val)
function ($val)
=> (int)$val

{
    return (int)$val;
}

[Дмитрий]

User 4550, каст к типу и короче и проще читать, и нет переусложнения как с intval, так как у intval есть второй аргумент

[Человек]

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

[Rsa97]

User 4550, Просто перепишите самостоятельно.
fn($x) => $y
заменяется на
function($x) { return $y }

[Дмитрий]

User 4550, почитайте для понимания
https://www.php.net/manual/ru/function.array-map.php

[Rsa97]

User 4550, Разница между приведением типа (int) и его заменой через settype только в том, что при приведении создаётся новое значение указанного типа, а при settype меняется тип указанной переменной.
Но синтаксис (int) используется во многих языках программинрвания и будет более понятен тем, для кого PHP не первый язык.
Именование переменных/функций желательно строить так, чтобы по названию можно было понять, что именно делается в функции или что находится в переменной. Ваше название conversion говорит о том, что внутри выполняется какое-то преобразование, но какое именно - не понять не посмотрев в текст функции.
Ну и, для вывода массива есть штатная функция implode, нет смысла городить свой цикл с конкатенацией строк.

[tukreb]

User 4550, просто оставлю это здесь
https://www.levijackson.xyz/posts/are-array_-funct...

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

[krypt3r]

> Но было бы не плохо взглянуть на полный код без использования стрелочных функций =)
User 4550,
$result= array_map('intval', $arr);

[Человек]

Rsa97,
Меня смущает $result,
я не могу понять куда его запихнуть в обычной функции.

array_map(
        static fn(mixed $val): string => gettype($val),
        $result,   // этот result тоже нужно возвращать через return?
    ),

[Человек]

krypt3r,
Сделал по вашему совету.
Моя попытка переписать код без стрелочных функций не увенчалась успехом =(

function allToInt(array $arr): array{

return array_map(
$arr = static function(mixed $val): int{
return (int)$val;
}

);
}


$result = allToInt([1.5, 2.6, 3, 4, 5]);

echo implode( 
', ',

$result = array_map(
static function(mixed $val): string{
return gettype($val);
}

),);

[Rsa97]

User 4550, $result вообще к стрелочной функции не относится. Это второй параметр функции array_map.

[Человек]

Rsa97, точно!
Посмотрите, пожалуйста, правильно ли я переписал ваш код?

function allToInt(array $arr): array{

return array_map(
static function(mixed $val): int {
return (int)$val;
},$arr,

);}


$result = allToInt([1.5, 2.6, 3, 4, 5]);

echo implode(
', ',

array_map(
static function(mixed $val): string{
return gettype($val);
}, $result,

),);

А как называется функция(allToInt) внутри, которой еще одна функция?
Это явно не рекурсия.

[Rsa97]

User 4550, Правильно.
Только рекомендую соблюдать нормальный стиль. Поставьте себе какой-нибудь линтер, например PHP_CodeSniffer, плагин для него в IDE и настройте стиль PSR12. Тогда ваш код будет удобнее читать. А если ещё и на будете лениться проходить rector'ом, то ещё получите подсказки по рефакторингу.

spoiler

function allToInt(array $arr): array
{
    return array_map(
        static function(mixed $val): int {
            return (int)$val;
        },
        $arr,
    );
}

$result = allToInt([1.5, 2.6, 3, 4, 5]);

echo implode(
    ', ',
    array_map(
        static function(mixed $val): string
        {
            return gettype($val);
        },
        $result,
    ),
);

[Человек]

Rsa97, а что такое rector'ом и линтер.
Где взять стиль PSR12?
А зачем IDE? Я весь код пишу в notepad++.
Я установил сервак apache и на него php8, мне этого пока хватает

[Rsa97]

User 4550, Линтер - система, следящая за соблюдением стиля кода (стайлгайд, styleguide). Самый, наверное, известный линтер для PHP - phpcs (PHP CodeSniffer).
IDE нужна для удобства разработки. Тот же VS Code с плагинами даёт подсветку кода, автодополнение при наборе, автоформатирование, постоянную проверку линтером.
PSR12 - официальный стайлгайд для PHP. Дополняет стандарт PSR1, заменяет PSR2.
Вот что получается, если проверить ваш код в phpcs на стандарт PSR12:

spoiler

------------------------------------------------------------------------------------------------------------------------------------------------------
FOUND 31 ERRORS AND 1 WARNING AFFECTING 16 LINES
------------------------------------------------------------------------------------------------------------------------------------------------------
  1 | WARNING | [ ] A file should declare new symbols (classes, functions, constants, etc.) and cause no other side effects, or it should execute
    |         |     logic with side effects, but should not do both. The first symbol is defined on line 3 and the first side effect is on line 13.
  3 | ERROR   | [x] Opening brace should be on a new line
  5 | ERROR   | [x] Line indented incorrectly; expected at least 4 spaces, found 0
  6 | ERROR   | [x] Line indented incorrectly; expected at least 4 spaces, found 0
  6 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
  6 | ERROR   | [x] Expected 1 space after FUNCTION keyword; 0 found
  7 | ERROR   | [x] Line indented incorrectly; expected at least 8 spaces, found 0
  7 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
  8 | ERROR   | [x] Line indented incorrectly; expected 4 spaces, found 0
  8 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
  8 | ERROR   | [x] No space found after comma in argument list
  8 | ERROR   | [x] Only one argument is allowed per line in a multi-line function call
  9 | ERROR   | [x] Empty lines are not allowed in multi-line function calls
 10 | ERROR   | [x] Line indented incorrectly; expected at least 4 spaces, found 0
 10 | ERROR   | [x] Closing brace must be on a line by itself
 16 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 17 | ERROR   | [x] Empty lines are not allowed in multi-line function calls
 18 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 19 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 19 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 19 | ERROR   | [x] Expected 1 space after FUNCTION keyword; 0 found
 19 | ERROR   | [x] Expected 1 space before opening brace; found 0
 20 | ERROR   | [x] Line indented incorrectly; expected at least 4 spaces, found 0
 20 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 20 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 21 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 21 | ERROR   | [x] Multi-line function call not indented correctly; expected 4 spaces but found 0
 21 | ERROR   | [x] Only one argument is allowed per line in a multi-line function call
 22 | ERROR   | [x] Empty lines are not allowed in multi-line function calls
 23 | ERROR   | [x] No space found after comma in argument list
 23 | ERROR   | [x] Closing parenthesis of a multi-line function call must be on a line by itself
 23 | ERROR   | [x] Expected 1 newline at end of file; 0 found
------------------------------------------------------------------------------------------------------------------------------------------------------
PHPCBF CAN FIX THE 31 MARKED SNIFF VIOLATIONS AUTOMATICALLY
------------------------------------------------------------------------------------------------------------------------------------------------------

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

spoiler

---------- begin diff ----------
@@ @@
 function allToInt(array $arr): array{

 return array_map(
-static function(mixed $val): int {
-return (int)$val;
-},$arr,
+static fn(mixed $val): int => (int)$val,$arr,

 );}

@@ @@
 ', ',

 array_map(
-static function(mixed $val): string{
-return gettype($val);
-}, $result,
+static fn(mixed $val): string => gettype($val), $result,

-),);
+),);
    ----------- end diff -----------

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