1

I just failed my first review for suggesting a user may get better answers on Code Review.

The question presents two options of working code, and asks:

What are the pros and cons? Or is there a third option?

To me, this question would attract opinionated answers. I suggested in a comment that they may get better quality answers on Code Review. I did not suggest that the question was off-topic for Stack Overflow. This led to a failed review.

Is this the desired result? How can I avoid future failed reviews while continuing to try to help new users?


Edit: The marked duplicate Be careful when recommending Code Review to askers outlines criteria that should be followed before recommending that a question be posted on code review. This question passes all of these criteria:

  • Real code has to be written.
  • The code has to work.
  • Broken code or troubleshoot bugs is off-topic.
  • Requests to explain others' code are off-topic.

This suggests that I correctly suggested that the question be posted on Code Review (I agree). Nominating that this question be reopened on this basis.

8
  • 9
    It's simple: don't post comments saying that this would be good for Code Review. Like, at all.
    – Makoto
    Commented Jul 18, 2019 at 15:47
  • 2
    Given the high, positive score this post had, I'm guessing it didn't have any down votes before you reviewed it. So while your first problem is that commenting causes the audit to fail, your other problem is that a question asking "which way is better" was deemed an unambiguously good question by the community. You got unlucky. (In my experience the voting on questions asking for comparisons or pro/con lists varies wildly. You may want to just be very careful of posts like that.)
    – BSMP
    Commented Jul 18, 2019 at 15:59
  • 1
    @gnat — following this checklist, the question should be recommended for Code Review. It's real, working code, and not someone else's. With no bugs or troubleshooting required.
    – Sean
    Commented Jul 18, 2019 at 16:22
  • 3
    @BSMP Looking at the timeline, it had two downvotes as of July 8. Still more positive than it should have been, but I'm surprised it was chosen as an audit target. Commented Jul 18, 2019 at 16:35
  • A question is not off-topic because it fits on code review. Unless you think it's too broad or should be closed for another reason, just leave it be. Else, vote/flag to close and possibly add a comment. Questions that fit on both are best left on the site they were posted on, even though they might fit better on another site.
    – Erik A
    Commented Jul 18, 2019 at 19:41
  • @ErikA At no point did I suggest it was off-topic for StackOverflow. My question states that in bold.
    – Sean
    Commented Jul 18, 2019 at 20:31
  • 1
    I didn't say you did, my point is, if it's not off topic here, never recommend other sites, since we want to leave it here
    – Erik A
    Commented Jul 19, 2019 at 1:33
  • 1
    @Sean. IMHO that Q is not a good candidate for CR. That checklist are not the only requirements for a CR Q, including: 1) On CR any and every aspect of the code is up for review, the OP only asks about one aspect. 2) the code must be complete working code (the OP isn't, as witnessed by the .... towards the bottom of the code block). Commented Jul 19, 2019 at 5:40

0

Browse other questions tagged .