6

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;
}
11
  • 3
    «не понравилось качество и стиль реализации» — не волнуйтесь. «Забейте» на эту компанию, раз они такие развернутые комментарии дают. В черный список её и поделом с ней. ;) 2 авг 2017 в 15:44
  • Согласен с предыдущим оратором; в то же время проверку на непригодность строки можно делать до разбиения её на "слова" и суммирования с последующим unset. Ещё можно strcspn использовать, но это уже совсем придирки.
    – Lyth
    2 авг 2017 в 16:17
  • Спасибо за коментарии и поддержку))) А может я неправильно понял задание, может нужно было реализовать с стиле ООП? Вроде написано "реализовать функцию".
    – plavv
    2 авг 2017 в 16:26
  • @AlexanderBragin как бы то ни было - можете помочь и написать ответ, что где как можно улучшить и сделать проффессиональнее. Метку инспекция-кода поставил. Вроде подходит под условия 2 авг 2017 в 16:36
  • @plavv постараюсь сейчас написать решение, как я это вижу с комментариями. И вот ведь люди, не пишут в ТЗ про то, какие числа могут быть в файле: целые/не целые, положительные/отрицательные... 2 авг 2017 в 17:16

5 ответов 5

5

Код

<?php

/**
 * Функция производит чтение данных из файла и возвращает обработанные данные согласно алгоритму
 * - Считать данные с файла
 * - Разбить данные по строкам в массив
 * - Отфильтровать массив таким образом, чтобы в нем остались лишь строки содержащие только числа
 * - Суммировать числа в каждой строке
 * - Отсортировать полученные суммы в порядке убывания
 * - Вернуть результат
 *
 * @param string $file
 * @return array
 */
function getResult($file) {
    $numbers = [];

    $handle = @fopen($file, 'r');
    if ($handle) {
        // Читаем файл построчно
        while (($buffer = fgets($handle, 4096)) !== false) {
            // Удаляет пробелы (или другие символы) из начала и конца строки
            $buffer = trim($buffer);

            if (preg_match('/^[\d ]+$/', $buffer)) {
                // Разбиваем строку на числа, складываем получившийся массив чисел и записываем в массив
                $numbers[] = array_sum(explode(' ', $buffer));
            }
        }
    }

    // Сортируем массив по возрастанию (без сохранения отношений с ключами)
    rsort($numbers);

    return $numbers;
}

$result = getResult(__DIR__ . '/datalist.txt');

print_r($result);

Комментарии к коду:

  • По входному файлу, в примере, делаю следующий вывод. Функция разработана с учетом того, что числа только положительные и целые, и внутри слов нету чисел, например, oT89ft. И число не может идти рядом со словом, например, 200NTfy (то есть будет ли он считаться числом после удаление «нечисел»).
  • Хоть в алгоритме указан шаг Разбить данные по строкам в массив, он пропускается, чтобы не «забивать» память (не происходило переполнение памяти). Потом что строк в файле может быть очень много. Чтение происходит построчно и в массив сразу вычисляется и записывается сумма.
  • Длина строки при чтении ограничена length - 1 байт
  • Не стал «доходить по паранойи»: сортирую массив встроенной в PHP функцией и не стал делать какой-нибудь composer-пакет с тестами.
  • Комментарии на русском... Желательно на англ. писать, если компании не против, где работать будете — потом пригодится :)

Дополнительная информация

При написании (форматировании кода) я стараюсь следовать PHP Standards Recommendations, в частности:

Изучите информацию на сайте — пригодится.

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

И не бойтесь ошибаться! Задавать любые вопросы на Stack Overflow!

4
  • Сейчас ещё некоторые нюансы внесу и добавлю некоторые ссылки дополнительные ссылки по оформлению PHP. Может пригодится. Проверю также работу в версии 5.3-5.6, а то у меня PHP 7 стоит, хоть и не стал особо ничего мудрить, но проверить нужно. 2 авг 2017 в 17:57
  • 1
    В задании сказано оставить "лишь строки содержащие только числа" - т.е. строку с любыми буквами надо пропускать целиком, а не пытаться выделить из неё числа.
    – Lyth
    3 авг 2017 в 8:34
  • @lyth точно! Вы правы. Переписал код. Спасибо, что заметили! 3 авг 2017 в 9:18
  • @lyth если по изложению и коду есть ещё какая-та критика (можно что-то улучшить), в хорошом смысле этого слова, напишите, пожалуйста — я сам постоянно учусь. Учту. 3 авг 2017 в 9:24
2

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

Но могу сделать пару замечаний по стилю.

  1. Имена переменных - почему некоторые переменные записаны в CamelCase, а другие с маленькой буквы.
  2. Отступы в комментариях. Комментарии легче читаются когда они по отступам не выделяются от кода (либо идут в конце строки).
  3. Возможный баг: в задаче сказано что нужно убрать все строки которые содержат не числа. Вы проверяете регуляркой только наличие латинских букв. Если в примере будут какие либо символы то функция будет работать некорректно (не проверял код).
  4. Незакрыт тэг <pre>, хотя возможно это просто не весь код.
3
  • >Как правило в php используется нижнее подчеркивание для имен переменных и функций эм... Вы серьезно?
    – DaemonHK
    2 авг 2017 в 17:07
  • 1
    @DaemonHK из-за наличия стандартной библиотеки с бешеной неконсистентностью каждый волен строить свои предположения
    – etki
    2 авг 2017 в 18:02
  • @DaemonHK Убрал про функции, ориентировался на стандартную библиотеку. В PSR-1 действительно прописано что должно быть в camelCase.
    – Arnial
    8 авг 2017 в 9:43
1

Можно было бы более лаконично выполнить задание:

<?php

$file = __DIR__ .'/datalist.txt';
$result = getResult($file);

echo '<pre>'; var_export($result); echo '</pre>';


function getResult($file) {
    $result = array_map(
        function ($item) {
            return array_sum(explode(' ', $item));
        },
        array_filter(
            file($file, FILE_IGNORE_NEW_LINES|FILE_SKIP_EMPTY_LINES),
            function ($item) {
                return preg_match('~^[\s\d]+$~', $item);
            }
        )
    );

    arsort($result);
    return $result;
}
1
<?php

// Путь к файлу данных
$file = __DIR__ . '/datalist.txt';

// Передаем данные в функцию и получаем результат
$result = getResult($file);

// Отображаем результат
printResult($result);

// functions
function getResult($fileName) {
    $content = file_get_contents($fileName);
    $rows = explode("\n", $content);

    $arrWithNums = array_filter($rows, 'getRowsThatContainNumber');

    $arrWithSum = array();
    foreach ($arrWithNums as $str) {
        $nums = explode(' ', $str);
        $arrWithSum[] = array_sum($nums);
    }

    arsort($arrWithSum);

    return $arrWithSum;
}

function getRowsThatContainNumber($row) {
    $rowWithoutSpaces = str_replace(' ', '', $row);
    if (ctype_digit($rowWithoutSpaces)) {
        return $row;
    }
}

function printResult($var) {
    echo '<pre>';
    var_export($var);
}
1
<?php
# Путь к файлу данных
$file = __DIR__ . '/datalist.txt';

# Передаем данные в функцию и получаем результат
$result = getResult($file);

# Отображаем результат
echo '<pre>';
var_export($result);
echo "</pre>\n";

function getResult ($file)
{
    if (file_exists($file)) {
        if ($lines = file($file, FILE_SKIP_EMPTY_LINES)) {
            foreach ($lines as $index => $line) {
                $line = trim($line);
                preg_match_all('/\d\d{0,}/', $line, $matches);
                if (!empty($matches[0]) && $line === implode(' ', $matches[0])) {
                    $lines[$index] = array_sum($matches[0]);
                } else {
                    unset($lines[$index]);
                }
            }

            if (empty($lines)) {
                return "Numbers don't found";
            } else {
                rsort($lines);
                return $lines;
            }
        } else {
            return 'File empty or has wrong structure';
        }
    } else {
        return 'File not found';
    }
}

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.