[MySQL][PHP] Nie dodaje i nie edytuje rekordow Opcje

[adek-]

Witam skrypt nie dodaje, nie edytuje i nie kasuje rekordów. Po tym jak dodam do bazy ręcznie dane przez phpmyadmin to mi je wyświetla, ale nie idzie nic z automatu... Jeszcze pewne rzeczy nie są dociągnięte typu upload pliku, ale chciałem już dodawać chociaż dane.
Proszę o pomoc w usunięciu problemu z dodawaniem.
PS drażni mnie trochę to rozbijanie na query i query. i te && $_POST, jest jakiś ładniejszy sposób dodawania rekordów, a zarazem bezpieczny?

Dokładniej chodzi mi o ten początek

[PHP] pobierz, plaintext 
if ($_POST['co'] == 'dodaj') {
if ($_POST['kraj'] && $_POST['wartosc'] && $_POST['rok_obrotu'] && $_POST['rok_obrotu2'] && $_POST['rodz_metalu'] && $_POST['srednica'] && $_POST['grubosc'] && $_POST['masa'] && $_POST['zdjecie_awers']) {
$query = "INSERT INTO monety (kraj, wartosc, rok_obrotu, rok_obrotu2, rodz_metalu, srednica, grubosc, masa, zdjecie_awers)";
$query .="VALUES ('" . $_POST['kraj'] . "', '" . $_POST['wartosc'] . "', '" . $_POST['rok_obrotu'] . "', '" . $_POST['rok_obrotu2'] . "' ";
$query .=", '" . $_POST['rodz_metalu'] . "', '" . $_POST['srednica'] . "', '" . $_POST['grubosc'] . "', '" . $_POST['masa'] . "', '" . $_POST['zdjecie_awers'] . "' );";
$wynik = mysql_query ($query);
}
 
print '<form METHOD="POST">Nowy rekord:';
print '<input type="hidden" name="co" VALUE="dodaj">';
print '<table><tr><td>Kraj:</td><td><input type="text" name="kraj"></td></tr>';
print '<tr><td>Wartosc:</td><td><input type="text" name="wartosc">zl</td></tr>';
print '<tr><td>Rok obrotu:</td><td><input type="text" maxlength="4" size="4" name="rok_obrotu">-<input type="text" maxlength="4" size="4" name="rok_obrotu2">lata</td></tr>';
print '<tr><td>Rodzaj metalu:</td><td><input type="text" maxlength="3" size="3"  name="rodz_metalu">(próba 500-999)</td></tr>';
print '<tr><td>Srednica:</td><td><input type="text" name="srednica">mm (np: 32.00 mm)</td></tr>';
print '<tr><td>Grubosc:</td><td><input type="text" name="grubosc">mm (np: 1.85 mm)</td></tr>';
print '<tr><td>Masa:</td><td><input type="text" name="masa">g (np: 15.55 g)</td></tr>';	
print '<tr><td>Zdjecie:</td><td><input type="text" name="zdjecie_awers"></td></tr>';
print '</table><input type="submit" VALUE="dodaj"></form>';
[PHP] pobierz, plaintext 

PS czy jest jakaś różnica w dodawaniu jeśli w bazie jest jeszcze jedno pole zdjecie_rewers, a nie jest tutaj uwzględnine? Czy aby dodało muszą być wszystkie pola jakie sa w bazie?


PONIZEJ CALY SKRYPT:

[PHP] pobierz, plaintext 
<?
mysql_connect ("localhost", "root", "xxx") or 
die ("nie mozna polaczyc sie z mysql");
mysql_select_db ("xxx") or 
die ("nie mozna polaczyc sie z mysql");
 
if ($_POST['co'] == 'dodaj') {
    if ($_POST['kraj'] && $_POST['wartosc'] && $_POST['rok_obrotu'] && $_POST['rok_obrotu2'] && $_POST['rodz_metalu'] && $_POST['srednica'] && $_POST['grubosc'] && $_POST['masa'] && $_POST['zdjecie_awers']) {
        $query = "INSERT INTO monety (kraj, wartosc, rok_obrotu, rok_obrotu2, rodz_metalu, srednica, grubosc, masa, zdjecie_awers)";
        $query .="VALUES ('" . $_POST['kraj'] . "', '" . $_POST['wartosc'] . "', '" . $_POST['rok_obrotu'] . "', '" . $_POST['rok_obrotu2'] . "' ";
		$query .=", '" . $_POST['rodz_metalu'] . "', '" . $_POST['srednica'] . "', '" . $_POST['grubosc'] . "', '" . $_POST['masa'] . "', '" . $_POST['zdjecie_awers'] . "' );";
        $wynik = mysql_query ($query);
    }
} elseif ($_POST['co'] == 'popraw') {
    if ($_POST['kraj'] && $_POST['wartosc'] && $_POST['rok_obrotu'] && $_POST['rok_obrotu2'] && $_POST['rodz_metalu'] && $_POST['srednica'] && $_POST['grubosc'] && $_POST['masa'] && $_POST['zdjecie_awers']) {
        $query = "UPDATE monety SET kraj='" .$_POST['kraj']. "', wartosc =";
        $query.= "'".$_POST['wartosc']."', rok_obrotu='".$_POST['rok_obrotu']."' WHERE nr='". $_POST['id']."';";
        $wynik = mysql_query ($query);
		//niedokończone poprawianie bo w zasadzie na razie nie zamierzam poprawiac zanim czegos tam nie wrzuce! :)
}
} elseif ($_GET['co']=='edytuj'){
    $query = "SELECT* FROM monety WHERE nr='".$_GET['id']."';";
    $wynik= mysql_query ($query);
    $rekord = mysql_fetch_assoc ($wynik);
    $nr = $rekord['nr']; $kraj=$rekord['kraj']; $wartosc=$rekord['wartosc']; $rok_obrotu=$rekord['rok_obrotu']; $rok_obrotu2=$rekord['rok_obrotu2']; $rodz_metalu=$rekord['rodz_metalu']; $srednica=$rekord['srednica']; $grubosc=$rekord['grubosc']; $masa=$rekord['masa']; $zdjecie_awers=$rekord['zdjecie_awers'];
    print '<form METHOD="POST">poprawa rekordu:';
    print '<input type="hidden" name="co" VALUE="popraw">';
    print '<input type="hidden" name="id" VALUE="'.$nr.'">';
    print '<table>';
    print '<tr><td>kraj:</td><td><input type="text" name="kraj" VALUE="'.$kraj.'"></td></tr>';
    print '<tr><td>wartosc:</td><td><input type="text" name="wartosc" VALUE="'.$wartosc.'"></td></tr>';
    print '<tr><td>rok obrotu:</td><td><input type="text" name="rok_obrotu" VALUE="'.$rok_obrotu.'">-<input type="text" name="telefon" VALUE="'.$rok_obrotu2.'"></td></tr>';
    print '<tr><td>rodz_metalu:</td><td><input type="text" name="rok_obrotu" VALUE="'.$rodz_metalu.'"></td></tr>';
    print '<tr><td>srednica:</td><td><input type="text" name="srednica" VALUE="'.$srednica.'"></td></tr>';	
    print '<tr><td>grubosc:</td><td><input type="text" name="grubosc" VALUE="'.$grubosc.'"></td></tr>';
    print '<tr><td>masa:</td><td><input type="text" name="masa" VALUE="'.$masa.'"></td></tr>';	
    print '<tr><td>zdjecie_awers:</td><td><input type="text" name="zdjecie_awers" VALUE="'.$zdjecie_awers.'"></td></tr>';		
    print '</table><input type="submit" VALUE="popraw"></form>';
 
} elseif ($_GET['co']=='skasuj') {
    $wynik = mysql_query
    ("DELETE FROM monety WHERE  nr= '".$_GET['id']."';");
}
 
print '<form METHOD="POST">Nowy rekord:';
print '<input type="hidden" name="co" VALUE="dodaj">';
print '<table><tr><td>Kraj:</td><td><input type="text" name="kraj"></td></tr>';
print '<tr><td>Wartosc:</td><td><input type="text" name="wartosc">zl</td></tr>';
print '<tr><td>Rok obrotu:</td><td><input type="text" maxlength="4" size="4" name="rok_obrotu">-<input type="text" maxlength="4" size="4" name="rok_obrotu2">lata</td></tr>';
print '<tr><td>Rodzaj metalu:</td><td><input type="text" maxlength="3" size="3"  name="rodz_metalu">(próba 500-999)</td></tr>';
print '<tr><td>Srednica:</td><td><input type="text" name="srednica">mm (np: 32.00 mm)</td></tr>';
print '<tr><td>Grubosc:</td><td><input type="text" name="grubosc">mm (np: 1.85 mm)</td></tr>';
print '<tr><td>Masa:</td><td><input type="text" name="masa">g (np: 15.55 g)</td></tr>';	
print '<tr><td>Zdjecie:</td><td><input type="text" name="zdjecie"></td></tr>';
print '</table><input type="submit" VALUE="dodaj"></form>';
 
$wynik = mysql_query ("SELECT * FROM monety;") or 
die ("blad w pytaniu");
print "<table cellpadding=5 border=1>";
print "<tr><td><b>kraj</b></td><td>wartosc</td><td><b>rok obrotu</b></td><td>rok obrotu2</td><td><b>rodz_metalu</b></td>";
print "<td>srednica</td><td><b>grubosc</b></td><td>masa</td><td><b>zdjecie_rewers</b></td></tr>\n";
while ($rekord = mysql_fetch_assoc ($wynik)){
    $nr=$rekord['nr'];
    $kraj =$rekord['kraj'];
    $wartosc= $rekord['wartosc'];
    $rok_obrotu= $rekord['rok_obrotu'];
    $rok_obrotu2= $rekord['rok_obrotu2'];
    $rodz_metalu= $rekord['rodz_metalu'];
    $srednica= $rekord['srednica'];
    $grubosc= $rekord['grubosc'];
    $masa= $rekord['masa'];		
    $zdjecie_rewers= $rekord['zdjecie_rewers'];					
    print "<tr><td>$kraj</td><td>$wartosc</td><td>$rok_obrotu</td><td>$rok_obrotu2</td><td>$rodz_metalu</td><td>$srednica</td><td>$grubosc</td><td>$masa</td><td>$zdjecie_rewers</td><td>";
    print "<a href=\"baza.php?co=skasuj&id=$nr\">skasuj</a></td><td>\n";
    print "<a href=\"baza.php?co=edytuj&id=$nr\">edytuj</a></td></tr>\n";
 
}
print "</table>";
 
?> 
[PHP] pobierz, plaintext 

Ten post edytował adek- 24.05.2011, 16:28:23

[wookieb]

Temat: Jak poprawnie zadac pytanie

[impro]

czy użytkownik ma dostateczne uprawienia do wykonywania takich poleceń
zajad mu najprostsze zapytanie jakie są do edycji.
prawdopodobnie literówka
Jeśli polecenie select działa to być może tkwi błąd w formularzu zobacz co zwraca użyj echo.

[PHP] pobierz, plaintext 
 
$query = "INSERT INTO monety (kraj, wartosc, rok_obrotu, rok_obrotu2, rodz_metalu, srednica, grubosc, masa, zdjecie_awers)";
//wcześniej z 
$kraj = $_POST['kraj'];
 $query = "INSERT INTO monety values ($kraj, $wartosc, $rok_obrotu, $rok_obrotu2, $rodz_metalu, $srednica, $grubosc, $masa, $zdjecie_awers)";
//lub chyba
 $query = "INSERT INTO monety values ($_POST['kraj'], $_POST['wartosc'])";
 
[PHP] pobierz, plaintext 

Ten post edytował impro 24.05.2011, 16:05:58

[toaspzoo]

w zapytaniach nie możesz używać tablic ($_POST['tablica'])

[impro]

też mi się tak wydaje poza tym i tak trzeba będzie odfltrować znaki specjalne.
Moja propozycja jest taka napisać najprostszy kod dodający do bazy danych i zobaczyć czy działa pierw nawet niech nie pobiera z formularza i szukać przyczyny.
Później dodać formularz jeśli psuje nam to działanie wina formularza i szukać przyczyny.

[adek-]

tak literówka była przy dodawaniu zdjęcia, zamiast zdjecie_rewers było zdjecie (IMG:style_emoticons/default/smile.gif)
teraz próbuję zrobić tak jak mówiliście, nie wiem czy dobrze zrozumiałem ale chodzi o to aby skrypt wyglądał tak?

zamiast tak

[PHP] pobierz, plaintext 
if ($_POST['co'] == 'dodaj') {
if ($_POST['kraj'] && $_POST['wartosc'] && $_POST['rok_obrotu'] && $_POST['rok_obrotu2'] && $_POST['rodz_metalu'] && $_POST['srednica'] && $_POST['grubosc'] && $_POST['masa'] && $_POST['zdjecie_awers']) {
$query = "INSERT INTO monety (kraj, wartosc, rok_obrotu, rok_obrotu2, rodz_metalu, srednica, grubosc, masa, zdjecie_awers)";
$query .="VALUES ('" . $_POST['kraj'] . "', '" . $_POST['wartosc'] . "', '" . $_POST['rok_obrotu'] . "', '" . $_POST['rok_obrotu2'] . "' ";
$query .=", '" . $_POST['rodz_metalu'] . "', '" . $_POST['srednica'] . "', '" . $_POST['grubosc'] . "', '" . $_POST['masa'] . "', '" . $_POST['zdjecie_awers'] . "' );";
$wynik = mysql_query ($query);
}
[PHP] pobierz, plaintext 

wyglądał tak:

[PHP] pobierz, plaintext 
$kraj=$_POST['kraj'];
$wartosc=$_POST['wartosc'];
$rok_obrotu=$_POST['rok_obrotu'];
$rok_obrotu2=$_POST['rok_obrotu2'];
$srednica=$_POST['srednica'];
$grubosc=$_POST['grubosc'];
$masa=$_POST['masa'];
$zdjecie_awers=$_POST['zdjecie_awers'];
 
if ($_POST['co'] == 'dodaj') {
if ($_POST['kraj'] && $_POST['wartosc'] && $_POST['rok_obrotu'] && $_POST['rok_obrotu2'] && $_POST['rodz_metalu'] && $_POST['srednica'] && $_POST['grubosc'] && $_POST['masa'] && $_POST['zdjecie_awers']) {
$query = "INSERT INTO monety (kraj, wartosc, rok_obrotu, rok_obrotu2, rodz_metalu, srednica, grubosc, masa, zdjecie_awers)";
$query .="VALUES ('$kraj','$wartosc','$rok_obrotu','$rok_obrotu2','$srednica','$grubosc','$masa','$zdjecie_awers' );";
$wynik = mysql_query ($query);
}
[PHP] pobierz, plaintext 

nie, to powyżej mi nie działa, chyba że, trzeba owe zmienne wkleić w inne miejsce np po $wynik = mysql_query ($query);, czy to nie ma różnicy?

Ten post edytował adek- 24.05.2011, 17:08:39

[impro]

[PHP] pobierz, plaintext 
$query = "INSERT INTO monety (kraj, wartosc, rok_obrotu, rok_obrotu2, rodz_metalu, srednica, grubosc, masa, zdjecie_awers)";
$query .="VALUES ('$kraj','$wartosc','$rok_obrotu','$rok_obrotu2','$srednica','$grubosc','$masa','$zdjecie_awers' );";
 
//tak
$kraj = 'Polska';
$query = "INSERT INTO monety VALUES ('".$kraj."')";
/sprawdź czy coś dodaje
 
[PHP] pobierz, plaintext 

[IceManSpy]

Jasne, że może używać tablic w zapytaniach np:

[PHP] pobierz, plaintext 
$sql = "select * from tabelka where id={$_POST['id']}"
[PHP] pobierz, plaintext 

Tylko jest to bardzo niebezpieczne.

Ten post edytował IceManSpy 24.05.2011, 17:18:12