13

This edit ended up being approved, but one of the reviewers rejected it as:

This edit defaces the post in order to promote a product or service, or is deliberately destructive.

Was this just a troll? All I did was take OP screenshot of their code and transcribe it into a code block. I fail to see how anyone could reject it on that ground.

35
  • 36
    IMO providing the code should be the responsibility of OP, not transcribed into the post by another user. There's too much risk of a manual mistake creeping in by transcribing rather than copy-pasting. I'd have voted to reject as well, though using a custom reason rather than defacement. Oct 9, 2018 at 15:04
  • 15
    I would probably have rejected such an edit. Does that mean I'm a troll?
    – user4639281
    Oct 9, 2018 at 15:08
  • 35
    In the second line you changed ' to " and removed a space - at that point I'd probably have stop checking and rejected the edit (assuming I bothered to do a line-by-line comparison at all). I guess that specific difference wouldn't change anything in the execution, but at the very least it's clear that you're not transcribing it exactly, thus there may be other errors. Oct 9, 2018 at 15:09
  • 3
    @TinyGiant I didn't think the fact that it was rejected was trolling. I thought the reason attached didn't make any sense and that the person was just rejecting edits for no reason. Oct 9, 2018 at 15:51
  • 20
    You assumed that because you didn't understand the reason there must be no reason, and jumped immediately to calling someone a troll. Don't you think there should be something between "I don't understand" and "TROLL!!!"? Maybe it's just me.
    – user4639281
    Oct 9, 2018 at 16:06
  • 6
    @TinyGiant then maybe I still don't understand the reason. Can you please elaborate on how this is either A) promotes a service or product or B) is deliberately destructive. After reading the answers below, I understand why this type of edit would be rejected, but that reason still does not fit. Oct 9, 2018 at 16:09
  • 3
    The label for that reason in the edit review queue is "spam or vandalism". I could see how someone would use that in a situation like this for a variety of reasons. It may not be the correct reason to use, but I can see how someone might have gotten there.
    – user4639281
    Oct 9, 2018 at 16:14
  • 10
    @TinyGiant I know I probably should just stay out of this, but I have a strong opinion. I believe that user-moderators always have a responsibility of clear communication to users they are moderating, and in this case the rejection reason was not so clearly communicated. Everyone who is ingrained enough in this community to be reading this meta post effectively has a leadership role at SO, and along with that comes the responsibility of clearly communicating to the newer users why we take the actions that we do. In this case the communication failure was very small, but a failure nonetheless. Oct 9, 2018 at 16:38
  • 10
    @Maximillian of course. As I said it may not have been the correct reason to choose, and it is everyone's responsibility to be as clear as possible. I'm taking issue with the jumping straight from "I don't understand" to "that person must be a troll".
    – user4639281
    Oct 9, 2018 at 17:11
  • 5
    If you don't understand something, you should attempt to expand your understanding. If you don't understand why a program you've written is behaving in a given way, do you assume that it is trolling you? Furthermore if you performed an action that you mistakenly thought to be correct then found a meta discussion where someone called you a troll because that person didn't understand your reasoning, how would that make you feel?
    – user4639281
    Oct 9, 2018 at 17:17
  • 5
    @vol7ron There's a big difference between copy-pasting code into a snippet or indenting it (which is generally really hard to mess up and not a lot of work for either editor or reviewer) and manually rewriting it from an image. Oct 10, 2018 at 8:40
  • 5
    @vol7ron I'm not saying "don't touch code at all", I regularly make edits to posts where code is not indented correctly or isn't using the correct markup, and your example of turning code blocks into snippets is also an example of a good edit. I'm just saying that it's not a good idea to transcribe someone else's image of code into text as it carries an inherent risk of introducing new mistakes (as per Dukeling's first comment). In that case, OP should fix their own post by copy-pasting their code into the question. Oct 10, 2018 at 9:35
  • 5
    @vol7ron if there is an error introduced by the editor then it is absolutely not more accessible. It is actually less accessible because anyone reading the question is going to assume the author posted the code as is, and won't bother to go to the revision history to find the link to the image of the code. It is just a bad idea. If the OP wants to transcribe their own code then that is on them.
    – user4639281
    Oct 10, 2018 at 15:38
  • 2
    See the answer below that details the numerous errors introduced to the code then tell me that all of those errors and a code block which makes no sense at all and replaces the image entirely is a good thing. It isn't and no amount of spin or hand waving about screen readers is going to make a code block that doesn't make any sense make sense.
    – user4639281
    Oct 10, 2018 at 16:19
  • 4
    Possible duplicate of When should I make edits to code?
    – pppery
    Jan 29, 2021 at 21:42

3 Answers 3

62

Thanks for putting in the effort to transcribe that image into text which can be copied elsewhere. That was pretty cool of you to do.

Most reviewers, however, would close the question instead of transcribing images, since there's a lot of risk involved in doing that transcription; notably, copy errors or your transcription introducing new problems.

I think it was good of you to take the effort, but the real responsibility is on the original author of the question. It is their question; edits are only for making sure it looks nice and neat. It is not our responsibility to transcribe everything, and it is often best to simply close the question until the author could edit it themselves. (Questions that lack a minimal, reproducible example within the text of the question itself can be closed as "a community-specific reason → needs debugging details".)

But thanks nonetheless.

1
  • 10
    ...or, occasionally, your transcription fixing errors unintentionally. ;-) Nov 16, 2018 at 16:00
41

Besides the general "don't transcribe image code into text" (with which you can agree or not), this specific example introduces a large number of new bugs and is more or less useless. Every reviewer who accepted this made a bad job. If anyone would be going to answer the question based on the transcribed code, they would state a large number of errors that simply weren't there in the image:

  • Line 2: Replaced " with ', removed whitespace at end of string.
  • Line 4: Changed ETAGM11 to ETAGM
  • Line 6: Added whitespace
  • Line 7: Changed INP1 to IMP1
  • Line 25: Changed input1.i = input2.j to input1.i = input2.i
  • Line 32: Changed oldinput.i to oldinput.1
  • Line 42: DAYA instead of DATA

Some of them might not be that important, but the majority drastically changes the program.

1
  • 8
    Which makes this suggested edit a good example of one of the reasons that the consensus is we don't transcribe code into questions for OPs.
    – Makyen Mod
    Oct 12, 2018 at 1:29
16

Only the person who rejected can tell you why they did so.

But I'm not sure I'd accept a "code-transcription" edit either. Most likely I'd skip, but it is something that I believe the original author should do, and most importantly transcription could introduce new/different errors that would render the question moot.

Your intentions are good, and your edit was approved in the end, but I have to question how a user who's not prepared to actually copy/paste their own code is going to be ready to support their question so it gets a useful answer anyway.

Until they do, the better move is to close the question so the user can work on it. Providing a working example is their responsability.

As such, an edit like this could be seen by some as a waste of time for other reviewers, since you are asking other users to put a lot of effort to validate your edit (they would have to compare your edit to the screenshot line by line to see it is actually good), for a question that would very likely not be up to scratch if its original user is not willing to do the minimum effort to provide a better quality post.

And the user how voted to reject must certainly was not "a troll". Assuming that someone who votes or act in a way you disagree could be described as such is not a great move.

5
  • 14
    I would go so far as saying that calling a user a "troll" is "unkind or unwelcoming"... Oct 9, 2018 at 15:45
  • 2
    To be fair, I didn't think it being rejected was a troll, but the reason for rejection is what made me think they were trolling. Oct 9, 2018 at 15:49
  • 4
    @SaggingRufus People who are trolling are called trolls, so you just called that reviewer a troll. Oct 9, 2018 at 17:48
  • 7
    @Mark They didn't. The use of the work might have not been the best idea, but they just came over to meta to make a question. No need to focus on that so much, IMO. I just mentioned in the end because I thought it could be useful, but it's not like Rufus went around lobbing personal accusations.
    – yivi
    Oct 9, 2018 at 17:50
  • 2
    It could be as simple as that the reviewer just accidentally picked the wrong close reason.
    – klutt
    Oct 10, 2018 at 10:21

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