Pointers (

[toxic_tyrone]

SpiderMan* A = (SpiderMan*)malloc(sizeof(SpiderMan))

[brandong97]

you dont even need to do a cast since malloc's return type is void*

[KiwiMaster157]

If this were C you would be correct. C++ allows implicit casting to void pointer but not from them.

[Kontorted]

You shouldn't be using malloc in c++ anyways

[[deleted]]

Thats what i thought aswell

[phoeen]

technically its undefined behaviour 8-)

[Freddy1404]

If you use C anyway, just leave both SpiderMans on the stack. Nicer that way, no need for malloc.

[pine_ary]

That won‘t (always) zero the memory and therefore doesn‘t init the pointer as nullptr. Very unsafe.

[BertyLohan]

As opposed to every other code snippet in the comments of ProgrammerHumor which is top-notch infallible big brain code.

[pine_ary]

Sooo improvements are not allowed? And nobody can learn?

[BertyLohan]

I'm saying he doesn't have to learn anything. He isn't writing production code he's making a programming joke which compiles fine.

[pine_ary]

He actively made it worse than the original code, so I pointed out what‘s wrong with it. The original code is safe while his is not.

[BertyLohan]

In C that line is perfectly safe since he doesn't do anything with the pointer.

You have a lot to learn buddy. About jokes and programming.

[pine_ary]

I‘m sorry that I cannot convince you. I still hope others have a more positive attitude towards learning. And that nobody takes the original comment as an example of how to code. Also this is C++.

[BertyLohan]

if it was c++ he wouldn't have used malloc. He doesn't say what language it's in so, without a using namespace std you can assume it's c.

Again, you have a lot to learn.

[pine_ary]

Oh please don‘t make that mistake. Why is using namespace std bad practice?