the answer is wrong for the question: "How do you compare two objects in JavaScript?"

[30-seconds]

the answer is wrong for the question: "How do you compare two objects in JavaScript?"

[skatcat31]

The one you've submitted in the PR seems to be working. I want to do some more in depth tests when I have some time or if you'd like to submit some tests for it in the PR as a comment on the thread I'd be more than happy to look it over. Either way, this is a helpful find!

[skatcat31]

right never reported test results did I... yeah it's working in all cases. Never found an edge case that could be considered a problem except for a SO if I went deep with auto generating object references

[flxwu]

Please open a PR with the proposed change in the answer's code sample. Thanks! 😊

[skatcat31]

i'm going to reopen this to allow for easier discussion from #100

[skatcat31]

Are you saying this error or that it returns true with 1 and 2 used as arguments?

[chenjigeng]

yes.and it will return true with { a:1 } amd { a: 2} used as arguments.The code can't work.

[skatcat31]

This is a confirmed problem. We might need to add better handling for this.

Or we could just use JSON.stringify(a) === JSON.stringify(b) as a short and sloppy...

But like I said it's sloppy and only really applies to POJO(Plain Old JavaScript Object)

[chenjigeng]

We can solve it by adding a if statement.You can see my code above.

[fejes713]

@chenjigeng thanks for noticing this. The PR is merged. ❤️