Skip to main content
Code Review

Return to Answer

Commonmark migration
Source Link

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

    Possible improvements to Risk board game?

    This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  2. Trivia game refactoring

    In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  3. Moving method from derived class to base

    This question was tagged with , and asked why the resulting code was still ugly after refactoring.

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

    This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  2. Trivia game refactoring

    In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  3. Moving method from derived class to base

    This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

replaced http://codereview.stackexchange.com/ with https://codereview.stackexchange.com/
Source Link

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game? Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

Added another example question
Source Link
200_success
  • 145.6k
  • 4
  • 114
  • 283

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been twothree refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been two refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

The examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

I'd like to amend my (削除) previous proposal to abolish and (削除ここまで). Instead, I propose to make them both synonyms of .

Since this Meta question was first raised, there have been three refactoring-related questions that I'm aware of:

  1. Possible improvements to Risk board game?

This question was not tagged for , nor did it ask for it. However, one of the answers applied five refactoring techniques to achieve a significant improvement in the code.

  1. Trivia game refactoring

In contrast, this question was tagged for , and asked whether a laundry list of 16 refactoring techniques could be applied. Of the two answers, one had nothing to do with refactoring, and the other proposed an even more important refactoring technique that the poster had missed.

  1. Moving method from derived class to base

This question was tagged with , and asked why the resulting code was still ugly after refactoring.

The first two examples above illustrate why the tag is problematic:

  • Most of the time, when the OP asks for "refactoring" without being more specific, it's just a buzzword asking for a general improvement.
  • Once a relevant refactoring technique has been identified, applying it is a very methodical procedure — often mechanical enough to be automated in an IDE. Therefore, if the OP asks for specific refactorings to be applied, it borders on asking for code to be written.
  • However, it is within the OP's role to point out any particular concerns about the code, e.g. repetitiveness, lack of flexibility, spaghettification — in other words, .
  • Determining the applicability of refactoring rules should be a task for the reviewers. The OP doesn't even have to know anything about refactoring. Therefore, it doesn't make sense for the question to be tagged .

I think makes sense, since most of the important refactoring techniques are meant to clean up Bad Code Smells.

The third example is the closest I've seen to a good use of the tag. Even there, however, would also work, since the question is about why the code is still ugly.

Source Link
200_success
  • 145.6k
  • 4
  • 114
  • 283
Loading

AltStyle によって変換されたページ (->オリジナル) /