zle wykonanie WHILE i wpisanie do bazy Opcje

[roghatt]

Najpierw kawałek kodu

[PHP] pobierz, plaintext 
<FORM ACTION="index.php?top=<?=$ADM_TOP_FAKTURA?>" METHOD="POST" enctype="multipart/form-data">
<?
echo "<table border=\"0\" cellspacing=\"4\">";
		echo "<tr>";
		echo "<td style=\"background-color: #dffcf8\" colspan=\"3\">";
?>
		Wpisz numer faktury: <INPUT TYPE="TEXT" NAME="nrfaktury" size="25">
<?
		echo "</td>";
		echo "</tr>";
while($row=mysql_fetch_array($wynik1))
{
		$id=$row['id'];
		$nazwa= $row['nazwa'];
		$cenan = $row['cenan'];
		$podatek = $row['podatek'];
		$nrfaktury = $_POST['nrfaktury'];
 
		echo "<tr>";
		echo "<td style=\"background-color: #dffcf8\">";
		echo $nazwa;
		echo "</td>";
		echo "<td style=\"background-color: #dffcf8\">";
		echo "$cenan";
		echo " zl.";
		echo "</td>";
		echo "<td style=\"background-color: #dffcf8\">";
?>
		Wpisz ilosc: <INPUT TYPE="TEXT" NAME="ilosc" size="5">
<?
		$ilosc = $_POST['ilosc'];
		echo "</td>";
		echo "</tr>";
 
if ($nrfaktury) {
mysql_query ("INSERT INTO faktura VALUES (' ','$nazwa', '$cenan', '$podatek', '$pkwiu', '$jm','$kreska','$nrfaktury','$ilosc');");
if (mysql_affected_rows()>0) echo("Faktura dodana.");
$result=dbquery($sql,$conn);
 
}
}
 
echo "</table>";
?>
<input type="hidden" name="test" value="test"><br>
<input type="submit" value="&nbsp;&nbsp;&nbsp;&nbsp;Dodaj&nbsp;&nbsp;&nbsp;&nbsp;">
</form>
[PHP] pobierz, plaintext 

Formularz wygląda tak.
Na samej górze pole do wpisania nr faktury.
Pod spodem jedno pod drugim spis produktów nazwa i cena (pobierane z innej tabeli). Obok produktu pole na wpisanie ilości.

Wszystko po kliknięciu na "Dodaj" powinno być wpisane do drugiej tabeli. Wszystko pięknie mi wpisuje, nazwę, cenę, i inne parametry. Numer faktury mi też wpisuje do tabeli, ale w polu "ilość" wpisywana jest ostatnia wartość wpisana przy ostatnim produkcie.
Czyli np. jak wpiszę przy produktach ilość:
Produkt1 ilosc:5
Produkt2 ilosc:6
Produkt3 ilosc:7

i w tabeli w polu ilość wszędzie jest cyfra 7. Po kliknięciu w "dodaj" wywołana jest ta sama strona z formularzem bo w nim znajduje się funkcja wpisywania do bazy.
Nie mogę sobie poradzić jedynie z wpisywaniem tej ilości przy każdym produkcie.
Mam 5 produktów i wyświetla się 5 razy "Faktura dodana" więc 5 razy jest do bazy dodawane ale jak pisałem wpisuje w polu "ilość" wszędzie ostatnią wartość.

Dodatkowo jaki parametr dodać aby nie wpisywało do bazy produktu w której nic nie wpiszemy w polu "ilość"
Myślałem, że:

[PHP] pobierz, plaintext 
if ($ilosc) {
mysql_query ("INSERT INTO faktura VALUES (' ','$nazwa', '$cenan', '$podatek', '$pkwiu', '$jm','$kreska','$nrfaktury','$ilosc');");
if (mysql_affected_rows()>0) echo("Faktura dodana.");
$result=dbquery($sql,$conn);
}
[PHP] pobierz, plaintext 

ale coś nie do końca działa.
Proszę o wskazówki.

Ten post edytował freelinkz 7.09.2009, 20:43:17

[nexis]

Proszę o wskazówki.

1. wyłącz register_globals
2. odwołuj się do danych z formularza poprzez $_POST['nazwa_zmiennej']
3. wyświetl sobie dane otrzymane z formularza i zobacz co w nich masz - print_r($_POST);

[thek]

Masz problem bo mieszasz wszystko ze wszystkim. Zauważ, że w pętli while część danych bierzesz z bazy ale jednocześnie w tej samej pętli część danych bierzesz z $_POST. Problem z $ilosc masz taki, że powinieneś sprawdzić czy dane z $_POST z tego pola istnieją najpierw przez isset. Formularz bowiem przesyła tylko wypełnione pola. Jeśli nic nie wpiszesz to nawet go nie prześle. A masz taki babol, że do zmiennej ilosc przypisujesz nieistniejące, czyli tak naprawdę nie wiesz co tam Ci się ustawi. Posprawdzaj co masz w zmiennych to sam zauważysz byki.

[maly_swd]

[PHP] pobierz, plaintext 
<FORM ACTION="index.php?top=<?=$ADM_TOP_FAKTURA?>" METHOD="POST" enctype="multipart/form-data">
<?
echo "<table border=\"0\" cellspacing=\"4\">";
		echo "<tr>";
		echo "<td style=\"background-color: #dffcf8\" colspan=\"3\">";
?>
		Wpisz numer faktury: <INPUT TYPE="TEXT" NAME="nrfaktury" size="25">
<?
		echo "</td>";
		echo "</tr>";
$i=0;
while($row=mysql_fetch_array($wynik1))
{
		$id=$row['id'];
		$nazwa= $row['nazwa'];
		$cenan = $row['cenan'];
		$podatek = $row['podatek'];
		$nrfaktury = $_POST['nrfaktury'];
 
		echo "<tr>";
		echo "<td style=\"background-color: #dffcf8\">";
		echo $nazwa;
		echo "</td>";
		echo "<td style=\"background-color: #dffcf8\">";
		echo "$cenan";
		echo " zl.";
		echo "</td>";
		echo "<td style=\"background-color: #dffcf8\">";
?>
		Wpisz ilosc: <INPUT TYPE="TEXT" NAME="ilosc[]" size="5">
<?
		$ilosc = $_POST['ilosc'][$i];
		echo "</td>";
		echo "</tr>";
 
if ($nrfaktury) {
mysql_query ("INSERT INTO faktura VALUES (' ','$nazwa', '$cenan', '$podatek', '$pkwiu', '$jm','$kreska','$nrfaktury','$ilosc');");
if (mysql_affected_rows()>0) echo("Faktura dodana.");
$result=dbquery($sql,$conn);
 
}
$i++;
}
 
echo "</table>";
?>
<input type="hidden" name="test" value="test"><br>
<input type="submit" value="&nbsp;&nbsp;&nbsp;&nbsp;Dodaj&nbsp;&nbsp;&nbsp;&nbsp;">
</form>
[PHP] pobierz, plaintext 

[nexis]

I posprzątaj swój kod trochę - np.:

[PHP] pobierz, plaintext 
<?php 
if (isset($_POST['nrfaktury'], $_POST['ilosc'])) {
    foreach ($_POST['ilosc'] as $id) {
        $query = sprintf("INSERT INTO `faktura` VALUES ('%d')", $id); // tutaj dodaj dodatkowe pola
        mysql_query($query) or exit(mysql_error());
    }
}
?>
<form action="<?= $_SERVER['PHP_SELF'] ?>" method="post">
    <table id="uzywaj_stylow_css">
        <tr>
            <td>
                Wpisz numer faktury: <input type="text" name="nrfaktury" />
            </td>
        </tr>
        <?php 
        $query = "Twoje zapytanie MySQL";
        $query = mysql_query($query) or exit(mysql_error());
        while ($row = mysql_fetch_array($query)) {
            printf('<tr><td>%s</td><td>%01.2f zł</td><td><input type="text" name="ilosc[%d]" /></td></tr>', $row['nazwa'], $row['cenan'], $row['id']);
        }
        ?>
    </table>
    <p>
        <input type="submit" value="Zapisz zmiany" />
    </p>
</form>
[PHP] pobierz, plaintext 

[roghatt]

maly_swd dziala :-) dzieki wielkie za pomoc
juz bede wiedzial o co chodzi.

nexis dzieki za wskazowki, to jest bardzo prosty skrypt i nie uzywalem do tego styli tylko pisalem jak leci, ale widze ze kod duzo mniejszy, bede sie juz do tego stosowal.

Ten post edytował freelinkz 7.09.2009, 21:18:02