[MySQL][PHP][OOP] Nie dodaje wpisu do bazy Opcje

[Turson]

Analizuję od godziny i nie mogę znaleźć przyczyny, dlaczego wpis nie jest dodawany do bazy, a komunikat się wyświetla.

*Struktura bazy:
wpis: `id` int(11) | `user_id` int(3) | `title` char(50) | `content` text | `data` date
*Połączenie z bazą jest.
*Zmienne post oraz z sesji poprawne
*Błędów żadnych nie wywala

Może ktoś rzuci świeżym okiem i od razu wychwyci

[PHP] pobierz, plaintext 
<h3>Dodaj wpis</h3>
<?PHP
function form(){
	echo'
		<div id="wpis_form">
			<form action="index.php?page=new" method="post">
				<input type="text" name="title" placeholder="Tytuł" required maxlength="30"><br>
				<textarea name="content" placeholder="Treść" maxlength="500" required></textarea><br>
				<input type="submit" name="sent" value="Dodaj wpis">
			</form>
		</div>
	';
}
class dodajWpis extends Connect{
 
	function Length($title,$content){
		if(strlen($title)>=5 && strlen($content)>=5){
			return true;
		}
	}
	function ifNotExists($title){
		$zap=$this->db->prepare("SELECT id FROM wpisy WHERE title=:title");
		$zap->bindValue(':title', $title, PDO::PARAM_STR);
		$zap->execute();
		if($zap->rowCount()==0){
			return true;
		}
	}
	function getId($title){
		$zap=$this->db->prepare("SELECT id FROM wpisy WHERE title=:title");
		$zap->bindValue(':title', $title, PDO::PARAM_STR);
		$zap->execute();
		return $zap->fetch()['id'];
	}
	function dodaj($title,$content,$user_id){
		$query = $this->db->prepare("INSERT INTO wpisy VALUES(null, '$user_id', :title, :content, CURDATE()");
		$query -> bindValue(':title', $title, PDO::PARAM_STR);
		$query -> bindValue(':content', $content, PDO::PARAM_STR);
		$query -> execute();
	}
 
}
$o=new dodajWpis();
if(isset($_SESSION["logged"]))
{
	if(isset($_POST["sent"]))
	{
		$title=trim($_POST['title']);
		$content=trim($_POST['content']);
		if($o->Length($title,$content))
		{
			if($o->ifNotExists($title))
			{
				$user_id=$_SESSION["user_id"];
				$o->dodaj($title,$content,$user_id);
				echo '<a href="index.php?page=wpis&id='.$o->getId($title).'">Wpis został dodany.</a>';
			}
			else{
				echo "Wpis o podanym tytule już istnieje!<br>";
				form();
			}
		}
		else{
			echo "Tytuł i/lub treść są za krótkie!<br>";
			form();
		}
	}
	else{
		form();
	}
}
else{
	header('Location:index.php?page=login');
}
?>
[PHP] pobierz, plaintext 

Ten post edytował Turson 28.10.2013, 18:12:21

[Wazniak96]

Do zapytań INSERT, UPDATE, DELETE używany exec, a nie execute

[Turson]

Tak, ale ja napierw binduję


@down
Lajtowo

Ten post edytował Turson 28.10.2013, 18:29:22

[Wazniak96]

Ahhh wybacz. Kolejny przykład, że jeszcze wiele muszę się nauczyć.

Jeszcze raz sorki

[nospor]

dlaczego wpis nie jest dodawany do bazy, a komunikat się wyświetla.

Dziwne pytanie.... komunikat sie wyswietla, bo go wyswietlasz bezwarunkowo, niezaleznie czy sie dodalo do bazy czy nie.... nie sadziszz, ze komunikat o dodaniu powinienes wyswietlac dopiero jak sie poprawnie doda? Do tego sluzy IF.
A jak sie nie doda poprawnie, to nalezy wyswietlic blad bazy, a wowczas dowiesz sie, czemu sie nie dodalo.... takie tam podstawy podstaw...

[Dejmien_85]

[PHP] pobierz, plaintext 
function dodaj($title,$content,$user_id){
$query = $this->db->prepare("INSERT INTO wpisy VALUES(null, '$user_id', :title, :content, CURDATE()");
[PHP] pobierz, plaintext 

Łot yz dys?

Ten post edytował Dejmien_85 28.10.2013, 18:50:55

[Turson]

Dziwne pytanie.... komunikat sie wyswietla, bo go wyswietlasz bezwarunkowo, niezaleznie czy sie dodalo do bazy czy nie.... nie sadziszz, ze komunikat o dodaniu powinienes wyswietlac dopiero jak sie poprawnie doda? Do tego sluzy IF.
A jak sie nie doda poprawnie, to nalezy wyswietlic blad bazy, a wowczas dowiesz sie, czemu sie nie dodalo.... takie tam podstawy podstaw...

Mówiąc o kumunikacie to miałej raczej na myśli, że błąd leży gdzieś w samym dodawaniu a nie wcześniej skoro przechodzi do dodawania (pozornie).
Ale faktycznie masz rację, więc zmodyfikowałem

[PHP] pobierz, plaintext 
function dodaj($title,$content,$user_id){
$query = $this->db->prepare("INSERT INTO wpisy VALUES(null, '$user_id', :title, :content, CURDATE()");
$query -> bindValue(':title', $title, PDO::PARAM_STR);
$query -> bindValue(':content', $content, PDO::PARAM_STR);
if($query -> execute()){
return true;
}
//dalej
$user_id=$_SESSION["user_id"];
if($o->dodaj($title,$content,$user_id))
{
echo '<a href="index.php?page=wpis&id='.$o->getId($title).'">Wpis został dodany.</a>';
}
else{
echo "Wystąpił nieoczekiwany błąd!";
}
[PHP] pobierz, plaintext 

I rzeczywiście bład...

Łot yz dys?

Możesz rozwinąć myśl? Podobna metoda działa mi prawidłowo w innej klasie.

[nospor]

I rzeczywiście bład...

Napisalem wyraznie: masz wyswielic blad bazy, a nie tekst "blad".... przeciez to ci nic nie mowi.... zajrzyj do manuala i zobacz jak PDO wyswietla bledy bazy.

[Turson]

Syntax error. Nie domknąłem nawiasa w VALUES()

Ten post edytował Turson 28.10.2013, 19:19:45