C#•7mo 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•7mo ago

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

restingphantomOP•7mo 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•7mo ago

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

restingphantomOP•7mo ago

i saw something on that on stack overflow

Pobiega•7mo ago

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

restingphantomOP•7mo ago

i just didnt understand where to put it yes that works

Pobiega•7mo ago

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

restingphantomOP•7mo ago

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

Pobiega•7mo ago

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

Angius•7mo ago

But, ideally, you should .Select() instead

Pobiega•7mo 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•7mo ago

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

Pobiega•7mo ago

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

Angius•7mo 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•7mo ago

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

Pobiega•7mo ago

why not? perfectly fine oh, nullable

leowest•7mo ago

nullable Guid?

Pobiega•7mo ago

😄 yeah remove the nullability

leowest•7mo ago

:catgun:

Angius•7mo ago

Sequential IDs gang represent

restingphantomOP•7mo ago

Guid gets set in the db

Pobiega•7mo ago

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

restingphantomOP•7mo ago

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

leowest•7mo ago

yes and they are pk so they should never be null

Pobiega•7mo ago

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

restingphantomOP•7mo 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•7mo ago

Tech2 💀 Not a list?

leowest•7mo ago

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

restingphantomOP•7mo ago

that name isnt my choise

leowest•7mo ago

specially with that many include

restingphantomOP•7mo ago

😅

Angius•7mo ago

Name is whatever, schema seems bad

restingphantomOP•7mo ago

how do you mean?

Angius•7mo ago

What if I want to list 67 skills?

restingphantomOP•7mo ago

skills?

Angius•7mo ago

Uh, techs

restingphantomOP•7mo 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•7mo ago

That makes more sense then Though the naming sucks ass

restingphantomOP•7mo 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•7mo ago

Yes A select

restingphantomOP•7mo ago

aha let me try ill add that later Thnx everyone

Gaming

Programming

✅ EF CORE, get request wont return related object