|
Dopo tanto penare son finalmente riuscito ad arrivare in fondo alla riorganizzazione del codice del mio "Codificatore". Come l'altra volta apprezzerei e prenderei in seria considerazione i suggerimenti delle anime pie che volessero darmene.
Sperando di fare cosa gradita a chi mi ha GIA' consigliato, ci tengo a riassumere in che modo i contributi che ho ricevuto mi hanno spinto a modificare il programma. E' un po' una "lenzuolata" di testo, per cui capirei se la saltaste a pie' pari.
Ho riorganizzato il “Codificatore” secondo alcuni dei suggerimenti che gentilmente avete voluto darmi. Altri suggerimenti li ho considerati, ma mi han dato l’idea di non essere particolarmente vantaggiosi: chi me li ha dati mi tiri pure pubblicamente le orecchie se ritiene che io abbia preso una cantonata agendo così.
“Utilizzare un po' gli oggetti” (marco_) – fatto! Ora la parte “operativa” del codificatore è implementata nella forma della classe CDF.
“Evitare variabili globali” (marco_) – fatto, almeno in parte. Le uniche variabili globali residue sono alcuni handle a elementi di interfaccia e un buffer di char.
“Evitare funzioni con nomi eccessivamente lunghi” (marco_) – per il momento ho preferito conservare i nomi lunghi perché in fase di sviluppo m’aiuta ad orientarmi meglio.
“Mischiare codice dipendente dal SO con l'algoritmo è orribile” (TheDarkJuster) – non succede più! La classe CDF usa solo funzioni e tipi standard.
“Evitare problemi con malloc/free” (marco_) – fatto! Ho completamente “raschiato via” ogni chiamata a malloc() e a free(), sostituendole con new.
“Usare std::string anzichè char*” (TheDarkJuster) – Ho controllato nella documentazione della libreria standard le funzionalità apportate dalla classe stringa. Devo dire che nel contesto di un programmino tanto semplice m’è sembrato inopportuno “caricare” una botta di codice del genere per usare giusto un paio di quelle funzionalità. Ho preferito mantenere la più classica gestione tipo array del C. Ho fatto lo stesso ragionamento per la classe vector – come sparare ai passeri con un cannone, nel caso del mio piccolo codificatore.
“Non serve specificare il tipo delle eccezioni lanciate dai metodi” (TheDarkJuster) – Fatto! Avevo inserito quella caratteristica più che altro perché volevo che un eventuale utilizzatore della classe CDF potesse essere immediatamente al corrente delle eccezioni lanciate senza avere dubbi. In effetti, credo proprio che lo stesso scopo di ottenga anche con la soluzione attuale: dei semplici commenti nel file header cdf.h.
“Personalmente ti sconsiglio di gestire i file direttamente in quella classe” (TheDarkJuster) – Fatto! La classe “lavora” esclusivamente su stringhe di caratteri.
“Normalmente i metodi e le proprietà pubbliche si scrivono con le iniziali maiuscole (ecc.)” (Roby94). Ci ho pensato parecchio, ma ho deciso di rimandare per la stessa ragione per la quale ho mantenuto i nomi lunghi: siccome mi sono abituato così, mi resta più semplice ricordare le cose e ragionarci su con un metodo che mi è ormai famigliare. Una volta finito tutto è probabile ch’io segua gli standard che mi consigli.
“Lasciare che sia la classe a decidere se codificare o decodificare non mi sembra un ottima idea” (Roby94) – fatto e non fatto al tempo stesso. La classe ha un’unica funzione elabora(), ma questa opera in codifica o in decodifica a seconda del tipo di dato che è stato passato in fase di inizializzazione/aggiornamento. Se è stata passata una stringa classificata come testo in chiaro, elabora() agisce come una funzione di codifica; se è stata passata una stringa classificata come testo in codice, elabora() agisce come una funzione di decodifica. Del resto, non avrebbe senso tentare di decodificare un testo già in chiaro, così come non avrebbe senso tentare di codificare un testo già in codice. Sta al chiamante specificare che tipo di testo sta fornendo alla classe. La classe, dal canto suo, marca come “testo in chiaro” il testo risultante da una decodifica, e come “testo codificato” quello risultante da una codifica. C’è, insomma, un minimo di automatismo che a me sembra piuttosto razionale.
ATTENZIONE! Sono un hobbista e l'affidabilità delle mie conoscenze informatiche è molto limitata. Non prendere come esempio il codice che scrivo, perché non ho alcuna formazione accademica e rischieresti di apprendere pratiche controproducenti. | 
