Ver Mensaje Individual
  #4 (permalink)  
Antiguo 15/02/2011, 01:05
Avatar de proxipor
proxipor
 
Fecha de Ingreso: diciembre-2009
Mensajes: 211
Antigüedad: 15 años
Puntos: 10
De acuerdo Respuesta: Clase ManiDB fácil manipulación de datos para novatos!

Cita:
Iniciado por Triby Ver Mensaje
Interesante, pero... cuando ejecutas la consulta, obtienes los resultados en una matriz (array), que despues conviertes en cadena, separandolos por comas y, finalmente, usas la funcion separa para volver a recuperar la matriz original, pero... que pasa si en alguno de los valores existe una coma?

empresa, direccion, telefono
Google Inc.,Somewhere in USA,1-800-google
Coma, S.A.,Calle y numero,#### -> te va a generar un error

Sugerencias:
- Utiliza consulta para devolver un recurso MySQL
- La funcion recupera puede ser util para leer todos los registros encontrados y sin necesidad sin "dar margen de error" al usuario

Disculpame, pero en realidad no le veo mucho sentido a esta clase, necesitas trabajarla un poco mas y ponerla a prueba para encontar los puntos fuertes y tambien los posibles fallos para ir mejorando.
Tienes mucha razon Triby!

No habia pensado eso, tratare de usar tus sugerencias y modificar eso de las comas (:

Saludos y Gracias!

Cita:
Iniciado por jonasanx Ver Mensaje
Opino igual que Triby, esta clase esta aun muy joven, tiene bastantes deficiencias.

1. ¿Que pasa cuando ya no necesito el enlace a la base de datos? ¿y si necesito conectarme a otra base de datos?
2. Falta el sanitizador de sentencias. La clase es susceptible a ataques de inyección.
3. Necesita una manejador de errores externo.
4. Se pueden crear múltiples instancias de la clase, necesitas agregar el patrón singleton.
... y otras de las cuales todavia no me doy cuenta.

Pero échale ganas, siempre se empieza por algo.
Yo nunca dije que fuera una maravilla, simplemente queria que la comunidad la adaptara a sus proyectos optimizandola ellos y compartiendo sus logros, aqui en el foro.

Si hay algun programador que tenga algo de tiempo libre y le pueda hacer algunas mejoras se lo agaradeceria.

Saludos!