dziś zajrzałem do działu programowanie obiektowe i na początek stworzyłem taką prostą klasę

[PHP] pobierz, plaintext 
<?php
class Clanteam {
 
	function dodaj($nazwa, $szef) {
		$result=mysql_query("insert into turniej ( nazwa, szef ) VALUES ('$nazwa', '$szef')");
	}
}
 
if($_GET['akcja']=='dodaj'){
$dodaj = new Clanteam;
$dodaj->dodaj($_POST['nazwa'], $_POST['szef']);
}
else{
echo'<form action="index.php?site=tur&akcja=dodaj" method="post">
<input type="text" name="nazwa">
<input type="text" name="szef">
<input type="submit">
</form>';
}
?>
[PHP] pobierz, plaintext 

Proszę sprawdzić czy jest wszystko w porządku, chce sprawdzić czy dobrze rozumuje.

wygląda ok

od siebie dodałbym :

$result=mysql_query("insert into turniej ( nazwa, szef ) VALUES ('.addslashes($nazwa).', '.addslashes($szef).')");

Na chwilę obecną ten kod nie ma wiele wspólnego z OOP poza słowem kluczowym "class".

Pomijając już, że jest to kod niepoprawny (brak modyfikatora dostępu - tutaj: public) i niebezpieczny (brak jakiegokolwiek filtrowania danych) to nie da się stworzyć kodu OOP składającego się z jednego obiektu z jedną metodą...

tak wiem, że nie zabezpieczony kod, dopiero wchodzę w programowanie obiektowe no i chce się na bierząco sprawdzać co robię źle