opinion

RicardGTX · #1 (**permalink**) 07/04/2016, 03:24

Hola chicos,
el siguiente codigo me funciona bien, pero creo que no es lo mas correcto.
El problema que veo es que hago dos consultas iguales para ingresar en dos tablas diferentes.
El primero ($insertar) lo hago para comprobar que no exista, si no existe le guardo.
El segundo ($insertar_2) lo hago para que vuelta a comprobar todos los registros de la tabla dibujos porque anterior he insertado uno nuevo con el primero ($insertar).

No se si es lo mas correcto esto.
Me he podido explicar bien ??

Código PHP:

Ver original$ver = mysql_query("SELECT * FROM artista");
    $ver2 = mysql_num_rows($ver);
    
    if($ver2 == 0){
        
        $dibujo = mysql_query("SELECT * FROM dibujos WHERE dibujo='$id'");
        $ver_dibujo = mysql_num_rows($dibujo);
        
        if($ver_dibujo == 0){
            $insertar = mysql_query("INSERT INTO tags (dibujo) VALUES ('$di')");
        }
        
        $dibujo = mysql_query("SELECT * FROM dibujos WHERE dibujo='$id'");
        $todos = mysql_fetch_array($dibujo);
        $insertar_2 = mysql_query("INSERT INTO arte (id_dibujo) VALUES ('$id')");
    
    }

saludos gracias

Malenko · #2 (**permalink**) 07/04/2016, 03:39

Tengo varias dudas de tu código.

- En la línea 6 y en la 13 haces la misma select contra dibujos y entre esas líneas de código lo único que haces es un insert en la tabla "tags", con lo que el resultado de la select de la línea 13 será el mismo que el de la línea 6.

- Porqué en la línea 4 no pones directamente en el if la comparación contra mysql_num_rows, sin necesidad de usar la variable intermedia?

- Usa variables más descriptivas. Por ejemplo, en lugar de "ver2" se entendería mejor el código si se llamara "numeroArtistas" y lo mismo con "dibujo" que se podría llamar "numeroDibujos".

RicardGTX · #3 (**permalink**) 07/04/2016, 03:45

Si, hago la misma consulta en la 6 y en la 13, porque despues de la consulta 6 inserto un registro, y si no hago la misma consulta en la 13, no recoge ese nuevo registro que inserto.

se entiende ?

saludos
gracias :)

Pondre nombre mas especificos en las variables :)

Malenko · #4 (**permalink**) 07/04/2016, 03:52

Cita:

Iniciado por RicardGTX

Si, hago la misma consulta en la 6 y en la 13, porque despues de la consulta 6 inserto un registro, y si no hago la misma consulta en la 13, no recoge ese nuevo registro que inserto.

se entiende ?

Pues no, no se entiende. Lo intentaré explicar lo más sencillo posible...

1) Haces una select de la tabla DIBUJOS
2) Haces un insert en la tabla TAGS
3) Haces una select de la tabla DIBUJOS

Puedes insertar todos los registros que quieras, pero mientras no sean en la tabla DIBUJOS el resultado del punto 1 y del punto 3 será siempre el mismo.

dashtrash · #5 (**permalink**) 07/04/2016, 03:57

Cita:

Iniciado por RicardGTX

Hola chicos,
el siguiente codigo me funciona bien, pero creo que no es lo mas correcto.
El problema que veo es que hago dos consultas iguales para ingresar en dos tablas diferentes.
El primero ($insertar) lo hago para comprobar que no exista, si no existe le guardo.
El segundo ($insertar_2) lo hago para que vuelta a comprobar todos los registros de la tabla dibujos porque anterior he insertado uno nuevo con el primero ($insertar).

No se si es lo mas correcto esto.
Me he podido explicar bien ??

Código PHP:

Ver original$ver = mysql_query("SELECT * FROM artista");
    $ver2 = mysql_num_rows($ver);
    
    if($ver2 == 0){
        
        $dibujo = mysql_query("SELECT * FROM dibujos WHERE dibujo='$id'");
        $ver_dibujo = mysql_num_rows($dibujo);
        
        if($ver_dibujo == 0){
            $insertar = mysql_query("INSERT INTO tags (dibujo) VALUES ('$di')");
        }
        
        $dibujo = mysql_query("SELECT * FROM dibujos WHERE dibujo='$id'");
        $todos = mysql_fetch_array($dibujo);
        $insertar_2 = mysql_query("INSERT INTO arte (id_dibujo) VALUES ('$id')");
    
    }

saludos gracias

No entiendo casi nada de este código.
- Partiendo de que todo lo que hay en el if, no se ejecutará en cuanto haya 1 artista en la tabla (seguro que esto tiene que funcionar así??)
- Se re-ejecuta una query sin necesidad.No, no hay que volver a hacer la query porque has insertado un registro.Puedes perfectamente hacer el num_rows, luego el fetch_array, y después los 2 inserts.Las condicion por la que ejecutas el insert no cambia si haces el fetch_array.
- Se hace el fetch_array, pero no se usa para nada.

RicardGTX · #6 (**permalink**) 07/04/2016, 04:00

hice unos cambios en el codigo para que se entienda mejor.

Hago dos veces la consulta en la tabla dibujos, porque entre medias de cada consulta si se inserta un nuevo registro en la tabla dibujos.
En el ejemplo anterior lo puse mal. Perdon.

Por eso mi duda de si esta bien o mal asi

Código PHP:

Ver original$artista = mysql_query("SELECT * FROM artista");
        $existe_artista = mysql_num_rows($ver);
        
        if($existe_artista == 0){
            
            $dibujo = mysql_query("SELECT * FROM dibujos WHERE dibujo='$id'");
            $existe_dibujo = mysql_num_rows($dibujo);
            
            if($existe_dibujo == 0){
                $insertar_dibujo = mysql_query("INSERT INTO dibujos (dibujo) VALUES ('$di')");
            }
            
            $tabla_dibujo_artista = mysql_query("SELECT * FROM dibujos WHERE dibujo='$id'");
            $mostrar_dibujo_artista = mysql_fetch_array($tabla_dibujo_artista);
            $insertar_tabla_dibujo_artista = mysql_query("INSERT INTO arte (id_dibujo) VALUES ('$id')");
        
        }

saludos

dashtrash · #7 (**permalink**) 07/04/2016, 04:38

Sigo sin entenderlo, sobre todo, que el if se ejecute sólo si no hay ningun artista en la tabla.
Aunque insertes un nuevo dibujo en la tabla, los datos habrán cambiado, pero el select sigue siendo sobre el dibujo cuyo id es $id.
Hay sólo dos opciones: O un dibujo con el id $id ya existía, por lo que la segunda query te va a devolver el mismo registro que la primera, aunque insertes 1000 dibujos más.
O ese id no existía, y es posible que el nuevo dibujo tenga ese id.Si ese caso es el que se quiere cubrir (que sería bastante raro), los datos que te devolvería la query son los mismos que ya tienes en $di (a menos que ese $di,sea, de nuevo, un error de código, y ahi también quieras decir $id).Y, en ese caso, hacer un segundo select sólo tendría sentido si la tabla posee valores por defecto para otras columnas.
Finalmente, sigues sin usar para nada el resultado $mostrar_dibujo_artista.Muy posiblemente, ese segundo select, lo quieres hacer fuera del if.

Malenko · #8 (**permalink**) 07/04/2016, 04:53

Creo que ayudaría saber que quiere hacer, porque con ese segundo código yo tampoco lo tengo claro. Hace consultas duplicadas, selects de datos que luego no usa....

RicardGTX · #9 (**permalink**) 07/04/2016, 05:40

Me olvido de mi codigo y explico lo que quiero hacer.

Tengo 3 tablas

artistas -> id, nombre
dibujos -> id_dibujo, dibujo
artistas_dibujos -> id_artista, id_dibujo

Lo que quiero hacer es lo siguiente.
Quiero grabar en artistas_dibujos el id del artista y el id del dibujo.

Malenko · #10 (**permalink**) 07/04/2016, 07:18

Cita:

Iniciado por RicardGTX

Me olvido de mi codigo y explico lo que quiero hacer.

Tengo 3 tablas

artistas -> id, nombre
dibujos -> id_dibujo, dibujo
artistas_dibujos -> id_artista, id_dibujo

Lo que quiero hacer es lo siguiente.
Quiero grabar en artistas_dibujos el id del artista y el id del dibujo.

ok, y que datos tienes de entrada?

RicardGTX · #11 (**permalink**) 07/04/2016, 07:49

De entrada tengo el nombre de la tabla artistas y el dibujo de la tabla dibujos.

saludos
gracias

Malenko · #12 (**permalink**) 07/04/2016, 08:05

Entonces es bien sencillo.

1) Select de la tabla de artistas para obtener el id al que pertenece el nombre que tienes
2) Si no te ha devuelto ningún registro...fin
3) Select de la tabla de dibujos para obtener el id al que pertenece el dibujo que tienes
4) Si no te ha devuelto ningún registro...fin
5) Haces un insert en la tabla artistas_dibujo con los dos Id's que tienes

No tiene mucha más historia

xerifandtomas · #13 (**permalink**) 07/04/2016, 08:58

Una prugunta respecto de tu lógica de bd.

Tienes 2 tablas y una 3 para relacionar estas dos, si entiendo bien.

La pregunta es
¿ un dibujo puede pertenecer a uno o a varios artistas ?
Si la respuesta es a varios artistas, nada, la relación (N:N) es correcta y puedes dejar de leer este comentario.

Si la respuesta es que un dibujo solo puede pertenecer a un artista (relacion 1:N) la tabla artistas_dibujos es inecesariamente y el id_artista deberia estar en la tabla dibujos.

RicardGTX · #14 (**permalink**) 07/04/2016, 09:17

Si eso lo tengo claro pero el problema me viene cuando pregunto si existe o no.
Os dejo el siguiente codigo comentado.
Espero que esta vez me haya explicado mejor.

Código PHP:

Ver original/* Compruebo primero que el "nombre_dibujo" no exista en la tabla "categorias", si no existe en la tabla "categorias", entro en el if y compruebo que el "nombre_dibujo" tampoco exista en la tabla "dibujos". 
Si no existe en la tabla "dibujos", lo grabo. 
Pero por otro lado quiero grabar el id del nombre_dibujo y el id del artista (el id del artista no tengo problema, de ejemplo fijo he puesto 95) en una tabla nueva que se llama "dibujos_artistas", se graba siempre excepto que exista en la tabla categorias.*/
 
$id_artista = 95;
 
$categorias = mysql_query("SELECT * FROM categorias WHERE dibujo='$nombre_dibujo'");
$total_categorias = mysql_num_rows($categorias);
 
if($total_categorias == 0){
    
    $comprobar_tabla_dibujos = mysql_query("SELECT * FROM dibujos WHERE dibujo='$nombre_dibujo'");
    $total_tabla_dibujos = mysql_num_rows($comprobar_tabla_dibujos);
    
    if($total_tabla_dibujos == 0){
        $insertar_dibujo = mysql_query("INSERT INTO dibujos (dibujo) VALUES ('$dibujo')");
    }
    
    $comprobar_tabla_dibujos_de_nuevo = mysql_query("SELECT * FROM dibujos WHERE dibujo='$nombre_dibujo'");
    /* duplico esta consulta, porque sino no me coge el id del ultimo dibujo añadido ( no se por que ) */
    $sacar_id_comprobar_tabla_dibujos_de_nuevo = mysql_fetch_assoc($comprobar_tabla_dibujos_de_nuevo);
    /* utilizo assoc para saco la linea para coger el id del dibujo*/
    $tgs_vi =  mysql_query("INSERT INTO dibujos_artistas (id_dibujo, id_artista) VALUES ('$id_artista', '$sacar_id_comprobar_tabla_dibujos_de_nuevo[id_dibujo]')");
}

saludos
gracias

xerifandtomas · #15 (**permalink**) 07/04/2016, 09:34

Para obtener el id del registro insertado puedes utilizar mysqli_insert_id

Código PHP:

Ver originalmysqli_insert_id($link)

RicardGTX · #16 (**permalink**) 07/04/2016, 09:36

esque eso es justo lo que uso para obtener la id del artista pero no se usar dos veces no?

Malenko · #17 (**permalink**) 07/04/2016, 09:37

Cita:

Iniciado por RicardGTX

Si eso lo tengo claro pero el problema me viene cuando pregunto si existe o no.
Os dejo el siguiente codigo comentado.
Espero que esta vez me haya explicado mejor.

Si existe el que? El autor? El dibujo? La relación autor-dibujo?

RicardGTX · #18 (**permalink**) 07/04/2016, 09:39

el nombre del dibujo que es porque lo que pregunto en los if

saludos

xerifandtomas · #19 (**permalink**) 07/04/2016, 09:42

Puedes utilizarlo cuantas veces necesites, eso sí, para obtener un mismo id lo normal es almacenarlo en una variable

Código PHP:

Ver original$id_my_insert=mysql_insert_id();

Teniendo en cuenta esto que nos dice el manual:

Cita:

Nota:
Como mysql_insert_id() actúa en la última consulta realizada, asegúrese de llamar a mysql_insert_id() inmediatamente después de la consulta que genera el valor.