[concettuale]scrittura di un metodo

[RaouL_BennetH]

Ciao a tutti :)

Avrei da risolvere un problemuccio che però credo di non riuscire ad inquadrare.

Su un'ipotetica griglia, ho 5 celle; La quinta cella deve darmi un risultato che è la differenza di alcuni valori.

Le mie possibili condizioni sono:

1) Ho solo le prime due celle che contengono un valore e quindi la quinta cella deve darmi la differenza di: cella2 - cella1

2) Ho solo la terza e quarta cella che contengono un valore e quindi la quinta cella deve darmi la differenza di: cella4 - cella1

3) Ho tutte e quattro le celle che contengono un valore e quindi la quinta cella deve darmi: (cella2 - cella1) + (cella4 - cella3)

Questo problema l'ho risolto con una serie infinita di "if - else", ma non mi piace assolutamente. Dato che è un'operazione che devo svolgere spesso su questo programma, intuisco che devo astrarre il concetto in maniera migliore.

Come posso fare?

Grazie mille.

RaouL.

[okay]

Ciao a tutti :)

Avrei da risolvere un problemuccio che però credo di non riuscire ad inquadrare.

Su un'ipotetica griglia, ho 5 celle; La quinta cella deve darmi un risultato che è la differenza di alcuni valori.

Le mie possibili condizioni sono:

1) Ho solo le prime due celle che contengono un valore e quindi la quinta cella deve darmi la differenza di: cella2 - cella1

2) Ho solo la terza e quarta cella che contengono un valore e quindi la quinta cella deve darmi la differenza di: cella4 - cella1

3) Ho tutte e quattro le celle che contengono un valore e quindi la quinta cella deve darmi: (cella2 - cella1) + (cella4 - cella3)

Questo problema l'ho risolto con una serie infinita di "if - else", ma non mi piace assolutamente. Dato che è un'operazione che devo svolgere spesso su questo programma, intuisco che devo astrarre il concetto in maniera migliore.

Come posso fare?

Grazie mille.

RaouL.

fai dei cicli for:

bool ok=false;
a=0
for i=0 i<5 i++
if(cella[i]>0)
a++;

if(a==4)
ok=true;//hai tutte le 4 celle con un valore quindi calcoli
if(ok){
cella[5]=(cella2 - cella1) + (cella4 - cella3)
}else{
for i=0 i<5 i++
for j=i+1; <5 i++
if(cella[j]>0 && cella[i]>0)
cella[5]=cella[j]-cella[i];
}


perlomeno per quanto hai scritto...

un'appunto è che sull'else cella[5] viene sovrascritto... per ora!... se i valori di cella i e j sono >0

[RaouL_BennetH]

più o meno corrisponde all'idea che mi ero fatto, solo che non so perchè non mi convince :(

Attualmente ho abbozzato così:




private void GetTotal(MyGridCellEventArgs e)
{
DateTime dt1, dt2, dt3, dt4;
bool b1, b2, b3, b4;


//prima condizione, le prime due celle contengono un valore e le altre no
if(grid.Rows[e.RowIndex].Cells[0].Value != null && grid.Rows[e.RowIndex].Cells[1].Value != null)
{
b1 = DateTime.TryParse(grid.Rows[e.RowIndex].Cells[0].Value.ToString(), out dt1);
b2 = DateTime.TryParse(grid.Rows[e.RowIndex].Cells[1].Value.ToString(), out dt2);

TimeSpan ts = dt2.SubTract(dt1);
grid.Rows[e.RowIndex].Cells[4].Value = ts.ToString();
}

//seconda condizione: solo la terza e quarta cella contengono un valore
if(grid.Rows[e.RowIndex].Cells[2].Vale != null && grid.Rows[e.RowIndex].Cells[3].Value != null)
{
b3 = ....
b4 = ....
TimeSpan ts = .....
}

//terza condizione: tutte e quattro le celle contengono un valore
{
b1 = .... out dt1;
b2 = .... out dt2;
b3 = .... out dt3;
b4 = .... out dt4;

TimeSpan ts1 = dt2 - dt1;
TimeSpan ts2 = dt4 - dt3;
TimeSpan total = ts2 + ts1;
}
//end of horror code pretty candidate to the daily wtf

[cionci]

Fai un metodo che ti fa la differenza fra due celle:

TimeSpan GetSubTotal(MyGridCellEventArgs e, int first, int second)
{
DateTime dt1, dt2;
if(grid.Rows[e.RowIndex].Cells[first].Value == null || grid.Rows[e.RowIndex].Cells[second].Value == null)
return new TimeSpan(0); //non so come esprimerlo, ritorna un time span in cui la differenza è zero
DateTime.TryParse(grid.Rows[e.RowIndex].Cells[first].Value.ToString(), out dt1);
DateTime.TryParse(grid.Rows[e.RowIndex].Cells[second].Value.ToString(), out dt2);

return dt2.SubTract(dt1);
}

private void GetTotal(MyGridCellEventArgs e)
{
TimeSpan total = GetSubTotal(e, 0, 1) + GetSubTotal(e, 2, 3);
grid.Rows[e.RowIndex].Cells[4].Value = ts.ToString();
}

Potrebbe andare ?

[RaouL_BennetH]

Potrebbe andare ?

Ecco, questo si che significa leggere nel pensiero ! :)

Si, credo che come concetto sia esattamente ciò che cercavo !


Grazie mille :)

RaouL.

[MEMon]

Ma se fai un metodo per leggere il valore della cella e uno per farne il totale non fai prima?:

int getCellValue(int n){
int value=grid.Rows[e.RowIndex].Cells[n].Vale
if(value!=null) return value;
else return 0;
}

int doCalc(int cell1, int cell2, int cell3, int cell4){
return (cell2-cell1)+(cell4-cell3);
}


Non va bene?
Corrisponde a quello che hai detto.

[cionci]

Avevo lasciato due != ;)

E' una specie di NullObject pattern ;)

[MEMon]

C'è qualcosa che mi sfugge...io non vedo la difficoltà mi sa.

[cionci]

C'è qualcosa che mi sfugge...io non vedo la difficoltà mi sa.
Guarda, io ho fatto circa la stessa cosa che hai fatto te...soltanto che il contenuto delle celle non è un int ;)

[MEMon]

aaaaaa ecco

[RaouL_BennetH]

Ma se fai un metodo per leggere il valore della cella e uno per farne il totale non fai prima?:

int getCellValue(int n){
int value=grid.Rows[e.RowIndex].Cells[n].Vale
if(value!=null) return value;
else return 0;
}

int doCalc(int cell1, int cell2, int cell3, int cell4){
return (cell2-cell1)+(cell4-cell3);
}


Non va bene?
Corrisponde a quello che hai detto.

Se fossero interi o double o float, andrebbe bene anche così.

Il fatto è che i valori che ho sono di tipo DateTime e per fare le differenze o addizioni necessito di un oggetto di tipo TimeSpan, il quale, non ammette lo stesso tipo di aritmetica dei tipi classici.

[MEMon]

Sisi ho capito, magari se lo specificavi anche nel primo post lo capivo prima :)

[RaouL_BennetH]

Sisi ho capito, magari se lo specificavi anche nel primo post lo capivo prima :)

:( :ops:

[MEMon]

:( :ops:

noooooo spero che non mi hai frainteso, volevo dire che io parto subito per la tangente, mi sono poi accorto dopo che ti serviva un qualcosa di concettuale! :p

[^TiGeRShArK^]

EDIT
non avevo capito una mazza :D