Progetto opensource JGush

[andremar]

Assolutamente ribadisco che apprezzo il tuo commento, anche perche mi hai fatto capire che un progetto viene considerato solamente se finito e quindi usabile altrimenti non viene considerato... Ti chiedo solo una cose, perche dici che è scritto male? (Vorrei capire meglio cosa devo migliorare)

Ok sto esagerando con i toni quindi comincio a scusarmi.
Ora ti faccio qualche osservazione tecnica (premetto che ho una certificazione Java dal 1999 e una certificazione JavaEE enterprise architect nel 2005) quindi dovrei essere qualificato per farle.

- i commenti sono random, un po' si uno po' no - fatti cosi sono assolutamente inutili

- la divisione in bean e dao la capisco, quella in widget e label no - la label alla fine e' uno specifico widget

- usi un sacco di inner classes, e questo rende difficile la navigazione del codice - in java una inner classes deve essere propriamente "inner" tu invece usi una classe come fosse un package. L'intero builder dovrebbe essere un package

- la classe widget prende una servlet request, che e' assolutamente atroce per l'incapsulazione (mai sentito parlare del principio di liskov?)

- i messaggi sono cablati (orrore!) e in italiano (peggio!) - almeno usa un resource bundle

- WidgetMethod (grande! una classe method...) con un metodo execMethod!!! Mamma! Manca solo MiaClasse con i metodi pippo e pluto?

- build_config viola le naming convention standard di Java

- Version(3,2,0)??? Sti numeri vanno nel build file di maven, non nel codice

- Ah gia, non ci sta la build di maven

- Uh, non e' marcata manco la versione

- Oh, e dove sta il binario per scaricare e provarlo?

- E perche' 3 progetti, uno per il demo una per il codice e uno per la documentazione (finta...)? Per fare aria? Riempire il tuo account github di progetti che uno solo sembra troppo poco? Dovrebbe star tutto in un solo progetto.

Note sul codice:

- short compileTemplate che ritorna -1 per l'errore! credevo fosse Java, non C (eccezioni mai sentite?)

- tra l'altro se ti fossi mai letto bene il manuale di java sapresti che uno short viene immediatamente promosso a int quindi e' assolutamente inutile ritornare da un metodo short - non risparmi nemmeno un bit fai solo vedere che non conosci bene java

- questa e' carina: if( null==method || "".equals( method ) )
Ora, gli esperti usa di solito "".equals(method) invece di method.equals("") per eveitare il null pointer quando method e' null. Qui tu usi la tecnica degli esperti ma si vede che no la hai capita perche hai messo anche il controllo (del tutto inutile) null==method.

Mi fermo qui che potrei continuare per ore ...

[Rikim]

- questa e' carina: if( null==method || "".equals( method ) )
Ora, gli esperti usa di solito "".equals(method) invece di method.equals("") per eveitare il null pointer quando method e' null. Qui tu usi la tecnica degli esperti ma si vede che no la hai capita perche hai messo anche il controllo (del tutto inutile) null==method.

con

if(null==method || "".equals(method))

si può generare un eccezione per essere sicuri che la stringa method sia valida (ad esempio in un metodo) e scrivere "".equals(method) o method.equals("") in questo caso è indifferente!

esempio:

public void pippo(String method){
     if(null==method || "".equals(method))
           throw new RuntimeException("method == null or is \"\"");
     
     //. . .
}