HR менеджер дал тестовое задание. Я выполнил. В ответ он сказал, что
я получил только краткий фидбек - не понравилось качество и стиль реализации.
Хочу вашего совета, что я не так написал. Вот задание:
Тестовое задание #1:
Версии ПО: PHP 5.3-5.6
Необходимо реализовать функцию, которая произведет чтение данных файла и вернет обработанные данные в указанном виде.
Функция должна:
- Считать данные с файла
- Разбить данные по строкам в массив
- Отфильтровать массив таким образом, чтобы в нем остались лишь строки содержащие только числа
- Суммировать числа в каждой строке
- Отсортировать полученные суммы в порядке убывания
- Вернуть результат
Пример кода:
# Путь к файлу данных $file = __DIR__ . '/datalist.txt';
# Передаем данные в функцию и получаем результат $result = getResult( $file );
# Отображаем результат echo '<pre>';
var_export( $result );
function getResult ( $file ) {
# @TODO Реализовать... }
Пример результата работы функции: array ( 82 => 16396, 19 =>
16169, 71 => 15864, 73 => 15224, 81 => 14244 ...
Кусок файла datalist.txt:
55 NTfy 591 405 kLj 48 644 768 164 ubd 837 oTft GPQV 163 ja 445 961
431 574 168
375 380 427 670 610 284 765 48 687 660 377 333 914 70 146 328 301 925 266
620 237 137 584 427 308 939 660 917 59 864
j hHo 279 tqpg 617 870 CoNJ 173 czgW 301 299 134 820 625 U 369 165 hutPN jiq 31 575 46 NS 397 378 954 764
Как я это реализовал:
<?php
# Путь к файлу данных
$file = __DIR__ . '/datalist.txt';
# Передаем данные в функцию и получаем результат
$result = getResult($file);
# Отображаем результат
echo '<pre>';
var_export($result);
function getResult($file) {
#Считать данные с файла.
#Разбить данные по строкам в массив
$line = file($file);
$i = 0;
foreach ($line as $value) {
$ArrayLine = explode(' ', $value);
foreach ($ArrayLine as $Record) {
$result[$i] = 0;
#Отфильтровать массив таким образом, чтобы в нем остались лишь строки содержащие только числа
$CheckString = preg_replace('~[^A-Za-z]+~', '', $Record);
if (strlen($CheckString) > 0) {
unset($result[$i]);
break;
};
$res = (int) $Record;
#Суммировать числа в каждой строке
$result[$i] += $res;
}
$i++;
}
#Отсортировать полученные суммы в порядке убывания
arsort($result);
return $result;
}
unset
. Ещё можноstrcspn
использовать, но это уже совсем придирки.