1. PHP / Говнокод #1796

    +151.7

    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, 09 Сентября 2009

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

    • ava striker 09.09.2009 15:10 # +2

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

      Хотя минус тут за == null (is_null() таки лучше). Еще за count() на каждом шаге цикла.
      Вероятно, тут удобно было бы использовать array_filter.
      Ответить

      • ava interested 09.09.2009 16:26 # 0

        Если делать unset, то ведь число элементов в массиве уменьшается... Тогда count будет (может) разные числа выдавать при каждой проверке... Не выйдет ли так, что проход завершится раньше того, как кончится массив?
        Ответить

        • ava guest 09.09.2009 17:05 # 0

          Как раз таки выйдет. Но на говнокод тянет слабо.
          Ответить

    • ava guest 09.09.2009 15:54 # +2

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

    • ava zerkms 10.09.2009 01:42 # 0

      array_filter + is_null
      Ответить

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