Как устранить Undefined array key после назначения переменных через explode()?

Question

[PageUp]

Всем привет!

Хочу перевести скрипты с php 5.2 на 8 но столкнулся с постоянными ошибками о неназначенных переменных. И если большинство сразу налету можно вписать в условие isset(), перед назначением, когда она назначается в цикле for например.
Ну или когда передается методом POST, GET или штучно назначается.

Но как быть с explode? У меня их очень много и переделывать их все на циклы нет возможности.

Вот такой вот код выдает ошибку в заголовке этой темы:

$pr_ip = explode(".", $ip_addr);
$my_ip = $pr_ip[0].$pr_ip[1].$pr_ip[2];

Как быть? Как менее болезненно устранить все эти Undefined array key в коде, везде, где есть explode()

Это ладно когда еще небольшая входящая может быть. Но у меня есть проверки файлов в которых допустим 60 ячеек с которых берутся данные. Где-то ячейки пустые, где-то с данными. Не писать же для каждой отдельно проверку.

Answer 1

[Ипатьев]

На "западном аналогичном этому сайте" только что случайно наткнулся на красивое решение.

$arrdata = array_pad(explode('|', $file[0]), 666, '');

где 666 - это количество колонок, которое должно быть после explode.
array_pad добьёт их пустыми строками. можно поставить нули при желании.

Comments

[PageUp]

Это если $file[0] окажется пустым, то $arrdata все ключи заполнит либо '', либо нулями, если их указать?
Или это добавляет к имеющемуся массиву эти значения?
Прочел об array_pad, но так и не понял, в конкретном случае, что оно сделает.

Ну и сразу вопрос, чем результат будет отличаться от $data=explode("|", $file[0] ?? ''); ?

[PageUp]

Попробовал, вроде и правда работает) Очень классно!
Только теперь с другой вот проблемой столкнулся, касаемо цифирных ключей
Ну это уже наверное другая тема.

[Ипатьев]

я же написал в ответе. array_pad добьёт недостающие колонки пустыми строками.
Например explode вернула 10 колонок, а надо 20.
Вот в array_pad написать 20 и в массиве будет 20 колонок - и ошибок не будет при обращении к любому элементу

[PageUp]

Ипатьев, Я этот метод запомнил, думаю обязательно еще пригодится, но все же принял волевое решение не пытаться обмануть PHP. Сел, запасся терпением и поехал построчно по всем пунктам, вооружившись isset() и empty(). Иначе это бесконечно будет продолжаться, одно подавляешь, другое всплывает. Лучше уж привести код к положенным стандартам, а не искать костыли.

Прошел все стадии, пришла стадия принятия)

[Ипатьев]

isset и empty - это не "положенные стандарты", а как раз наоборот - костыли. И я об этом в каком-то из комментариев уже писал.

Но главное - это сразу становится понятным, если немного подумать головой.
Эти ошибки, Undefined array key - они не для того, чтобы программист задолбался.
А вы их воспринимаете именно так. Вы считаете, что создатели языка заставляют вас везде писать isset и empty. Но это же глупость - писать код только для того, чтобы задавить сообщение об ошибке!

Задача этих ошибок не в том, чтобы программист все время как обезьяна везде писал isset и empty.
Любые сообщения об ошибках - служат для помощи программисту.
Данная ошибка подсказывает, что программист пытается обратиться к переменной, или элементу массива которых нет.
И увидев эту ошибку, программист не должен тупо затыкать ей рот через isset! А должен разобраться - почему вдруг нет нужной переменной.

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

А то что вы делаете сейчас - это то же самое подавление ошибок, вид сбоку.

Все что я написал - нетрудно понять просто логикой. Но если вам обязательно нужно, то вот вам статья с англоязычного ресурса, https://phpdelusions.net/articles/null_coalescing_abuse

Причем array_pad - это тоже костыль. И по идее, надо приводить свои файлы в порядок, чтобы в них не было пустых ячеек. Но на данном этапе использование array_pad оправдано, поскольку приводит все массивы к единому виду. То есть делает то, что вы хотели изначально - чтобы if($arrdata[19]>0){ не вызывало ошибок. Потому что в $arrdata всегда будет нужное число колонок.

[PageUp]

Задача этих ошибок не в том, чтобы программист все время как обезьяна везде писал isset и empty.
Любые сообщения об ошибках - служат для помощи программисту.
Данная ошибка подсказывает, что программист пытается обратиться к переменной, или элементу массива которых нет.
И увидев эту ошибку, программист не должен тупо затыкать ей рот через isset! А должен разобраться - почему вдруг нет нужной переменной.

На словах все очень хорошо звучит. А то я этого не понимаю.

А теперь ситуация, которых может быть уйма.

На сайте может создаваться файл, может не создаваться, он может быть с данными, а может быть пустой, в зависимости от действий пользователя.

Мы через explode считываем данные этого файла назначая переменные, через ключи, что назначаются в explode.

И какие варианты? Мы проверяем либо что файл не пустой, либо мы потом проверяем есть ли переменная назначенная или нет.

В любом случае мы делаем эту проверку. И таких моментов сотня может быть, так как весь сайт использует сплошные файлы. Так понятней объяснил? А то красиво, конечно, говорить, не зная сути дела, как на самом деле обстоит обстановка. Если есть решение как в таких ситуациях поступать дабы избежать isset или empty, я с радостью выслушаю. По поводу file_exists писать не стоит, файлы всегда в наличии, ну в большинстве случаев, а вот внутренности могут пустыми. Причем внутренности разделены на строки и разделены на столбцы. Данные перемешиваются, это могут быть как string так и int что тоже надо постоянно проверять.

[PageUp]

Ипатьев, файлы не пустые и так. Ячейка может быть пустой в зависимости от того, какое действие сделал пользователь. Например, в нее занеслось Apple, а если он не сделал какое-то действие, то будет "". Вы же не предлагаете в КАЖДОЙ ячейке по умолчанию вписывать default, только для того что бы она была не пустой. Тем более некоторые ячейки создаются в процессе, могут дополняться путем пуша или .=

В общем с чужой колокольни всегда виднее

P.S. и не обязательно теперь под каждой темой моей заходить в комментарии и везде писать что я типикал говнокодер и всячески переходить на личности.

[Ипатьев]

я уже писал, что предлагаю
Это написано в ответе, который вы комментируете

[PageUp]

Ипатьев, так вот именно что вы пишите совершенно несоответствующее действительности. То что ничего не решит. А то я не знаю таких элементарных вещей. Ну прям уже аж до смешного, ей богу. Да хоть вы нулями все ячейки заполните сразу, есть ячейки которые должны быть пустыми, так как ячейка string и с нее считывается только текстовая информация, а не числовая, и это еще одну проверку так или иначе создаст затем. Говорю же, со стороны всегда виднее. А я и так делаю все что возможно что бы код был чище, а потому и отказался от ?? '' и от array_pad, так как они плодят только еще больше ошибок, смешивая текстовые и стринговые столбцы

[Ипатьев]

Тем более тогда надо завести эталонный массив и один раз заполнить его данными, полученными из файла.
И потом спокойно обращаться к его элементам без костылей.
Это если делать по уму.

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

[PageUp]

Ипатьев, этот массив меняется в минуту раз 10 и таких массивов сотни, разного плана.

[PageUp]

Ипатьев,

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

Как раз именно от вас такой посыл прямо СОЧИТСЯ в каждой ветке. Я уже устал читать о том какой вы ГЕНИЙ, а кругом сплошные говнокодеры. Я никогда не завышал своих знаний, никогда не стеснялся спросить совета или задать вопрос, но когда человек прям везде тебе говорит ты дно, ты не умеешь кодить, криворукий и т.д., но при этом потом спрашивает, "а что такое ячейки?" все желание говорить пропадает, так как человек учит тому, с чем сам по всей видимости не так много опыта имел.

[Ипатьев]

Пишите. Пишите как придумали.
Я ОЧЕНЬ надеюсь, что вам этот код придется в дальнейшем поддерживать и модернизировать.
И это будет меня очень, очень радовать )

[PageUp]

Голова дана человеку не только для того, чтобы в неё есть.

Флаг в руки и барабан на шею. Безумству храбрых поем мы песню.

Никаких "вариантов" тут встречать не надо. Если человек в принципе умеет ходить, и знает что такое алгебра с литературой.
Вы вообще в состоянии понять, что это три разных действия, а не одно?

Это конечно ад, но уговаривать автора переписывать эту "процедурку" на нормальное MVC с DI я конечно не возьмусь.
поэтому

АХАХАХАХАХАХАХА!
Этот ответ выдает джуна сразу! )))

Первый пример - это говнокод самого автора, если следить за его выступлениями на этом ресурсе. Причем при сравнении он имеет в виду именно значение переменной, а не результат isset. И его уже натыкали в это носом. Но поскольку до него очень долго всё доходит, он решил уточнить, правильно ли все понял.

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

Как раз модернизированием я сейчас и занимаюсь. Это код с 2013 года, написанный на php 5.2 и мне необходимо его привести к рабочему виду для php 8 без warningов даже.

[PageUp]

Ипатьев, А вы даже подписались на меня как я понимаю, не пропускаете ни одного моего комментария или темы, дабы видеть мои вопросы, и тешить свое ЧСВ. И удивлюсь если это не будет сопровождаться очередными колкостями от вас в мой адрес. Это уже и правда какая-то клиника, если честно. Всего хорошего.

[Ипатьев]

Вы такой забавный :)
Просмотрели кучу моих ответов,и пришли сказать мне, что это оказывается я на вас подписался. Хотя, внезапно, такого функционала на этом сайте нету :)

[PageUp]

Ипатьев, ну тем не менее в соседней теме вообще по другому моему вопросу не поленились зайти в комментарии и написать вообще другому человеку (к чему это) что я говнокодер.

И при всем при этом, это я забавный. А ответы и комменты почитать мне прямо интересно стало

Я ОЧЕНЬ надеюсь, что вам этот код придется в дальнейшем поддерживать и модернизировать.
И это будет меня очень, очень радовать )

Это же сколько нужно иметь ЧСВ и желчности, что бы аж до такого доходить, вместо того что бы пожелать удачи в трудной задачи и все. Потому и вывод насчет подписки напросился, если вы прям ждете как я буду править и хотите злорадствовать или смеяться с этого.

Говорю же, это уже нездоровый интерес какой-то

[Ипатьев]

При случае посмотритесь в зеркало.

[PageUp]

Ипатьев, Я то по факту все написал и указал конкретные действия и ответы. Все, что получил в ответ "кто обзывается, тот сам так и называется."

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

[Ипатьев]

Вы раскопали 5% от моих ответов, и с плохо скрываемым сарказмом назвали их "конкретными". Скромно умолчав про остальные.
Вы даже не вспомнили, как я буквально за уши вас тащу к свету, и трачу кучу времени, чтобы пробиться через ваше детсадовское самомнение и зазнайство, каждый раз объясняя элементарные вещи.
Вы совершенно забыли про свои собственные тирады на тему "меня никто не понимает, моя ситуация уникальная!!!".
Вы не вспомнили то бешеное сопротивление, которое оказываете любой попытке что-то вам объяснить.
Вы вдруг резко забыли причину, по которой я и написал свою последнюю реплику. Которая была вызвана вашим очередным отказом делать по-человечески.
Вы внезапно вообще не обращаете внимания на СВОЮ манеру общения, когда вместо "ну хорошо, допустим такой вариант, но у меня вот такая ситуация" вы сходу выдаете свое фирменное "МНЕ ЭТО НЕ ПОДОЙДЕТ!!!". Чтобы потом, после траты кучи времени и нервов, все-таки согласиться.

При таком подходе-то конечно, вы у нас будете беленький и пушистенький, без единого пятнышка. А вокруг одни негодяи с завышенным ЧСВ :)

[PageUp]

Ипатьев, Да ваше ЧСВ почти в каждом вашем комментарии прослеживается с первого же раза, как вы ответили мне. Да что там говорить, достаточно просто прочесть ваш статус: Потомок старинного рода Ипатьевых-Колотитьевых

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

[PageUp]

Ипатьев, Не нужно врубать манипуляцию и обобщать, я не говорил про всех, вы эти приемчики себе оставьте, касаемо ЧСВ я говорил имея в виду только вас лично.

Answer 2

[toxa82]

Писать, обрабатывать все варианты входящих данных. Можешь конечно засобачить, но это говнокод. Даже 5.2 матерился на это нотайсами, сейчас, если не ошибаюсь, это переделали в 8 на варнинги.

Comments

[PageUp]

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

Сейчас вот, после того как я правлю код под PHP 8, мой аккуратный код стал больше похож на говнокод.
Если раньше переменная просто назначалась $a=$input[1]; легко и просто.
То теперь эта элементарная вещь превращается именно, что в говнокод с кучей ненужных условий и строк, делая файл объемнее раз в 5 if(isset($input[1])){$a=$input[1];}
А если таких $input[1].....$input[101], каждую проверь. Просто жесть!
Что бы просто назначить переменную, вызывать целую функцию isset().

Как же у меня горит, прям словами не передать. Все кругом с новыми версиями все упрощают, что бы чище было и проще кодировать, эти же на PHP все наоборот усложняют. С каждой новой версией, вся страница в ошибках сразу становится у всех.

[toxa82]

Тогда всё еще хуже, говнокодер это вы. Ваши пару строк кода работают и срут нотисами и варнингами замедляя работу. Обернув в проверку на не пустую переменную и на то что она содержит валидные данные вы бы улучшили и ускорили свой код. Та и вы даже не знаете что в пхп поменялось за 12 лет. Ваш пример с isset можно записать $a=$input[1] ?? null

[PageUp]

toxa82,ну так я потому и спрашиваю тут, как можно малой кровью отделаться, так как не знаю, что нужно прописать, что бы не было этих ошибок. У меня не было их никогда. Не нужно наговаривать. Они появились только после перехода на 8. Если бы мне было плевать на ошибки, я бы их отключил и забил. Потому сюда и пришел, так как нужно править. И я удивился, что то что раньше не вызывало ошибок никогда, теперь всю страницу ими засирает.

[PageUp]

toxa82, С какого перепугу у меня был говнокод, если он не вызывал ни одного варнинга, не говоря уже об ошибках. Если php позволял так просто назначать переменные, значит можно.

Сейчас стал ругаться какого-то черта. Зачем спрашивается каждую входящую переменную проверять. Ну пустая она, да и бог с ним, может мне так и надо, будет пустая значит один результат, назначится каким то данным, значит другой результат.

[toxa82]

Вызывало если уровень ошибок включал в себя E_NOTICE. Он у вас не включает их поэтому вы их не видели. В PHP8 эти ошибки стали уровня E_WARNING поэтому вы их стали видеть. Удалите этот уровень тоже, но это вообще плохой вариант решения. Лучше заморочиться и исправить код.

[toxa82]

PageUp, поставил бы уровень ошибок E_ALL оценил бы. Подавление нотайсов это не значит что у тебя нет косяков в коде. Потому необъявленные переменные и ключи массивов и вынесли в варнинги

[PageUp]

toxa82, не, скрывать конечно не вариант. Надо исправлять. Мне всегда самому спокойней, зная, что файл с логами пустой не потому что отключен, а потому что не на что ругаться.

Загрузка сайта с кучей огромных скриптов и функций у меня занимала 0,2 секунды. Практически как папки на ПК открывались, не думаю, что там были какие-то нагрузки на производительность. Но геморроя сейчас подкинули просто ужас. Я не представляю как я буду все эти сотни explode() сейчас переписывать на какие-то бесполезные циклы, которые как мне еще дольше работают и вызывают доп.нагрузку. А все для того, что бы просто проверить назначение переменной. Как по мне, так это абсурд. Но выбора нет похоже. А чем заменить то explode() в моей ситуации можно? Ну может есть какой-то быстрый и более изящный вариант?

[toxa82]

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

[PageUp]

toxa82, вот у меня сейчас выдет в одном месте Warning: Undefined array key 83

Это 83 переменных, что бы каждую проверить input[0]... input[83] в любом случае придется вызывать цикл и в него вписывать isset. с увеличивающейся $num который вписывать в input[$num]. Как по мне это просто ужасно.

[toxa82]

Ты эксплодом чтоли цсв строку парсишь?

[PageUp]

toxa82,

Я бы эти 2 строки обернул бы в проверку что переменная с ип адресом не пустая

обернуть все в isset() ничего не даст по сути. Ну с IP возможно и решит, но в другом месте нет. Так как будет допустим $mass = '1|3|4||||4|||'

Делаешь $stroka = Explode('|', "$mass")

Ну да, $stroka будет не пустая, но только вот $stroka[5] и другие после нее будут пустыми и это снова варнинг.

[toxa82]

PageUp, они будут пустыми, но они будут существовать в массиве и варнинга не будет

[PageUp]

toxa82, файлы, которые содержат строки, разбитые на ячейки разделителями, которые и определяешь в explode. Чаще всего это | разделитель.
И когда нужно узнать какая ячейка какие данные имеет, обращаешься к переменной полученной из explode("|", $file)

[PageUp]

toxa82, у меня есть варнинг. Говорю ж, сейчас прям страницу засрало 83 строками. Где я в переменную, через разделители прописываю другие переменные, полученные из explode. Так он на все, что пустые ругается.

[PageUp]

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

[Ипатьев]

PageUp, не надо врать.

допустим $mass = '1|3|4||||4|||'
Делаешь $stroka = Explode('|', "$mass")

$stroka[5] и другие после нее НЕ будут пустыми. Пустой будет только $stroka[10]. Что ЕСТЕСТВЕННО, потому что в строке только 9 разделителей. И РНР не должен вам создавать по explode бесконечный массив.

Одно слово - наркодер.

[PageUp]

Ипатьев, о боже..... я это написал просто на ходу, не считая даже разделители, просто ради примера. Какой же ты душный то.

[Ипатьев]

Ну то есть у вас получается, что пхп должен после explode создать бесконечный массив.
Вы это. Завязывайте с наркотиками.

[PageUp]

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

[PageUp]

Ипатьев, мысленно дорисуй end и подставь. Ну емое. Ты понимаешь что я хотел показать лишь количество разделителей, что бы просто было видно как образуются переменные. Это я еще точку с запятой поставил и кавычки, представляю как бы у тебя пригорело, если бы я и их не написал)))

И это кто еще из нас наркоша) Сидит рандомный пример на серьезном виде разбирает, дабы указать кому-то на его низкий уровень знаний. От чего сам выглядишь только мега глупо.

[Ипатьев]

На самом деле это хорошо, что вы так нервничаете.
Это вас подсознательно эээ... копулирует мысль о том, что вы что-то делаете неправильно.
Но вы пока не готовы это принять. Это нормально. Стадия отрицания.
Со временем придет и понимание, и принятие )

[PageUp]

Ипатьев, было бы от чего нервничать. Просто тут и так работы выше крыши, сидеть править миллион строк. Да еще какой-то непонятный чел сидит и просто душнит и душнит и душнит

[Сергей delphinpro]

С какого перепугу у меня был говнокод, если он не вызывал ни одного варнинга, не говоря уже об ошибках.

Говнокод коварен – он работает!

Прочувствуйте всю суть этой фразы.

[toxa82]

9 разделителей - 10 элементов массива. Некоторые будут пустые, но они будет существовать и обращение к ним не вызовет ошибку.

toxa@vinga:~$ php -r "var_export(explode('|', '1|3|4||||4|||'));"
array (
  0 => '1',
  1 => '3',
  2 => '4',
  3 => '',
  4 => '',
  5 => '',
  6 => '4',
  7 => '',
  8 => '',
  9 => '',
)

И вот что я имел ввиду про IP:

if (!empty($ip_addr) && filter_var($ip_addr, FILTER_VALIDATE_IP)) {
    $pr_ip = explode(".", $ip_addr);
    $my_ip = $pr_ip[0].$pr_ip[1].$pr_ip[2];
} else {
    $my_ip = '';
}

[PageUp]

Сергей delphinpro, работать и "не выдавать ошибок" с включённым показом ошибок, это разные вещи.

[toxa82]

PageUp, Для разработки рекомендуют этот уровень

// Добавлять в отчёт все ошибки PHP
error_reporting(E_ALL);
// Добавлять в отчёт все ошибки PHP
error_reporting(-1);

[PageUp]

toxa82, да да, на нем как раз сейчас вот и правлю сижу. За пример с IP благодарю, в принципе рабочий вариант, вполне себе. Но вот с массивом, там все равно назначаются пустые переменные, которые нужно затем перепроверять. Я сделал по другому малолсть. Цикл на повышение сделал перед explode с назначением ="". А после него уже идет заполнение через explode. И получается, что каждый key имеет значение в любом случае. Потом в проверке оборачиваешь его в isset() и вроде норм.

[Ипатьев]

Опять какая-то шизофрения.

• В соседнем топике никаких циклов, а только безумная радость, что узнал про оператор подавления ошибок.
  
• Здесь "в нем и сижу", а рядом - "подавляю ошибки через @". Смысл тогда делать E_ALL? Тогда уж проще отключить варнинги и не писать собаку к каждому оператору присвоения.
  
• Если каждый key имеет значение в любом случае, то тогда зачем isset? Загадка.
  
• C IP адресом опять шизофрения. Уж где-где, а в реальном IPv4 всегда 4 октета. И если их вдруг меньше, то надо проверять, из какой помойки берется IP, а не что-то выковыривать с переменными.
  
  

Все больше убеждаюсь, что наркотики - это зло. Раздвоение личности - это очень плохой симптом.

[PageUp]

toxa82, странно, но он все равно ругается, что не пустые переменные. Все они назначаются через explode. Но предупреждение появляется на все ключи, что не найдено.

$arrdata = explode('|', $file[0]);

if($arrdata[19]>0){...
if($arrdata[20]>0){...
if($arrdata[21]>0){...

Undefined array key 19
Undefined array key 20
Undefined array key 21

В файле | 50 штук. Соответственно все 49 $arrdata должны быть назначены. Но не назначается. Собственно потому я и пошел сюда в поисках ответов. Я тоже думал, что должно назначаться, но вот нет. Почему, не пойму.

Я, конечно, оборачиваю в Isset(),

if(isset($statistic_data[19])>0){....

но все же, по идее и без него должна переменная быть рабочая уже. Так ведь?

[toxa82]

Покажите весь ваш код. Сделайте var_dump($arrdata) и посмотрете что у вас получается после explode. Ну и входящую переменную в explode тоже посмотрите.

[PageUp]

toxa82, var_dump()

array(2) { [0]=> string(0) "" [1]=> string(0) "" }

Я так понимаю, она назначает строки, а я от него требую проверки как к числу. Наверняка в этом причина.
Если через for прогнать все arrdata с присвоением нулей, то тогда предупреждения пропадают.

Для примера я взял файл не с 50-ю. Для примера поменьше, где 2. На всякий случай.
Ps строка одна, нулевая, и именно к ней идет обращение.

[Ипатьев]

Подсказка: в файле может быть больше одной строки.

[Ипатьев]

Но особенно здесь конечно разует вот это

о боже..... я это написал просто на ходу, не считая даже разделители, просто ради примера. Какой же ты душный то.
Написано час назад

и

Ой, не назначается! Собственно потому я и пошел сюда в поисках ответов. Я тоже думал, что должно назначаться, но вот нет. Почему, не пойму.
Написано 9 минут назад

Вы прогрессируете прямо на глазах :)

[toxa82]

if(isset($statistic_data[19])>0)
Это херня, делайте

if (!empty($statistic_data[19]) && $statistic_data[19] > 0)

empty() проверяет что переменная существует и не пустая.

[Ипатьев]

Я так понимаю, она назначает строки, а я от него требую проверки как к числу.

Это чушь. Учитесь читать сообщения об ошибках. Undefined array key не имеет никакого отношения к числам.

И кстати о числах. if(isset($statistic_data[19])>0) делает не то что вы думаете.

[PageUp]

toxa82, Ага, понял

[Ипатьев]

toxa82,

if (!empty($statistic_data[19]) && $statistic_data[19] > 0)

Это херня, скорее всего отрицательных чисел там нет, поэтому делайте
if (!empty($statistic_data[19]))
empty() проверяет что переменная существует и не пустая.

но короче всего будет
if ($statistic_data[19] ?? 0)

Answer 3

[MIX2MAX]

а чем не угодило типовое решение

set_error_handler(function(int $errno, string $errstr) {
    if ((strpos($errstr, 'Undefined array key') === false) && (strpos($errstr, 'Undefined variable') === false)) {
        return false;
    } else {
        return true;
    }
}, E_WARNING);

Comments

[MIX2MAX]

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

[Ипатьев]

А зачем вот это всё?
Если так хочется говнокодить, то достаточно просто поставить собаку перед обращением к массиву.

Answer 4

[PageUp]

Кому нужно будет решение, то вот. Нашел на западном аналогичном этому сайте. Работает как надо. Все ошибки касательно explode и не назначенных переменных пропали.

$data=explode("|", $file[0] ?? '');

Comments

[Ипатьев]

Это фигня а не решение.
И какое еще "западный сайт" если я это писал это в соседнем топике?

[Ипатьев]

Что вообще за бред?
В комментариях выше вы жалуетесь, что у вас ругает за обращние к $data, а не к $file[0]

[PageUp]

Ипатьев, ну за $data ругает, потому что $file[0] не всегда полный, а соответственно $data не от куда наполняться через Explode. Как то так. А западный сайт, это stackoverflow. Вы про ?? '' немного в другом ключе написали, уже во время проверки его писать, а не при установке массива, если я не ошибаюсь. Либо пропустил этот комментарий.