Mensajes

[5]

Hola de nuevo!!! Gracias maxim_o!!! Pero tu ejemplo del final con numeros no esta bien, te explico.

El codigo es C++ por eso hago el paso por referencia del entero "n"(no hace falta pasarle un puntero) y funciona perfecto.

El problema con numeros es:

Yo tengo el array 1 3 5 7 9, llamo a eliminar y quiero borrar por ejemplo 5, entonces me retorna 1 3 5 7 (sin el 9) ya que como he dicho, la "n" si que se resta una unidad, pero el array cambia de tamaño(uno menos) pero no cambia sus elementos.
En cambio, coloqué las llamadas a visualizar dentro de la funcion "eliminar" y al visualizar "auxiliar" se ve 1 3 7 9 y lo mas gracioso es que si visualizo "x" tambien se ve 1 3 7 9, EL PROBLEMA es que al retornar a main y llamar a visualizar con "producto" (que deberia ser igual que "x" debido a la llamada de la funcion "eliminar"), se visualiza como dije al principio 1 3 5 7.

¿Alguien me puede ayudar?... no lo entiendo

Gracias por la ayuda durasno!! El codigo es C++ y efectivamente es un paso por referencia el int &n. La verdad que no sabia que para copiar los arreglos tenias que ir elemento a elemento, asike muchas gracias. Yo pense que como los nombres de los arreglos (x  y auxiliar en mi caso) son direcciones de memoria del primer elemento del arreglo, pense que asignando x = auxiliar; me serviria(que por cierto no da ERROR).

Lo que me extraña mucho es que si te fijas en las funciones visualizar que tenemos justo debajo de x = auxiliar; , ambas lo muestran correctamente, con el elemento eliminado. E problema es que cuando vuelvo a la funcion "main" y tengo una nueva llamada a visualizar(justo el case 8 del switch) con el arreglo "producto" que deberia ser el mismo que el arreglo "x" de la funcion eliminar, pues este arreglo muestra un elemento menos(ya que pase la n por referencia) pero no elimina el que yo keria, sino el ultimo.

¿Alguien sabe porque?

[El problema es que aunque paso a la funcion el arreglo como puntero, no me elimina la estructura del arreglo(aunke la "n" si disminuye una unidad). Realmente parece que si la elimina(si miramos las llamadas a visualizar), pero no llega bien a la funcion main]

Buenos dias chic@s. Tengo una duda con un programa. Realmente le he hecho de dos formas, de la primera no funciona y de la segunda si. Como estoy aprendiendo programacion, me gustaria que alguien me explique porke narices no funciona la primera(creo que es relacionado con el paso por punteros o direcciones de memoria, pero no lo se...). Aqui dejo el codigo. No os asusteis porke sea muy largo, con que mireis la ultima funcion llamada "eliminar"(elimina una estructura de un arreglo) que es donde esta el problema, ya vale(dejo el programa por si alguien le kiere ejecutar y para que se entienda mejor). El problema es que aunque paso a la funcion el arreglo como puntero, no me elimina la estructura del arreglo(aunke la "n" si disminuye una unidad). Realmente parece que si la elimina(si miramos las llamadas a visualizar), pero no llega bien a la funcion main. Supongo para alguien que sepa programar bien la duda sera estupida, pero yo no lo entiendo

Gracias a todos!!!

Código (cpp) [Seleccionar] Expandir

/*****   Preproceso*/
#include <stdio.h>
#include <stdlib.h>
#include <string.h>
#include <conio.h>
#define N 50
#define ELEM 20
#define UU char(163)/*Para los acentos*/

/***** Variables globales*/
/***** int const N=50;*/
struct Producto{
       int codigo;
       char nombre[N];
       float precio;
       int stock;
       };

/***** Declaracion de funciones o procedimientos*/
int menu (void);
void introducir (struct Producto x[],int &n);
void visualizar (struct Producto x[],int n);
int existe_codigo (struct Producto x[],int n, int cod);
void comprar (struct Producto x[],int n);
void vender (struct Producto x[],int n);
void modificar (struct Producto x[],int n);
int submenu (void);
void ordenar_cod (struct Producto x[],int n);
void ordenar_pre (struct Producto x[],int n);
void ordenar_sto (struct Producto x[],int n);
void ordenar_nom (struct Producto x[],int n);
void grabar_fichero (struct Producto x[],int n,char* nom);
void cargar_fichero (struct Producto x[], int &n, char* nom);
void cambiar (struct Producto x[], int &n, char* nom);
void eliminar (struct Producto x[], int &n);

/***** Programa principal*/

int main (){
    /***** Var. Locales*/
    int opcion=0, opcion2 =0;
    int n = 0;
    char nombre[ELEM];
    struct Producto producto[ELEM];
    printf ("C%cal es el nombre de la tienda??", UU);
    gets(nombre);
    cargar_fichero (producto, n, nombre);
   
    do{
       opcion = menu();
       switch(opcion){
             case 1:
                  introducir(producto, n);
                  break;
             case 2:
                  visualizar(producto, n);
                  break;
             case 3:
                  comprar(producto, n);
                  break;
             case 4:
                  vender(producto, n);
                  break;
             case 5:
                  modificar(producto, n);
                  break;
             case 6:
                  opcion2 = submenu();
                  switch(opcion2){
                                  case 1:
                                  ordenar_cod(producto, n);
                                  break;
                                  case 2:
                                  ordenar_nom(producto, n);
                                  break;
                                  case 3:
                                  ordenar_pre(producto, n);
                                  break;
                                  case 4:
                                  ordenar_sto(producto, n);
                                  break;           
                  }
                 
                  break; 
             case 7:
                  cambiar(producto, n, nombre);
                  break;
             case 8:
                  eliminar(producto, n);
                  visualizar (producto,n);/*Llamada para ver el ERROR*/
                  break;
       }
         
    } while (opcion !=0);
    grabar_fichero(producto, n, nombre);
    return 0;
}


/***** Cuerpo de las funciones*/
int menu (void){
    int opcion;
    do{
       printf ("\n\n1. Introducir Producto");
       printf ("\n2. Listar Productos disponibles");
       printf ("\n3. Comprar");
       printf ("\n4. Vender");
       printf ("\n5. Cambiar Precio");
       printf ("\n6. Ordenar");
       printf ("\n7. Cambiar tienda");
       printf ("\n8. Eliminar producto");
       printf ("\n0. Salir\n\n");
       fflush (stdin);
       scanf ("%d", &opcion);
       
    }while((opcion<0) || (opcion>8));
    return opcion;
}


int submenu (void){
    int opcion;
    do{
       printf ("\n¿Como quieres ordenar?");
       printf ("\n1. Codigo");
       printf ("\n2. Nombre");
       printf ("\n3. Precio");
       printf ("\n4. Stock\n\n");
       fflush (stdin);
       scanf ("%d", &opcion);
       
    }while((opcion<1) || (opcion>4));
    return opcion;
}
   

void introducir (struct Producto x[],int &n){
     int controlar;
     do{
     printf ("\nIntroduzca Codigo: ");
     fflush (stdin);
     scanf ("%d", &x[n].codigo);
     controlar = existe_codigo (x, n, x[n].codigo);
     if (controlar != -1)
        printf("\nEl producto ya existe!!!");
     }while (controlar != -1);
     
     printf ("\nIntroduzca Nombre Producto: ");
     fflush (stdin);
     gets (x[n].nombre);
     printf ("\nIntroduzca Precio: ");
     fflush (stdin);
     scanf ("%f", &x[n].precio);
     printf ("\nIntroduzca Stock: ");
     fflush (stdin);
     scanf ("%d", &x[n].stock);
     n++;
}


void visualizar (struct Producto y[],int n){
     int i;
     for (i=0; i<n; i++){
         printf ("\n%10d\t%20s\t%10.2f\t%10d", y[i].codigo, y[i].nombre, y[i].precio, y[i].stock);
     }
}


int existe_codigo (struct Producto x[],int n, int cod){
     int i, r = -1;
     for (i=0; i<n; i++){
         if (x[i].codigo == cod){
            r = i;
            break;
           
         }
     }
     return r;/*****Retorna la posicion si existe y -1 si no existe*/
}


void comprar (struct Producto x[],int n){
     int codigo, result, cant;
     printf ("\nQue producto desea comprar?(indique codigo)");
     fflush (stdin);
     scanf ("%d", &codigo);
     result = existe_codigo (x, n, codigo);
     if (result == -1){
          printf ("\nEl producto no existe!!!");
     }
     else{
          printf ("\nCuanta cantidad desea comprar??");
          fflush (stdin);
          scanf ("%d", &cant);
          x[result].stock = x[result].stock + cant;
     }
}


void vender (struct Producto x[],int n){
     int codigo, result, cant;
     printf ("\nQue producto desea vender?(indique codigo)");
     fflush (stdin);
     scanf ("%d", &codigo);
     result = existe_codigo (x, n, codigo);
     if (result == -1){
          printf ("\nEl producto no existe!!!");
     }
     else{
          printf ("\nCuanta cantidad desea vender??");
          fflush (stdin);
          scanf ("%d", &cant);
          x[result].stock = x[result].stock - cant;
     }
}


void modificar (struct Producto x[],int n){
     int codigo, result;
     float precio;
     printf ("\nQue producto desea modificar precio?(indique codigo)");
     fflush (stdin);
     scanf ("%d", &codigo);
     result = existe_codigo (x, n, codigo);
     if (result == -1){
          printf ("\nEl producto no existe!!!");
     }
     else{
          printf ("\nValor del nuevo precio: ");
          fflush (stdin);
          scanf ("%f", &precio);
          x[result].precio = precio;
     }
}


void ordenar_cod (struct Producto x[],int n){
     bool ordenado = false;
     int i;
     struct Producto aux;
     int j = 0;
     while (j<n-1 || ordenado == false){
           ordenado = true;
           for (i=0; i<n-1; i++){
                     if (x[i].codigo > x[i+1].codigo){
                     aux = x[i];
                     x[i] = x[i+1];
                     x[i+1] = aux;
                     ordenado = false;
                     }
           }                     
           j++;
     }
}


void ordenar_pre (struct Producto x[],int n){
     bool ordenado = false;
     int i;
     struct Producto aux;
     int j = 0;
     while (j<n-1 || ordenado == false){
           ordenado = true;
           for (i=0; i<n-1; i++){
                     if (x[i].precio > x[i+1].precio){
                     aux = x[i];
                     x[i] = x[i+1];
                     x[i+1] = aux;
                     ordenado = false;
                     }
           }                     
           j++;
     }
}


void ordenar_sto (struct Producto x[],int n){
     bool ordenado = false;
     int i;
     struct Producto aux;
     int j = 0;
     while (j<n-1 || ordenado == false){
           ordenado = true;
           for (i=0; i<n-1; i++){
                     if (x[i].stock > x[i+1].stock){
                     aux = x[i];
                     x[i] = x[i+1];
                     x[i+1] = aux;
                     ordenado = false;
                     }
           }                     
           j++;
     }
}


void ordenar_nom (struct Producto x[],int n){
     bool ordenado = false;
     int i;
     struct Producto aux;
     int j = 0;
     while (j<n-1 || ordenado == false){
           ordenado = true;
           for (i=0; i<n-1; i++){
                     if (strcmp (x[i].nombre , x[i+1].nombre) > 0){
                     aux = x[i];
                     x[i] = x[i+1];
                     x[i+1] = aux;
                     ordenado = false;
                     }
           }
           j++;
     }
}


void grabar_fichero (struct Producto x[], int n, char* nom){
     FILE *fichero = NULL;
     int i;
     fichero = fopen(nom, "wb");
     if (fichero == NULL){
        printf ("\nERROR: No se ha podido abrir %s", nom);
        getch();
        exit(0);
     }
     else{
         for (i=0; i<n; i++){
             fwrite(&x[i], sizeof(struct Producto), 1, fichero);
         }
         fclose(fichero);
     }
}


void cargar_fichero (struct Producto x[], int &n, char* nom){
     FILE *fichero = NULL;
     strcat(nom,".dat");
     fichero = fopen(nom, "rb");
     if (fichero == NULL){
        printf ("\nADVERTENCIA: No se ha podido abrir %s", nom);
        getch();
     }
     else{
             fread(&x[n], sizeof(struct Producto), 1, fichero);
             while(!feof(fichero)){
                 n++;
                 fread(&x[n], sizeof(struct Producto), 1, fichero);
             }
             fclose(fichero);
     }
}


void cambiar (struct Producto x[], int &n, char* nom){
     grabar_fichero(x, n, nom);
     printf ("\nC%cal es el nombre de la tienda a la que quieres cambiar??", UU);
     fflush (stdin);
     gets(nom);
     n=0;
     cargar_fichero (x, n, nom);
}


void eliminar (struct Producto x[], int &n){
     int pro_elim, result, i,j=0;
     struct Producto auxiliar[ELEM];
     printf ("\nCodigo del producto que deseas eliminar??");
     fflush (stdin);
     scanf ("%d", &pro_elim);
     result = existe_codigo (x, n, pro_elim);
     if (result == -1){
          printf ("\nEl producto no existe!!!");
     }
     else{
          for(i=0; i<n; i++){
                if (i!=result){
                      auxiliar[j]=x[i];
                      j++;
                }
          }
          n--;
          x = auxiliar;
     }
     visualizar (auxiliar,n);/*Llamada para ver el ERROR*/
     visualizar (x,n);/*Llamada para ver el ERROR*/
}


Funcion "eliminar" corregida y SI FUNCIONA:

Código (cpp) [Seleccionar] Expandir

void eliminar (struct Producto x[], int &n){
     int pro_elim, result, i,j=0;
     struct Producto auxiliar[ELEM];
     printf ("\nCodigo del producto que deseas eliminar??");
     fflush (stdin);
     scanf ("%d", &pro_elim);
     result = existe_codigo (x, n, pro_elim);
     if (result == -1){
          printf ("\nEl producto no existe!!!");
     }
     else{
          for(i=0; i<n; i++){
                if (i!=result){
                      auxiliar[j]=x[i];
                      j++;
                }
          }
          n--;
          for(i=0; i<n; i++){
                   x[i]= auxiliar[i];
          }
     }
     visualizar (auxiliar,n);/*Llamada para ver el ERROR*/
     visualizar (x,n);/*Llamada para ver el ERROR*/
}

El problema esta en:

Código (cpp) [Seleccionar] Expandir

x = auxiliar;

que lo he cambiado por lo siguiente y si funciona:

Código (cpp) [Seleccionar] Expandir

for(i=0; i<n; i++){
          x[i]= auxiliar[i];
          }

Has probado a meter el código en la etiqueta code?

En la parte derecha tienes un listbox que pone GeSHi, hay seleccionas el código que quieras y lo pegas entre etiquetas.

Pruébalo, no te fallará  

Gracias overxfl0w13!! Funciono!! estoy empezando en este mundo y era la primera vez que metia codigo... :S

me lo ha escrito en cursiva despues del nomb_array

Buenas a tod@s!!!

Pues estaba escribiendo un nuevo tema para preguntar una duda y al copiar mi codigo del dev-C++ y pegarle aqui, habia parte del codigo que no me lo pegaba como estaba originalmente. Sobretodo me fije en que no me copiaba las "i" que van dentro de los corchetes para acceder a elementos de una lista en bucles (ejemplo for(i=0;i<20;i++) nomb_array; ) Aqui me ha pasado lo mismo...

Alguien me puede echar una mano e indicarme como pegar TODO mi codigo en los mensajes sin problemas??

Gracias