C#•17mo ago

getting warning: CS8618 - Non-nullable variable must contain a non-null value when exiting construct

I guess, it's not a big deal since the code is getting compiled but I still want to understand why I'm getting this warning and how can I resolve this. I was using a generic struct

truct Rectangle<T> 
{
    T length;
    T width;

    public T Length{
        readonly get{ return length; }
        set{ length = value; }
    }
    public T Width{
        readonly get{ return width; }
        set{ width = value; }
    }

    public Rectangle(T L, T W) // <-- this is where I am getting the warning
    {
        Length = L;
        Width = W;
    }
}

truct Rectangle<T> 
{
    T length;
    T width;

    public T Length{
        readonly get{ return length; }
        set{ length = value; }
    }
    public T Width{
        readonly get{ return width; }
        set{ width = value; }
    }

    public Rectangle(T L, T W) // <-- this is where I am getting the warning
    {
        Length = L;
        Width = W;
    }
}

29 Replies

ero•17mo ago

With the code you've provided, what's stopping me from creating a Rectangle<Rectangle<List<Rectangle<StringBuilder>>>>?

JibzOP•17mo ago

sanity?

ero•17mo ago

So nothing, basically That's the issue you're having I can pass any nullable class as T

Thinker•17mo ago

The actual error occurs because the compiler can't tell that Length and Width aren't null if their corresponding fields aren't null.

ero•17mo ago

This is obviously a glaring issue, and is if I do pass a nullable type, then it's your responsibility to make sure length and width are not null when the constructor exits

Thinker•17mo ago

What you can do is either ignore the warning because it doesn't really matter in this case, or add [NotNullWhenNotNull(nameof(length))] to Length, and the same for Width.

JibzOP•17mo ago

so what can I do to stop you?

ero•17mo ago

This is because you explicitly specified those two to be non-null

Thinker•17mo ago

Maybe constraint T to INumber<T> if Ero thinks that's more sane

ero•17mo ago

Yup, exactly that

JibzOP•17mo ago

INumber?

ero•17mo ago

You may also consider using auto-properies for Length and Width You may also consider renaming Length to Height

Thinker•17mo ago

Basically so that T can only be a number, because using string for length and height doesn't really make sense.

ero•17mo ago

public readonly record struct Rectangle<TUnit>(
  TUnit Width,
  TUnit Height) where TUnit : INumber<TUnit>;

public readonly record struct Rectangle<TUnit>(
  TUnit Width,
  TUnit Height) where TUnit : INumber<TUnit>;

Should get you where you want

Thinker•17mo ago

struct Rectangle<T> where T : INumber<T>
{
    public T Length { get; set; }
    public T Width { get; set; }

    public Rectangle(T length, T width)
    {
        Length = length;
        Width = width;
    }
}

struct Rectangle<T> where T : INumber<T>
{
    public T Length { get; set; }
    public T Width { get; set; }

    public Rectangle(T length, T width)
    {
        Length = length;
        Width = width;
    }
}

(if you prefer the full type over a record) Although the record will also give you equality methods, which might be something you want

JibzOP•17mo ago

The type or namespace name 'INumber<>' could not be found

ero•17mo ago

Is anyone surprised Can you tell us what type of project you're making?

Thinker•17mo ago

INumber is in System.Numerics so you'll have to import that

ero•17mo ago

What tutorial are you following? Do you know what .net version your project is using?

JibzOP•17mo ago

Yeah dotnet 7

ero•17mo ago

Ah, that's good then Importing System.Numerics should do it

JibzOP•17mo ago

well.. this is frustrating. I'm using INumber as struct Rectangle <T> where T : INumber<T> but the warning doesn't go away 😓

ero•17mo ago

Try where T : unmanaged, INumber<T> Might need to turn them around I wonder if it's possible to just constrain it to notnull instead of unmanaged Though I think unmanaged is a better fit

Thinker•17mo ago

yeah that has nothing to do with the INumber<T> thing at all again this is the actual issue

JibzOP•17mo ago

right, so this error can't be removed?

Thinker•17mo ago

If you were using auto-properties then it would go away That's the easiest way

JibzOP•17mo ago

nvm. Yeah, its gone! thanks both of you!

Thinker•17mo ago

Accord•17mo ago

Was this issue resolved? If so, run /close - otherwise I will mark this as stale and this post will be archived until there is new activity.

Gaming

Programming

getting warning: CS8618 - Non-nullable variable must contain a non-null value when exiting construct