21
Not a problem
If it would be a problem, you wouldn't be allowed to use someone else's libraries either.
That rule is there to prevent people from dumping code that they don't understand and/or 80% copied from the internet. We can't review that for a whole lot of reasons explained in other meta posts, enough of them already visible in the 'Related' section ...
14
Let me know if I am allowed to do this
You're allowed to do this, and it has been done before. However, it's not a particularly good idea with regards to question clarity and quality. This is one of those cases where if you can do it better, you probably should do it better.
Examples that such a scheme has been done before:
JSON Test Harness: Part 1
JSON ...
9
Unfortunately, your question is based on a faulty premise. The code to be reviewed consists of a makeBst() function with one test case. That test case produces incorrect output:
class BinaryTreeNode {
constructor(value) {
this.value = value;
this.left = null;
this.right = null;
}
}
function makeBst(arr){
if(!arr || arr.length <= ...
8
Traditionally, answers to off-topic questions (especially to those questions that are obviously off-topic) are not considered valid answers for the purposes of answer invalidation rollbacks.
We'd rather have an on-topic question and inconvenience an answerer than having to roll back the question into an off-topic state just to avoid invalidating an answer.
...
8
We have an existing FAQ that discourages adding updated code.
You also should not append your revised code to the question. Posting improved code is an answering activity, not a questioning activity. An example of such a problematic edit is Revision 2 of Optimize vector rotation. While the addendum to the question does not necessarily invalidate ...
7
In general, comments are ephemeral.... they are not designed to have long-lasting content. Comments are to seek clarification on posts, and point out short term issues. Once the issues are addressed, the comments should be removed.
In the case of this specific question, things became a real mess.... in part, because the question started off "broken", and ...
7
Moderators delete answers for a variety of reasons.
Here on Code Review, there is even at least one more reason than on most other Stack Exchange sites.
One of the reasons specific to Code Review is the "Independent and unjustified solution". Answers that provide a completely independent solution and don't justify why the solution is better are not a review....
6
Vote to close zombies which should be closed. That makes it easier to find the ones which could be answered.
4
An answer does not invalidate any other answers. Answers are independent of each other and are responses to questions. (If you are referencing another answer and that answer changes it is up to you to update your answer accordingly.) It seems that the second answer simply adds an alternative approach to the existing answer. The fact that the second answer ...
4
I think that Matt said it best here when he said "readers seeing your post for the first time, don't need to see which part was in the original post, and which parts were added." Granted he was talking about edits to the question that were done for other reasons, But the point is made clear that Stack Exchange sites are not forums and I would take it a ...
4
Lacking answers, I've suggested an edit to the api tag to clarify its intended usage (code that implements and/or wraps an API, not code that uses one). Feel free to improve it further along those lines.
Allowing the tag to be used on any question containing code that simply talks to an API would make the tag way, way too broad. It would include just about ...
4
Indentation vs conventions
I recently declined the following suggested edit: removing-leading-white-space-in-question-code-blocks.
While the indentation was a good suggestion, the editor took it a step further and also changed whitespaces in the code. This to me indicates deviating from the OP's intend (not indent, or maybe also indent .. mhmm). By doing ...
3
The point is that normal users generally won't see these questions and it might be good to bring them back into focus, so that someone can try to answer.
I support the idea of bounties on zombie questions, but for the opposite reason! :) I have [c++] in my favorite tags, and I typically look at the "unanswered questions" tab when looking for something to ...
3
I made a query questions-in-need-of-a-friendly-helpful-comment to try to find questions where moderating comments are invited. Feel free to tweek this query at own will. Note that the database this query runs on a database that gets refreshed with realtime data periodically.
Your concerns are justified; we recently had an influx of such questions.
3
Please keep suggestions about the code in answers.
Comments are about improving the question in ways that the OP is allowed, and so suggesting changes to the code in comments sends mixed messages that it's ok to change the code.
A simple answer containing just the following is allowed.
I think you want a cascade effect.
This has the benefit that we ...
2
Since gamification is baked into the system: in your own profile, you can choose two badges to track progress towards them: one "normal" badge and one tag badge. (Alternatively you can track a "normal" badge and a privilege).
Pick a tag badge, and then click on it: a little UI element pops up with a button to "Go get it". This opens the corresponding tag ...
1
Focus on activity of recent questions
I would focus on actively reviewing, voting, commenting, closing recent questions to avoid creating zombies in the making. As the question count goes up, the percentage of zombies naturally goes down.
That being said, if I find interesting old questions without up-voted answers, I might edit them to get them bumped. ...
1
The Photo SE Electric Boogaloo : Now this is kind of far fetched, but Photo SE has a Photo of the Week thing.
We could maybe ask StackExchange for a similar widget but that would show a small list of zombies, or something like that (I've thought less about this point than others)
1
The Walking Dead Prison : If you haven't watched Walking Dead, don't go past season 1 (citation needed), because the season two is loonnnggg. In the season 3, they end up in a prison where they need to work together to kill zombies are survive.
We could have a chat room specifically for zombie killing. The reasoning behind this is :
We have zombies that ...
1
The Rising Dead Challenge : Rising Dead is a video game where you kill plenty of zombies for whichever reason they must explain later than after the 10 first minutes of the game where I decided it wasn't a game for me.
We do a monthly friendly competition about who kills the most zombies. Using an SEDE query I've created (The set date is just as an example,...
1
I cannot imagine any question where it's - given that it is suitable for at least one of them and these two sites are the only choices - ambiguous if it should be posted on Codereview or Codegolf. The latter one is purely for fun, and the only objective is to write as short code as possible by any means possible. Performance, readability, memory usage and ...
Only top voted, non community-wiki answers of a minimum length are eligible
