Что не так с циклом?

Question

[Flexx0]

Что не так с циклом? В БД приходит дубль последней записи.

$id_tr = $_POST['id_tr'];
extract($_POST);
$yyy = (count($_POST)) / 4;

for ($i = 0; ; $i++) {
   if ($i == $yyy ) {
      break;
   } else {
		$sql = "INSERT INTO wow (id_tr, day_no, day_title, day_desc)
		VALUES ('$id_tr', '${"day_no".$i}', '${"day_title".$i}', '${"day_desc".$i}')";
		$conn->query($sql);
   }  
}

Comments

[Дмитрий]

echo '<pre>' . print_r($_POST, true);
Кинь сюда вывод

[FanatPHP]

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

вопрос надо задавать не в форме "вот у меня говнокод, он не работает", а "вот у меня задача, как написать для неё код?"

Answer 1

[Владимир Могила]

Привет, у тебя в форе отсутствует условие для выхода из цикла

т.е сейчас написано от и = 0, делать увеличение и на 1 каждый раз

правильнее написать думаю
for ($i = 0; $i <= $yyy ; $i++) {
$sql = "INSERT INTO wow (id_tr, day_no, day_title, day_desc)
VALUES ('$id_tr', '${"day_no".$i}', '${"day_title".$i}', '${"day_desc".$i}')";
$conn->query($sql);
}

Comments

[FanatPHP]

а break; по-твоему - это не выход из цикла?

[Владимир Могила]

FanatPHP, тоже выход, но возможно не срабатывает условие if

[FanatPHP]

Если бы не срабатывал, то был бы бесконечный цикл