PHP OOP+MySQLi?

Du mangler $ på linje 16.

du har glemt et $ ja
der ud over er jeg ikke sikker på du kan kalde din klasse list, da der er en php-function som hedder det i forvejen

du har glemt dit $db


$id = $db->mysqli_real_escape_string($id);

Nu kommer jeg med et fuldkommen radikalt forslag. Noget som aldrig er prøvet før...

Er i klar?

Her kommer det...

Har i prøvet at kigge i dokumentationen: http://php.net/manual/en/book.mysqli.php

Jeg vil i princippet mene det er bedre at OOP funktion sørger for, at gøre variablen valid eller fejler, hvis den ikke er det.

Gør du det ekternt ender du jo med i teorien tusinde steder, hvor der står

$val = clean($val);
$val2 = clean($val2);
$object->method($val, $val2);

i stedet for bare:
$object->method($val, $val2);

Og samtlige steder, hvor du glemmer at clean variablerne har du SQL injection risiko. Der er sikkert et fornuftigt modargument mod disse påstande.

Det jeg sagde om at rykke SQL'en ind i metoden gik dog ikke på selve variablen men på hele sql'en:
print $db::getuserinfo("SELECT * FROM users WHERE id='". mysql_real_escape_string($id) ."'");

Hvor din løsning godt nok tillader dig at skrive variable queries, kræver det samtidig også en viden over alt om database tabellerne. Og hvis du nogensinde retter bare lidt i den kan du have mange fejlsteder

linje 29 blev :: erstattet med ->
linje 16 fik et $ foran result

Jeg får så denne fejl, og nu er jeg virkelig på herrens mark..

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: Access denied for user 'specieslist'@'localhost' (using password: NO) in /home/public_html/test/class_getuserinfo.php on line 29

Warning: mysql_real_escape_string() [function.mysql-real-escape-string]: A link to the server could not be established in /home/public_html/test/class_getuserinfo.php on line 29

Fatal error: Call to undefined method mysqli_result::mysql_fetch_array() in /home/public_html/test/class_getuserinfo.php on line 16

ordet list som class, bliver dog ikke brugt, den hedder noget helt 2, har dog ændret det for ikke at røbe for meget at sidens info

ser ud til du bruger: mysql_real_escape_string og ikke mysqli_real_escape_string
og det samme med din fetch.

Aha, jamen, så blev jeg det klogere

Så mysqli_fetch_array og mysqli_real_escape_string ?

nu har jeg ik brugt mysqli så meget, men vil jeg mene ja.

Hejsa.

Nå, nu smider jeg koden her ud engang mere, med et nyt problem....

 <?php
 
 //Inkluder Databasen klassen.
 require_once( "databasen.php" );
 
  class specieslist {
 
   private $connection;
 
   public function getuserinfo($query)
   {
   //Get the MySQL connection.
   $this->connection = Database::getConnection();
   $result = $this->connection->query( $query );
 
   while ($row = $result->mysqli_fetch_array())
   {
    print "Your ID is: ". $row['id'] ."<br />";
   }
 
   //close the connection again!
   Database::closeConnection($this->connection);
   }
  }
 
  $id = strip_tags($_GET['id']);
 
  $db = new database();
  $id = getConnection->mysqli_real_escape_string($id);
 
 
  $splist = new specieslist();
  print $splist->getuserinfo("SELECT * FROM users WHERE id='". $id ."'");
 
 ?>

Fejlen lyder sådan her:
Parse error: syntax error, unexpected T_OBJECT_OPERATOR in /home/specieslist/public_html/test/class_getuserinfo.php on line 29

Uforventet T_OBJECT_OPERATOR?