Blank

Blank

Joined
Jul 07, 2007

Activity Stream

Blank commented on a Page, Another ?page= (id) Script  -  Sep 03, 2007

I\'d just be worried about throwing up errors if i ran that. It only checks if the get field is set once. The way to do this without incurring errors would be to have all the checks for the pages inside the \"if (isset($_GET[\'page\'])){\". I\'d also suggest using switch instead of \"if\" statements for the page numbers.

 Respond  
Blank commented on a Page, Simple Seen System  -  Sep 03, 2007

just a suggestion, but i added a timer that saved the hash db every 30 seconds, in case my computer crashed or lost power.

 Respond  
Blank commented on a Page, Includes using switch statment  -  Jul 07, 2007

nice and simple. ;)
it took me a while to get into switch, only got into it because my \"if\" statements were extremely long and messy.

 Respond  
Blank commented on a Page, database driven page system  -  Jul 07, 2007

just a suggestion, but you should be using mysql_real_escape_string on any user based inputs (or at the very least, an \'if(!is_numeric($editid)) die(\"Invalid input\");\'), because they could easily compromise your system by SQL injection.

 Respond  
Blank commented on a Page, PHP gunzip, unzip File Extractor  -  Jul 07, 2007

Is this script a *nix only script?

 Respond  
Blank commented on a Page, BBCODE  -  Jul 07, 2007

just a suggestion, but to decrease the size of the script, consider sticking the search and replacements into arrays. also, just using str_replace (or as Hawkee suggested, str_ireplace), means that you aren\'t actually validating the input, meaning that the user could possibly break up the format of the output page by adding too many closing or opening tags. A preg_replace would be better, that way if there are any mismatches, they aren\'t replaced at all.

 Respond  
Are you sure you want to unfollow this person?
Are you sure you want to delete this?
Click "Unsubscribe" to stop receiving notices pertaining to this post.
Click "Subscribe" to resume notices pertaining to this post.