[GENERALE] Regole di buona programmazione

[PGI-Bis]

Originariamente inviato da marco.r

mi ricorda qualcosa...
http://thedailywtf.com/Articles/The_...From_Hell.aspx

Gli è andata ancora di lusso. Io ho visto il "gestionale" del magazzino di una società da 200 milioni di euro di fatturato l'anno: un foglio excel usato a mo' di form.

[DanieleC88]

Originariamente inviato da PGI-Bis

Gli è andata ancora di lusso. Io ho visto il "gestionale" del magazzino di una società da 200 milioni di euro di fatturato l'anno: un foglio excel usato a mo' di form.

Che sia usato fino a questo livello mi stupisce un po', lo ammetto, ma in effetti ho visto diverse volte chi pensa che Excel sia la panacea (e, per me che ci ho dovuto mettere le mani da Java, la fonte) di tutti i mali...

[banryu79]

Originariamente inviato da marco.r

mi ricorda qualcosa...
http://thedailywtf.com/Articles/The_...From_Hell.aspx

Sono esterefatto: non tanto per il casino in se, quante per il fatto che l'autore doveva essere tedesco o giù di li (dai pochi nomi intellegibili di alcuni form).

Io tra i clienti con cui ho a che fare ho dei tedeschi: sono i più "rognosi" perchè sono di una precisione maniacale e senza una direttiva letta da un manuale non si spostano di un millimetro.
Percui sono rimasto basito leggendo 'sta roba. (Sempre ammesso che questi in realtà non avessero pure un secondo database: quello che teneva traccia della nomenclatura dei form del progetto )

[malocchio]

Originariamente inviato da gugoXX

Una buona norma e' quella di mettere le guardie ai parametri di ingresso delle funzioni.

Altrimenti il software finisce per combinare casini come questo

[Gold]

Io in questi giorni devo (purtroppo) mettere le mani su questa della roba...

Un breve stralcio per voi (Notare la finezza al primo rigo)

if (cod_tipo_uo.equals("21") || !cod_tipo_uo.equals("21")){
				if (itemComboTip.equals("")){
					tipoPtf = "";
					sottoTtipolPtf = "";
					if (cod_tipo_uo.equals("A3")|| cod_tipo_uo.equals("94")){
						if (!JFWebContextUtil.consumeFlowArea(request, "CodItem").isEmpty() && 
								JFWebContextUtil.consumeFlowArea(request, "CodItem")!= null	){
							String val = (String) JFWebContextUtil.consumeFlowArea(request, "CodItem").toArray() [0];
							val = val.substring(3,val.length());
							JFWebContextUtil.informFlowArea(request, val, "tipoRicerca");
/**/ 
			 				if (!val.equals("TUTTI")){					
								tipoPtf = val.substring(0,2);
								if (val.length()== 5){
									sottoTtipolPtf = val.substring(3,5);
								}else if (val.length()== 2){
									sottoTtipolPtf = "";
								}	
				
								if (sottoTtipolPtf.equals("")){
									JFWebContextUtil.informFlowArea(request,cod_tipo_uo +'-'+ tipoPtf, "tipoRicerca");					
								}else JFWebContextUtil.informFlowArea(request,cod_tipo_uo +'-'+ tipoPtf+'-'+sottoTtipolPtf, "tipoRicerca");
							}				
/***/							
						}	
					}else JFWebContextUtil.informFlowArea(request, "TUTTI", "tipoRicerca");
				}

Auguri a me

[banryu79]

if (cod_tipo_uo.equals("21") || !cod_tipo_uo.equals("21")){...}

E' un classico caso di pentimento repentino aggravato da fobia della cancellazione del sorgente con scappellamento a destra

[yorkeiser]

Originariamente inviato da banryu79

if (cod_tipo_uo.equals("21") || !cod_tipo_uo.equals("21")){...}

E' un classico caso di pentimento repentino aggravato da fobia della cancellazione del sorgente con scappellamento a destra

Con indentazione antani brematurata

[_Claudio]

Originariamente inviato da Gold

if (cod_tipo_uo.equals("21") || !cod_tipo_uo.equals("21")

Mio Dio ma... ma... è terribile!!!

[banryu79]

Originariamente inviato da _Claudio

Mio Dio ma... ma... è terribile!!!

No, no, stia tranquillo si guarisce subito e completamente: basta una settimana di sodomizzazione forzata con apparato scartavetrante

Comunque si vede che chi ha scritto quel sorgente non sa nulla di refactoring, con una semplificazione poteva ridurre il tutto a questa perla:

if (true != false) {...}

Almeno si salvava dalla "identazione antani brematurata"
Certo che vista in questa forma denuncia un incredibile scetticismo...

[_Claudio]

Ora mi è ancora più chiaro il perchè i lavori IT sono svalutati e considerati stupidi e inutili, e una laurea tecnica nel settore è considerata poco meno che nulla.
Finchè c'è gente magari laureata disposta ad essere pagata 900€ al mese per scrivere certe boiate.

Almeno lo facessero gratis, potrebbero presentarsi e dire:"io sono laureato in ..., ho studiato anni ma comunque non so un'emerita ceppa di quello che fate, non sono in grado di comprendere neppure sotto tortura, scriverò del codice pessimo dettatomi dal pupazzo gnappo che tengo nella mia 24 ore, ma lo farò gratis per dignità"
Noi "professionisti seri" vivremmo tutti più felici (e ricchi).

[Frank1962]

Il mio professore di Linguaggi & Compilatori al 2° semestre del 3° anno della triennale ci disse " .....ma cosa credete? iscriversi a un corso di laurea di informatica non serve a diventare dei buoni programmatori, per questo basta comprarsi un manuale...." ....ancora devo capire a che cosa serve la famigerata laurea in informatica, ma non me lo poteva dire al 1° semestre del 1° anno sta roba

[cdimauro]

Originariamente inviato da _Claudio

Mio Dio ma... ma... è terribile!!!

Macché terribile. Ci si lamenta sempre della teoria, preferendole la pratica, e una volta tanto che si vede applicata una tautologia si grida allo scandalo.

[_Claudio]

Originariamente inviato da cdimauro

Macché terribile. Ci si lamenta sempre della teoria, preferendole la pratica, e una volta tanto che si vede applicata una tautologia si grida allo scandalo.

Dici bene, chi ha scritto quell'accrocchio sicuramente ha voluto dimostrare proprio questo...

[_Claudio]

Chi ci dice invece che invece vale la logica di Lukasiewicz nel valutare la equals e quindi quell'enunciato porta informazioni?

Mai assumere il mondo chiuso, e mai basarsi sul FOL per descrivere il mondo intero...

tsè, principianti

[gugoXX]

(ammesso che si possa fare) Il massimo sarebbe un overload della .equals(), che cambia il valore durante il primo confronto, oppure che esegue qualcosa di "importante".

[PGI-Bis]

E' chiaramente un codice destinato ad essere eseguito in un contesto concorrente! L'if viene eseguito se e solo se un Thread cambia il valore di cod_tipo_uo esattamente tra il primo ed il secondo if.

EDIT: No, c'è un || e io ho letto &&. E' l'età.

EDIT 2: Non viene eseguito se un Thread cambia il valore tra il primo e il secondo if.

[malocchio]

Originariamente inviato da PGI-Bis

E' chiaramente un codice destinato ad essere eseguito in un contesto concorrente! L'if viene eseguito se e solo se un Thread cambia il valore di cod_tipo_uo esattamente tra il primo ed il secondo if.

La nuova frontiera della sincronizzazione

[banryu79]

Lo ripeto e lo ribadisco: è solo scetticismo.
Giustamente chi gli garantisce la validità del principio "Tertium non datur"? Non si sa mai...

[PGI-Bis]

Originariamente inviato da banryu79

Lo ripeto e lo ribadisco: è solo scetticismo.
Giustamente chi gli garantisce la validità del principio "Tertium non datur"? Non si sa mai...

In effetti l'introduzione del booleano "maybe" lo troverebbe preparato.

[khelidan1980]

Originariamente inviato da _Claudio

Ora mi è ancora più chiaro il perchè i lavori IT sono svalutati e considerati stupidi e inutili, e una laurea tecnica nel settore è considerata poco meno che nulla.
Finchè c'è gente magari laureata disposta ad essere pagata 900€ al mese per scrivere certe boiate.

Almeno lo facessero gratis, potrebbero presentarsi e dire:"io sono laureato in ..., ho studiato anni ma comunque non so un'emerita ceppa di quello che fate, non sono in grado di comprendere neppure sotto tortura, scriverò del codice pessimo dettatomi dal pupazzo gnappo che tengo nella mia 24 ore, ma lo farò gratis per dignità"
Noi "professionisti seri" vivremmo tutti più felici (e ricchi).

Guara imho quelle perle di cui sopra sono spesso eredità di codice legacy risalente agli anni del "boom" dove si prendeva tizio qualunque e lo si metteva a fare lo sviluppatore dopo un corso java di sei mesi ad andar bene...