Forum PHP.pl > $_POST w OOP PDO

Pomoc - Szukaj - Użytkownicy - Kalendarz

Forum PHP.pl > Forum > PHP > Object-oriented programming

Sakil

2.08.2012, 15:49:04

Witam.
Robię prostą listę 'todo'.
Tak wygląda html z formularzem:

[HTML] pobierz, plaintext 
<!DOCTYPE html>
<html>
	<head>
		<link rel="stylesheet" type="text/css" href="index.css" />
		<meta charset="UTF-8" />
		<title> Todo list </title>
	</head>
	<body>
		<div id="wrapper">
			<h1> Todo list </h1>
			<form action="dodaj.php" method="POST">
				<input type="text" name="todo" value="Co masz zrobić?" />
				<input type="submit" value="Dodaj" />
			</form>
		</div>
	</body>
</html>
[HTML] pobierz, plaintext

Dodaj.php:

[PHP] pobierz, plaintext 
<?php
error_reporting(E_ALL);
class DB_CONN
{
	public $todo = $_POST['todo'];
	public $DBH;
	public $STH;
 
 
	function __construct($path_to_db)
	{
		try
		{
			$this->DBH = new PDO("sqlite:".$path_to_db);
			echo "Succesfully connected<br />";
		}
		catch(PDOException $e)
		{
			echo $e->getMessage();
		}
	}
 
	function __destruct()
	{
		$this->DBH = null;
	}
 
	function add_todo()
	{
		try
		{
			$this->STH = $this->DBH->prepare("insert into todo_list (todo) values(':todo')");
			//var_dump($this->DBH->prepare("insert into todo_list (todo) values(:todo)"));
			$this->STH->bindParam(':todo', $this->todo);
			$this->STH->execute();
		}
		catch(PDOException $e)
		{
			echo $e->getMessage();
		}
	}
}
 
$driver = new DB_CONN("D:/Programy/moja_db.db");
$driver->add_todo('asd');
print_r($_POST);
?>
[PHP] pobierz, plaintext

PHP wywala errora: Parse error: syntax error, unexpected '$_POST' (T_VARIABLE) on line 5
Linijka 5:

[PHP] pobierz, plaintext 
public $todo = $_POST['todo'];
[PHP] pobierz, plaintext

Szukałem jakiś błędów w kodzie, ale jedynym średnikiem jaki mógłbym pominąć był ten przy error_reporting. Jeśli zmienię $todo na cokolwiek innego poza $_POST lub $_GET to wszystko działa. Poza tym jeśli wywalę tą linijkę poza klasę też wszystko działa.
Pracuję na PHP 5.4.4 na localhoscie.

markonix

2.08.2012, 17:09:49

To niedozwolona (na szczęście) konstrukcja.
Toż to straszna głupota gdyby takie coś dało się robić.

Dane do klasy przekazujesz z zewnątrz jako parametry metod.

Sakil

2.08.2012, 17:29:26

Jakie niebezpieczeństwo się z tym wiążę? Mógłbyś podać jakiś realny przykład, bo po prostu nie mogę sobie tego wyobraźić

markonix

2.08.2012, 17:37:19

Nie napisałem o żadnym niebezpieczeństwie tylko zupełnie bezsensownej składni.
U góry ustala się wyłącznie atrybuty klasy, domyślne parametry można ustalić w kostruktorze, a z zewnątrz idą przez parametry metod.
Tutaj będzie to logicznym aby dodać to do add_todo.

To jest wersja lo-fi głównej zawartości. Aby zobaczyć pełną wersję z większą zawartością, obrazkami i formatowaniem proszę kliknij tutaj.