WEB API with Entity Framework Core using the JOIN statement

Question

Hi all,
I have written a WEB API with Entity Framework that read form two tables using the JOIN statement.

In order to obtain the wanted result, I have used vary solutions. The "Include" statement didn't work (I don't know why), so I have used the classic JOIN, and the procedure works fine.

But, I have needed to use a DTO model to "export" the joined data from the api.

Is this the correct way to accomplisce this kinde of work (reading data from joined tables)?

Is there more efficient way?

This is my Model

// MODEL (Giocata + TipoGiocata)    

    public class Giocata
    {
        public int GiocataId { get; set; }
        public string NomeGiocata { get; set; }
        public DateTime DataGiocata { get; set; }
        public string Numeri { get; set; }
        public bool Attiva { get; set; }

        // FK TipoGiocataId
        public int TipoGiocataId { get; set; }
        public TipoGiocata TipiGiocata { get; set; }
    }

    public class TipoGiocata
    {
        public int TipoGiocataId { get; set; }
        public string NomeTipoGiocata { get; set; }

        // FK Giocate
        public int GiocataId { get; set; }
        public ICollection<Giocata> Giocate { get; set; }
    }

// MODEL GiocataDTO

    public class GiocataDTO
    {
        // Giocate
        public int GiocataId { get; set; }
        public string NomeGiocata { get; set; }
        public DateTime DataGiocata { get; set; }
        public string Numeri { get; set; }
        public bool Attiva { get; set; }

        // TipoGiocata
        public int TipoGiocataId { get; set; }
        public string NomeTipoGiocata { get; set; }
    }

This is my WEB API:

        public async Task<ActionResult<IEnumerable<GiocataDTO>>> GetGiocate()
        {
            List<GiocataDTO> listaGiocateDTO = new List<GiocataDTO>();

            var listaGiocate = await _context.Giocate
                .Join(
                    _context.TipiGiocata,
                    giocata => giocata.TipoGiocataId,
                    tipogiocata => tipogiocata.TipoGiocataId,
                    (giocata, tipogiocata) => new
                    {
                        GiocataId = giocata.GiocataId,
                        NomeGiocata = giocata.NomeGiocata,
                        DataGiocata = giocata.DataGiocata,
                        Numeri = giocata.Numeri,
                        NomeTipoGiocata = tipogiocata.NomeTipoGiocata,
                        Attiva = giocata.Attiva
                    }
                ).ToListAsync();

            foreach (var giocata in listaGiocate)
            {
                GiocataDTO giocataDTO = new GiocataDTO();

                giocataDTO.GiocataId = giocata.GiocataId;
                giocataDTO.NomeGiocata = giocata.NomeGiocata;
                giocataDTO.DataGiocata = giocata.DataGiocata;
                giocataDTO.Numeri = giocata.Numeri;
                giocataDTO.NomeTipoGiocata = giocata.NomeTipoGiocata;
                giocataDTO.Attiva = giocata.Attiva;

                listaGiocateDTO.Add(giocataDTO);
            }

            return listaGiocateDTO;
        }

Thank you for your help.
Igor.

Answer 1

Why create the anonymous object in the select rather than the desired?

Answer 2

Like this?

        [HttpGet]
        [Route("[action]")]
        [Route("api/giocate/GetGiocate")]
        public async Task<ActionResult<IEnumerable<Giocata>>> GetGiocate()
        {
            List<Giocata> listaGiocate = new List<Giocata>();

            listaGiocate = await _context.Giocate
                .Join(
                    _context.TipiGiocata,
                    giocata => giocata.TipoGiocataId,
                    tipogiocata => tipogiocata.TipoGiocataId,
                    (giocata, tipogiocata) => new
                    {
                        GiocataId = giocata.GiocataId,
                        NomeGiocata = giocata.NomeGiocata,
                        DataGiocata = giocata.DataGiocata,
                        Numeri = giocata.Numeri,
                        NomeTipoGiocata = tipogiocata.NomeTipoGiocata,
                        Attiva = giocata.Attiva
                    }
                ).ToListAsync();

            return listaGiocate;
        }

If I do it, I receive the error in line 6:

Error CS0029: Cannot implicitly convert type 'System.Collections.Generic.List<<anonymous type: int GiocataId, string NomeGiocata, System.DateTime DataGiocata, string Numeri, string NomeTipoGiocata, bool Attiva>>' to 'System.Collections.Generic.List<IreneLottoApi.Models.Giocata>'

Answer 3

Hi Bruce-SqlWork,
your answer inspired me the solution: why using an anonymous object in the select?

So, I have modified like this and now seems better:

        [HttpGet]
        [Route("[action]")]
        [Route("api/giocate/GetGiocate")]
        public async Task<ActionResult<IEnumerable<GiocataDTO>>> GetGiocate()
        {
            var listaGiocate = await _context.Giocate
                .Join(
                    _context.TipiGiocata,
                    giocata => giocata.TipoGiocataId,
                    tipogiocata => tipogiocata.TipoGiocataId,
                    (giocata, tipogiocata) => new GiocataDTO
                    {
                        GiocataId = giocata.GiocataId,
                        NomeGiocata = giocata.NomeGiocata,
                        DataGiocata = giocata.DataGiocata,
                        Numeri = giocata.Numeri,
                        NomeTipoGiocata = tipogiocata.NomeTipoGiocata,
                        Attiva = giocata.Attiva
                    }
                ).ToListAsync();

            return listaGiocate;
        }