Newsgroup: it.comp.lang.c++
Titolo: [OT]C++ Utils

Chi vuole dare un occhiata alla mia lib??
Il link č in firma.
Se avete commenti, ben vengano :)
Tnk
-- 
Kleidemos
C++ Programmer
kleidemo...@altervista.org
http://kleidemos.altervista.org/

Nessun commento???


 
Ehi, oggi e' l'ultimo dell'anno... lascia alla gente il tempo di 
divertirsi. Poi il tempo di riprendersi :-P
Vedrai che qualche commento prima o poi arrivera' nei prossimi giorni.
-- 
Roal Zanazzi

Tnk
-- 
]\/[ad]\/[age (
www.mindreamz.net
)
"Un Maestro insegue semplicemente la sua visione di Eccellenza in
qualunque cosa sta facendo e lascia agli altri decidere se sta 
lavorando o giocando. A lui sembra sempre di fare entrambe." - Anonimo

 
Poca roba (le ultime parole famose...) :-)
- i testi della pagina web hanno qualche imprecisione (lo so, lo so, 
sono un rompi balle...)
es. " It is usable, modificable and redistribuible free."
lo scriverei cosi': "It is freely usable, modifiable and redistributable"
A &copy 2003 credo manchi un punto e virgola.
"multicharter" -> "multicharacter"
Cosa vuol dire "Improved of 50% searching (now using map<string, 
string>)" ? Dato che il miglioramento non e' sempre il 50% (puo' essere 
di piu' o di meno, dipende anche dal numero di elementi nel container), 
io starei piu' generico.
- un nome piu' _significativo_ per il file zip non starebbe male.
- Personalmente preferisco l'estensione .h, invece di .hpp, ma non vado 
oltre, questo e' campo da guerra di religione ;-)
- Ma porc! Ecco cosa dovevo fare in questi giorni: reinstallare Visual 
Studio e C++Builder!!!! Mi sono completamente dimenticato... vabbe', vai 
di notepad.
- Come ho gia' avuto modo di scriverti, l'uso delle macro N_BEG e N_END 
per il namespace non mi piacciono (cosi' potresti togliere, per ora, 
define.hpp)
- "Utils" e' un po' troppo generico come namespace. Credo che ci sia 
qualche migliaio di programmatori C++ nel mondo che ha un suo namespace 
Utils, utils, o simile. Perche' non "Kleidemos"?
- using namespace Utils; in utils.hpp e' ... da maleducati. ;-)
Altamente consigliato: toglierlo.
- Mat.hpp:
Manca il namespace std!
Perche' qui nomi e commenti sono in italiano e altrove in inglese? Se 
vuoi che questa libreria sia usata in giro per il mondo, meglio usare 
l'inglese.
"alpha" in converti() e' un char[], mentre in convertiBeta10() e' una 
string, forse meglio unificarli.
Nel controllo if(base > alpha.size()) io restituirei un'eccezione e nei 
commenti/doc della funzione scrivi che la base puo' essere un valore 
fino a TOT.
- Str.hpp:
filter_text(), perche' il parametro "words" e' un puntatore? Meglio un 
reference.
Anche string ha bisogno di std::, come vector.
Potresti provare ad usare qualche algorithm (es. remove_if() )...
- Parse.hpp:
Toglierei tutti quegli "using"...
Nei costruttori usa le initializer list:
   CParse(){ _filename = "prog.conf"; };
   ->
   CParse(): _filename("prog.conf") {};
Non ho molto capito come dovrebbe essere usata la classe CParse,
perche' Read() legge da file nella map e Write scrive su file, ma non 
nella map?
In Find() NON usare l'operator[] della std::map!!! Se non esiste un 
elemento con la chiave specificata, ne viene inserito uno! RTFM!
Altre eventuali:
- un programmino demo che faccia vedere "sul campo" le feature della 
libreria non sarebbe male, anche semplice, da console.
Direi che per ora basta :-P
-- 
Roal Zanazzi

 io starei piu' generico.
Concordo.
Inoltre, lasciando stare la "semantica della frase" dovrebbe essere qualche
cosa tipo:
"Searching is now 50% faster"
oppure
"50% performance improvement when searching"
oppure
"Improved searching"
almeno queste suonano un tantino meglio.
sempre della serie, siamo rompiballi ;-)  e... OT ;-)
Tommy