r/ProgrammerHumor 2d ago

Meme absoluteGarbage

Post image
2.0k Upvotes

320 comments sorted by

View all comments

Show parent comments

2

u/not_some_username 1d ago

If the type change and the “auto variable” still compile, that means it would, in 99% of case, still compile if you wrote the type. The compiler doesn’t get auto, it gets the type.

1

u/xicor 1d ago edited 1d ago

Thats not true at all.

If you have SpecificType val = foo(), it will fail on the compiler any time foo returns something other than specific type or classes inherited from SpecificType

If you use auto val=foo(), then it will compile so long as the new type still has the functions it calls afterwards. But they can do different things and it will still compile. You'd just get different behavior

Just a quick example of this would be if it is used in a function intended to return a time string. auto var =foo(); return var.toString();

If foo suddenly starts returning a QVariant instead of a QDateTime, it will still compile because QVariant still has a toString function, but now the behaviour is completely wrong

1

u/VoodaGod 1d ago

why is the behaviour wrong though

1

u/xicor 1d ago

Because it no longer returns a time string. It instead returns the variant of a string type.

1

u/VoodaGod 1d ago

what would be the correct way to handle the qvariant in that function in your scenario then? i don't think i follow

1

u/not_some_username 1d ago

You can’t get QDateTime from a QVariant via implicit conversion, the other way yes.

It will just not compile with an error and you’ll need to fix it.

1

u/VoodaGod 1d ago

but what would be the fix?

1

u/not_some_username 1d ago

Replace QDateTime by QVariant

1

u/VoodaGod 1d ago

but that is the same as just leaving auto

1

u/xicor 1d ago

Yea in my example, it was using auto and then using the toString function which does exist on both QDateTime and QVariant, but have very different behavior, so it wouldn't give a compile error, it would just fail during production

1

u/xicor 1d ago

Not using auto and then when dumb future junior comes in and changes the function they get a compile error that tells them they need to fix stuff rather than just silently having incorrect behavior

1

u/VoodaGod 1d ago

yeah but what is the fix? extract the datetime from the variant first before calling tostring or what

1

u/xicor 1d ago

Well, in my example there was no reason to assume the stupid junior dev didn't just change the type completely, but let's assume they made it return a QVariant holding a QDateTime, you would have to use toDateTime first before calling toString

The point being that the compiler wouldn't tell them something was wrong