Как избавиться от notice Unhandled Exception?

Question

[cyberche]

echo "<pre/>";
error_reporting(E_ALL);
function dataFromFile($h){
    if (!file_exists($h)){
        throw new Exception('Файл '.$h.' не найден');
    }try{
     return file($h);
    }catch(Exception $e){
        $e->getMessage();
    }
    return true;
}
function arrayExplode(array $array):array{
    return array_map(function($explode){return explode(':',$explode);},$array);
}
function arrayConstructor(array $array):array{
    return array_map(function($construct){return $res =['wall'=>$construct[0],'token'=>$construct[1]];},$array);
}
/////////////////////////////
$data = arrayConstructor(arrayExplode(dataFromFile('data.array')));

Unhandled Exception:6
Unhandled \Exception:21
Unused element: 'arrayExplode' - обязательно ли мне результаты выполнения функции возвращать в переменную? И на сколько критично этого не делать?

Comments

[FanatPHP]

Попробуй задать более осмысленный вопрос
какая связь между Как избавиться от notice Unhandled Exception и обязательно ли мне результаты выполнения функции возвращать в переменную?
и что ты вообще здесь хочешь сделать?

Answer 1

[FanatPHP]

Эээ. Я кажется понял этот вопрос.
Тебе твой код сообщает, что 'Файл '.$h.' не найден'.
То есть надо проверить, есть ли файл для обработки, и если есть, то указывать к нему правильный путь.
Для этого надо выкинуть кустарную проверку на ошибки, и получить нормальную ошибку РНР. В ней будет будет подробно написано, что тебе делать

Для этого выкинуть функцию getDataFromFile и испльзовать file() напрямую

функциональщину я бы тоже выкинул, она тебя только запутывает

$data = file('data.array', 6);
foreach($data as $i => $row) {
    $array = explode(':', $row);
    $data[$i] = ['wall'=>$array[0],'token'=>$array[1]];
}

Comments

[FanatPHP]

Лентюй, ну это уже совсем какая-то содомия будет, если даже отсутствие прав не будет ошибкой

[cyberche]

function arrayConstructor($fileHandle):array{
    return array_map(function($construct){
        return $res = [
            'wall'=>$construct[0],
            'token'=>$construct[1]
        ];},(array_map(function($explode){
        return explode(':',$explode);}, file($fileHandle,FILE_IGNORE_NEW_LINES))));
}

циклы быстрее получаются, очень любопытно.

[FanatPHP]

не циклы, а цикл
у меня тут 1 цикл, а у тебя два

[cyberche]

FanatPHP, подскажите, а хранить обработанные данные в файле формата json строки это нормальная практика? Просто не хочу использовать SQL

[FanatPHP]

нормальная практика - это агрументировать свои действия с помощью логических доводов, а не слова "не хочу".
В любом случае, перекладывать из пустого в порожнее - из одного формата в другой - будет абсолютно бессмысленно.

Ты явно себе представляешь РНР гаррипотером, а функции - такими заклинаниями. сказал array_map, и хопа, цикла как не бывало. сказал json_decode - и хопа, РНР родил тебе массив прямо из электромагнитного излучения, вообще не напрягаясь. Главное чтобы функция была. А то что она внутри себя будет делать то же самое, что ты наколупал на коленке - это тебе почему-то в голову не приходит

[cyberche]

FanatPHP, про циклы под капотом стандартных функий я понял, но не думаю, что для хранения 400-500 записей с 5 полями нужна SQL. Файл из которого импортируются номера и токены нужен только для начальной конфигурации скрипта, в 00.00 минут, данные в файле обновляются. Хотя может и имеет смысл сразу складывать json в файл конфигурации.

[FanatPHP]

Ты можешь объяснить, зачем складывать в Джейсон? Чем тебя текущий формат не устраивает?

[cyberche]

FanatPHP, в формате json я собираюсь хранить логи: id, кошелек, баланс, детали платежа, кол-во полученной крипты. На каждый день создается свой лог-файл.

[FanatPHP]

забудь про свою боязнь баз данных
это та информация которая должна храниться в БД.

я не знаю, какое отношение эти логи имеют к файлу из вопроса, то то что ты описал хранится в таблице БД

[cyberche]

FanatPHP, понял. Спасибо за ответы, буду читать про SQL. Да я особо не боюсь, просто никогда не доводилось работать с БД. Вообще благодаря Вам я переписал ряд функций. Разобрался c парсингом и т.д. Подскажите, увидел либу guzzle, но с классами никогда дел не имел, в guzzle под капотом тот же CURL?

[FanatPHP]

Да, у guzzle под капотом тот же CURL

Answer 2

[Андрей Шевкалюк]

Не обязательно. Вы можете отключить нотисы так:
error_reporting(E_ALL & ~E_NOTICE);

Comments

[FanatPHP]

добавь к своей подписи слово junior
потому что такая туннельность зрения, когда человек видит даже не одну строчку, а одно слово в этой строчке, свойственно именно этому классу пользователей

[FanatPHP]

да, и разумеется, стремление изгадить все приложение, пытаясь решить несуществующую проблему - тоже

[Андрей Шевкалюк]

FanatPHP, я тут ответил на конкретный вопрос, а не решал проблему и не советовал, как сделать приложение лучше. Вопрос был не "Почему у меня вылезает куча ошибок памагитиии", а как убрать Notice. Надеюсь, что автор вопроса все-таки понимает, что у него происходит в коде

[FanatPHP]

Автор-то понимает, а вот ты совсем нет
и предлагаешь ты не "ответ на вопрос" - тут ту себе льстишь неимоверно - а, как я и писал, комментарий к одному знакомому слову, которое увидел во всем вопросе

[cyberche]

Андрей Шевкалюк, ну я только 3 день изучаю PHP для своих целей, т.к. кликермен не подходит для решения текущих задач. Насчет понимания: у меня есть файл с номерами телефона и токенами к номерам - "номер:токен".На счет понимания - считываю данные из файла в массив, далее разбиваю массив строк и создаю новый ассоциативный массив для дальнейшей работы с ним. Если Вы можете мне подсказать почему появляется "предупреждение Unhandled Exception" и как его убрать буду весьма признателен. Так же если Вы подскажите, где и что я делаю неправильно, то это было бы очень кстати.

[Spartak (Web-StyleStudio)]

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

[cyberche]

Spartak (Web-StyleStudio),
00000000000:qweqweeqweewqewqewqeqwewq
00000000000:hghttyutyutyuty
00000000000:tryryryrtyrtyrty
00000000000:rtyrytyrtyrtyrt
00000000000:ryyrytryty45646yrt
00000000000:rytryrtyrtyrtyrty4
00000000000:rtyryrtyrtykuyiyuiy
00000000000:ertertertertert3

[Spartak (Web-StyleStudio)]

function arrayExplode(array $array):array{
    return array_map(function($explode){return explode(':',$explode);},$array);
}

В вашем файле кроме ":" есть ещё и перевод строки

[FanatPHP]

Spartak (Web-StyleStudio), и что?

[cyberche]

Spartak (Web-StyleStudio), спасибо, добавил параметр.