Error en Ciclo "For" (C)

[Geg1]

Buenas a todos! Desde ya muchas gracias por su atención!
No encuentro la forma de evitar que ocurra esto en mi "lista de pedidos":

Código (c) [Seleccionar] Expandir


#include <stdio.h>
#include <stdlib.h>

typedef struct fecha{
   int MM;
   int AA;
   int DD;
}tfecha;

typedef struct pedido{
   char nombrecliente[20],domicilioentrega[20];
   tfecha fechadepedido;
}tpedido;

//----------------------------------------------------

void cargarpedido (tpedido *R);
void mostrarpedido (tpedido R);
void cargarvectorpedidos (tpedido V[]);
void mostrarvectorpedidos (tpedido V[]);

//----------------------------------------------------

int main()
{
   tpedido X[5];
   int op;
   while(op!=4)
   {
       printf("1.- Ingresar encomienda\n");
       printf("2.- Anular encomienda\n");
       printf("3.- Mostrar lista de encomiendas\n");
       printf("4.- Salir\n");
       printf("Indica la opcion: ");
       scanf("%d", &op);
       system("cls");
 switch(op)
         {

           case 1: cargarvectorpedidos(X);
           break;
           case 2: break;
           case 3: mostrarvectorpedidos(X);
           break;
           case 4: break;
       }

   }
}
//-----------------------------------------------------

void cargarpedido(tpedido *R){
tpedido A;
printf("\nIngrese nombre del cliente:\n");
scanf("%s",&A.nombrecliente);
fflush(stdin);
printf("\nIngrese domicilio de entrega:\n");
gets(A.domicilioentrega);
fflush(stdin);
printf("\nIngrese fecha de ingreso de la encomienda:\n");
printf("\n\n");
printf("\Ingrese numero de dia \n");
fflush(stdin);
scanf("%d",&A.fechadepedido.DD);
printf("\Ingrese numero de mes \n");
fflush(stdin);
scanf("%d",&A.fechadepedido.MM);
printf("\Ingrese numero de a%co\n",164);
fflush(stdin);
scanf("%d",&A.fechadepedido.AA);
fflush(stdin);
*R=A;
}

//--------------------------------------------------------

void cargarvectorpedidos(tpedido V[]){
       int cantidad,i;
       tpedido aux;
       printf("Ingrese la cantidad de encomiendas a cargar \n");
       fflush(stdin);
       scanf("%d",&cantidad);

       for(i=0;i<cantidad;i++){
       printf("\nIngrese los datos para la encomienda %d:\n",i+1);
       cargarpedido(&aux);
       V[i]=aux;
       system("cls");
       }

}

//------------------------------------------------------------


void mostrarvectorpedidos(tpedido V[]){
int j;
system("pause");
system("cls");
printf("\n Listado de encomiendas:\n");
for(j=0;[color=red][b]j<5[/b][/color];j++){
       mostrarpedido(V[j]);
}
system("pause");
system("cls");
}

//--------------------------------------------------------------

void mostrarpedido(tpedido R){

       printf("\nNombre del cliente: %s", R.nombrecliente);
       printf("\nDomicilio de entrega: %s", R.domicilioentrega);
       printf("\nFecha de ingreso de la encomienda: %d/%d/%d",R.fechadepedido.DD,R.fechadepedido.MM,R.fechadepedido.AA);
       printf("\n");

   }




En Rojo, creo yo, en donde esta el error... ya que al ingresar (Ejemplo) un pedido, y quererlo mostrar por pantalla, sale esto:







No se como lograr que solo muestre la cantidad puesta anteriormente asignada al entero "cantidad"


Por otra parte, necesito ayuda para anular un registro y asi crear la función "Anular pedido", digamos que estoy perdido y no se por donde empezar para crear dicha función, si me ayudan con un planteo se los agradecería muchísimo!

[MAFUS]

¿Has visto listas enlazadas?
Lo que planteas es un problema que se resuelve con ellas.

Si no las has visto y solo puedes usar arrays tendrás podrías resolver el problema poniendo un campo más en tu estructura diciendo si el registro es útil. Cada vez que incluyas un registro buscas el primero que no sea útil (esté vacío) y lo incluyes allí. Para visualizar repasas toda la lista y solo muestras los marcados como útiles. Para borrar solo debes marcar el registro de interés como inútil.
También podrías hacer lo mismo sin ese campo pero creando un array paralelo, que tendrán que ser recorridos al mismo tiempo, en el que solo guardaría la información de si el registro es válido.

[Geg1]

¿Has visto listas enlazadas?
Lo que planteas es un problema que se resuelve con ellas.

Si no las has visto y solo puedes usar arrays tendrás podrías resolver el problema poniendo un campo más en tu estructura diciendo si el registro es útil. Cada vez que incluyas un registro buscas el primero que no sea útil (esté vacío) y lo incluyes allí. Para visualizar repasas toda la lista y solo muestras los marcados como útiles. Para borrar solo debes marcar el registro de interés como inútil.
También podrías hacer lo mismo sin ese campo pero creando un array paralelo, que tendrán que ser recorridos al mismo tiempo, en el que solo guardaría la información de si el registro es válido.

No he visto tal cosa pero me podrias explicar mas a fondo la nueva estructura que planteas? Gracias por tu respuesta!!

[Kenji-chan]

el problema que tienes esta en la funcion

Código [Seleccionar] Expandir


void mostrarvectorpedidos(tpedido V[]){
int j;
system("pause");
system("cls");
printf("\n Listado de encomiendas:\n");
for(j=0; j<5; j++){
        mostrarpedido(V[j]);
}
system("pause");
system("cls");
}


en la que tu ciclo for(); j<5 se repte 5 veces independiente del numero de pedidos que has ingrsado por ejemplo tu ingresaste 1 y te lo imprime pero tambien imprime otros 4 con contenido basura o no definido

lo que puedes hacer es mantener un contador que almacene el numero de pedidos ingresados que sera el mismo que le pases al ciclo for() como j<contador; para que solo te imprima la cantidad de pedidos ingresados

haora para las listas enlasadas te recomiendo este video
https://www.youtube.com/watch?v=vldM-3PYAmo

[MAFUS]

Algo así (solo se muestran las modificaciones al original):

Código (c) [Seleccionar] Expandir


typedef struct pedido{
    char nombrecliente[20],domicilioentrega[20];
    tfecha fechadepedido;
    int esValido; // <-- inclusión
}tpedido;

void inicializar_lista(tpedido lista[]) { // <-- Nueva función. Debe ser el primer código que se
    int i;                                //         ejecute dentro de main(), después de la
                                          //         declaración de las variables
    for(i=0; i<5; ++i)
        lista[i].esValido = 0;
}

void cargarvectorpedidos(tpedido V[]){
    int cantidad, i;
    tpedido aux;
   
    printf("Ingrese la cantidad de encomiendas a cargar \n");
    fflush(stdin);
    scanf("%d", &cantidad);

    for(i=0; i<cantidad; i++){
        printf("\nIngrese los datos para la encomienda %d:\n",i+1);
        cargarpedido(&aux);
        V[i]=aux;
        V[i].esValido = 1; // <-- inclusión
        system("cls");
    }
}

void mostrarvectorpedidos(tpedido V[]) {
    int j;
   
    system("pause");
    system("cls");
    printf("\n Listado de encomiendas:\n");

    for(j=0; j<5; j++) {
        if(V[j].esValido) // <-- inclusión
            mostrarpedido(V[j]);
        }
    }
   
    system("pause");
    system("cls");
}


int main()
{
    tpedido X[5];
    int op;
   
    inicializar_lista(X); // <-- Llamada a la nueva función
    while(op!=4)
    // ...
}

[Geg1]

el problema que tienes esta en la funcion

Código [Seleccionar] Expandir


void mostrarvectorpedidos(tpedido V[]){
int j;
system("pause");
system("cls");
printf("\n Listado de encomiendas:\n");
for(j=0; j<5; j++){
        mostrarpedido(V[j]);
}
system("pause");
system("cls");
}


en la que tu ciclo for(); j<5 se repte 5 veces independiente del numero de pedidos que has ingrsado por ejemplo tu ingresaste 1 y te lo imprime pero tambien imprime otros 4 con contenido basura o no definido

lo que puedes hacer es mantener un contador que almacene el numero de pedidos ingresados que sera el mismo que le pases al ciclo for() como j<contador; para que solo te imprima la cantidad de pedidos ingresados

haora para las listas enlasadas te recomiendo este video
https://www.youtube.com/watch?v=vldM-3PYAmo

Entiendo, pero ese contador es justamente el entero "cantidad" como lo relaciono con la funcion "mostrarVectorPedidos"?

Disculpen es que soy bastante nuevo en esto de la programacion! 

[MAFUS]

Sácalo de la función y conviértelo en global, así lo tendrás visible para todas las funciones.

[Geg1]

Sácalo de la función y conviértelo en global, así lo tendrás visible para todas las funciones.

Uf, que me hago un mate en la cabeza jajaj, solo el entero "cantidad" dices que saque de la funcion "Cargarvectorpedidos"?

[Kenji-chan]

la variable 'cantidad' que tienes en la función void cargarvectorpedidos(tpedido V[]) declarala en la funcion main() y despues solo pasar la direccion de la variable cargarvectorpedidos(x, &cantidaad); y la revises como puntero en la funcion void cargarvectorpedidos(tpedido V[], int* cantidad); y despues solo lo pasas como reerencia en la uncion mostrarvectorpedidos(X, cantidad); y la usas en el ciclo for(); j<cantidad

si tienes mas dudad puedes ver este vídeo
[youtube=640,360]
https://youtu.be/fFjmbymw2Ug
[/youtube]

· No hacer doble post, usa el botón "modificar"
>aquí las reglas del foro
-Engel Lex

[Geg1]

la variable 'cantidad' que tienes en la función void cargarvectorpedidos(tpedido V[]) declarala en la funcion main() y despues solo pasar la direccion de la variable cargarvectorpedidos(x, &cantidaad); y la revises como puntero en la funcion void cargarvectorpedidos(tpedido V[], int* cantidad); y despues solo lo pasas como reerencia en la uncion mostrarvectorpedidos(X, cantidad); y la usas en el ciclo for(); j<cantidad

si tienes mas dudad puedes ver este vídeo
[youtube=640,360]
https://youtu.be/fFjmbymw2Ug
[/youtube]

· No hacer doble post, usa el botón "modificar"
>aquí las reglas del foro
-Engel Lex

No entiendo a que te refiieres con " la revises como puntero"