Откуда берется неизвестная строчка?

Question

[Андрей Андрианов]

Всезнающий ALL !

Вывожу в CSV массив простым способом

$i=0 ;
$str = '' ;
foreach ( $newarr as $Row ) {
if (($Row[1]) && ($Row[1] != '')) {
$str .= $i .";". $Row[1] .";". $Row[2] .";". $Row[3] .";". $Row[4] .";". $Row[5] .";
" ;
++$i; 
  }
}

В середине получаю пустую запись
27;96261;340;
;;;;;;;;;
28;96356;130;

Откуда она берется - не понятно.
Сделана проверка на пустое значение .
Даже если бы какое-то значение - то $i дало бы знать
Пробовал несколько раз - всегда между 27 и 28 строками.

P.S. Перепроверю на более большом массиве (раз в 5 больше)
если ничего не вылезит - то значит в 27 строчке из-за незакрытой кавычки сбой

Comments

[Максим]

я не сильно волоку в этом но вы проверяете на не равенство $Row[1]&& $Row[1] - это верно ? и ещё в самом конце у вас " ; по моему лишнее

[Талян]

Максим Ленский, неравенство, это !=, а у автора && - логическое "И"

[Максим]

Талян, ну так то да ...это не так звучит "если row[1] и row[1] не равно " ?

[Талян]

Максим Ленский, if($row) равнозначно if($row!=false).

Сперва проверетс первое условие , потом второе (после &&), при чем если первое не выполнено, то второе даже проверяться не будет.

Автор сделал так чтобы не было ошибки unknown var $row[1] при проверке $row[1]!="" если она по какой-то причине не существует.

То есть если $row[1]==false, то if($row[1]!="") выдаст ошибку о несуществующей переменной.

[Александр]

Максим Ленский, сравнение написано криво, как и весь код, но синтаксических ошибок там нет (2 раза проверяется что $Row[1] не пустой).

[Максим]

Талян, это так читается ??

[Александр]

Талян,

Автор сделал так чтобы не было ошибки unknown var $row[1] при проверке $row[1]!="" если она по какой-то причине не существует.

Смею заметить, что не сделал. Т.к. эта ошибка проверяется через isset. А у него простое преобразование к bool типу, что бросит warning.

[Талян]

Александр, вы правы, но с точки зрения минимального уровня E_NOTICE:

[Александр]

Максим Ленский,

это так читается ??

Читается это примерно так:
Если преобразовав $Row[1] к bool типу он равен true И $Row[1] не пустая строка, то {...}, иначе пропустить.

[Александр]

Талян, Я точно не помню, какой там уровень ошибки, т.к. никогда с ней не сталкиваюсь :)

[Андрей Андрианов]

Товарищи дорогие !
не туда , наверное , смотрите (имею ввиду проверку)

1. если бы даже проверка была не правильной тогда бы $i дало значение и
в $str была бы циферка вначале .

2. если проверка правильная и в цикл не заходит пустое(неправильное) значение , то
откуда берется пустая строка тогда ?

Какой экспериментус еще поставить ?

[Александр]

Андрей Андрианов,

Какой экспериментус еще поставить ?

Я бы вам посоветовал еще поучиться. Учитывая, какой вы ужасный код пишете.

[Андрей Андрианов]

Александр , пожалуйста не отвлекайте форум своей философией.

[Андрей Андрианов]

Владимир,

вывожу массив в CSV именно простым способом , сразу с переводом на новую строку

$str .= $i .";". $Row[1] .";". $Row[2] .";". $Row[3] .";". $Row[4] .";". $Row[5] .";
" ;

---

отрывок CSV с которого я получаю вначале массив
из которого удаляю повторы и далее вывожу опять в CSV

26;962617;340;"Скульптурирование";2019-05-26 12:00:00;Фурштатская, 12;;;;462;;TP;
27;962615;340;"Скульптурирование.;2019-05-19 12:00:00;Фурштатская, 12;;;;462;;TP;
28;963560;130;"Слишком";2019-05-04 12:00:00;Наб.реки Фонтанки 119;;;;379;;TP;
29;968520;306;"Трогательно";2019-05-11 11:00:00;ул. Венская, 5;;;;453;;TP;

( ничего плохого ;) тут не видно )

[Андрей Андрианов]

конечно могу еще раз перепроверить файл и удалить эту строчку
но все-таки - откуда она взялась :)

Мысль Arman хороша , но тогда бы $i дала о себе знать в строчке

[Дмитрий]

Андрей Андрианов, если получаете массив через file() , то добавьте флаг игнорировать пустые строки

[Андрей Андрианов]

Владимир
тоже на это смотрел
но вначале массив из CSV получаю так

$event_str = str_getcsv($Row, ";");
$event[] = $event_str;

поэтому нарезка в массив идет по ";"

хотя конечно всяко может быть ...

[Александр]

Владимир,

при чем если первое не выполнено

$var1 в вашем примере должен быть false. Он говорит про этот кейс.

[Виталий Хоменко]

> пожалуйста не отвлекайте форум своей философией.

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

[Андрей Андрианов]

Всех благодарю !

строчка вылезала из-за того что в Row[1] не было закрытой "
вместо нее стояла .

но строка резалась по шаблону ";"
и почему-то эта точка в массиве сыграла роль.

Убрал кавычки из этого поля массива - ОК
точка стоит до сих пор и не мешает .
(пока не мешает)

Далее , (если кому-то интересно)
в этот же момент пришлось откатить с 7.1 на 5.6
почему - долгая история .

Ответы
synapse_people - не все так просто и сразу
Arman - возможно , потому что стоит функция по удалению повторов в многомерном массиве
(может она косячит)
Талян - дело не столько в проверке Row , как почему $i не давала значение

[Андрей Андрианов]

Виталий Хоменко,

покажите на явную ошибку здесь
$i=0 ;
$str = '' ;
foreach ( $newarr as $Row ) {
if (($Row[1]) && ($Row[1] != '')) {
$str .= $i .";". $Row[1] .";". $Row[2] .";". $Row[3] .";". $Row[4] .";". $Row[5] .";
" ;
++$i;
}
}

Answer 1

[Arman]

Подозреваю один из $Row[N] просто содержит набор ";" =)

Comments

[Андрей Андрианов]

Arman , Подозреваю один из $Row[N] просто содержит набор ";" =)
это мысль .. проверю ...

[Евгений]

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

[Евгений]

Или 29 строка... если вывод с $i=0

[Arman]

Андрей Андрианов, Евгений интересную мысль подсказал, попробуй ++$i; вынести вне условия, тогда ясно будет сколько прошло итераций

Answer 2

[synapse_people]

вот и зачем этот говнокод, когда есть fputcsv????

Comments

[synapse_people]

Владимир, да я понял в чем у него проблема, просто код не очень
А вообще, знаков ; получается 9, что не может быть правдой, с учетом того, что их должно быть 6 исходя из кода, значит или смотрелка кривая, или автор чет путает, или скрывает какой-то левый код, котого нет в коде из топика

[synapse_people]

Владимир, да и эта функция экранирует данные внутри массива, вдруг там будет эта дичь с ";"

Answer 3

[Талян]

попробуйте trim()
if (($Row[1]) && (trim($Row[1]) != ''))

Comments

[Александр]

Если внимательно вчитаться в кусок скрипт, то он никаким образом не может выдать, то что автор привел в качестве результата. Т.к. $i не может быть пустой строкой.
Проблема в другой части скрипта, которую мы не видим.