[C#] - Cappottare un ciclo

[gugoXX]

do
{
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
if (pippo != null){
FaiQualcosaconPippo(pippo);
}
DataLayer.Commit();
} while (pippo!=null);


E' possibile riscriverlo facendo il confronto
pippo!=null
una volta sola?

[0rph3n]

ecco la mia ennesima cazzata :sofico:

DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
while (pippo!=null)
{
FaiQualcosaconPippo(pippo);
DataLayer.Commit();

DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
}
DataLayer.Commit();

[kernel::panic]

Così ti eviti di fare 2 volte il controllo, ma sono cmq più righe di codice:

while(true)
{
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
try
{
if(pippo == null)
break;
else
FaiQualcosaconPippo(pippo);
}
finally
{
DataLayer.Commit();
}
}


Ciao ;)

[gugoXX]

ecco la mia ennesima cazzata :sofico:

DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
while (pippo!=null)
{
FaiQualcosaconPippo(pippo);
DataLayer.Commit();

DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
}
DataLayer.Commit();



Grazie.
Pero' mi sembra un po' come il tagliarsi una gamba perche' ho un unghia incarnita :D:D
Scherzo, purtroppo non mi sembra piu' elegante.

[gugoXX]

[QUOTE=kernel::panic;21629967]Così ti eviti di fare 2 volte il controllo, ma sono cmq più righe di codice:

while(true)
{
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
try
{
if(pippo == null)
break;
else
FaiQualcosaconPippo(pippo);
}
finally
{
DataLayer.Commit();
}
}

Grazie anche a te.
Anche questa purtroppo non mi sembra guadagni in leggibilita'.
Mi sa che una riposta che mi piaccia davvero non si riesca a trovare.
Facciamo cosi'. Sposto il controllo
if (parametro!=null)
dentro la funzione, che tanto ci sta anche bene perche' mi e' stato segnalato come assente dai test automatici.
Me lo tengo cosi'


do
{
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
} while (pippo!=null);

...

public void FaiQualcosaconPippo(parametro)
{
if (parametro==null) return;
// continua
}

[cionci]

while(true)
{
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
if (pippo == null){
DataLayer.Commit();
break;
}
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
}

[gugoXX]

...

Grazie.
Ci devo pensare.
Vedere 2 Commit per una BeginTransaction non mi esalta

[cionci]

Capisco...così ?
while(true)
{
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
if (pippo == null){
break;
}
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
}
DataLayer.Commit();

[cionci]

Anche così:

DataLayer.BeginTransaction();
while(true)
{
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
if (pippo == null){
break;
}
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
DataLayer.BeginTransaction();
}
DataLayer.Commit();
o così:

DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
while(pippo != null)
{
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
}
DataLayer.Commit();

[0rph3n]

o così:

DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
while(pippo != null)
{
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
}
DataLayer.Commit();

copione :Prrr: :ciapet:

[cionci]

copione :Prrr: :ciapet:
Non avevo visto :D

[gugoXX]

Facciamo cosi'.
Le metto tutte e 4.

Poi genero un numero casuale di volta in volta e vado ad eseguire la corrispondente.

[cionci]

Poi genero un numero casuale di volta in volta e vado ad eseguire la corrispondente.
:asd:
Comunque a parte gli scherzi, la migliore è la tua con l'if spostato nella funzione ;)

[morskott]

sparo la mia try{
while (true){
DataLayer.BeginTransaction();
RispostaDaDatabase pippo = DomandaAlDatabase(parametri);
if (pippo==null) throw new MiaException(ex);
FaiQualcosaconPippo(pippo);
DataLayer.Commit();
}
}catch(MiaException ex){
DataLayer.Commit();
}(sempre che col C# puoi fare questo tipo di gestione delle eccezioni)

[cionci]

Usare le eccezioni così sinceramente non mi piace molto ;)