Clase Contador.... No funciona....

[RAAM08]

Hola estoy realizando un contador PHP y MySQL, pero me manda un error que aun no logro solucionarlo. Este el código de mi clase Contador:

class Contador
{
public function visitas()
{
try
{
$conBD = new ConexionBD();
$conBD->setConectar();
$cSQL = $conBD->getSQL("SELECT * FROM visitas");
if(mysql_num_rows($cSQL) == 0)
{
$conBD->setSQL("INSERT INTO visitas (num_visitas) VALUES(1)");
$conBD->setSQL("INSERT INTO ipvisita (ipvisita) VALUES ('".$_SERVER['REMOTE_ADDR']."')");
}
else
{
if($row=$conBD->getSQL($cSQL))
{
$num = $row['num_visitas'];
$cons = $conBD->getSQL("SELECT * FROM ipvisita WHERE ipvisita='".$_SERVER['REMOTE_ADDR']."'");
if($visitadas['ipvisita'] != $_SERVER['REMOTE_ADDR'])
{
$conBD->getDatos("INSERT INTO ipvisita (ipvisita) VALUES ('".$_SERVER['REMOTE_ADDR']."')");
$visitas2 = $row['num_visitas']+1;
$conBD->setSQL("UPDATE visitas SET num_visitas=".$visitas2);
}
return true;
}
else
{
echo "Error";
}


}
echo $visitas2;
}
catch(Exception $ex)
{
throw new Exception($ex->getMessage());
}
}
}

y el error que me arroja es este:
Fatal error: Uncaught exception 'Exception' with message 'No se puede ejecutar la consulta...Resource id #6' in C:\Archivos de programa\Apache Software Foundation\Apache2.2\htdocs\MiWeb\paqL\Contador.ph p:42 Stack trace: #0 C:\Archivos de programa\Apache Software Foundation\Apache2.2\htdocs\MiWeb\index.php(140): Contador->visitas() #1 {main} thrown in C:\Archivos de programa\Apache Software Foundation\Apache2.2\htdocs\MiWeb\paqL\Contador.ph p on line 42

La línea 42 es la del throw....

Necesito su ayuda...!!!!!!!!!!!

[GatorV]

Sin conocer tu clase "conexionBD" es un poco complicado, pero por el error parece ser que el problema esta en la sintaxis de una de tus consultas.

Saludos.

[rudy69]

segun vi, el error esta al final, en esta parte

Código PHP:

  }
            echo $visitas2;
        }
        catch(Exception $ex)
        {
            throw new Exception($ex->getMessage());
        }
    }
} 
   

al final haces otro throw y no lo catch nada, en terminos humanos entendibles xD, cuando haces catch, capturas algo que haz lanzado con throw, y en el catch, haces un throw que no lo captura nada, en ese caso la solucion seria:

Código PHP:

  }
            echo $visitas2;
        }
        catch(Exception $ex)
        {
            echo $ex->getMessage();
        }
    }
} 
   

te aconsejaria que primero leas y entiendas bien el codigo y saber que hace antes de usarlo, esa linea veo un error garrafal que lo ve hasta un humano XD

[RAAM08]

Hola mi clase ConexionBD es esta:
class ConexionBD
{
private $servidor = "servidor";
private $usuario = "user";
private $contrasena = "pass";
private $bd = "bd";
private $linkServer;
private $linkBD;
private $cSQL;

public function setConectar()
{
$this->linkServer = mysql_connect($this->servidor,$this->usuario,$this->contrasena);
if($this->linkServer)
{
$this->linkBD = mysql_select_db($this->bd,$this->linkServer);
if(!$this->linkBD)
{
throw new Exception("No se puede recuperar la BD");
}
}
else
{
throw new Exception("El servidor no se encuentra disponible, sus datos son incorrectos");
}
}

public function setCerrar()
{
mysql_close($this->linkServer);
}

public function setSQL($SQL)
{
try
{
$cSQL = mysql_query($SQL,$this->linkServer);
if(!$cSQL)
{
throw new Exception("No se puede ejecutar la consulta.. ".$SQL);
}
}
catch(Exception $ex)
{
throw new Exception($ex->getMessage());
}
}


public function getSQL($SQL)
{
try
{
$cSQL = mysql_query($SQL,$this->linkServer);
if(!$cSQL)
{
throw new Exception("No se puede ejecutar la consulta...".$SQL);
}
else
{
return $cSQL;
}
}
catch(Exception $ex)
{
throw new Exception($ex->getMessage());
}
}

public function getDatos($cSQL)
{
return mysql_fetch_array($cSQL);
}
}

Lo siento pero no veo el error, desearia que me dijeran dond esta mi errror !!!!!!!!!

[GatorV]

No veo el caso de usar el bloque try/catch si lo vas a capturar y luego volver a lanzar, ese es un error lógico.

Por otro lado veo que el error se produce en getSQL o en setSQL, y viendo tu código el problema esta en esta linea: if($row=$conBD->getSQL($cSQL)) $cSQL tu la defines más arriba como el resultado de SELECT * FROM visitas y luego quieres volver a lanzarla como consulta segun veo tu clase.

Para descargar datos segun tu clase tienes que usar getDatos.

Saludos.

[RAAM08]

Hola pues aún así no he podido hacer que el código funcione he hecho lo siguiente:
class Contador
{
public function visitas()
{
$visitas2 = '';
try
{
$conBD = new ConexionBD();
$conBD->setConectar();
$cSQL = $conBD->getSQL("SELECT * FROM visitas");
if(mysql_num_rows($cSQL) == 0)
{
$conBD->setSQL("INSERT INTO visitas (num_visitas) VALUES(1)");
$conBD->setSQL("INSERT INTO ipvisita (ipvisita) VALUES ('".$_SERVER['REMOTE_ADDR']."')");
}
else
{
$cSQL = "SELECT * FROM visitas";
if($row=$conBD->getDatos($cSQL))
{
$num = $row['num_visitas'];
$cons = $conBD->getDatos("SELECT * FROM ipvisita WHERE ipvisita='".$_SERVER['REMOTE_ADDR']."'");
if($visitadas['ipvisita'] != $_SERVER['REMOTE_ADDR'])
{
$conBD->setSQL("INSERT INTO ipvisita (ipvisita) VALUES ('".$_SERVER['REMOTE_ADDR']."')");
$visitas2 = $row['num_visitas']+1;
$conBD->setSQL("UPDATE visitas SET num_visitas=".$visitas2);
}
return true;
}
else
{
echo "Maricela";
}
}
echo $visitas2;
}
catch(Exception $ex)
{
echo $ex->getMessage();
}
}
}
Si me pudieran decir como hacerle, es que aún no se muy bien acerca de POO....

[GatorV]

Esque no es cuestión de que uses o no POO, el problema es la lógica de tu script, es lo que tienes incorrecto, te recomendaría que primero hicieras el diagrama de lo que quieres realizar por pasos y luego trates de aplicarlo en una lógica POO.

[rudy69]

Necesitas aprender ciertos TIP's que te ayudan a ahorrar codigo, usas mucho try/catch en partes que te podrias ahorrar unas 4 lineas.

Tambian trata de usar las etiquetas del foro para mejor lectura, como ejemplo te resumo una funcion, en como lo podrias hacer y ahorrar lineas.

Código PHP:

  public function setConectar()
{
    $this->linkServer = mysql_connect($this->servidor,$this->usuario,$this->contrasena);
    if($this->linkServer)
    {
        $this->linkBD = mysql_select_db($this->bd,$this->linkServer);
        if(!$this->linkBD)
        {
            throw new Exception("No se puede recuperar la BD");
        }    
    }
    else
    {
        throw new Exception("El servidor no se encuentra disponible, sus datos son incorrectos");    
    }    
} 
   

Código PHP:

  public function setConectar()
{
    $this->linkServer = mysql_connect($this->servidor,$this->usuario,$this->contrasena) or die('no se pudo conectar');
    
    $this->linkBD = mysql_select_db($this->bd,$this->linkServer) or die('no se pudo seleccionar la base de datos bla bla bla');
} 
   

[jam1138]

Iniciado por rudy69

Necesitas aprender ciertos TIP's que te ayudan a ahorrar codigo

La utilidad y legibilidad de dicho código debería prescindir del ahorro de líneas, ¿no crees?.

Tu alternativa de código presentaría un mensaje fofo de error al usuario, como programador ni te llegas a enterar siquiera como para hacer algo. Lanzando una excepción, por el contrario, podrás cacharla, presentarle algo mucho más amigable al usuario y, lo mejor, podrás hacer algo al respecto, como enviarte un correo con un log del error. El código es mucho más útil.

Ahí como tip . Un saludo

[rudy69]

jam1138, que sepa yo, los programadores estudian y desarrollan nuevas tecnicas con un muchos fines, uno de ellos es el ahorro de lineas, y segun tengo entendido los objetos estan para eso, ahorro de lineas.

Le he recomendado eso porque no vi que usara el bloque try/catch mas que para decir 'Failed: bla bla bla', que es normal y la ruta correcta para alguien que empieza, haciendo cosas sencillas.

jam1138 paso de tu comentario

[GatorV]

@rudy69.

Estas equivocado, el fín es la optimización del código, no el ahorro de lineas, la idea es hacer la clase lo más robusta y tolerante a cambios.

El ejemplo que expones si el servidor tiene los errores desactivados solo mostrara una pantalla en blanco, nada amigable para el usuario, en cambio en los bloques try/catch puedes "cachar" ese error y mostrar algo más amigable al usuario.

Saludos.

[jam1138]

Iniciado por rudy69

segun tengo entendido los objetos estan para eso, ahorro de lineas.

... será todo, menos eso . Que pena te lo tomaste tan mal ... programador. Paso igual.

[rudy69]

A ver, no quiero romper el hilo de este post que es para ayudarle a nuestro companyero, pero parece que no me han entendido muy bien, o es que no se en que mundo estoy.

En muchos tutoriales que he leido dice claramente que los objetos son hechos para hacer tareas comunes entre otras cosas mas, no me interesa entrar en su completa deficinion, obviamente eso conyeba un ahorro de lineas, no estoy diciendo que los objetos tienen que ser minimizados, el objeto tiene que tener todo lo necesario (robusto, tolerante a cambios y todo lo que se le ocurra a GatorV) con un codigo lo mas optimizado posible, si estoy mal, pasadme un tutorial que diga algo diferente que me he leido los erroneos, existen frameworks, que ayudan a ahorrar tiempo y por consiguiente, lineas a la hora de programar, no se en que mi definicion falla, obviamente no lo dice todo porque tampoco es un manual, y jam1138, no soy programador, soy solo un aficionado, al llamarme asi ofendes a los programadores que son de verdad, mis conocimientos no pasan de basico.

Y continuando con el post, or die VS try/catch, me parece que nuestro companyero dijo que esta empezando, que try/catch es mejor, lo es, pero para empezar a programar manejando errores se puede comenzar usando or die en mi opinion, queda a opcion de RAAM08 decidir cual usar.

[GatorV]

Mmm solo quisiera saber donde viste que or die() es un buen método para el manejo de errores.

Por otro lado también me gustaría ver un framework que explicitamente diga que una de las ventajas es el ahorro de lineas.

Si eres aficionado creo te gustaría investigar más del tema ya que tienes unos conceptos erroneos de lo que es la Programación orientada a Objetos.

Saludos.

[rudy69]

http://framework.zend.com/manual/en/...ction.overview

Ahi lo dice de manera implicita :D, soy malo explicando cosas, y para entenerme se me hace que tenes que leer una o dos veces.

Cito directamente de la web:

Whatever your application needs are, you're likely to find a Zend Framework component that can be used to dramatically reduce development time with a thoroughly tested foundation.

that can be used to dramatically reduce development time., tiempo = tiempo gastado digitando, con lineas me refiero a muchas cosas, es lo que quiero que entiendan!! xD

Yo personalmente lo he visto, estoy desarrollando una aplicacion, primer objeto que utilice fue para conectar a la bd, me redujo codigo, segundo problema fue, necesitaba imprimir unas 20 tablas, ahi use otro objeto, tercer problema, necesitaba cientos de formularios, otro objeto, lo que antes necesitaba 3 paginas para tratar de tener todo ordenado y separado, usando objetos hoy me cabe en una y hasta mas cosas, teniendo todo siempre lo mas ordenado posible.

[RAAM08]

hola, gracias a todos por sus comentarios pero lo único que necesitaba saber es que es lo que estaba haciendo mal se que no se nada de programación y que todos los que estan en este foro y que me dieron algunos tips saben muchisimo más que yo. Pero me urge corregir mi error!!!!!!!!!!!!!!!!!!

El error no se cuál es. No le entiendo muy bien a la lógica de Programacion y pues necesito hacer que el contador me funcione porq ya he intentado de muchas formas pero no encuentro uno que me funcione correctamente.....

Ayuda!!!!!1

[GatorV]

@rudy69: No se en que mundo lineas = tiempo, pero sí ahi dice reduce development time alias el tiempo de desarrollo, no es implicito como comentas que sea en lineas de código, ya que puedes aún hacer un sistema con 3,000,000 lineas de código pero el tiempo de desarrollo será menor al que si lo hicieras sin Framework.

@RAAM08: Si no sabes de programación lo mejor es que contrates a un programador que desarrolle eso por ti, ya que tu error es logico, no es un error tanto con la sintaxis, tienes que desarrollar tu logica correctamente y usar tu clase de la forma correcta para realizar los inserts y updates.

Viendolo de manera por encima tu clase debe de hacer algo asi:

Código:

Checar filas en la tabla ipvisitas donde sea igual a la IP del visitante
Si es mayor a 0
       Actualizar tabla visitas con un visitas +1
Si es igual a 0
       Insertar en tabla la IP de la visita la IP del visitante
Terminar

Pero tu debes de definir bien que es lo que deseas que haga tu clase.

Saludos.

[RAAM08]

Bueno, pero antes de darme rendida me gustaría saber si aquí en el foro hay un código que ya funcione correctamente con POO y utilizando la clase de ConexionBD que tengo yo. Si me podría decir como encontrarla porq ya visite todos los temas y aún no encuentro una asi como les digo

[GatorV]

Dudo que encuentres una clase que use tu clase, tu debes de programarla, por eso mi insistencia a que hagas el diagrama de flujo de lo que tiene que hacer tu clase para que en base a eso puedas diseñarla.

Saludos.

[rudy69]

RAAM08, concuerdo con GatorV, necesitas recrear todo de nuevo que tiene fallas logicas,

GatorV, concuerdo con todo lo que me has dicho solo en una cosa que dices tu que yo digo, y es en lineas = tiempo, NO! digo, tiempo es directamente proporcional a lo que escribes

tiempo = X * lineas, donde X representa las herramientas que usaras a la hora de escribir tus programas, entre mas efectiva sea X, las variables tiempo - lineas seran menores

PD: Se me hace que GatorV anda pidiendo trabajo xD, con eso de la crisis economica :P

[GatorV]

Lo que pasa es que no puedes medir el tiempo de desarrollo así, es algo muy diferente el tiempo que te tardas en escribir una clase, al tiempo que te tardas a desarrollarla.

Si trabajaras en esto podrías darte cuenta que el tiempo que inviertes no solo es en escribir lineas, también hay que probar, hacer testing, logica, diagramas, etc. No es solo el sentarte y escribir lineas de código y todo va a estar bien, la tarea de desarrollo va mucho más alla, y un framework te ahorra en esa area, en el desarrollo, no en el tiempo lineal.

Saludos.

[rudy69]

Y dale, estoy deacuerdo contigo en eso, pero hablo de la parte en que te tardas en escribirla especificamente.

No puedes sentarte a pensar como desarrollarla si no sabes que herramientas vas a usar, cuando no sabes que lenguaje vas a usar, y peor cuando no conoces bien el lenguaje, lo primero que se puede aprendes es a saber utilizar las herramientas de un lenguaje, y nuestro companyero RAAM08, pienso personalmente que es donde falla, te pones a escribir artas cosas que posiblemente una semana despues te des cuenta de lo que has escrito ha sido algo horrorofico sin ofender, es algo a lo que estoy acostumbrado ver en mis codigos, y tambien pueda que esto tambien esto que dije te paresca mal y le veas la falla, no estoy intentando explicar todo en 1 parrafo, intento ser especifico refiriendome a un tema.

Ahora, no quiero alargar mas esto que me aburri xD, esto no es lo mio, soy solo un aficionado, tu que has estudiado me parece, de esto, has de saber muchisimo mas, lo mio es mas las matematicas, estudie de Ing Civil, y si me hablas de mecanica de solidos/fluidos creo que ahi si te podria ganar xD

[GatorV]

Solo para finalizar, para eso existen los IDEs (Integrated Development Enviroment) la idea de ellos es ayudarte a escribir código de forma más rapida al autocompletar variables, bloques de funciones, o ifs, ciclos o try/catch. Por eso es que no "tardas más en escribir" si lo haces con un IDE (como todos deberían hacerlo) te ahorras mucho más tiempo que en estar escribiendo y luego quitando los die().

Saludos.

[thepancher]

CONSEJO... Metan lo que sea PHP en [ HIGHLIGHT="php" ] [ /HIGHLIGHT ] (Obviemante sin los espacios dentro de los corchetes)

Asi el codigo es mas LEIBLE xD

saludos.