PHP-hispano.net

Esta bien el script, pero si la conexion a la base de datos esta hecha, lo volvería a hacer? :S
Creo que tendrias que comprobar si se ha abierto una conexion ya

Veamos...

COMENTARIO: Cuando veas (uno|dos) es que no me apetece escribir y resumo asi, eso dignificaria que hay dos cosas, una llamada uno y otro llamada dos.

Haces cosas raras en Obt(POST|GET)(1|2):
1) if ($_(POST|GET)) <- Ke pretendes hacer con esto?
2) Usa explode en vez de split :S
3) $pro[$v] = ke? de donde sale $por? no la veo definida en ningun lado

No se si te enseñaron que cosas como conexion a las bases de datos y demas son cosas MUY costosas, tu abres una conexion en cada funcion!! y para que pones esto:

return @$result;
mysql_free_result($result);

La arroba es estupida ahi, ademas el metodo acaba con el return, lo de despues de los returns no se ejecuta :S.

while (list($llave, $valor) = each($contenido))

while (list($llave, $valor) = each($contenido)) <- te presento a mi amigo, el bucle foreach.

No me apetece seguir mirando, no esta muy bien programado y sumamente ineficiente :S

Uy, el voto...

Si bueno, lo interesante es el concepto. La programación es mejorable.
Se podría comprobar si la conexión a sido abierta.

Y contestando los comentarios de Airis:

con:

if ($_POST) {
}

Compruebo que exista algún valor enviado por POST.

Con lo de foreach, los dos bucles son igual de buenos.

Y respecto a $pro, ¿desde cuando es necesario declarar previamente las variables?

Un salido.

Un apunte. En lo del return tienes toda la razón, se me fue la pinza.

ponte un error_reporting(E_ALL); y veras como no es tan majo lo de no hacer $pro = array(); primero, es necesario declarar los arrays para poder meter elementos ... Y la manera correcta de comprobar si hay algo en un array es con la funcion empty(), no como lo haces.

No soy el rey de la ortografia, pero creo que es Prohibido (en el inicio del archivo).

Con los comentarios estoy de acuerdo, esos if en get y post deberias especificar que quieres controlar, si tienen elementos, o que, utilizando funciones... mira count, is_array, empty, etc, etc.

Por otro lado, mas que un conjunto de funciones, personalmente, lo haria en un conjunto de clases... (de conexion, parametros, etc).

Saludos!

Ya he dicho que la programación era mejorable...

Pero para mi lo que cuenta es el concepto. El concepto es el concepto. Una pizza sin buena pasta no vale nada.

Soys más quisikillosas que las garrapatas... que si código por aki... que si ortografía por aya..

Me estuve planteando hacerlo con clases. pero la verdad...Al menos en mi caso... Todo lo que considero que forma parte del kernel de un website no me gusta hacerlo con clases. Son manias, lose...Para todo lo demás, mastercard.

Un saludo,.

considerate afotunado, no viste los fallos k le sacaron a mi script que era cortito xD, creo que se batieron records en lineas escritas en comentarios

yo creo k cuanto mas perfecto es mejor :P

Cuando esta mal la base no merece la pena ponerse a analizar los errores del codigo...

estaba perfecto, solo me modificasteis tonterias en plan comillas y cosas asi, xk el script funcoino desde el principio

Vale. Vale...
Aki pongo una versión más eficiente....

¿Mejor ahora?

Hola, veo que ahora compruebas si la conexión está o no abierta, y haces caso de la mayoría de cosas que te ha dicho Arias.. aunque ¿Por dónde está documentado?.

Un saludo

Esta version no esta documentada, pero se basa en los mismos nombres de funciones que hay en la versión anterior.

Si leeis el articulo: http://php.blogcindario.com/ficheros/php/database.html

Podeis intuir mas o menos que hace cada función.

Es muy simple. Pero no tengo tiempo de documentarlo.

Un saludo.

Usas Objetos y tienes esto, no te parece que deberias tener algo como esto otro?

Me equivoque en algo, seria mas bien como este codigo.

Edit: Nota que reemplaze el $_POST que me habia quedado en el primer codigo que corregi por $superGlobal.

Yo trabajé mucho tiempo con cosas como ésta. Sirven para aprender. A veces conviene que la función te devuelva un string con sólo el SQL... de esa forma te independizas de la conexión y conseguis un código mas flexible

No se parece a éste concepto? Evitemos crear la rueda dos veces
http://phplens.com/lens/adodb/docs-adodb.htm#autoexecute
http://phplens.com/lens/adodb/docs-active-record.htm

Hola Mijt,

Pues tienes toda la razón del mundo. Ni habia pensado en ello. Lo mejoraré en las proximas versiones.

Gracias por tu anotación

Ho0ola aston,

Si, la verdad es que es muy similar. No lo conocia, pero me parece que hacen cosas similares.
Con un pequeño matiz. Esta clase también te genera el array con los datos. Para eso esta OBTPost que retorna el array para poder utilizarlo al hacer el insert.

Seguramente el que tu dices es más eficiente. No te lo discuto.

Un saludo.

como la mayoria comenta, es mejor creo usar una clase para la conexion, haci se hace mas liviano tu codigo =), he chekeado tu codigo y ta bien , aunke le faltan algunas cosillas :=), pero vale!!!

Esta bastante bien

«Podeis intuir mas o menos que hace cada función.» En un primer vistazo no, la verdad. Y los nombres de las variables podrian ser un poco más explicativos ($ret, $c, $k, $v...) y luego $tabla, $contenido, $datos... (¿por qué no $t, $d, $gjisqw? ya puestos)
De todos modos a mí la idea me gusta, y ha sido en mi opinion demasiado criticado. Está documentado lo que siempre es un punto a favor.
Por cierto estoy completamente deacuerdo con "Airis" en el uso de split: con un explode es suficiente, y si vas a usar split, al menos usa preg_split. split es la menos eficiente de las tres.

Aparte de las cosas mencionadas, le falta seguridad en lo referente al soporte $_POST y $_GET (en la documentación se menciona cómo solucionarlo, pero debería estar incluído en la clase)

¿Y los ejemplos?

A ver si se aprueba para que sigas mejorandolo....

cuando repites algo muchas veces, vease "if (!$this->abierto) $this->OpenMySQL()" lo mejor es hacer una funcion que contenga eso mismo en invocarla. Yo la llamaria "ensureOpen" o algo asi... Mirate la funcion mysql_real_escape_string, las magic quotes y algo de orientación a objetos, que el ese diseño de clases no es muy afortunado ... Por cierto mi tabla "Cliente" tiene como clave primaria "DNI", no "id_cliente", es una pena que la clase no funcione 8-) ...

if (condicion) { codigo, mas codigo } else { codigo, otro codigo } => codigo; if (condicion) { mas codigo } else { otro codigo}

foreach ($c as $k => $v) => si no vas a usar $k pues simplemente => foreach ($c as $v)

Otro dia sigo mirando....

Y como nota para todos: leed algo de refactoring, os cambiará la manera de programar...

La funcion de INSERT, debería ser mas flexible, tu antepones de que yo, tengo un campo con ID (AUTO INCREMENT), pero supongamos de que no... que solo utilizo para Relacionar...

Saludos.

Muchas sugerencias y pocas actualizaciones en mucho tiempo...

Desde el 2006!
El archivo mas viejo...
Esperemos que voten y se elimine...
Porque parece que ya no hay mas actualizaciones sobre este archivo y aunque yo habia votado a favor...cambio mi voto porque me parece que ya no va a ser actualizado mas el archivo.