Over on Stack Overflow, I frequently see exchanges like the following:
Questioner: I have to do [algorithmically complex task]. I've written [unworkably naive algorithm that will need to be scrapped], but it's too slow and I [got bored/got a "time limit exceeded" message]. Can you help?
Commenter: Working code belongs on Code Review. Voting to close.
Is this actually the kind of thing that should be moved to Code Review?
- The existing code is irrelevant. It does not need to be improved; it needs to be replaced entirely. Frequently, the questioner explicitly says that they think the code is useless or that they're only posting it to show that they tried.
- Anything like style cleanup, consistent variable names, or avoiding deprecated APIs is completely tangential to the question.
- Code that fails to meet its performance requirements, especially by such a huge margin as these algorithms usually do, can hardly be considered "working".
- If you submitted code like this for code review in your actual job, you would be told to stop wasting other engineers' time with code you know isn't ready.
It really seems like these questions are a better fit for Stack Overflow, not Code Review. Should such questions actually be moved here, or should they stay on Stack Overflow?