this post was submitted on 24 Sep 2024
5 points (100.0% liked)
C Sharp
1537 readers
1 users here now
A community about the C# programming language
Getting started
Useful resources
- C# documentation
- C# Language Reference
- C# Programming Guide
- C# Coding Conventions
- .NET Framework Reference Source Code
IDEs and code editors
- Visual Studio (Windows/Mac)
- Rider (Windows/Mac/Linux)
- Visual Studio Code (Windows/Mac/Linux)
Tools
Rules
- Rule 1: Follow Lemmy rules
- Rule 2: Be excellent to each other, no hostility towards users for any reason
- Rule 3: No spam of tools/companies/advertisements
Related communities
founded 2 years ago
MODERATORS
you are viewing a single comment's thread
view the rest of the comments
view the rest of the comments
But that's correct behaviour because if it isn't included in the query it is null. So you should check it and the warnings are correct.
But like you say irritating if you're doing it straight after an include haha
I think .NET and EF should handle navigation property nullabiloty better out of the box, especially when you explicitly set the foreign key to long?.
That said, while I understand these relationships in my own projects, adding clear annotations is very helpful for colleagues who may have to provide support in the future. That could even be me!
So there’s no perfect solution in this stack, but anything that makes a developer’s intent clearer helps. Privilege of least surprise, right?
Exactly.
What we need is someone to create an analyzer that recognises when you are writing an ef query or if you've included and disabled the warnings.
The actual issue is using the same model for the database and query results. I.e. it's not nullable in the database but could be after a query. But there's no nice way to implement this without mapping everything to a dto.