SRocks

Dont use singleton pattern if you want concurrent access (Another reason for not using singleton is it is hard to unit test with this pattern) Use Factory pattern instead. A singleton pattern ensures that you always get back the same instance of whatever type you are retrieving, whereas the factory …

The code is clumsy. Why dont you seperate html and php? why are you taking a "</div>" at line number 49? You are rendering a table by calling the function getContent() from line no 73, right? Its O.K. only. There is no restriction that you should maintain any order for …

Instead of enclosing $fn in '', enclose it in "". insert into ........ ("$fn","$ln",.......)

Update query does not return any database row...So you should have checked for T/F value instead of (mysql_num_rows($result) ==1) //this statement is valid only if your query returning database row..

Try d following snippet... <?php // you have to open the session first session_start(); //remove all the variables in the session session_unset(); // destroy the session session_destroy(); ?> " session_destroy() destroys all of the data associated with the current session. It does not unset any of the global variables associated …