D'Alessandro Matteo 30 ---------------------- La probabilita' era meglio definirla come costante due return in una funzione non dovrebbero esserci D'Alvito Renata 29 ------------------ Definisce array di taglia non costante un po' di disordine inutili if (poco ottimizzato) La funzione di stampa non dovrebbe essere responsabile di acquisire il nome del file non si mette un exit in una funzione D'Ascenzi Riccardo 29 --------------------- Un po' troppe variabili inutili nAtoms, nAtoms2, nAtoms3, ... due return in una funzione La funzione di stampa non dovrebbe essere responsabile di acquisire il nome del file De Angelis Camilla 30 --------------------- poco ottimizzato (inutile if e inutile uso di puntatori per array) De Feo Simone 29 ---------------- warning in compilazione (& davanti a una stringa) non c'e' un messaggio che aiuti l'utente a capire cosa ha sbagliato nell'input il for non e' la struttura piu' adatta per controllare la fine del programma due return in una funzione Del Castello Giorgio 28 ----------------------- Un po' troppe variabili inutili inutilissimi if nel ciclo che controlla il decadimenti inutile salvataggio degli atomi decaduti inutile uso di puntatori per navigare nell'array Delicato Daniele 30 ------------------- warning (implicit close) due return in una funzione close invece di fclose De Marco Francesco 29 --------------------- warning in compilazione (time non definita) non ottimizzato (inutili if) inutile uso di puntatori invece di array De Matteis Edoardo 21 --------------------- 3 warning in compilazione (unused expressions) infatti non finisce mai non c'e' un messaggio per l'utente se sbaglia input definisce array di taglia variabile il for non e' la struttura adatta a controllare l'uscita dal programma ed e' anche scritto male non aggiorna il contenuto degli array (aggiorna l'indice) scrive su file a ogni iterazione genera N numeri casuali nella funzione che dovrebbe estrarne uno De Mei Vincenzo 27 ------------------ Non termina la stringa contenente il nome del file e il nome risulta composto di caratteri a caso exit sovrabbondanza inutile di variabili poco ordinato poco ottimizzato (if inutili e un else vuoto) usa un secondo array in cui mette i dati del primo senza alcun motivo De Paoli Adriano 25 ------------------- I dati sono scritti in un file di nome diverso da quello indicato dall'utente (.txt) La funzione che provoca il decadimento non rispetta le prescrizioni Non funziona affatto come dovrebbe: al massimo decade un atomo al secondo Desideri Marcantonio 28 ----------------------- L'i-- nel for conta doppio il numero di atomi decaduti due return in una funzione uso sbagliato del puntatore per navigare l'array De Simoni Marta 28 ------------------ n1, n2, n3, .... ma quante variabili servono? due return in una funzione la funzione che scrive non deve avere anche la responsabilita' di acquisire il nome del file De Tommasi Marco 29 ------------------- disordinato poco ottimizzato (un inutile if - bastava seguire le prescrizioni per evitarlo) Di Fruscia Lorenzo 28 --------------------- prima conta i decadimenti con un inutile if e poi li sottrae dal campione (!) inutile if dopo un else uso di puntatori sbagliato per navigare in array Di Giulio Antonio 20 -------------------- warning in compilazione (return ignorato) loop infinito + segmentation fault malloc? attende davvero un secondo prima di procedere al decadimento! poco ottimizzato (inutile if e conta i decadimenti da sottrarre poi dal numero di atomi) realloc? due return in una funzione Di Pace Fabrizio 30 ------------------- due return iin una funzione l'if dentro il loop per l'acquisizione del nome del file praticamente non fa nulla Di Pasqua Antonio 30 -------------------- due return in una funzione uso inopportuno di puntatori per navigare un array Domeniconi Siria 18 ------------------- non compila, vari errori dichiara le funzioni nel main (gravissimo!) inutile if nel decadimento due return in una funzione la funzione di stampa non deve richiedere anche il nome del file Dunlop Ryan william 29 ---------------------- una certa sovrabbondanza di variabili x = x - a invece che x-=a due return in una funzione uso inopportuno di puntatori per navigare un array Ercolino Andrea 24 ------------------ warning (time non dichiarata) segmentation fault dichiara un array con il risultato di una funzione! in pratica simula due volta il decadimento con una logica poco comprensibile Falzetti Andrea 28 ------------------ warning (time non dichiarata) il for non e' la struttura adatta poco ottimizzato (if inutile) la funzione di scrittura assume una taglia fissa per l'array che scrive tutto Faraone Andrea INSUFFICIENTE ---------------------------- non compila non e' consentito piu' di un tentativo d'inserimento dati corretti non fa scegliere il nome del file La simulazione consisterebbe nell'estrazione di 20 numeri casuali! no comment per il resto Ferrante Giacomo 29 ------------------- warning (time non dichiarata) poco ottimizzato (inutile if) due return in una funzione Ferrante Matteo 22 ------------------ un po' di disordine dichiara array di lunghezza implicita, ma con un solo elemento la variabile di controllo del ciclo e' usata come un indice dell'array, ma e' una cosa diversa riusa t che e' riassegnata per poi essere subito cancellata dal ciclo (stupisce che non ci sia un segmentation fault) Infatti nel file di uscita non c'e' nulla Ferrara Annalara 25 ------------------- N=N-atomi_decaduti disordinato non usa costanti l'estrazione del numero casuale la fa usando float che trasforma in interi passa &atomi_sopravvissuti[0] invece di limitarsi a passare atomi_sopravvissuti uso sbagliato del puntatore per navigare l'array Ferri Annachiara 27 ------------------- non compila (conflicting type per una funzione) + 1 warning disordine due return in una funzione Fiacco Davide 27 ---------------- non usa costanti balletto tra variabili (bastava usarne una invece di tre) uso improprio di puntatori per navigare array Fiorillo Matteo 26 ------------------ warning (scanf("%s, &nome)) Gli atomi non decadono mai (per forza, perche' decadono solo quando non ci sono!) una costante in piu' (19) e una in meno (0.001) sarebbe stato meglio Foglietta Marco INSUFFICIENTE ----------------------------- warning (campione[0];) int campione[] = {n}; dovrebbe essere l'assegnazione di n a campione chiama prob(); per poi fare if(prob() == 1) campione[0] = campione[0] - 1; Alla funzione di stampa passa campione[0] (che del resto e' l'unico elemento "usato" nell'array) Se a non e' maggiore di 19, controlla se a sia maggiore di 19 La scrittura su file non esiste Foglietti Agnese INSUFFICIENTE ------------------------------ segmentation fault campione non e' inizializzata nel ciclo del decadimento si sottrae uno a campione (con logica inversa rispetto a quella indicata) quindi si memorizza in un array N-campione (che a questo punto puo' anche essere negativo) e poi si ricontrolla campione per decudere se il ciclo e' finito l'array ha tante componenti quanti sono gli atomi ogni componente e' riscritta a ogni ciclo di decadimento se il numero e' minore di 19 controlla se sia maggiore di 19. Se poi vale 19 non si sa... Franciamore Samuele 30 ---------------------- non troppo ordinato l'fclose dovrebbe stare nella porzione in cui il file e' stato aperto Francioni Mirko INSUFFICIENTE ----------------------------- segmentation fault decade un atomo al secondo? la funzione di decadimento richiede un parametro (?) dichiata array di taglia variabile in un array il cui indice e' l'atomo mette il risultato della funzione di decadimento la funzione di decadimento esatre un numero random compreso tra 0 e 101 che si confronta con il numero di atomi sopravvissuti (parametro) moltiplicato per 19 e si cerca di verificare che N<19*N, il che' ovviamente, e' sempre vero tranne che per N=0. non sa aprire un file di testo Frosina Lorenzo INSUFFICIENTE ----------------------------- se uno sbaglia l'inserimento non sa che deve ripeterlo poi non finisce mai la funzione di decadimento prevede un parametro (numero di atomi) sia nel caso di un decadimento che nel caso in cui questo non sia avvenuto esegue le stesse operazioni il numero di atomi no diminuisce mai perche' si tolgono gli atomi decaduti (o meglio che dovrebbero essere decaduti) sempre al campione iniziale e non a quello rimasto inutile funzione per estrarre un numero casuale genera n volte un deacdimento, ma alla fine usa solo l'ultimo risultato la funzione di scrittura e' usata anche per acquisire il nome del file, ma all'utente non viene chiesto Galbato Gabriele 30 ------------------- if di troppo la funzione di scrittura non dovrebbe chiedere il nome del file Galizia Nicolo' 18 ------------------ warning (format) in caso di errore bisogna reinserire tutti i dati non direi che decade nulla o quasi riassegna il puntatore a file ogni volta che si reinserisce il nome t=t+1, N=N-1, j=j+1 assegna a ogni componente di atoms il valore di N sottratto di uno tenta di scrivere su file a ogni iterazione non lo fa per tutti gli atomi, ma solo per uno per volta alla funzione di scrittura passa solo un valore per volta inutile controllo del valore casuale per decidere il decadimento (inutile perche' inutile la sua definizione) Garcia Lucia 30 --------------- if di troppo la funzione di scrittura non dovrebbe chiedere il nome del file Ghimenti Federico 30 -------------------- if di troppo due return in una funzione non ordinatissimo Ghio Davide 30 -------------- La funzione di scrittura su file e' definita con un parametro che e' un array di lunghezza fissa qualche variabile di troppo la funzione che scrive il file ne chiede anche il nome Giammei Marco 30 ---------------- warning (time non dichiarata) Giordo Virginia 30 ------------------ warning (non void senza return) non comunica all'utente cosa ha sbagliato, se l'input non e' corretto inutile if per decidere il decadimento due return (e due if alternativi, senza else) in una funzione Giujusa Leonardo INSUFFICIENTE ------------------------------ warning (= invece di ==) memorizza una stringa in un puntatore a file fopen("%d" se e' nullo dice che c'e' un errore, ma poi va avanti tutto il lavoro sta in una funzione (di nome inopportuno fun) variabili non inizializzate usate in cicli while uso di = e == tutto sbagliato Granatelli Gaetano INSUFFICIENTE -------------------------------- non compila apre il file in lettura invece che in scrittura segnala eventuali problemi nell'apertura del file, ma poi va avanti chiama una funzione, che per argomento avrebbe un array, mettendo come argomento un'altra funzione (!) genera un inutile array di numeri casuali, ma restituisce uno solo di essi (l'ultimo) usa questo valore in modo del tutto inappropriato dentro un for vuoto return scriteriati Gravina Luca 30 --------------- inutile if nel decadimento Graziotto Lorenzo 30 -------------------- inutile if nel decadimento inutile uso della funzione casuale Griffoni Irene INSUFFICIENTE ---------------------------- warning (= e ==) incomprensibile ciclo dopo input non c'e' di fatto alcun decadimento (ci sarebbe una funzione mai invocata) questa funzione genera n numeri casuali ciascun numero e' moltiplicato per 0.19 poi il numero e' sostituito da un numero x che e' un parametro della funzione (!) Hasanuzzamankamrul Venus 29 --------------------------- inutile if dopo else I decadimenti avvengono a blocchi di 19%: a ogni istante o non decade alcun atomo oppure ne decade il 19% La funzione di stampa non dovrebbe chiedere il nome del file Ierani Marco INSUFFICIENTE -------------------------- warning (formato) fa lo scanf del puntatore a file esegue il ciclo 100 volte invoca una funzione senza parentesi, trattandola come una variabile se avesse usato bene la funzione, in ogni caso avrebbe tolto un atomo dal campione in assenza di decadimenti e il 19% di essi se ci fossero stati la variabile che avrebbe dovuto tener conto di questi e' diversa nei due casi apre il file con nome fissato e con parentesi acute e ci scrive un solo numero Illiani Giulia 30 ----------------- una funzione per leggere una stringa pare eccessiva anche altre funzioni un po' superflue Isopi Giovanni 30 ----------------- rispettando le prescrizioni si poteva evitare l'if nel decadimento Iuliano Maria Grazia 30 ----------------------- come argomento della funzione stampa usa un array di taglia fissa Modenese Davide 25 ------------------ warning (format) segmentation fault malloc()!! alloca un intero non rispetta le prescrizioni sulle parentesi alcuni if inutili rialloca N volte la memoria due return nella stessa funzione la funzione che scrive il file non dovrebbe chiederne il nome il controllo con strlen e' impossibile