Ciao,

Però se tolgo l'istruzione con aggregate e poi eseguo
foreach(BarcaplanningDTO c in test)
{
c.Imbarco.Aggregate("",(current,next) => current + next);
}

è normale che questa funzioni e la prima no...stai facendo due cose differenti

Nella prima soluzione hai un IQueryable che si traduce in sql. Aggregate è un metodo sconosciuto in sql ed è per questo che viene generata l'eccezione.

Una possibile soluzione (non l'unica, va valutato il contesto) è di chiamare prima AsEnumerable, questo farà in modo che la tua query sql venga eseguita ed il resto delle operazioni (compreso aggregate) verranno eseguite in memoria e non su sql.



/Ciao

Ciao,

un paio di domande :)

nella classe del dominio (per intenderci, la classe utilizzata per ListEventiBarche) la proprietà Imbarco è dichiarata come virtual?

nella classe BarcaplanningDTO, ti servono sia Imbarco che Imbarcostring ?

Non è la causa dell'errore e quindi l'affrontiamo alla fine, al posto di aggregate meglio string.join

UPDATE:
E' probabile che l'errore sia generato a causa del lazy loading di ef.
Ci sono diverse soluzioni per la tua esigenza e per capire quale soluzione sia la migliore per il tuo contesto servirebbero altre info...allo stato attuale per es. potrebbe andare bene anche questo



A livello generale due consigli a livello di performance
- verificare sempre (e capire realmente) come una query linq si traduce nella richiesta/e (non sempre è vero il rapporto 1:1) al database e per fare questo puoi guardare qui
- in questo caso quando disponibile meglio utilizzare la variante asincrona, await _db.ListEventiBarche....ToListAsync()

/Ciao

---

Modificato da scioCoder il 18 marzo 2019 13:53 -

Ciao Moreno,

ok, il tuo ragionamento mi torna sul codice che ho postato ieri ma la stessa cosa vale anche sull'ultimo?

Grazie
/Ciao

Rispondo prima alle domande di Alessio:
nella classe del dominio (per intenderci, la classe utilizzata per ListEventiBarche) la proprietà Imbarco è dichiarata come virtual?
Si è dichiarata virtual come vedi:
public class EventoBarca
{
[Key, Column(Order = 0)]
public int EventoID { get; set; }
[Key, Column(Order = 1)]
public int BarcaID { get; set; }

public virtual Evento Evento { get; set; }
public virtual Barca Barca { get; set; }

public virtual Fornitore Skipper { get; set; }
public virtual ICollection<Tesserato> Tesserati { get; set; }
public virtual Imbarco Imbarco { get; set; }
}

nella classe BarcaplanningDTO, ti servono sia Imbarco che Imbarcostring ?
Mi servono entrambi perchè sto provando a fare la seguente cosa:
Una gridview con 31 colonne pari ai giorni del mese e le righe un elenco di barche.
Vorrei mostrare con delle caselle colorate i giorni in cui le barche sono impegnate e dato che una barca può essere impegnata in più intervalli di giorni nel mese scelto, raggruppo nella query BarcaID.
Poi tengo una proprietà List<string> Imbarco perchè ogni barca ha un imbarco e quindi se in una riga ci sono barche diverse devo collezionare gli imbarchi... mentre in string Imbarcostring mi tocca appiattire la lista in una singola stringa per poi poter filtrare la gridview per nome imbarco usando il "contains" (chissà quanti metodi migliori ci sono)
Ma per rispondere anche a Moreno, i ToList() sono anche di più di quelli che avevo riportato.
Ecco la query completa
query = _db.ListEventiBarche
.Where(r => r.Evento.Event_date.Month == mese || r.Evento.Event_end_date.Month == mese)
.GroupBy(r => r.Barca.BarcaID)
//.AsEnumerable()
.Select(r => new BarcaplanningDTO
{
BarcaID = r.Key,
Titolo = r.Select(x => x.Evento.Title).ToList(),
Modello = r.Select(x => x.Barca.Modello).FirstOrDefault(),
Nome = r.Select(x => x.Barca.Nome).FirstOrDefault(),
Regione = r.Select(x => x.Barca.Regione).FirstOrDefault(),
Letti = r.Select(x => x.Barca.Letti).FirstOrDefault(),
Imbarco = r.Select(x => x.Imbarco.Nome).ToList(),
//Imbarcostring = (r.Select(x => x.Imbarco.Nome).ToList()).Aggregate("",(current,next) => current + next),
Occupata = r.Select(x => 3).ToList(),
Ordine = 3,
Dateint = r.Select(x => new Intervallodate { EventoID = x.EventoID, Event_date = x.Evento.Event_date, Event_end_date = x.Evento.Event_end_date }).ToList()
}).ToList();


foreach (BarcaplanningDTO c in query)
{
c.Imbarcostring = c.Imbarco.Aggregate("", (current, next) => current + next);
}

Così come le vedete funziona tutto. Se tolgo i due commenti e naturalmente il foreach mi trovo l'errore che ho riportato