[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

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]

Originariamente inviato da 0rph3n

ecco la mia ennesima cazzata

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
Scherzo, purtroppo non mi sembra piu' elegante.

[gugoXX]

[quote=kernel:anic;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]

Originariamente inviato da cionci

...

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]

Originariamente inviato da cionci

o così:

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

copione

[cionci]

Originariamente inviato da 0rph3n

copione

Non avevo visto

[gugoXX]

Facciamo cosi'.
Le metto tutte e 4.

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

[cionci]

Originariamente inviato da gugoXX

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

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