46

The question: PHP: Optimize If Statement

To me this looks like a clear cut case for migration to Code Review. It's code that works, and the question is "Is there a better way of writing the following block of code to easily be maintainable?"

So I made a comment that it should be posted to CR instead. The OP replied with "thanks for the link". A diamond moderator then commented that "it needs a redesign rather than code review", to which I replied "...so?"

The comment thread's contents (not word for word because the comments are now removed, but the content is accurate):

I think this belongs to Code Review instead. - Juhana

Thanks for the link. - OP

@Juhana It needs a redesign more than a code review. - Moderator ♦

@Moderator ...so? - Juhana

When I reloaded the page, the comments were gone.

Ok, if my last comment was too blunt, I apologize. I was merely trying to say that if code requires redesign (rather than cosmetic polish) it would still be on-topic at CR and the amount of required work was irrelevant. But why delete all comments instead of just that one?

Then I posted this comment:

@Moderator Did you just wipe out the comments? Why? - Juhana

...and it was immediately deleted. I would have appreciated a brief explanation first.

Did I do something wrong?

13
  • 18
    looks like a good post to migrate to CodeReview to me, I flagged it as such.
    – Malachi
    Commented Jul 14, 2014 at 16:09
  • 5
    That question looks perfect for Code Review to me...
    – Izkata
    Commented Jul 14, 2014 at 16:11
  • 9
    Yep, perfectly reasonable for Code Review. I don’t know why the comments would have been deleted.
    – Ry- Mod
    Commented Jul 14, 2014 at 16:14
  • 3
    @minitech, we don't want people cross posting and I assume the Mod who deleted them was thinking the same thing, less dups if the user waits for the migration.
    – Malachi
    Commented Jul 14, 2014 at 16:15
  • 7
    @Malachi: Moderators can migrate posts, though.
    – Ry- Mod
    Commented Jul 14, 2014 at 16:18
  • 1
    @Malachi Yeeaah, but you can still read deleted comments in the notification inbox. Especially a short one like that.
    – Izkata
    Commented Jul 14, 2014 at 16:19
  • @minitech, sounds like the OP already had Cross posted.
    – Malachi
    Commented Jul 14, 2014 at 16:19
  • 7
    If the reason was to prevent cross-posting, deleting would have been counterproductive since the OP had already seen the comment. Delete comments + an additional "don't cross-post" comment I would have understood.
    – JJJ
    Commented Jul 14, 2014 at 16:20
  • @Malachi: <del>No, they don’t have any other accounts.</del> Apparently they used a different account. How fantastic.
    – Ry- Mod
    Commented Jul 14, 2014 at 16:20
  • @minitech: Yeah, I've noticed that. I have no idea why that was done...
    – Jamal
    Commented Jul 14, 2014 at 16:28
  • 3
    FYI: The 2nd Monitor (CR's main chatroom) has a feed for all codereview-se questions asked on MSO. Commented Jul 14, 2014 at 16:35
  • 2
    Given the direction this has gone, I'm putting this here so the question is "linked" rather than just "related": Be careful when recommending Code Review to askers
    – Izkata
    Commented Jul 14, 2014 at 23:33
  • 3
    Deletion of comments without explanation seems heavy-handed and won't help the poster of the comment learn to do better.
    – Tim
    Commented Jul 15, 2014 at 13:38

4 Answers 4

19

As a quite active user on CR I will tell you what I think:

Comments like yours induce cross-posting. That is an issue discussed on CR Meta, as well as here on SO meta. These comments are generally disliked by moderators (from what I hear in CR chat) as they increase adminstrative cost.

Additionally to the already highlighted problems with cross-posting your first comment became obsolete the moment OP acknowledged it by commenting back "thanks for the link".

Obsolete comments are subject to deletion. This has always been that way and is SE-Wide policy.

On the "why the removal"-comment:

It is not constructive to the post at hand. Inquiries concerning moderator decisions should be taken to Chat or Meta. I assume it got deleted for that exact reason.

4
  • 1
    I've never heard or read that obsolete comments can be deleted. Can you please show me some posts showing why that's the policy? (I don't doubt you're right; I'd just like to learn more.)
    – Kevin
    Commented Jul 14, 2014 at 21:13
  • 5
    @Kevin chech out the comment privilege help. "Comments are temporary post-it notes". You might also check the flag dialogue of a comment. Among too-chatty and off-topic there's also obsolete.
    – Vogel612
    Commented Jul 14, 2014 at 21:21
  • I think the goal was migration, not crossposting. Also if we would have to delete obsolete comments we could delete a lot (say at least 50%) of the comments. I guess that nobody wants to do that really. Commented Jul 15, 2014 at 8:34
  • 4
    @Trilarion we don't have to delete obsolete comments. They just may be deleted any time. Also, even if the goal was migration, this comment still warrants cross-posting by OP, and has in fact done that (see below answer)
    – Vogel612
    Commented Jul 15, 2014 at 8:35
11

The question has already been cross-posted, but with a title change (by me). Although the OP indeed didn't ask for a design review (which are off-topic), it doesn't belong on SO. The code itself does look okay to me, but that's probably not saying much since PHP isn't my area of expertise. That said, I'll let the community decide if the question should stay open, as this one doesn't need my mod hammer.

As for the comment-wiping, that may have been done because they're either no longer needed, or because the moderator disagrees. They're only comments anyway, so it doesn't matter if they get removed. You could still request an explanation behind the removal on Meta, though.

2
  • 1
    Migration was not possible? And I wonder why you asked the question on CR? Do you want to have it answered for yourself or to you want to have it answered for the original poster? Commented Jul 15, 2014 at 8:36
  • 1
    Ah I just saw that you didn't ask the question on CR, just edited it. Sorry, my fault. Commented Jul 15, 2014 at 10:46
8

I don't think this question is actually offtopic on SO in the first place, which is a prerequisite of any migration.

The question seems to essentially be: "is there a conscice language construct for representing this structure?"

It's slightly clumsily asked but I don't see that as making it OT here.

1
  • 2
    I'd disagree that being off topic is a prerequisite. If something is on topic on site A, but will get significantly higher quality answers on site B, there's nothing wrong with a migration, despite it being under the 'off-topic' reasons. I wouldn't say the population of such questions is very large, but it does exist.
    – corsiKa
    Commented Jul 15, 2014 at 17:17
8

The entire post, question and answers, seem to match Code Review.

I have flagged the post, and I don't know if it would help to have others flag the question.

The reason for not wanting the comments is because of cross posting and duplicates from this. The user is new to the Stack Exchange system and will very likely cross post when given the suggestion that this post would be a better fit for Code Review. And then there would be a duplicate that would be closed on Code Review when this original is migrated.

So that is probably why they deleted the comments.

You must log in to answer this question.

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