Thread: C project

Make a structure that will hold all of the cities values

Visto che sei italiano usiamo la nostra lingua madre :P
Ho dato un'occhiata veloce al codice, hai scelto di usare un vettore per memorizzare i vari prodotti, e non una coda, per cui quello che dovresti fare, trovato il "j" che corrisponde al prodotto (linea 181) devi fare un secondo ciclo che vada da j al termine del vettore (city[i-1].num_prod) e copiare il prodotto successivo nella posizione corrente e solo successivamente diminuire il numero di prodotti per quella data città. Non è necessario ma per correttezza dovresti anche "resettare" l'ultima posizione del vettore non più utilizzata.
Sono stato chiaro?
OT: di dove sei? Che materia è?

edit: linea 188 non 181

Your use of m, n, and p make me sad

Devi sapere che qui se la tirano tutti e non poco
Passando a noi, avevo letto decisamente di fretta il codice! Spiegami cosa intendi dire per cancellare un prodotto: data una città da input ed un determinato prodotto, cancellare questo dalla città?
Se la risposta è sì, dalla riga 186 alla riga 189 dovresti modificare così:

Code:

while(j<city[i-1].num_prod)
    {
      if(strcmp(prodotto,city[i-1].listprod[j].type)==0){
          while(j<city[i-1].num_prod-1){
                 city[i-1].listprod[j].code_n=city[i-1].listprod[j+1].code_n;
                 strcpy(city[i-1].listprod[j].type,city[i-1].listprod[j+1].type);
                 city[i-1].listprod[j].prezzo=city[i-1].listprod[j+1].prezzo;
                 j++;
          }
     }
      j++;
    }

La domanda te l'ho posta perchè non capisco a cosa ti serva il flag eliminato.
Con questo codice:
-hai trovato il prodotto da eliminare;
-lo hai eliminato copiandoci sopra il prodotto successivo (se è l'ultimo prodotto semplicemente diminuisce il numero dei prodotti).
Ora è più chiaro?
Fammi sapere

PS: come dovresti affrontare il caso in cui nella città ci siano due prodotti uguali?
PPS: cosa studi?

Translation:
I read your code too much quickly. What do you mean by deleting a product?
If you mean a certain product must be deleted from a certain city (both given by input) you could try this code:
....
I'm asking this cause i don't understand why you should use the flag "eliminato".
With this code:
-you found the position of the product to erase;
-you erase it coping the next product in it and so on (if it's the last one, you just reduce the number of product in that city).
Now is it clearer?
Let me know.

PS: how should you manage the case two product with the same name are present?
PPS: what do you study?

Please post in English.

Ok, sorry, have I to translate?

Originally Posted by ZaC

Ok, sorry, have I to translate?

That would probably be a good idea.

Originally Posted by laserlight

That would probably be a good idea.

already done

Originally Posted by thenewbiecoder

sei un grande!

ehehe thanks

Originally Posted by thenewbiecoder

facevo lo stesso ciclo... sbagliando la condizione ._.

rincretinito completamente mettevo (j==k), con k=cit[i-1].num_prod... lo so, merito solo schiaffi, ma avevo un mal di testa assurdo oggi

That's happen don't you worry

Originally Posted by thenewbiecoder

comunque studio Informatica, esame di Prog 1... e non ci sono code e liste nel programma, per questo ho dovuto usare questo metodo!

Messina by chance?

Originally Posted by thenewbiecoder

grazie mille ^_^

No problem we are here for this... really?!


PS: I sudgest you to use for instead while, it's clearer. And also typedef if it's allowed in your exam.

I spotted only a logic error, you flag as erased not the sent product but the last one, this is meaningless... I think that you don't need that flag ("eliminato").
Example:
You have to send the products a,b,c to the city x. Number of products = 3.
You send b and now in the x product list you have: a,c,c (b erased, c copied in the position of b)
The number of products now = 2.
Now you flag c as erased but it's useless.

The other thing I noticed is the use of typedef. I sudgested you to use it so you don't have to repeat struct a_product evrywhere so...

Code:

typedef struct a_product{
... 
} product;

and

Code:

#
typedef struct a_city{
...
        product listprod[m];
...       
} city;

and so on...
Tomorrow morning I will read it better now i'm a little tired so I could miss some error.

edit:
I told you I could miss something...
From line 181 to 195:

Code:

                            for(j=0;j<city[i-1].num_prod;j++)
                                {
                                    if(strcmp(prodotto,city[i-1].listprod[j].type)==0)
                                    {
                                        for(;j<city[i-1].num_prod-1;j++) //note nothing inizilized!
                                        {
                                            city[i-1].listprod[j].code_n=city[i-1].listprod[j+1].code_n;
                                            strcpy(city[i-1].listprod[j].type,city[i-1].listprod[j+1].type);
                                            city[i-1].listprod[j].prezzo=city[i-1].listprod[j+1].prezzo;
                                        }
                                    }
                                }