Sagi

Call it without arguments: mysql_affected_rows(). The optional parameter is connection resource and not recordset resource.

You can send the form in post method to the same page, and display it like this: <?php echo $make; ?> And not: <?$make?> Just make sure that when you display it, you clean the string from XSS. You can use htmlentities. For example: $make = htmlentities($_POST['make'], ENT_QUOTES, 'UTF-8');

You use :: to access static methods, and -> to access instance methods. If you have DB class, you can have DB object reference as a property in user class: public $db; You instantiate the DB object in the user class constructor: $this->db = new DB(); But, you don't want to have multiple connections to the datebase, so you can use design pattern called singleton. In short, you have only one DB object and you don't have to pass it from one method to another. There are other options, it is just one of them for DB object. Read more about singleton design pattern, and patterns in general. If you have any questions you can ask. Also check PDO, it's probably better than building your own DB class: http://il.php.net/manual/en/book.pdo.php

Use file_exists to check if the file exists, then include the file only if it does. if (file_exists("file.php")) { require_once "file.php"; } else { echo "File not found"; }

* Change $this->$title_prefix to $this->title_prefix, you don't have variable $title_prefix. * You didn't declare property "table_prefix" and "title_suffix" in class base or super classes (db, config).