3
\$\begingroup\$

Please see this question : https://codereview.stackexchange.com/q/36987/14625

I invited the 2 users to comment on this thread.

From my perspective, Code Review is there to Review working code, that code is not working.

\$\endgroup\$
4
  • \$\begingroup\$ That is certainly why CR is here. As per the FAQ: "What questions are on-topic for this site? ... 6.) To the best of my knowledge, does the code work?" There doesn't seem to be much to discuss, unless I'm missing something? It's unfortunate that the OP will not get the helping he is looking for, but that's how it goes. \$\endgroup\$
    – Corbin
    Dec 9, 2013 at 20:25
  • \$\begingroup\$ "Users unhappy with voting to close/move" Users have been unhappy with the rules of Stack Exchange since the very beginning. "From my perspective, Code Review is there to Review working code, that code is not working." Yes, I full heartily agree...why do we have this discussion again? ... Oh wait, was there a comment from OP that I missed/was deleted in the mean time? \$\endgroup\$
    – Bobby
    Dec 9, 2013 at 20:58
  • 2
    \$\begingroup\$ And if you think I meant "this question belongs here on CR" then it's a misunderstanding. I meant "it does belong as much on SO as it does belong on CR...and it does not belong on CR", migrating or sending it to SO would do nothing, it would just be closed there is it would be closed here. Badly phrased, sorry. \$\endgroup\$
    – Bobby
    Dec 9, 2013 at 21:00
  • 1
    \$\begingroup\$ @Bobby that was indeed how I understood you, np \$\endgroup\$
    – konijn
    Dec 10, 2013 at 17:10

2 Answers 2

6
\$\begingroup\$

Bobby's comment to the question is:

@tomdemuyt: Please no! This question does belong on SO as much as it does belong on CR. It would just be closed on SO. – Bobby 49 mins ago

The poster clearly says (emphasis mine) "I want that it should reload or stay on same page with reload to same id." This means that there is a problem to be solved, the code does not work as it is intended to, one of the things in the Don't ask list for Code Review is:

How to add a feature to or solve a problem with your code

Just because a question would be closed for "... must demonstrate a minimal understanding of the problem being solved ..." on Stack Overflow doesn't mean that we should accept them here. There is a reason for why it would be closed on Stack Overflow. We might be friendlier here than the ones on SO, but off-topic is still off-topic.

\$\endgroup\$
4
\$\begingroup\$

There is plenty to criticize about the code: the use of HTTP GET for a non-idempotent delete operation, the use of the deprecated mysql_query() suite of functions, the careless use of string interpolation that could lead to SQL injection and HTML injection.

Unfortunately, the question is poorly formulated. I can't figure out exactly what the situation is, but it's clear that the poster is asking for a problem to be solved. There may be other fora for questions like this, but Code Review isn't it.

\$\endgroup\$

You must log in to answer this question.

Not the answer you're looking for? Browse other questions tagged .