DE NARDIS FRANCESCO INSUFF ========================== non compila disordinato 100*rand()/RAND_MAX puo' assumere valori incontrollati gli if sono senza else dove dovrebbero avercelo in alcuni casi (seppur rari) i valori degli array non sono determinati l'unica funzione non e' finita un po' copiato da De Santis, eh? DE SANTIS LUIGI 28 ================== warning (formato) det=det-x --> det -= x perche' passa s come puntatore? s si puo' aggiornare nel main la percentuale minima non e' nulla, ma 0 a=a+x --> a += x DI GIULIO ANTONIO 27 ==================== Il for non e' il ciclo piu' adatto, in questo caso Molti if si possono eliminare: basta fare, e.g., buche[0] = smallHole(); break! Ha scritto tre funzioni praticamente uguali, invece di scriverne una con un parametro DI MARIO VALENTINA 23 ===================== non compila tot = tot - x --> tot -= x else tot=tot!!!! if (..) tot=tot!!! il controllo relativo al passaggio sulle buche grandi non funziona non rispetta le prescrizioni (c'e' una sola funzione) cerca di calcolare il modulo di un float (errore) le costanti non sono tipograficamente diverse dalle variabili DI MAURO LUCIA PIA 22 ===================== non compila (non passa un array alla funzione, ma una sua componente, peraltro inesistente) ** grave ** lo stato delle gomme non e' inizializzato break!!! il for non e' il ciclo piu' adatto (infatti richiede un break) non molto ordinato la funzione che calcola il deterioramento non e' il massimo DOMENICONI SIRIA 30 =================== warning (formato) poco coerente internamente (le probabilita' sono ora espresse in percentuale, ora in assoluto) D'ONOFRIO SIMONE 30 =================== warning (formato) buon programma: qualcosa si poteva fare meglio (e.g. si poteva azzerare l'array in buchePresenti e si poteva calcolare il deterioramento con una sola riga). FARAONE ANDREA INSUFF ===================== non compila il for non e' il ciclo adatto (e fa un passo in meno) ripete n volte le stesse assegnazioni di variabili (!) moltiplica per 1 !! non fa alcuna estrazione di numeri casuali la probabilita' di rovinare gli pneumatici cresce col tempo definisce le funzioni nel main (!!!) FESSINA MASSIMILIANO 25 ======================= int tempo[1] = {0}; il for e' inadatto x = x - a --> x -= a inutile uso di puntatori exit ! non molto ordinato non rispetta le prescrizioni (ed e' costretto a passare puntatori alle funzioni) FIORILLO MATTEO 30 ================== un po' di disordine FIRMANI LORENZO 29 ================== disordinato un po' troppi if FIUMI CHIARA 27 =============== warning (formato) a una funzione non passa l'array, ma tre puntatori all'altra non passa l'array, ma il suo ultimo elemento break non azzera la variabile det cui poi aggiunge il deterioramento della gomma FOGLIETTA MARCO INSUFF ====================== non compila int buche[] = {}; while (1 < 8000) deterioramento(buche, buche); chiama bucagrande(buche); e poi la richiama ogni volta per assegnarne il valore di ritorno a una variabile o per confrontarla con altri valori! if alternativi senza else funzione di nome funzione mai usata no more comments... FOGLIETTI AGNESE 24 =================== warning (= invece di ==) grave non ordinato passa il tempo alla funzione che calcola il deterioramento break funzione numBuche di una lunghezza straordinaria... else del tutto fuori luogo FRANCIONI MIRKO 23 ================== warning su array (gravi) int b[2] per usare b[2] il for non e' il ciclo adatto inutile passaggio di variabili a funzioni disordine la variabile det non e' azzerata, ma si toglie da essa quantita' in una funzione FRASCA LORENZO 30 ================= warning (time non dichiarata) inutile uso dei puntatori FROSINA LORENZO 21 ================== invece di fare un array di 3 elementi, dichiara 3 array di 500 elementi! il programma e' tutto in una funzione continua a usare inutilmente array 1/15 fa zero il for non e' il ciclo adatto (ma lo salva perche' se no non ne uscirebbe) enorme complicazione per gestire l'array break if esclusivi senza else GAJANO SAFFI GIOVANNI 22 ======================== invece di fare un array di 3 elementi, dichiara 3 array di 500 elementi! il for non e' il ciclo adatto l'if nel for si ripete inutilmente piu' volte e puo' dar luogo a ripetizioni non dovute se gli pneumatici si danneggiano troppo continua a viaggiare le condizioni di presenza di buche non sono esclusive GAVETTI CLAUDIO INSUFF ====================== non compila array tridimensionale! Le cui taglie, peraltro, non sono definite! disordine usa == per assegnare le variabili buca restituisce 0 o 1 indipendentemente dal tipo di buca! no more comments... GIANGRECO ELISA 27 ================== warning (formato) passaggio di array alle funzioni non ortodosso controllo ritardo richiede il passaggio di un puntatore probabilita' calcolata come rand%100+1 le probabilita' di buca non sono eslcusive GIANNANDREA GIULIA INSUFF ========================= ben 8 warning! disordine alle funzioni non passa array, ma elementi degli stessi oltre il relativo bordo! 4/60 = 0 Non T+=, ma T= no more comments... GIUJUSA LEONARDO INSUFF ======================= warning (pointer usato come intero) chiama 8000 volte la funzione buche e dopo un return 0... un controllo di una variabile non assegnata (che forse doveva essere una funzione) GRADI FRANCESCO INSUFF ====================== non compila chiama 8000 volte una funzione che si limita a generare i tre numeri che determinano la presenza di buche l'usura invece la calcola una sola volta, dopo questo ciclo il numero casuale vale sempre 0 disordine confusione massima tra probabilita' GRANATELLI GAETANO INSUFF ========================= warning (= invece di ==) grave il for e' inadatto percorre ogni tratto dx di strada per un tempo pari a quello per percorrere tutti gli 8 km un meccasnismo ultracontorto per assegnare la presenza di buche ne provoca il mancato funzionamento gia' al primo step perche' riscrive nello stesso array due numeri tra loro molto diversi e dal significato diverso per non parlare del deterioramento... GRIFFONI IRENE INSUFF ===================== il programma resta appeso all'infinito la stessa funzione e' dichiarata fuori e dentro il main un inutile ciclo porta q a passare da 8000 a 16 senza fare altro, in 500 passi lo stesso fa il tempo, in un ciclo a parte dopo di che si usano le variabili che dovrebbero contenere l'usura degli pneumantici come se contenessero dati ragionevoli, ma nessuno l'ha mai assegnato no more comments... IERANI MARCO INSUFF =================== 5 warning e 20 errori di compilazione credo costituiscano un nuovo record inutile cercare di capire cosa fa... INGELIDO PAOLO INSUFF ===================== warning (= invece di ==) grave non arriva mai... disordine ripete 8000 volte (con un for che non e' certo il ciclo adatto) un altro ciclo do fino a quando non distrugge le gomme (ecco perche' non arriva mai). trascuriamo i dettagli fini perche' se no... LIBERATI PAOLA 30 ================= il for non e' il ciclo piu' adatto, pero' almeno si esce in maniera pulita MODENESE DAVIDE 30 ================== il for non e' il ciclo piu' adatto break (se non fosse per il resto toglierei un punto per questo) NUCCETELLI ANDREA 26 ==================== warning (formato) malloc!! Per 3 elementi, poi... 100-1+1 probabilita' compresa tra 1 e 99 mancano le parentesi graffe complicata gestione del deterioramento con vari if innestati TATOLI LUCA 26 ============== segmentation fault puntatore a intero non assegnato (onde il segmentation fault) numero casuale generato tra 0 e 99 non usa parentesi graffe