[PHP][Geo - #RemoteExecution]

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta
Que grande hielasangre!!
Ya le pasaremos a fermino para implementarlo en underc0de!
Te dejo +1 por el aporte!

Saludos!
ANTRAX

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta
Esta para usarlo

Estaba pensando que en vez de .sql podríamos hacerlo con sqlite asi no crear una db al vicio en el mysql! En estos días veo de hacerlo con sqlite

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta
Esta para usarlo

Estaba pensando que en vez de .sql podríamos hacerlo con sqlite asi no crear una db al vicio en el mysql! En estos días veo de hacerlo con sqlite

Si usa PDO lo haces cambiando una línea nada más (PD: Aún no vi el code)...

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta
Esta para usarlo ;)

Estaba pensando que en vez de .sql podríamos hacerlo con sqlite asi no crear una db al vicio en el mysql! En estos días veo de hacerlo con sqlite ;)

Si usa PDO lo haces cambiando una línea nada más :P (PD: Aún no vi el code)...

Pégale una mirada, y subí las correcciones que creas ;)

<?php
/*
 * DBconfig.php
 * 
 * Copyright 2014  <hielasangre@fuckingmachine>
 * 
 * Clase que contiene la configuracion del sitio.
 * self::$var si es variable 
 * this->metodo
 */

class DBconfig{
	public static $dbDriver= 'mysql';  // se declaran como public static, para no tener que iniciarlo con new
	public static $dbFile = 'sqlite.db'; // Solo si el driver es sqlite
	public static $host = 'localhost';
	public static $user = 'root';
	public static $password= '';
	public static $database= 'geografica';
	
	public static function dsn(){
		if (self::$dbDriver == 'sqlite'){
			return self::$dbDriver . ':' . self::$dbFile;
		}
		return self::$dbDriver . ':host=' . self::$host . ';dbname=' . self::$database;
	}

}

No tienes permitido ver los links. Registrarse o Entrar a mi cuenta
en lo personal tengo mis dudas respecto a un par de cositas, que voy a aclarar:

por el archivo path.php de la carpeta lib:

crear una clase de esa forma me parece algo no solo innecesario sino incoherente, analicemos un poco las funciones y veamos, encontramos funciones como loadLib y loadConfig, las funciones de por si van completamente en contra del segundo criterio de calidad y el cuarto criterio.

sobre el segundo criterio de calidad:

por definicion:
Robustez es la habilidad de una entidad de software de funcionar aún en condiciones anormales (No especificadas).

desde el vamos estás utilizando require once, lo cual da por sentado el hecho de que exista el archivo que estas requiriendo, el resultado de esto es que ante la falta de dicho archivo nos encontraremos con un error fatal de ejecución, lo que deberías hacer es analizar si el archivo existe y de lo contrario decirle al usuario el problema, ya que además si tienes desactivado display_errors (asumiendo que estás en modo de producción) el resultado va a ser pantalla en blanco sin explicaciones y con detención de ejecución. entonces en la situación de que el archivo de configuracion o alguno de los archivos falle tu programa no tiene una salida concreta.

Pero este en realidad es el menor de los problemas, también está el 4º criterio de calidad

por definición:
Reusabilidad es la habilidad de una entidad de software de ser reusado en conjunto o en parte en la construcción de nuevas entidades

ahora, la pregunta es, como puedes reusar una clase que carga archivos específicos internos? y si no querías cumplir con dicho criterio, con qué motivo hiciste una clase? podrían haber sido dos funciones simples sin necesidad de cargar la memoria con una estructura de tal tipo (innecesaria en realidad).

las cosas deben ser usadas cuando valgan la pena, esa clase si bien tiene el proposito de normalizar o estandarizar las cargas, debería requerir tanto el archivo como el directorio de la ubicación de lo que se decea cargar, no cargar deliberadamente archivos en particular.

Entonces en el cambio de contexto donde se decee cargar otro archivo habrá que alterar el código de la clase path, que es en realidad una dependencia del principal, por el contrario un buen código requeriría la alteración unicamente del contexto.

Sobre el archivo GEO.php deberías analizar el uso que le das a los apostrofes de parseo y a los apostrofes de especificación de literales.

No me parece mal que uses apostrofes de parseo, o apostrofes de literales pero usar apostrofes de parseo y luego concatenación es innecesario, y dado que lo que tu escribes si concatenas son literales no es necesario parsear la cadena, dando un trabajo innecesario al interprete de php que desde el vamos que ya tiene sus propios problemas no es necesario darle más carga innecesaria.

Además crear una clase modelo de objetos que pueden cargar archivos, está mal estructurada, ni siquiera debería ser una clase ¿por qué? si quieres que sea una clase en este casi SI debes implementar singleton dado que de lo contrario podrías crear 30 instancias fantasmas (por dar un numero) que intenten cargar archivos, pero como usas require_once no harán absolutamente nada, ¿por qué razón podría yo crear muchas instancias de un cargador de archivos?, y si usas singleton es porque en realidad no necesitas una clase lo haces de puro gusto.

DB.php quizá este sea el asunto principal.

El controlador PDO ya de por si tiene una estructura similar a una clase, de hecho la forma de trabajar es crear una instancia de pdo obteniendo un objeto hecho y derecho, crear una clase interface intermediaria es innecesario por donde se lo mire, a menos que claro está te topes con lo que noté luego, la clase es únicamente para dotar a PDO de Singleton, algo aberrante, lo que estás haciendo es forzar a la clase a solo poder obtener una instancia, y esto ya de por si es un tema no poco menor, una clase que solo se puede instanciar una vez, trate de analizar lo que estoy expresando, es una clase bastante extraña, dado que el objetivo de una clase es representar un modelo de un objeto, que luego al instanciarlo se transforma en el objeto en particular, pero que sentido tiene crear un modelo de algo que solo hay un objeto particular, en otras palabras, se crean modelos de las cosas dada la variedad que existen, pero si solo existe un objeto de ese tipo, crearle una clase es un poco... dada la explicación anterior usar singleton correctamente es bastante dificil ya que solo se puede usar en algunos casos particulares sin concluir en un código ridículo, dotar a una clase que controla bases de datos (y recalco las S) con singleton, es completamente ridículo, ignoro como se te ocurrió tal cosa, pero aunque solo uses una base de datos, dotar de singleton limita al contexto particular en el que se usa dicho código, por lo que el resultado es una violación clara a uno o más de los criterios de calidad.

Ahora bien, no es mi intención tachar a tu código de horrible ni nada por el estilo, yo mismo suelo evitar la implementación de varios criterios dependiendo del caso en el que esté programando, generalmente no aplico criterios para tirar para arriba, dado el gasto de tiempo que conlleva. ¿Entonces por qué hago esta respuesta?, esto no es una queja de tu código, simplemente son explicaciones de cosas que eventualmente se podrían mejorar, y marcar cosas que son bastante ridiculas y que ocacionalmente hasta yo cometo semejantes ridiculeces por el hecho de que aveces uno no se pone a analizar el código. Entonces diré por que respondo todo ésto, mis textos de explicación suelen ser dificiles de entender, y generalmente suele creerse que el código no puede ser analizado bajo los criterios de calidad que explico en mi blog y que hay bastantes textos, esto pasa al principio y se que hay gente en el foro que tiene estas dudas, esto es casi una demostración de que realmente se pueden aplicar los criterios de calidad al código, y una ilustración para aquellos a los que la teoría les complique la vida.

Saludos!

pd: no es mi intencion ofenderte, concidero que tu código hielasangre está relativamente bien, y que es utilizable, y si quiciera implementarlo es muy probable que usara tu código.

pd2: en lo personal no me agrada para nada el uso de php embebido de la forma en que está en index.php pero tampoco es mala, preferiría etiquetas de php lo más cortas posible.

pd3: interesante aporte al código original fermino.