Witam. Proszę o podpowiedź. Mam dwa formularze w zasadzie identyczne. Zmieniłem w drugim tylko nazwy zmiennych i przestał działać. Chciałbym zrozumieć dlaczego. Analizowałem to przez dwie godziny i nie pojmuję. Co może powodować to, że nie wysyła się POST z drugiego formularza? Po kliknięciu wykonaj w drugim przypadku strona się tylko przeładowuje.

[PHP] pobierz, plaintext 
<?
		$towar=$_POST[towar]; 
		$towar1=$_POST[towar1];
		$cena=$_POST[cena];
		$towar2=$_POST[towar2];
		$cena2=$_POST[cena2];
?>
 
 
<div style="width:160px;border:solid 0px black;">
		<div style="padding-bottom:5px;">Ustalanie ceny:</div>
		<?php
 
	echo "<form action='magazyn.php' method=post>";
 
			echo "<br>";
			echo "Towar <input style=width:75px; type=text name='towar1'>";
			echo "<br>";
			echo "Cena <input style=width:75px;margin-left:8px; autocomplete=off type=text name='cena'>";
			echo "<br>";
			echo "<input style=margin-left:60px;margin-top:4px; type=submit value='Wykonaj'>";
			echo "</form>";
 
?>
		</div>
 
<?
if($_POST[cena]){
$odpowiedz1 = mysql_query("update ceny set $towar1=$cena WHERE nrkonta = '$nrkonta';");}
 
?>	
 
<div style="width:160px;border:solid 0px black;">
		<div style="padding-bottom:5px;">Ustalanie ceny2:</div>
		<?php 
 
	echo "<form action='magazyn.php' method=post>";
 
			echo "<br>";
			echo "Towar <input style=width:75px; type=text name='towar2'>";
			echo "<br>";
			echo "Cena2 <input style=width:75px;margin-left:8px; autocomplete=off type=text name='cena2'>";
			echo "<br>";
			echo "<input style=margin-left:60px;margin-top:4px; type=submit value='Wykonaj'>";
			echo "</form>";
 
?>
		</div>
 
<?
 
if($_POST[cena2]){
$odpowiedz2 = mysql_query("update ceny set $towar2=$cena2 WHERE nrkonta = '$nrkonta';");}
 
?>	
[PHP] pobierz, plaintext 

w $_POST[towar] itd brakuje ' '

Niestety niczego to nie zmieniło. Pierwszy formularz działa. Drugi nie.

[PHP] pobierz, plaintext 
<form action='magazyn.php' method=post> // 'post'
 
<input style=width:75px;margin-left:8px; autocomplete=off type=text name='cena2'>
<input style=margin-left:60px;margin-top:4px; type=submit value='Wykonaj'>
[PHP] pobierz, plaintext 

Od kiedy inputa tak się robi?

[HTML] pobierz, plaintext 
<input type="text" name="cena2">
<input type="submit" value="Wykonaj">
[HTML] pobierz, plaintext 

[PHP] pobierz, plaintext 
if($_POST[cena])
[PHP] pobierz, plaintext 

robisz takie bledy, ze szok

pamiętaj o "" bo bez tego nie będzie Ci działać... up może tak ale jak wrzuci to do odpowiednich "" choć style powinno się określać klasa lub po id

[PHP] pobierz, plaintext 
$odpowiedz2 = mysql_query("update ceny set $towar2=$cena2 WHERE nrkonta = '$nrkonta';");}
[PHP] pobierz, plaintext 

$towar2=$cena2 czegoś takiego nie ma w sql sądzę że $towar2=$cena2 nie masz kolumny o nazwie z zmiennej towar2 ale może się mylę, tak to powinienem napisać sory

Poprawiłem to na co mi zwróciliście uwagę i kod wklejam poniżej. Pierwszy form działa, drugi dalej nie. Wydaje mi się, że problem jest ze zmiennymi przesyłanymi postem w drugim formularzu. Nie m dlaczego nie działają. Kiedy je zamienię z jednego formularza do drugiego wtedy pierwszy nie działa, a drugi tak. Bez problemu podmienia wpis w bazie. Używam styli w kodzie bo tak mi wygodniej na szybko coś przesunąć. Normalnie używam class.

[PHP] pobierz, plaintext 
 
 
<?
		$towar=$_POST['towar']; 
		$towar1=$_POST['towar1'];
		$cena=$_POST['cena'];
		$towar2=$_POST['towar2'];
		$cena2=$_POST['cena2'];
?>
 
 
<div style="width:160px;border:solid 0px black;">
	<div style="padding-bottom:5px;">Ustalanie ceny:</div>
	<form action='magazyn.php' method='post'>
	Towar <input type='text' name='towar1'>
	Cena <input autocomplete='off' type='text' name='cena'>
	<input type='submit' value='Wykonaj'>
	</form>
	</div>
<?
if($_POST['cena']){
$odpowiedz1 = mysql_query("update ceny set $towar1=$cena WHERE nrkonta = $nrkonta;");}
?>	
 
<div style="width:160px;border:solid 0px black;">
 
	<div style="padding-bottom:5px;">Ustalanie ceny2:</div>
	<form action='magazyn.php' method='post'>
	Towar <input type='text' name='towar2'>
	Cena2 <input autocomplete='off' type='text' name='cena2'>
	<input type='submit' value='Wykonaj'>
	</form>"
 
</div>
<?
if($_POST['cena2']){
$odpowiedz2 = mysql_query("update ceny set $towar2=$cena2 WHERE nrkonta = $nrkonta;");}
?>
 
[PHP] pobierz, plaintext 

1)
zrob
print_r($_POST);
a bedziesz widzial co idzie a co nie, NIe ma co zgadywac :/

2) Uzywasz zmiennej $nrkonta ktorej nigdzie nie widze

3) Sprawdz czy nie ma bledow bazy.....
Temat: Jak poprawnie zada pytanie


Twoj problem sprowadza sie do prostej analizy danych i zamiast to zrobic to lecisz na forum.

Nie do końca się zastosowałeś

[PHP] pobierz, plaintext 
$odpowiedz1 = mysql_query("update ceny set $towar2=$cena2 WHERE nrkonta = $nrkonta;");}
[PHP] pobierz, plaintext 

Turson a co ci tu nie pasuje? Nie slyszales ze nazwe kolumny tez mozna przekazac do tekstu budujacego zapytanie tak samo jak jej wartosc?

Najpierw pomyślałem, że autor po prostu niepotrzebnie wstawił znak dolara i nie chciało mi się wierzyć, że kolumna nazywa się `$towar`, ale dopiero teraz widzę, że cena update'owana w zależności od danych z formularza, więc ok

Zrobiłem jak nospor polecił. Post jest wysyłany z formularza, ale nie jest przypisywany do zmiennych $towar2=$_POST['towar2'] i $cena2=$_POST['cena2']. Zmienna $nrkonta jest zdefiniowana na samym początku. Nie rozumiem dlaczego działa w pierwszym przypadku.

Tak, tak, oczywiscie..... i wlasnie przed 5cioma minutami zniknela grawitacja na ziemi

var_dump($towar2);
pokazuje? Pokazuje. Jezeli nie pokazuje, znaczy ze
1) $_POST['towar2'] nie ma nic
2) W zlym miejscu to sprawdzasz. zamiast po przypisaniu, to robisz to przed.
3) Podales nam nie caly kod i nie widzimy wszystkich bledow jakich robisz

a co ma kolumnę o nazwie $_POST['towar2'] ? bo nie wydaje mi się dlatego na to zruciłem uwagę, on ma przecież ustawić dla tej wartości, a nie kolumny o nazwie jablko , tylko raczej towar

no chyba ze tak własnie ma i wy to wiecie bo ja tego np nie wiem

pomijając fakt że nad takim rozwiązaniem trudno o kontrole bo to że tak sie da to ja wiem, nie wyraziłem się tam jasno, tzn mój skrót myślowy został źle odebrany...

@com naprawde nie widzisz obok pola CENA2:
Towar <input type='text' name='towar2'>
Poza tym pobiera ją wyraźnie z posta, poza tym pierwszy form mu dziala a ma tak samo zrobione.... to sie nazywa kojarzenie faktow

A to ze o kontrole trudno to inna bajka, ktora nie jest tutaj omawiana.

Na początek chciałem podziękować za poświęcony czas. Uczę się php sam i z doskoku czasami, ale lubię to Zastosowałem się do rad nospor i faktycznie dane z post są przypisywane do zmiennych. W złym miejscu to sprawdzałem. Mimo to kod nie działa. Uprościłem go maksymalnie i wklejam poniżej cały plik. Odkryłem, że problemem jest chyba pętla rysująca tabelę z bazy. Kiedy ją wytnę formularz robi wpisy w bazie bez problemu. Co może to powodować?

[PHP] pobierz, plaintext 
 
<?php
ob_start();
session_start();
$nrkonta=$_SESSION['nrkonta'];
include("includes/check.php");
include("includes/baza.php"); 
 
		if($_POST['wybor1']=='maga'){$wybor1='magazyn';}
		if($_POST['wybor1']=='skle'){$wybor1='sklep';}
		$cena=$_POST['cena'];
		$towar=$_POST['towar'];
 
		$kom1="witamy";
 
?>
 
<div class="cala">
<div class="lewa">
 
<?php
include("includes/info.php");
include("includes/menumagazyn.php");
?>
 
</div>
 
<div class="srodek">
 
	<div style="border:solid 0px black;">
 
		<?php // część odpowiedzialna za zapamiętywanie kierunku transferu towarów
		if($_SESSION['ref']==1){$wybor1=sklep; $_SESSION[ref]='';}
		if($_SESSION['ref']==2){$wybor1=magazyn; $_SESSION[ref]='';}
 
 
	?>	
			<form action='magazyn.php' method='post'>
			<select name='wybor1'>
			<option value='start' name='start'>Wyświetl stan:</option>
			<option value='magazyn' name='maga'>Magazynu</option>
			<option value='sklep' name='skle'>Sklepu</option>
			</select>
			<input style=margin-left:60px;margin-top:4px; type='submit' value='Wykonaj'>
			</form>
 
	</div>
 
 
			<br>
		<div id="srodek-lewa" style="float:left;border:solid 0px black;">
 
		<? //pętla wyświetlająca zawartość magazynu
 
		if(empty($_POST['wybor1']) and empty($_POST['wybor']) and empty($wybor1) and empty($_POST['cena'])){echo $kom1; exit();}
		if($wybor1==start){echo $kom1; exit();} // wyświetla komunikat poczatkowy
 
		if(!empty($wybor1)){echo 'Aktualny stan:';
		if($wybor1=='magazyn'){echo 'Magazynu';}
		if($wybor1=='sklep'){echo 'Sklepu';}
 
		echo '<br>';
		echo '<br>';
 
		$resResult1 = mysql_query( "SELECT * FROM $wybor1 where nrkonta=$nrkonta" );
		$arrRow1 = mysql_fetch_array( $resResult1, MYSQL_ASSOC );
		ksort($arrRow1);
 
		echo '<table  style="float: left; width: 400px;">';
		echo '<tr>';
		foreach( $arrRow1 as $strKey1 => $strValue1)
		{
		if(($strKey1 != 'id') and ($strKey1 != 'nrkonta') and ($strValue1 != 'id') and ($strValue1 != 'nrkonta') and (!empty($strValue1)))
	{
 
 
		$resResult = mysql_query( "SELECT $strKey1 FROM ceny where nrkonta=$nrkonta" );
		$arrRow = mysql_fetch_array($resResult);
		$x=$arrRow["$strKey1"];
 
		echo "<tr><td style=padding-bottom:11px;>$strKey1:</td>";
		echo "<td style=padding-bottom:11px;>$strValue1</td>";
		echo "<td style=padding-bottom:11px;>$x $waluta</td></tr>";
	}
 
		}
		echo '</tr>';
		echo '</table>';
		}
		?>
		</div>
 
	<div id="srodek-prawa" style="border:solid 0px black;float:right;">		
 
<div style="width:160px;border:solid 0px black;">
			<div style="padding-bottom:5px;">Ustalanie ceny:</div>
			<form action='magazyn.php' method='post'>
			<input type='text' name='towar'>
			<input autocomplete='off' type='text' name='cena'>
			<input type='submit' value='Wykonaj'>
			</form>
</div>
<?
if(!empty($_POST['cena'])){$cena = str_replace(',', '.', $cena);}
if($_POST['cena']){
if($wybor1==sklep){$_SESSION['ref']=2;}else{$_SESSION['ref']=1;}
$odpowiedz4 = mysql_query("update ceny set $towar = $cena WHERE nrkonta = $nrkonta;");
header("Location: magazyn.php");
}
?>	
	</div>
</div>
</div>
 
 
 
 
</body>
</html>
 
 
[PHP] pobierz, plaintext 

[PHP] pobierz, plaintext 
$_SESSION[ref]
$_SESSION[ref]
[PHP] pobierz, plaintext 

znowu brakuje "

[HTML] pobierz, plaintext 
<input style=margin-left:60px;margin-top:4px; type='submit' value='Wykonaj'>
[HTML] pobierz, plaintext 

style umieśc w "

[PHP] pobierz, plaintext 
	if(empty($_POST['wybor1']) and empty($_POST['wybor']) and empty($wybor1) and empty($_POST['cena'])){echo $kom1; exit();}
[PHP] pobierz, plaintext 

coś mi się tu wydaje, że powinno być OR (||) a nie AND. Gdy user nie uzupełni któregoś z pól raczej powinno się zatrzymać skrypt

[PHP] pobierz, plaintext 
if($wybor1==start)
[PHP] pobierz, plaintext 

Masz stałą start bo nie widziałem nigdzie deklaracji. Umieść też w uszkach, a do tego === zamiast ==

@iqique włacz wkoncu odrobine samodzielnosci :/

Podalem ci co masz robic, podalem ci linka do tematu, ktory to wszystko wyjasnia. Czy naprawda za kazdym razem trzeba ci pisac: a teraz wez sprawdz czy ta zmienna jest ustawiona czy nie?

Mowisz ze nie wykouje ci sie update. On jest w warunku. Zobacz, czy warunek jest spelniony, jesli tak, zobacz jak wyglada wygeneroane zapytanie, zobacz czy zapytanie nie rzuca bledem. Ile razy mozna to samo powtarzarzac ?

nospor
Dalej się nie rozumiemy, wiec wyjaśnię to łopatologicznie:
przykładowa składnia Update:
UPDATE table_name SET column1=value1,...WHERE ....;

column1 - musi być nazwą kolumny z bazy danych, wiec jak w input podam jabłko to takiej kolumny mi nie znajdzie, bo w bazie mam kolumnę towar a w niej rekord o wartości jablko a nie milion kolumn o nazwach towarów, bo jeśli tak ma autor to znaczy ze ma źle zaprojektowaną bazę.. Z czym na pewno się zgodzisz..

dlatego też sie poprawiłem w tamtym poście żeby nie było wątpliwości :

ba, bo jest wręcz niebezpieczne, jeśli użytkownik może dać update kolumny jakiej chce, bo wystarczy, że poda ją w formularzu