excellent class, it was just what i was looking for.

Recommend this page to a friend!

      Reduced MySQLi  >  All threads  >  excellent class, it was just what i...  >  (Un) Subscribe thread alerts  
Subject:excellent class, it was just what i...
Summary:Package rating comment
Author:Camilo Sperberg
Date:2009-06-09 21:42:55


Camilo Sperberg rated this package as follows:

Utility: Good
Consistency: Good
Examples: Good

  1. excellent class, it was just what i...   Reply   Report abuse  
Picture of Camilo Sperberg Camilo Sperberg - 2009-06-09 21:42:55
excellent class, it was just what i was looking for. A class that returns all the data within an array, and logs some usage such as number of queries and time taken to do the requests. I could notice only some "bad" things:
1- if you want to select a complete table ("SELECT * FROM table"), without args, it would throw an exception, So, to resolve that quickly and _very_ dirty, I would make $bind = TRUE only if no args are passed.
2- microtime() just gives the current second data. It means that if your query takes more than a second, it will give you an inaccurate time. Solved by making $time = time()+microsecond(true);
3- $types isn't defined at a beginning: it would raise a NOTICE_ERROR if you have the show_startup_errors (or something like that) == TRUE in your php.ini.
So, the resulting code in:
private function execute_query($arg_array = NULL)

will be:
private function execute_query($arg_array = NULL) {
//determine the types of arguments
$sql_query=array_shift($arg_array); // the first element is returned to sql_query and then removed
$types = '';
foreach ($arg_array as $v) {
switch ($v) {
case is_string($v):
case is_int($v):
case is_double($v):
try {
if (isset($arg_array[0])) {
array_unshift($arg_array, $types);
else $bind = TRUE;
if ($bind) {
$time=(time()+microtime(true)) - $time_start;

$this->log[]=array("query" => $sql_query,"time" => $time.' seconds');

Greetings and thanks for such a great class!


For more information send a message to info at phpclasses dot org.