semplificare con una sola if

[Kahm]

salve a tutti
mi ritrovo questo pezzo di codice, che cosi a leggerlo è bruttissimo, vorrei semplificarlo scrivendo una sola IF
in pratica deve prendere un valore da una data nullable.
prima controlla se ci sono record, poi se il numero è >0 e poi se non è null il dato

consigli?

                        string lastUpdate = "";
                        var L = db.DREAM_Log.AsNoTracking().OrderByDescending(ee => ee.Usr_DateMod).Where(ee => ee.IdRecord == Esame.IDEsame);


                        if (L != null)
                        {
                            if (L.Count() > 0)
                            {
                                if (L.First().Usr_DateMod.Value != null)
                                { lastUpdate = L.First().Usr_DateMod.Value.ToShortDateString(); }
                            }
                        }

[URANIO]

unifica le if... non capisco quale sia il problema

[Kahm]

unifica le if... non capisco quale sia il problema

se faccio una IF con AND e AND
e non ci sono record, oppure è null il valore
il compilatore mi da subito errore in quanto esegue tutti nello stesso momento

[optime]

usa AndAlso invece di And

[URANIO]

se faccio una IF con AND e AND
e non ci sono record, oppure è null il valore
il compilatore mi da subito errore in quanto esegue tutti nello stesso momento

In c# l'ìoperatore and && equivale all'Andalso di VB.NET, quindi se il primo operatore è falso non continua e quindi non va in errore

[alka]

vorrei semplificarlo scrivendo una sola IF
in pratica deve prendere un valore da una data nullable.
prima controlla se ci sono record, poi se il numero è >0 e poi se non è null il dato
consigli?

Innanzitutto, puoi eliminare in sicurezza la prima condizione if (L != null) che non serve assolutamente a nulla, in quanto il valore sarà sempre diverso da null.

In secondo luogo, osservando la presenza di db.DREAM_Log.AsNoTracking(), immagino che si tratti di una espressione LINQ applicata ad Entity Framework, quindi occhio perché quando esegui prima Count() e poi First(), in realtà stai eseguendo la query due volte (inutilmente), la prima per calcolare il conteggio e la seconda per recuperare di nuovo i record prendendo il primo.

Puoi sintetizzare il tutto con una istruzione sola che vada a recuperarti direttamente il valore che stai cercando dall'eventuale unico record - se esiste - che soddisfa la tua condizione dopo aver applicato l'ordinamento; in breve qualcosa del genere:

var usrDateMod = db.DREAM_Log.AsNoTracking()
	.OrderByDescending(ee => ee.Usr_DateMod)
	.Where(ee => ee.IdRecord == Esame.IDEsame)
	.Select(ee => ee.Usr_DateMod)
	.FirstOrDefault();

var lastUpdate = usrDateMod != null ? usrDateMod.Value.ToShortDateString() : string.Empty;

Ovviamente non ho verificato se il codice compila ed è corretto sintatticamente perché non ho a disposizione i tipi con cui stai effettivamente lavorando, ma dovrebbe rendere l'idea.

Ciao!

[Kahm]

Innanzitutto, puoi eliminare in sicurezza la prima condizione if (L != null) che non serve assolutamente a nulla, in quanto il valore sarà sempre diverso da null.

ho messo delle if in quanto se non trovava il record dava errore
dici che mettendo una firsOrdefault posso evitare una IF?

[alka]

ho messo delle if in quanto se non trovava il record dava errore

La if() che verifica se una espressione LINQ è diversa da null è sempre inutile: con la sintassi LINQ, vai a definire una espressione che esprime (in questo caso, trattandosi di Entity Framework) una query da eseguire sul DB, quindi è inutile costruirla con la sintassi LINQ e andare a verificare se è inizializzata, perché l'espressione viene comunque creata.

dici che mettendo una firsOrdefault posso evitare una IF?

Più che pensare alle condizioni da evitare, rileggi le mie osservazioni sopra e nel messaggio precedente e analizza il codice proposto per capire quello che avviene e determinare quindi in autonomia quello che serve (o non serve).