C#•8mo ago

✅ EF CORE, get request wont return related object

42 Replies

Venture

c#
using System.ComponentModel.DataAnnotations;
using System.ComponentModel.DataAnnotations.Schema;

namespace HTXL_Back_end.Models;

[Table("Venture")]
public class Venture
{
    [Key]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public Guid? Id { get; set; }
    
    public string Name { get; set; }

    public Portfolio? Portfolio { get; set; }
}

c#
using System.ComponentModel.DataAnnotations;
using System.ComponentModel.DataAnnotations.Schema;

namespace HTXL_Back_end.Models;

[Table("Venture")]
public class Venture
{
    [Key]
    [DatabaseGenerated(DatabaseGeneratedOption.Identity)]
    public Guid? Id { get; set; }
    
    public string Name { get; set; }

    public Portfolio? Portfolio { get; set; }
}

Pobiega•8mo ago

can you show the code that makes the actual query? and is Portfolio here an owned entity, or a normal relational object?

restingphantomOP•8mo ago

c#

// GET: api/Venture
        [HttpGet]
        public async Task<ActionResult<IEnumerable<Venture>>> GetVentures()
        {
          if (_context.Ventures == null)
          {
              return NotFound();
          }

            return await _context.Ventures.ToListAsync();
        }

c#

// GET: api/Venture
        [HttpGet]
        public async Task<ActionResult<IEnumerable<Venture>>> GetVentures()
        {
          if (_context.Ventures == null)
          {
              return NotFound();
          }

            return await _context.Ventures.ToListAsync();
        }

venture has one or zero portfolios

Pobiega•8mo ago

ok, I see no include - this is probably just a missed include call

restingphantomOP•8mo ago

i saw something on that on stack overflow

Pobiega•8mo ago

try await _context.Ventures.Include(x => x.Portfolio).ToListAsync();

restingphantomOP•8mo ago

i just didnt understand where to put it yes that works

Pobiega•8mo ago

EF doesnt fetch relational objects by default, you need to explicitly state that you want them included

restingphantomOP•8mo ago

would it also be possible to have some of the portfolio relations being returned like that

Pobiega•8mo ago

yep! .ThenInclude(x => ...)

Angius•8mo ago

But, ideally, you should .Select() instead

Pobiega•8mo ago

^ the ideal way to do this, especially in a web api, is to .Select into a DTO you dont want to return your actual entity models directly to the user

restingphantomOP•8mo ago

I know, but for my Proof of concept its fine for now

Pobiega•8mo ago

sure thing. then .Include(...).ThenInclude(...) will work

Angius•8mo ago

For a proof of concept I'd be selecting into anonymous objects tbh Defining them later Rather than including all over the place

leowest•8mo ago

also Guid? I dont think u want that on a pk

Pobiega•8mo ago

why not? perfectly fine oh, nullable

leowest•8mo ago

nullable Guid?

Pobiega•8mo ago

😄 yeah remove the nullability

leowest•8mo ago

:catgun:

Angius•8mo ago

Sequential IDs gang represent

restingphantomOP•8mo ago

Guid gets set in the db

Pobiega•8mo ago

vogen ids all the way baby yeah but its nullable remove the nullability and you're fine

restingphantomOP•8mo ago

yeah, otherwise it will set it to necisairy in the db and then I wont be able to post without ID

leowest•8mo ago

yes and they are pk so they should never be null

Pobiega•8mo ago

guid is a value type, it has a default non-null value (0000-000000...)

restingphantomOP•8mo ago

c#
 return await _context.Ventures
            .Include(x => x.Portfolio)
            .ThenInclude(x => x.Tech1)
            .Include(x => x.Portfolio)
            .ThenInclude( x => x.Tech2)
            .ToListAsync();

c#
 return await _context.Ventures
            .Include(x => x.Portfolio)
            .ThenInclude(x => x.Tech1)
            .Include(x => x.Portfolio)
            .ThenInclude( x => x.Tech2)
            .ToListAsync();

btw like this?

Angius•8mo ago

Tech2 💀 Not a list?

leowest•8mo ago

consider reading https://learn.microsoft.com/en-us/ef/core/performance/efficient-querying#project-only-properties-you-need

restingphantomOP•8mo ago

that name isnt my choise

leowest•8mo ago

specially with that many include

restingphantomOP•8mo ago

😅

Angius•8mo ago

Name is whatever, schema seems bad

restingphantomOP•8mo ago

how do you mean?

Angius•8mo ago

What if I want to list 67 skills?

restingphantomOP•8mo ago

skills?

Angius•8mo ago

Uh, techs

restingphantomOP•8mo ago

ow no thats also what I thought when I saw that tech one is technology one and has sub technologies called tech2 so you choose category tech1 sub category tech2

Angius•8mo ago

That makes more sense then Though the naming sucks ass

restingphantomOP•8mo ago

yup, I know but thats how my internship did it

c#
public async Task<ActionResult<IEnumerable<Venture>>> GetVentures()
    {
        if (_context.Ventures == null) return NotFound();

        return await _context.Ventures
            .Include(x => x.Portfolio)
            .Include(x => x.Portfolio).ThenInclude(x => x.Tech1)
            .Include(x => x.Portfolio).ThenInclude(x => x.Tech2)
            .Include(x => x.Portfolio).ThenInclude(x => x.Sdgs)
            .Include(x => x.Portfolio).ThenInclude(x => x.Industry)
            .Include(x => x.Portfolio).ThenInclude(x => x.Investments)
            .Include(x => x.Portfolio).ThenInclude(x => x.Investments).ThenInclude(x => x.InevestmentType)
            .Include(x => x.Portfolio).ThenInclude(x => x.Investments).ThenInclude(x => x.Series)
            .ToListAsync();
    }

c#
public async Task<ActionResult<IEnumerable<Venture>>> GetVentures()
    {
        if (_context.Ventures == null) return NotFound();

        return await _context.Ventures
            .Include(x => x.Portfolio)
            .Include(x => x.Portfolio).ThenInclude(x => x.Tech1)
            .Include(x => x.Portfolio).ThenInclude(x => x.Tech2)
            .Include(x => x.Portfolio).ThenInclude(x => x.Sdgs)
            .Include(x => x.Portfolio).ThenInclude(x => x.Industry)
            .Include(x => x.Portfolio).ThenInclude(x => x.Investments)
            .Include(x => x.Portfolio).ThenInclude(x => x.Investments).ThenInclude(x => x.InevestmentType)
            .Include(x => x.Portfolio).ThenInclude(x => x.Investments).ThenInclude(x => x.Series)
            .ToListAsync();
    }

I feel like there is a better way of doing this

Angius•8mo ago

Yes A select

restingphantomOP•8mo ago

aha let me try ill add that later Thnx everyone

Gaming

Programming

✅ EF CORE, get request wont return related object