Mensajes

Hola de nuevo releyendo me he encontrado con esto: El campo BANCO es un string que representa el nombre del BANCO. (los datos concretos de pago como numero de cuenta se encuentran alojados en otra tabla a la cual se accede a través del número de control)
Lo que me raya la cabeza es el contenido del paréntesis, que significa, por más que intento comprender no entiendo lo que se pide.

Lo que te está pidiendo de forma indirecta es que tengas otra tabla con los datos concretos del pago, llamémosla InfoPago. Esta tabla tendrá como te dicen pues el número de cuenta y podría tener otras cosas como el titular o lo que consideres/te indiquen.
Y que la relación entre InfoPago - Pago será 1:N. Cada pago apunta a un único registro de InfoPago mientras que cada registro de InfoPago puede ser referido desde diferentes pagos.

El uso de esto es para no copiar todos los datos en cada pago. Guardas en una tabla el número de cuenta, titular,... y le das un número de control. Así cuando quieras decir que un pago fue realizado por un número de cuenta, en vez de copiar todo (número de cuenta, titular,...) solo tendrás que copiar el número de control y eso te permitirá saber siempre qué número de cuenta, titular,... hizo ese pago.

Por si acaso diré que numeroControl tiene toda la pinta de ser la PK de InfoPago y una FK de Pago hacia InfoPago.

Visto que todas mis recomendaciones anteriores fueron ignoradas, me limitaré a decirte el procedimiento que tienes que seguir:

Código [Seleccionar] Expandir


PEDIR telefono
indice := 0
MIENTRAS indice < contactosTotales && telefono != contactos[indice].telefono HACER
  indice := indice + 1
FIN MIENTRAS
// Si indice = contactosTotales, el numero buscado no existe
SI telefono = contactos[indice].telefono ENTONCES
  PEDIR nuevoTelefono
  contactos[indice].telefono = nuevoTelefono
FIN SI
Ahora tienes que adaptar este psedocódigo a tu problema mirando a ver cómo puedes representar las variables que aparecen.
Suerte.

No sería necesario guardarlo al revés creo yo, solo necesito lo que puse en la primera entrada(no estoy seguro xdd)

Pues entonces con tener la relación hecha en el otro sentido ya vale. Además la importante es la que te permite borrar el pago automáticamente cuando borras el pedido relacionado. Así que ya sabes: esa no hace falta y la otra la estabas poniendo al revés. Ahora te toca hacerlo en MySQL.

Las claves foráneas quedarían así Id_vendedor en pedido, id_pedido en cliente, id_pago en pedido?
No tengo muy claro como crear la estructura con estas claves

Mira a ver si tal y como tú dices se cumplen las normas que te puse antes sobre cómo establecer las relaciones.
Ya te adelanto que la segunda no lo cumple.
Y la tercera ya te lo puse yo con la clave de id_pedido en Pago, no sé si de verdad necesitas guardarlo al revés también.

Otras cosa que veo es en la línea 241, con ese n-1 estás dejando la última posición sin "limpiar".
Seguramente tengas algún problema en la llamada recursiva de tipus3d() (línea 407).

Lo mejor que puedes hacer es usar algún depurador o hacerlo a la vieja ausanza con unos printf() por ahí para ver cómo van cambiando las variables.

Lo que veo a simple vista es que el array vmin tiene una longitud de n que no sé cuánto será puesto que depende del fichero pero en la función tipus3d() le asignas 100 veces el valor 0 a vmin[100].

Mira a ver si corrigiendo eso, el error se arregla. No he mirado mucho más puesto que el código es muy grande y hay que decir que la identación no ayuda mucho a identificar los bloques de código.

Suerte.

PD: Para pasar un puntero a una función, en vez de usar &(*p), puedes usar directamente p. Los operadores & y * se anulan al final.

Entre las demás tablas y relaciones, habría que crearlas normal o como has hecho tú?

No sé a qué te refieres con crearlas normal o como las he hecho yo...

Si te refieres a lo de crear una nueva tabla para la relación, eso es necesario únicamente para las relaciones con cardinalidad N:M. Digamos que la regla general es:

• Cardinalidad N:M -> Nueva tabla formada por las PKs de las dos tablas [+ discriminantes] como PK de la nueva, otros atributos de la relación y FKs de cada PK original a su tabla de origen.
  
• Cardinalidad 1:N -> La tabla con cardinalidad N contiene una FK a la PK de la tabla con cardinalidad 1.
  
• Cardinalidad 1:1 -> Una tabla contiene la FK a la PK de la otra. Cuál? Depende del problema muchas veces. La que más información aporte.
  
  Si te refieres al ON DELETE CASCADE, pues dependerá de lo que quieras eliminar en cascada o no.

Así es, una relación de cardinalidad N:M constituye una nueva entidad (tabla) que estará formada por las claves primarias de ambas tablas originales (que formarán conjuntamente la clave primaria de la nueva tabla) y los atributos de relación que pudiese haber (pudiendo ser alguno de ellos un discrimante y entonces habría que incluirlo en la clave primaria de la nueva tabla).

Código (sql) [Seleccionar] Expandir


CREATE TABLE [if not exists] Pedido (
  id_pedido INT PRIMARY KEY [auto_increment],
  ...
);

CREATE TABLE [if not exists] Producto (
  id_producto INT PRIMARY KEY [auto_increment],
  ...
);

-- Tabla resultante de la relacion:
CREATE TABLE [if not exists] Inventario (
  id_pedido INT,
  id_producto INT,
  cantidad INT [default 1], -- Cantidad del producto id_producto que hay en el pedido id_pedido, [por defecto: 1]
  PRIMARY KEY(id_pedido, id_producto),
  [CONSTRAINT FK_Inventario_Pedido] FOREIGN KEY (id_pedido) REFERENCES Pedido(id_pedido),
  [CONSTRAINT FK_Inventario_Producto] FOREIGN KEY (id_producto) REFERENCES Producto(id_producto)
);
Por ejemplo a esta relación le vendría bastante bien el atributo de relación <cantidad> para saber cuántos productos de cada tipo hay en cada pedido.
Las partes entre corchetes [] son opcionales y si se ponen tienes que quitar los corchetes.


El DELETE ON CASCADE tienes que incluirlo en la FOREIGN KEY de la tabla que quieres que se elimine en cascada, es decir, de la tabla opuesta a la que tú vas a borrar. En este caso si quieres que al eliminar un Pedido, se elimine un Pago, Pedido tiene que ser la tabla independiente (que no contiene la FK) y Pago será la dependiente.

Código (sql) [Seleccionar] Expandir


CREATE TABLE [if not exists] Pago (
  id_pago int primary key [auto_increment],
  ...
  id_pedido int,
  [constraint FK_Pago_Pedido] foreign key (id_pedido) references Pedido(id_pedido) ON DELETE CASCADE
);

Yo diría que me siento más identificado con la programación de bajo nivel o nivel medio. Empecé con una mezcla de C y C++ hasta que aprendí a diferenciarlos (algo que parece muy tonto pero mucha gente los usa indiferentemente) y aunque he probado algún lenguaje más, estos siguen ahí presentes.
Me gusta saber cómo funcionan las cosas y los lenguajes de menor nivel de abstracción te permiten entender mejor el funcionamiento directo con memoria. Hay que decir que ahora tengo que trabajar con Java y echo de menos mis queridos punteros (aunque mucha gente los odia) y otras tantas cosas que se podían hacer en C/C++ y que Java no permite.

Además pasar de bajo nivel a alto nivel es más sencillo que a la inversa. Alguien que sabe trabajar con memoria puede olvidarse de ella y dejarse llevar pero alguien que no tiene ni idea de cómo funciona la memoria tiene más problemas para empezar a trabajar con ella.

Lo cierto es que de la enorme lista de lenguajes que hay en la encuesta, aunque he oído sobre muchos de ellos, he probado muy poquitos. Me da pereza eso de cambiar de lenguaje cada poco tiempo (y cada mucho también ). Coger un lenguaje, aprender la sintaxis básica y cuando empiezas a hacer proyectos complejos, coges y empiezas a probar otro... Se vuelve un poco repetitivo, a mi parecer. Te limitas a hacer siempre unas entradas/salidas de datos estándar y unos algoritmos básicos.

Tengo pensado probar algunos más pero seguramente en los que más me especialice sean C++, Java y algún otro que vaya descubriendo del estilo.

PD: Por si no se ha notado por la ausencia de mención: no me llama la atención la programación Web. Respeto a los que les guste obviamente y seguro que si me meto con ello le acabaría cogiendo el gustillo pero de momento es algo que no me llama.

El problema principal es la separación que estás haciendo en tokens o partes.
Buscas hasta la primera coma (,) para localizar el nombre del libro pero de seguido guardas el nombre del autor también sin haber buscado hasta la siguiente coma. Por lo tanto, el nombre del libro y el autor siempre son la misma cadena.
A partir de ahí ya se descuadran todos los cálculos (que tampoco entiendo esos fgets() con longitud 4 y 5...) y se guarda todo mal.

Además otro problema es que la cadena que usas para guardar cada línea del fichero temporalmente tiene una longitud de 50 y hay líneas que superan ese número de caracteres por lo que los caracteres que no entran en una línea, se leen en la siguiente.

Otro otro tema importante cuando se trabaja con memoria dinámica es que esa memoria tienes que liberarla manualmente usando free().

Y otro tema es usar return para salir en caso de error. En la función main() se puede hacer porque de un return acaba el programa pero en otras funciones auxiliares, llamar a return lo que hace es devolver el control a la función que llamó a esta. Es más correcto utilizar exit().
Además el valor 0 se suele asociar a que todo ha finalizado correctamente. Para errores es mejor usar -1. En conjunto: exit(-1).

Otros consejos son:

• Utilizar una constante para la longitud de la cadena que te permita modificarlo rápidamente sin tener que ir mirando por todo el código.
  
• No utilizar variables globales, como en tu caso el puntero l.
  
• Para separar una cadena en base a un patrón o varios tienes la función strtok().
  
• Para "vaciar" una cadena no es necesario poner '\0' a todas sus posiciones. Basta con hacerlo en la posición 0.
  
  Te dejo otra forma de hacerlo con las modificaciones antes mencionadas:
  
  Código (c) [Seleccionar] Expandir
  
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

#define SIZE 100

typedef struct {
char *nombre;
char *autor;
int cantidad;
int precio;
} Libro;


void liberarMemoria(Libro *libros, int numeroLibros){
for(int i = 0; i < numeroLibros; ++i){
free(libros[i].nombre);
free(libros[i].autor);
}
free(libros);
}

void guardarLibro(Libro *libros, char *temporal, int numeroLibros){
char *token = strtok(temporal, ",");
libros[numeroLibros-1].nombre = (char*)malloc((strlen(token)+1) * sizeof(char));
strcpy(libros[numeroLibros-1].nombre, token);

token = strtok(NULL, ",");
libros[numeroLibros-1].autor = (char*)malloc((strlen(token)+1) * sizeof(char));
strcpy(libros[numeroLibros-1].autor, token);

token = strtok(NULL, ",");
libros[numeroLibros-1].cantidad = atoi(token);

token = strtok(NULL, ",");
libros[numeroLibros-1].precio = atoi(token);
}

void mostrarLibros(Libro *libros, int numeroLibros){
for(int i = 0; i < numeroLibros; ++i){
printf("***** INFORMACION LIBRO %d *****\n", i+1);
printf("Nombre: %s\n", libros[i].nombre);
printf("Autor: %s\n", libros[i].autor);
printf("Cantidad: %d\n", libros[i].cantidad);
printf("Precio: %d\n", libros[i].precio);
printf("\n");
}
}

int main(){
Libro *libros = NULL;
char temporal[SIZE];
FILE *fichero = fopen("libros.txt", "r");

int numeroLibros = 0;
while(fgets(temporal, SIZE, fichero)) {
++numeroLibros;
libros = (Libro*)realloc(libros, numeroLibros * sizeof(Libro));
guardarLibro(libros, temporal, numeroLibros);
}

mostrarLibros(libros, numeroLibros);

liberarMemoria(libros, numeroLibros);
return 0;
}
  
  PD: Cuando vayas a insertar código utiliza etiquetas de Código GeSHi y selecciona el lenguaje que corresponda a tu código. En tu mensaje anterior ya te las he incluido yo pero para otra ocasión.
  
  PD 2: Aunque no es relevante, el menú que tienes no funciona. Si vas a pedir una opción tendrás que devolverla con un return.