Wyświetla się tylko jeden wynik z tablicy

Question

Witam,

Oto kod z którym mam problem:

$item = $connect->query("SELECT * FROM shopping WHERE id_user = '$user_id'");	
if (!$item) throw new Exception($connect->error);
$nr_rows = $item->num_rows;
if($nr_rows>0){
    echo '<center>Lista przedmiotów w koszyku:<form action="" method="post"><table border="3"><tr><td>ID:</td><td>Nazwa:</td><td>Cena</td><td>Usuń:</td></tr>';
    if ($item){
	while ($buy = $item->fetch_assoc()){
		global $tablica;
	        $tablica = array('id'=>$buy['id_item']);
		echo '<tr><td>'.$buy['id'].'</td><td><a href="shop.php?id='.$buy['id_item'].'">'.$buy['name'].'</a></td><td>'.$buy['price'].' zł</td><td><input type="checkbox" name="delete" value="'.$buy['id'].'" ></td></tr>';									
	}
}else{
    echo '<div class="error">Błąd!</div>';
}



for($i=0;$i<$nr_rows;$i++){
$tablicaid = $tablica['id'];
 if ($connect->query("INSERT INTO user_buy VALUES (NULL, '$user_id', 'Zakup', '$tablicaid', 'Zakup', 'Niezweryfikowane')")){
											
}else{
	throw new Exception($connect->error);
}

Problem polega na tym, że podczas wrzucania rekordów do bazy z tablicy jest wyciągane tylko jedno id i powielane, a w tablicy te ID są różne.

ID w tablicy są różne. Sprawdzałem.

Czy ktoś wie o co tu może chodzić?

Answer 1

powinno byc $tablica .= array ...
edit: myliłem się, poprawna odpowiedz w komentarzach

Comments

Kiedy wstawię w tą pętle:

echo $tablica;

wyświetla mi sie:

38393939393939

Więc id jest dodawane na 100%.

W ogóle nie za bardzo wiem po co robisz tą dodatkową tablicę. Nie prościej od razu w tej pętli, która wyświetla dane z bazy, zrobić dodawanie rekordów?

Jeśli dobrze Cię rozumiem, to muszę to mieć w tablicy, bo każdy przedmiot ma inne ID, a jak mam przedmiotów kilaka, to gdzieś to ID muszę trzymać

Edit:

właśnie się zorientowałem, że rzeczywiście jest tak jak mówisz. Tam jest tylko jeden rekord. Nie zorientowałem się, że jak dam echo w pętli to mi się będzie nadpisywać. W takim wypadku co mam zrobić, żeby wszystkie te rekordy tam siedziały?

---

Ja bym to zrobił bardziej w ten sposób:

<?php
$result = $connect->query("SELECT * FROM shopping WHERE id_user = '$user_id'");    
$items = $result->fetch_all(MYSQLI_ASSOC);
if (!empty($items)) {
    echo '<center>Lista przedmiotów w koszyku:<form action="" method="post"><table border="3"><tr><td>ID:</td><td>Nazwa:</td><td>Cena</td><td>Usuń:</td></tr>';
    foreach ($items as $item) {
        echo '<tr><td>'.$item['id'].'</td><td><a href="shop.php?id='.$item['id_item'].'">'.$item['name'].'</a></td><td>'.$item['price'].' zł</td><td><input type="checkbox" name="delete" value="'.$item['id'].'" ></td></tr>';                                                              
        $connect->query("INSERT INTO user_buy VALUES (NULL, '$user_id', 'Zakup', $item['id'], 'Zakup', 'Niezweryfikowane')");
    }
}

A jeśli już musisz z jakiegoś powodu dzielić na dwie pętle to coś tego typu:

<?php
$itemIds = [];
$result = $connect->query("SELECT * FROM shopping WHERE id_user = '$user_id'");    
$items = $result->fetch_all(MYSQLI_ASSOC);
if (!empty($items)) {
    echo '<center>Lista przedmiotów w koszyku:<form action="" method="post"><table border="3"><tr><td>ID:</td><td>Nazwa:</td><td>Cena</td><td>Usuń:</td></tr>';
    foreach ($items as $item) {
        echo '<tr><td>'.$item['id'].'</td><td><a href="shop.php?id='.$item['id_item'].'">'.$item['name'].'</a></td><td>'.$item['price'].' zł</td><td><input type="checkbox" name="delete" value="'.$item['id'].'" ></td></tr>';                                                               $itemIds[] = $item['id'];
    }
}

 
foreach ($itemIds as $id) {
    $connect->query("INSERT INTO user_buy VALUES (NULL, '$user_id', 'Zakup', '$id', 'Zakup', 'Niezweryfikowane')");
}

W zasadzie to jakby jeszcze pokombinować mógłbyś w jednym zapytaniu dodać wszystkie id. Poza tym lepiej byłoby bindować parametry do zapytań.

To taka poprawka na szybko, pisane bez testowania. Chciałem Cię tylko trochę naprowadzić jak to zrobić, a nie dać gotowca.

---

Dzięki za pomoc, nigdy bym się chyba nie domyślił, że w taki sposób trzeba to zrobić. :/

---

Nie trzeba koniecznie w taki, można równie dobrze zamiast fetch_all() fetchować każdy rekord po kolei w pętli, tylko po co. Tak jak pokazałem wydaje mi się łatwiej, ale no sposobów na zrobienie tego może być kilka ;)

---

Szczerze Ci powiem, że tablicami w PHP nie zajmowałem się wcześniej stąd mój brak wiedzy, nawet nie umiem sobie wyobrazić jakby wyglądało to z fetchowaniem każdego rekordu po kolei. Dopiero po Twoim przykładzie trochę to ogarnąłem.