Cześć !

Piszę sobie prosty skrypt tabeli piłkarskiej i mam problem z aktualizowaniem wyników do bazy. Skrypt składa się z dwóch plików

Pierwszy odpowiada za wyświetlanie informacji z bazy w polach formularzy.

[PHP] pobierz, plaintext 
<form action="anglia.php" method="post">
<table>
 
<?php
 
include('config.php');
 
 
   try
   {
      $pdo = new PDO('mysql:host='.$host.';dbname='.$dbname.'', "$user", "$pass", array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
      $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
//Drużyna ID=1      
      $stmt = $pdo -> query('SELECT * FROM anglia WHERE ID=1');
      foreach($stmt as $row)    
      echo '<tr><td><input type="text" name="miejsce1" value="'.$row['miejsce'].'" style="width: 40px;" /> <td><input type="text" name="druzyna1" value="'.$row['druzyna'].'" /></td> <td><input type="text" name="mecze1" value="'.$row['mecze'].'" style="width: 40px;" /></td> <td><input type="text" name="punkty1" value="'.$row['punkty'].'" style="width: 40px;" /></td> <td><input type="text" name="bramki1" value="'.$row['bramki'].'" style="width: 40px;" /></td></tr><br />';                  
 
 
      
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
   }   
 
?>
 
</table><br />
<input type="submit" value="Edytuj" />
</form>
[PHP] pobierz, plaintext 

Drugi z plików odpowiada za dopisywanie nowych wyników do bazy. Z nim mam właśnie problem, ponieważ nie dopisuje tylko wywala następujący błąd:

Połączenie nie mogło zostać utworzone: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INTO `anglia` (`miejsce`, `druzyna`, `mecze`, `punkty`, `bramki`)VALUES( ''' at line 1

Kod tego pliku wygląda następująco:

[PHP] pobierz, plaintext 
<?php
 
//Zapisywanie do bazy
include('config.php');
 
 
   try
   {
      $pdo = new PDO('mysql:host='.$host.';dbname='.$dbname.'', "$user", "$pass", array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
      $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
{
$_POST['miejsce1'] = $row['miejsce1'];
$_POST['druzyna1'] = $row['druzyna1'];
$_POST['mecze1'] = $row['mecze1'];
$_POST['punkty1'] = $row['punkty1'];
$_POST['bramki1'] = $row['bramki1'];
 
 
$pdo -> exec('UPDATE INTO `anglia` (`miejsce`, `druzyna`, `mecze`, `punkty`, `bramki`)VALUES(
                ''.$_POST['miejsce1'].'',
                ''.$_POST['druzyna1'].'',
                ''.$_POST['mecze1'].'',
                ''.$_POST['punkty1'].'',
                ''.$_POST['bramki1'].'')'); 
    
}
      
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
   }  
?>
[PHP] pobierz, plaintext 

Z góry dzięki za pomoc

Pozdrawiam
Marcin

Witaj,

Masz błąd w składni SQL

Tak przy okazji, to powiedz mi, czemu służy zapis:

[PHP] pobierz, plaintext 
<?php
$_POST['miejsce1'] = $row['miejsce1'];
$_POST['druzyna1'] = $row['druzyna1'];
$_POST['mecze1'] = $row['mecze1'];
$_POST['punkty1'] = $row['punkty1'];
$_POST['bramki1'] = $row['bramki1'];
?>
[PHP] pobierz, plaintext 

Bo trochę nie rozumiem

Pozdrawiam

Masz racje to jest źle, chciałem żeby wartości z value, dodawane były do bazy.

Wtedy powinno wyglądać to tak:

[PHP] pobierz, plaintext 
<?php
$_POST['miejsce1'] = $row['miejsce'];
$_POST['druzyna1'] = $row['druzyna'];
$_POST['mecze1'] = $row['mecze'];
$_POST['punkty1'] = $row['punkty'];
$_POST['bramki1'] = $row['bramki'];
?>
[PHP] pobierz, plaintext 

Ale niestety w tym przypadku również pojawiają się błędy. Poza tym nie wiem jak rozwiązać dodawanie wielu różnych drużyn, ponieważ przy wyświetlaniu nie ma problemu bo można dać WHERE ID=liczba to właśnie przy aktualizowaniu pojawią mi się schody.

Możecie coś poradzić, aby wykonać to jak najprościej

sprawdź kod formularza, którym dodajesz dane ... czy gdzieś nie ma " i ' ...

DREEMus możesz napisać jaśniej o co chodzi ? Mam tylko te dwa pliki i bazę danych, nic poza tym.

a spróbuj tak ...

[PHP] pobierz, plaintext 
<?php
$pdo -> exec('UPDATE INTO `anglia` (`miejsce`, `druzyna`, `mecze`, `punkty`, `bramki`) VALUES ("'.$_POST['miejsce1'].'", "'.$_POST['druzyna1'].'", "'.$_POST['mecze1'].'", "'.$_POST['punkty1'].'", "'.$_POST['bramki1'].'")');
?>
[PHP] pobierz, plaintext 

Dalej mam błąd:

Połączenie nie mogło zostać utworzone: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near 'INTO `anglia` (`miejsce`, `druzyna`, `mecze`, `punkty`, `bramki`) VALUES ("", ""' at line 1

Może ktoś coś poradzić ?

moze zadam glupie pytanie, ale co to jest "update into" ?

Daj może print_r($_POST); bo wygląda jakby nie odbierał danych z formularza.

Dlatego, że już kombinowałem na wszystkie sposoby dopisując cokolwiek



Dałem to na początku pliku, który ma dodawać, ale teraz pojawia mi się błąd:

Array ( [miejsce1] => 4 [druzyna1] => Manchester City [mecze1] => 10 [punkty1] => 3 [bramki1] => 28-1 ) Połączenie nie mogło zostać utworzone: SQLSTATE[42000]: Syntax error or access violation: 1064 You have an error in your SQL syntax; check the manual that corresponds to your MySQL server version for the right syntax to use near '(`miejsce`, `druzyna`, `mecze`, `punkty`, `bramki`)VALUES( '', ' at line 1

mistrzu..... no ale nie ma czegos takiego jak update into...
jest insert into i tak ma byc.

A jak ci nie dzialalo insert into to robisz echo mysql_error(); i wiesz gdzie masz blad

Już wiem, że nie ma INSERT INTO działa, tylko problem w tym, że dodaje mi nowe rekordy a nie aktualizuje tego, który chce zmienić. Ja chce tylko je zaktualizować.

kurcze....to zajrzyj do manuala i zobacz jak wyglada skladnia UPDATE bo teraz jestes daleki od poprawnosci....

No ok, ale dlaczego przy INSERT INTO wstawia mi puste komórki a nie pobiera tych z formularza ?

no bo jesli robisz to tak:

[PHP] pobierz, plaintext 
<?php
$_POST['miejsce1'] = $row['miejsce1'];
$_POST['druzyna1'] = $row['druzyna1'];
$_POST['mecze1'] = $row['mecze1'];
$_POST['punkty1'] = $row['punkty1'];
$_POST['bramki1'] = $row['bramki1'];
?>
[PHP] pobierz, plaintext 

To nic dziwnego ze ci z forma nie pobiera.....

A jesli robisz inaczej to pokaz jak robisz

Tamto zmieniłem, bo błąd ten zauważyłem zaraz po tym jak napisałem wątek:

Teraz zmieniłem składnie UPDATE, ale wyskakuje mi błąd:

Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING in /home/buncol/ftpl/tabele/anglia.php on line 20

Plik dodawania wygląda następująco:

[PHP] pobierz, plaintext 
<?php
 
//Zapisywanie do bazy
include('config.php');
 
 
   try
   {
      $pdo = new PDO('mysql:host='.$host.';dbname='.$dbname.'', "$user", "$pass", array(PDO::MYSQL_ATTR_INIT_COMMAND => "SET NAMES utf8"));
      $pdo -> setAttribute(PDO::ATTR_ERRMODE, PDO::ERRMODE_EXCEPTION);
 
{
$_POST['miejsce1'] = $row['miejsce'];
$_POST['druzyna1'] = $row['druzyna'];
$_POST['mecze1'] = $row['mecze'];
$_POST['punkty1'] = $row['punkty'];
$_POST['bramki1'] = $row['bramki'];
 
 
$pdo -> exec("UPDATE `anglia` SET `miejsce` = '.$_POST['miejsce1'].',`druzyna` = '.$_POST['miejsce1'].',`mecze` = '.$_POST['mecze'].',`punkty` = '.$_POST['punkty1'].',`bramki` = 'bramki1' WHERE `id`='1'");
 
}
      
   }
   catch(PDOException $e)
   {
      echo 'Połączenie nie mogło zostać utworzone: ' . $e->getMessage();
   }  
?>
[PHP] pobierz, plaintext 

o zgrozo....
no nie widzisz ze mieszasz juz kody?

[PHP] pobierz, plaintext 
<?php
$pdo -> exec("UPDATE `anglia` SET `miejsce` = '$_POST[miejsce1]',`druzyna` = '$_POST[miejsce1]',`mecze` = '$_POST[mecze]',`punkty` = '$_POST[punkty1]',`bramki` = 'bramki1' WHERE `id`='1'");
?>
[PHP] pobierz, plaintext 

no i znowu masz:
$_POST['miejsce1'] = $row['miejsce'];
$_POST['druzyna1'] = $row['druzyna'];
$_POST['mecze1'] = $row['mecze'];
$_POST['punkty1'] = $row['punkty'];
$_POST['bramki1'] = $row['bramki'];

przeciez dane masz miec z forma a nie z jakiego $row.... no mysl troche chlopie

nospor ale ja to powinno wyglądać ? Ja dopiero się uczę a teraz to już mam zupełny mętlik w głowie.

no skoro dane idą ci z forma (z posta) to na grzyba dajesz to:
$_POST['miejsce1'] = $row['miejsce'];
$_POST['druzyna1'] = $row['druzyna'];
$_POST['mecze1'] = $row['mecze'];
$_POST['punkty1'] = $row['punkty'];
$_POST['bramki1'] = $row['bramki'];
?

$_POST jest automatycznie wypelnianiane danym z formularza, jesli takowy byl wyslany

Usunąłem to, ale nadal mam tutaj problem:

[PHP] pobierz, plaintext 
<?php
 
$pdo -> exec("UPDATE `anglia` SET `miejsce` = '.$_POST['miejsce'].',`druzyna` = '.$_POST['druzyna'].',`mecze` = '.$_POST['mecze'].',`punkty` = '.$_POST['punkty'].',`bramki` = '.$_POST['bramki'].' WHERE `id`='1'");
?>
[PHP] pobierz, plaintext 

Pojawia się błąd: Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING

kurde.... zaraz zamkne ci temat!
Przeciez dwa moje posty wyzej podalem ci poprawną wersje na ten kod php. nie marnuj naszego czasu!

[PHP] pobierz, plaintext 
<?php
$pdo -> exec("UPDATE `anglia` SET `miejsce` = '$_POST[miejsce1]',`druzyna` = '$_POST[miejsce1]',`mecze` = '$_POST[mecze]',`punkty` = '$_POST[punkty1]',`bramki` = '$_POST[bramki1]' WHERE `id`='1'");
?>
[PHP] pobierz, plaintext 

ps: ja juz na dzisiaj mam dosc tego tematu. bye, bye

nospor nie rozumie dlaczego się tak denerwujesz Coś co dla Ciebie jest oczywiste dla początkującego wcale nie musi. Zwróć uwagę, że to dział Przedszkole


Poza tym i tak mam błąd składni: Parse error: syntax error, unexpected T_ENCAPSED_AND_WHITESPACE, expecting T_STRING or T_VARIABLE or T_NUM_STRING

W tej linijce:

[PHP] pobierz, plaintext 
<?php
 
$pdo -> exec("UPDATE `anglia` SET `miejsce` = '.$_POST['miejsce'].',`druzyna` = '.$_POST['druzyna'].',`mecze` = '.$_POST['mecze'].',`punkty` = '.$_POST['punkty'].',`bramki` = '.$_POST['bramki'].' WHERE `id`='1'");
 
?>
[PHP] pobierz, plaintext 

jestes niesamowity. znowu uzyles swojego starego kodu zamiast mojego, ktory juz poraz drugi ci podalem

temat zamykam bo ewidentnie lejesz na nas i nasz czas mając w 4 literach to co ci podajemy
w razie pytan i watpliwosci zapraszam na PW, bo moze nadal nie rozumiesz oco mi chodzi

edit: po rozmowie na PW i zrozumieniu bledu - otwieram

Dzięki nospor

Uwagi sobie wezmę do serca

A jeszcze mam jedno pytanie.

W jaki sposób zabezpieczyć ten 2 plik odpowiedzialny za dodawanie, ponieważ jeśli go uruchomię z przeglądarki to zeruje mi wszystkie wyniki :/

Czy nie trzeba będzie połączyć ich w jeden ?

A gdzie Ty widzisz problem
PHP daje Ci wiele możliwości wykonania Twojego pytania

Ja bym zrobił formularz jako index.php
dodawania nowych jako dodaj.php
i pobieranie oraz pokazanie jako pokaz.php

wtedy się nie pogubisz z tym wszystkim