coding: 10/19/18

HR менеджер дал тестовое задание. Я выполнил. В ответ он сказал, что
я получил только краткий фидбек - не понравилось качество и стиль реализации.
Хочу вашего совета, что я не так написал. Вот задание:
Тестовое задание #1:
Версии ПО: PHP 5.3-5.6
Необходимо реализовать функцию, которая произведет чтение данных файла и вернет обработанные данные в указанном виде.
Функция должна: - Считать данные с файла - Разбить данные по строкам в массив - Отфильтровать массив таким образом, чтобы в нем остались лишь строки содержащие только числа - Суммировать числа в каждой строке - Отсортировать полученные суммы в порядке убывания - Вернуть результат
Пример кода:
# Путь к файлу данных $file = __DIR__ . '/datalist.txt'; # Передаем данные в функцию и получаем результат $result = getResult( $file ); # Отображаем результат echo '

'; 
     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

Как я это реализовал:
# Путь к файлу данных
$file = __DIR__ . '/datalist.txt';
# Передаем данные в функцию и получаем результат
$result = getResult($file);
# Отображаем результат
echo '';
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;
}

    

Ответ

Код
/**
 * Функция производит чтение данных из файла и возвращает обработанные данные согласно алгоритму
 * - Считать данные с файла
 * - Разбить данные по строкам в массив
 * - Отфильтровать массив таким образом, чтобы в нем остались лишь строки содержащие только числа
 * - Суммировать числа в каждой строке
 * - Отсортировать полученные суммы в порядке убывания
 * - Вернуть результат
 *
 * @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, в частности:

PSR-1: Basic Coding Standard
PSR-2: Coding Style Guide

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

Есть основное приложение WinForms в котором ведётся парсинг интернет ресурса с записью необходимых данных в двумерный массив string[5000, 50]
Где 1й столбец строки это ссылка, а остальные столбцы данные с ресурса.
Как только набирается строка c данными , данные этой строки сгруппировано выводятся в окне приложения.
Как мне эти данные передавать в массив другого приложения WinForms по интернету и так же выводить ?
Главный вопрос, как передавать эти данные c записью в массив ?
Самостоятельно искал, читал немного про сокеты, но пока каша в голове и не знаю про подводные камни.
Хотелось бы как то тупо, при заполнении строки массива в главном приложении, сразу эту строку массива передавать в массив удалённого приложения.

Ответ

Я мог бы вам предложить один из вариантов организации данного функционала:
Логика :
1) Серверная часть
Ваш Parser после анализа полученных данных и формирования массива значений (рекомендую использовать коллекции в место статичного массива) сериализуется в json(или XML формат) строку и добавляет их в базу данных MySQL (локальную или удалённую) (с одновременной проверкой наличия там таких данных) - SQL запрос для организации структуры базы данных для ваше случая должна быть примерно таковой
CREATE DATABASE IF NOT exists parsersystem; use parsersystem; CREATE TABLE IF NOT exists Procedures ( number int AUTO_INCREMENT, id int, jsonData TEXT, ts_create timestamp NOT NULL DEFAULT '0000-00-00 00:00:00', ts_update timestamp NOT NULL DEFAULT CURRENT_TIMESTAMP ON UPDATE CURRENT_TIMESTAMP, PRIMARY KEY (number) );
где number это автоинкриментное поле (уникальный номер записи в БД)
id - уникальный специфичный номер записи (заданный пользователем)
jsonData - сериализация любых данных в том числе и массива ваших значений
ts_create - дата и время создание записи
ts_update - дата и время обновления записи
MySQL сервер настраивается таким образом чтобы к нему имели доступ клиенты через интернет либо локальную сеть (в зависимости от ваших нужд)
2) Клиенты
Клиентское приложение будь то Desktop, Android или другое, не имеет связи непосредственно с вашим серверным Parser приложением, а обращается к открытой MySQL базе (закрытой по желанию паролем), на предмет наличия новых записей, это становится возможным при использовании поля ts_create - которое отражает время создание записи Логика:
клиент периодически делает запрос к базе данных примерно такой
SELECT * FROM Procedures WHERE ts_create > TIMESTAMP (время последней добавленной клиентом записи) ORDER BY ts_create DESC
таким образом клиент получит только новые записи из таблицы на стороне клиента происходит десереализация jsondata и формирование своей локальной копии массива (идентичного тому что на стороне сервера) клиенту следует хранить дату и время последней добавленной записи для последующих запросов
Преимущество такого подхода :
независимость клиентской и серверной части приложения, (данные хранятся отдельно в БД) нет необходимости в организации специфического протокола общения (сервер -клиент) при использовании подхода передачи при помощи сокетов
Предположим что мы имеем некий класс описывающий модель наших данных
public class Procedure(){ public String Name { get; set; } public List Data { get; set; } }
А также мы на стороне сервера имеем набор данных (коллекцию)
public List procedures;
Для записи в базу данных такой конструкции стоит применять примерно такой метод:
public AddProceduresToDataBase(List procedureList) { string Connect = "Port=3306; Database=parsersystem;Data Source=127.0.0.1;Password=mysql_password; MySqlConnection myConnection = null; try { myConnection = new MySqlConnection(Connect);
myConnection.Open(); } catch (Exception e) { myConnection.Close(); Console.Out.WriteLine("Ошибка доступа к базе данных"); }
foreach (var proc in procedureList) { var json = new JavaScriptSerializer().Serialize(proc); // Экранируем спец символы json = json.Replace(Convert.ToChar(0171) + "", "") .Replace(Convert.ToChar(0187) + "", "") .Replace("\\\"", "") .Replace("\\","\\\\"); MySqlCommand myCommand_add = new MySqlCommand(@"INSERT INTO Procedures(id,jsondata,ts_create,ts_update) VALUE (" + сгенерить нужно ID + "+ json + "',null,null);", myConnection); myCommand_add.ExecuteNonQuery(); }
myConnection.Close(); //Обязательно закрываем соединение!
}
Для чтения коллекции из базы данных стоит использовать примерно такой метод
public List GetProcedures() { string Connect = "Port=3306; Database=parsersystem;Data Source=127.0.0.1;Password=mysql_password; MySqlConnection myConnection = null; try { myConnection = new MySqlConnection(Connect);
myConnection.Open(); } catch (Exception e) { myConnection.Close(); Console.Out.WriteLine("Ошибка доступа к базе данных"); }
string CommandText = "SELECT * FROM Procedures "; MySqlCommand myCommand = new MySqlCommand(CommandText, myConnection);
MySqlDataReader MyDataReader; MyDataReader = myCommand.ExecuteReader();
List procedures = new List(); String jsonData;
while (MyDataReader.Read()) { try { jsonData = MyDataReader.GetString(2); // Читаем jsondata из 2го столбца таблицы Procedure newProcedure = new JavaScriptSerializer().Deserialize(jsonData); procedures.Add(newProcedure); } catch { Console.Out.WriteLine("Ошибка разбора JSON данных о процедуре"); } } MyDataReader.Close(); myConnection.Close(); return procedures;
}
ПС. естественно методы нужно дорабатывать, внедрить проверку на наличие уже такого рода данных в БД, расширить функционал чтения для получения новых записей , но это я уже оставляю вам. ПС. так же тут в принципе показаны примеры сериализации и десиарилизации данных в JSON

Страницы

Поиск по вопросам

пятница, 19 октября 2018 г.

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Азербайджанский язык(lang='az'):

Без указания языка текста, или с неправильным языком:

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

Ответ

пятница, 19 октября 2018 г.