"PHP" / Говнокод #1796 Ссылка на оригинал

0

  1. 1
  2. 2
  3. 3
  4. 4
  5. 5
for ($i = 0; $i < count($oldCLI); $i++) {
	if ($oldCLI[$i] == null) {
		unset($oldCLI[$i]);
	}
}

unset($oldCLI[$i]); - это как я понимаю контрольный выстрел в голову

Запостил: Sadie Sadie, (Updated )

Комментарии (5) RSS

  • Ну так это очистка массива от ненужных элементов. Или Вы полагаете, что если значение массива == null, то такого элемента не существует?

    Хотя минус тут за == null (is_null() таки лучше). Еще за count() на каждом шаге цикла.
    Вероятно, тут удобно было бы использовать array_filter.
    Ответить
    • Если делать unset, то ведь число элементов в массиве уменьшается... Тогда count будет (может) разные числа выдавать при каждой проверке... Не выйдет ли так, что проход завершится раньше того, как кончится массив?
      Ответить
      • Как раз таки выйдет. Но на говнокод тянет слабо.
        Ответить
  • все чаще говнопостят код люди, практически не имеющие нужной компетенции в вопросе, который они так хотят обосрать
    Ответить

Добавить комментарий

Переведи на "PHP", guest!

    А не использовать ли нам bbcode?


    8