To expand on Flexo's answer, let's take a closer look at what the specific suggested edit actually changed:
It added backticks around the words "Service", "Court" and "Machine" in this sentence:
I have abstract class (Service) with two derived classes (Court and Machine).
These words are (presumably) class names, which are a bit of a borderline case: some users like to always mark them up as code, others prefer to leave them unformatted when they appear merely as names in prose, and not as part of an actual code fragment. Personally, I'd normally respect the original author's choice and not edit them either way, nor would I approve an edit that only added (or removed) backticks around class names. Still, if the rest of the edit was good, I also wouldn't reject just because of a detail like this.
(The edit also removed the parentheses, which is another stylistic choice. To me, the sentence seems readable with or without them, which again means that I would not personally edit them, but I also wouldn't reject an otherwise good edit just for doing so.)
It added backticks around "a pointer array of Service" in:
I also have a pointer array of Service, and [...]
This is unquestionably wrong. The phrase "a pointer array of Service" is not code, at least not in any programming language I know of, but merely a prose description of a data structure. Just adding backticks around the class name "Service" might have been valid (and certainly should have been done for consistency, if the earlier mentions of class names were also backticked), but there's no excuse (other than ignorance or inattentiveness, that is) for marking up "a pointer array" as code.
It added backticks around "if" in:
I'm using an if.
This is the only part of the edit that I would consider (marginally) useful, because without code markup, the keyword if
looks confusingly like the English conjunction "if", making the sentence potentially difficult to parse. Granted, in this particular case, the meaning seems clear enough even without the backticks.
It replaced quotes with backticks around "segmentation fault" in:
I receive a "segmentation fault" and [...]
If you consider the quotes around "segmentation fault" to imply that it's a direct quotation of an error message, then this change might almost be considered valid: one of the valid uses for code markup is for verbatim program output, especially if it contains actual code fragments (like error messages often do) or otherwise is formatted in a way that makes it more readable in a monospace font (and/or as a preformatted text block).
Still, in the particular case, I don't think that reasoning applies. Not only is "segmentation fault" a perfectly valid phrase in English prose text, describing a certain type of runtime error, but even if it was a direct quote, it still doesn't contain any embedded code or anything else that would benefit from the monospace styling.
It's also worth briefly noting a few things that the edit did not do:
- It didn't add the missing indefinite article into "I have abstract class [...]".
- It didn't capitalize (or, for that matter, add consistent backticks to) the class names in "both court and machine".
- It didn't fix the run-on sentence by (e.g.) replacing the comma after "machine" [sic] with a full stop. (In this case, the comma is clearly wrong both grammatically and stylistically.)
- It also didn't fix the run-on sentence in the next paragraph (e.g.) by replacing the "and" after "segmentation fault" with a full stop. (This is a more borderline case, since the original is at least grammatically valid, but I'd still consider a sentence break here a significant improvement.)
In short, the edit did (almost) nothing to improve the post, while the changes it did do were either actively harmful (backticking "a pointer array of Service") or were (at best) neutral style changes that disagreed with the original author's perfectly valid stylistic choices. In other words, the edit as a whole was slightly harmful, and mostly useless, with no real redeeming features.
You might legitimately disagree with some of this, if you're one of those people who strongly prefers to see class names marked as code. Even then, though, the suggested edit is at best incomplete, and partly just plain wrong. I could see picking the "Edit" option in review (as Josh KG eventually did, although their further edit really didn't improve things much at all) and fixing the remaining issues with the post, but certainly not accepting the suggested edit as is.
Also, even if you really, really like to use backticks for class names, it's important to note that others do disagree, and that for such legitimately controversial style choices, it's generally best to avoid needless editing and to respect the original author's choice even if it doesn't match your own personal preference.
pointer array of Service
is not code, and neither issegmentation fault
, and they should not have been formatted as such. I am otherwise in complete agreement with you and with @TigerhawkT3segmentation fault
is output.