Hola, voy a hacerte muchas correcciones y mi respuesta no es ofenderte sino ayudarte a elaborar mejor código, fácil de mantener y algunas cosas que no deberían hacerse, aunque no se cual es tu caso pero si es para sacar estadisticas de una BD de datos gigantes lo siguiente no aplica.

Primero en la base de datos, veo campos que debería estar en otras tablas para mayor facilidad de reemplazar datos, ocupar menos espacio, por ejemplo los campos: estado, municipio, parroquia deberían ser tablas aparte para ahorrar espacio en la BD, permitir en caso de que quieran reemplazar contenido cambiar un campo y listo (se puede hacer con sentencias update para la situación actual pero hay riesgo de equivocarse y modificar cosas que no deberían ser), si tienes problemas para diseñar las tablas puedes usar DBDesigner que permite crear bd a través de modelos.

Me parece que tienes muchos campos que se pueden separar en tablas, permitiendo simplificar las setencias SQL y permitir fácil comprensión (al contrario de como está ahorita).

Por ahí vi un campo de fecha como VARCHAR, para eso existe el campo DATE , al usar DATE puedes realizar operaciones de comparación de fecha, tal vez el cliente te pida este tipo de cosas en un futuro.

**Del lado de PHP:

Veo que al principio haces un:
$consulta="select * from propietario where ced_cat='$cod'";
pero solo utilizas el primer campo (con el [0] que asumo que es el id) si es así en el SELECT solo deberías llamar al campo id para accelerar la operación de la bd (se que tal vez el sistema sea pequeño y no influye pero es recomendable para futuros proyectos o si la BD crece)


La sentencia SQL deberías construirla con la funcion sprintf (busca en google o la documentacion oficial de PHP) para evitar ataques SQL INJECTION en tu aplicación, ademas puedes de esa manera enviar mejor los datos a través de variables, si no lo haces de esta manera, no utilices "'$variable'" para agregar la informacion de la variable al string, utiliza la concatenación "'".$variable."'" ó "'{$variable}'" , de esta manera aseguras la correcta concatenación de los datos.

Esto no afecta pero es recomendable usar las palabras reservadas por Posgres en mayúscula para leer las setencias con mayor facilidad.

Hace unos meses hice una modificacion al phppgadmin para generar aplicaciones PHP teniendo el diseño de la base de datos hecha, puedes chequearlo en http://sourceforge.net/projects/ppa-appgen/ actualmente por falta de tiempo no doy soporte pero si estoy en el canal #phppgadmin de freenode puedes escribirme. Esta aplicacion puede servirte para generar una miniaplicación y leer las sentencias PHP/Postgres para las operaciónes básicas.

--

Miguel Ángel Useche Castro
Visita http://www.skatox.com/ : Blog de Informática, Linux, Playstation 3, 
Windows, Software Libre, Videojuegos, Java y mucho mas....

_______________________________________________
Lista de Correo l-talug
Grupo de Usuarios Linux de Tchira - Venezuela
Para enviar un Correo a la lista: [email protected]
Para suscribirse o desuscribirse: 
http://listas.velug.org.ve/mailman/listinfo/l-talug
Visitanos en el Canal IRC #talug en irc.freenode.net

Responder a