Suseitalia.org (Beta platform)

c'è qualcosa che non va... non gli piace il codice :P
eccolo qui:
http://pastebin.com/m2401caf1

Ciao

L'unica critica di fondo che si può fare al tuo programma riguarda l'uso della struttura data.
L'uso di una struttura del genere rende un incubo qualsiasi operazione statistica (anche il solo semplice sort) su tale campo.

Da che programmazione è programmazione i campi data / ora vengono salvate sempre in un unico campo numerico con lo 0 fisso ad una determinata data (se non ricordo male 1/1/1970) questo per rendere semplicissime le operazioni sulle date (specialmente le comparazioni).

Se vuoi evitare la conversione della data puoi usare una stringa nel formato AAAAMMGG o un long int con Anno*10000 + Mese*100 + Giorno

Ciao Ciao, Moreno

Moreno ha ragione. In C soprattutto è impensabile fare un ordinamento su delle strutture allocate staticamente (costa tempo e memoria). Io fossi in te, prima di completare questo progettino,andrei un altro pochino avanti nello studio. Di solito per questo genere di operazioni si usa l'allocazione dinamica della memoria (attraverso i puntatori) e una struttura dati che si presti bene al tipo di operazioni da fare (ad esempio una lista, se ordinata potrebbe ridurre ulteriormente il numero di calcoli, magari per operazioni che vengono effettuate con una certa frequenza).
Senza dimenticare che faciliterebbero non poco l'implementazioni di qualsiasi algoritmo di ricerca/ordinamento.

l'aritmetica dei puntatori l'ho finita questa mattina, ma forse sono tonto, non vedo come potrei trovarne vantaggio qui.
@ moreno.
ho cercato qualcosa che mi facesse vedere come gestire le date come suggerito ma ho trovato solo questo:
http://thedevelopers.netsons.org/c-funzione-di-gestione-data/  (questo controllo sulla data è anche più sofisticato del mio :) )
qui invece ho trovato praticamente la stessa struttura che ho fatto:
http://www.mrwebmaster.it/cplusplus/guide/gestione-date_290.html 

potresti farmi un esempio? Ad onor di cronaca sono interessato per mio sapere personale, dato che l'esercizio chiedeva espressamente di usare quel tipo di struttura!

Io parlavo di aritmetica dei puntatori nell'altro topic, comunque pensavo ti sarebbe tornata utile nel caso tu avessi voluto definire la dimensione dell'array a run time. Ad esempio:

<br />
studente *s;<br />
int num_studenti;<br />
printf("Inserire numero studenti: ");<br />
scanf("%d",&num_studenti);<br />
s = (studente*) malloc(num_studenti*sizeof(studente)); 

   
In alcuni codici puoi trovare anche il malloc senza niente davanti, perchè avviene un casting implicito, comunque per avere la certezza assoluta che tutto funzioni metterlo non è certo di troppo, anzi. A questo punto per accedere ai vari studenti ti basterebbe usare l'aritmetica dei  puntatori:

<br />
for(i=0;i<br />
       (s+i)->nome=....<br />

   

PS: sto abbozzando questo codice così su due piedi, potrebbe esserci qualche errore sintattico, ma la logica è giusta. Se hai qualche roblema posta pure.

Ciao

Non mi stavo riferendo alle routines di input ed analisi della data ma proprio alla variabile di memorizzazione della stessa.

Invece della struttura data sarebbe meglio usare una variabile data di tipo time_t definito in time.h

#ifndef _TIME_T
#define _TIME_T
typedef long time_t;
#endif

ed usare le routines mktime e localtime per operare sulla variabile (per maggiori info http://www.cplusplus.com/reference/clibrary/ctime/ )

Se vuoi ad esempio ordinare i tuoi allievi in ordine di data di nascita con la tua struttura ti spari mentre con variabili time_t è una baggianata data che a numero maggiore corrisponde data maggiore.

Oltre ai consigli precedenti, ne aggiungo uno di stile:
se hai:

<br />
...<br />
if ( ... ) {<br />
     ...<br />
     return 1<br />
}<br />
else {<br />
     ...<br />
     return 2<br />
}<br />

 

puoi sempre semplificare il codice con:

<br />
...<br />
if ( ... ) {<br />
     ...<br />
     return 1<br />
}<br />
...<br />
return 2<br />

Quote: l'aritmetica dei puntatori l'ho finita questa mattina, ma forse sono tonto, non vedo come potrei trovarne vantaggio qui.
@ moreno.
ho cercato qualcosa che mi facesse vedere come gestire le date come suggerito ma ho trovato solo questo:
http://thedevelopers.netsons.org/c-funzione-di-gestione-data/  (questo controllo sulla data è anche più sofisticato del mio :) )
qui invece ho trovato praticamente la stessa struttura che ho fatto:
http://www.mrwebmaster.it/cplusplus/guide/gestione-date_290.html 

potresti farmi un esempio? Ad onor di cronaca sono interessato per mio sapere personale, dato che l'esercizio chiedeva espressamente di usare quel tipo di struttura! 
Se è richiesto l'uso di una struttura, semplicemente ignora i commenti precedenti, che fanno riferimento a convenzioni che, eventualmente, potrai imparare più avanti. Ora pensa ad imparare il C :-)
Quanto all'uso dell'allocazione dinamica in C, aspetta ad usarla quando te la spiegheranno. E in ogni caso è un'esperienza dolorosa a confronto con altri linguaggi.
Quanto ad ordinare per data, con la struttura non ottieni il massimo dell'efficienza, ma non è un'operazione così complicata. Devi semplicemente usare un algoritmo di ordinamento tre volte, in sequenza:
- Ordina per anno
- Ordina per mese
- Ordina per giorno
Ciao

Quote: Oltre ai consigli precedenti, ne aggiungo uno di stile:
se hai:

<br />
...<br />
if ( ... ) {<br />
     ...<br />
     return 1<br />
}<br />
else {<br />
     ...<br />
     return 2<br />
}<br />

 

puoi sempre semplificare il codice con:

<br />
...<br />
if ( ... ) {<br />
     ...<br />
     return 1<br />
}<br />
...<br />
return 2<br />

   
 Vero, ma non mi piace a livello di leggibilità. Preferisco la prima versione e un return di default per evitare un warning

Non da warning ("gcc -Wall -ansi ...").
Era solo per accorciare il codice dove possibile.