私は最近、従来の FizzBuzz ソリューションの作成を依頼されたインタビューを受けました。
1 から 100 までの数字のリストを出力します。
- 3 と 5 のすべての倍数について、数字は「FizzBuzz」に置き換えられます。
- 残りの 3 の倍数はすべて「Fizz」に置き換えられます。
- 残りの 5 の倍数はすべて「バズ」に置き換えられます。
私のソリューションは役割のために Java で書かれていましたが、これは必須ではありませんでした。インタビュアーはTDDのいくつかの証拠を知りたがっていたので、その精神で、私は独自のFizzBuzz単体テストを作成することにしました:
public class FizzBuzzTest {
@Test
public void testReturnsAnArrayOfOneHundred() {
String[] result = FizzBuzz.getResultAsArray();
assertEquals(100, result.length);
}
@Test
public void testPrintsAStringRepresentationOfTheArray() {
String result = FizzBuzz.getResultAsString();
assertNotNull(result);
assertNotSame(0, result.length());
assertEquals("1, 2", result.substring(0, 4));
}
@Test
public void testMultiplesOfThreeAndFivePrintFizzBuzz() {
String[] result = FizzBuzz.getResultAsArray();
// Check all instances of "FizzBuzz" in array
for (int i = 1; i <= 100; i++) {
if ((i % 3) == 0 && (i % 5) == 0) {
assertEquals("FizzBuzz", result[i - 1]);
}
}
}
@Test
public void testMultiplesOfThreeOnlyPrintFizz() {
String[] result = FizzBuzz.getResultAsArray();
// Check all instances of "Fizz" in array
for (int i = 1; i <= 100; i++) {
if ((i % 3) == 0 && !((i % 5) == 0)) {
assertEquals("Fizz", result[i - 1]);
}
}
}
@Test
public void testMultiplesOfFiveOnlyPrintBuzz() {
String[] result = FizzBuzz.getResultAsArray();
// Check all instances of "Buzz" in array
for (int i = 1; i <= 100; i++) {
if ((i % 5) == 0 && !((i % 3) == 0)) {
assertEquals("Buzz", result[i - 1]);
}
}
}
}
私の結果の実装は次のようになりました。
public class FizzBuzz {
private static final int MIN_VALUE = 1;
private static final int MAX_VALUE = 100;
private static String[] generate() {
List<String> items = new ArrayList<String>();
for (int i = MIN_VALUE; i <= MAX_VALUE; i++) {
boolean multipleOfThree = ((i % 3) == 0);
boolean multipleOfFive = ((i % 5) == 0);
if (multipleOfThree && multipleOfFive) {
items.add("FizzBuzz");
}
else if (multipleOfThree) {
items.add("Fizz");
}
else if (multipleOfFive) {
items.add("Buzz");
}
else {
items.add(String.valueOf(i));
}
}
return items.toArray(new String[0]);
}
public static String[] getResultAsArray() {
return generate();
}
public static String getResultAsString() {
String[] result = generate();
String output = "";
if (result.length > 0) {
output = Arrays.toString(result);
// Strip out the brackets from the result
output = output.substring(1, output.length() - 1);
}
return output;
}
public static final void main(String[] args) {
System.out.println(getResultAsString());
}
}
ソリューション全体で、ある夜遅くに約 20 分かかりました。コードを提出する前に、必要以上に長い間コードを神経質にチェックすることも含まれていました :)
私が最初に提出したものを見直してください: オーバーエンジニアリングを避けるために、早い段階で「倍数」の計算を generate() メソッドにマージすることに決めましたが、今では間違いだったと思います。また、別個の getResultAsArray/generate メソッドは明らかに OTT でした。getResultAsString は、一方が他方に委譲するだけなので、main() メソッドとマージすることもできます。
私はまだTDDにかなり慣れていないので、この場合はがっかりしたかもしれません。特にTDDの実践に関して、このアプローチを改善できる他の方法を探していますか?
アップデート
以下の非常に有用な提案に基づいて、より「TDD に適した」ものであると考えられるものへの回答を作り直しました。
変更点:
FizzBuzz ロジックを出力生成から分離して、ソリューションをよりスケーラブルにする
テストを簡素化するために、テストごとに 1 つのアサーションのみ
いずれの場合も、ロジックの最も基本的なユニットのみをテストする
ストリング構築を確認する最終テストも検証済み
コード：
public class FizzBuzzTest {
@Test
public void testMultipleOfThreeAndFivePrintsFizzBuzz() {
assertEquals("FizzBuzz", FizzBuzz.getResult(15));
}
@Test
public void testMultipleOfThreeOnlyPrintsFizz() {
assertEquals("Fizz", FizzBuzz.getResult(93));
}
@Test
public void testMultipleOfFiveOnlyPrintsBuzz() {
assertEquals("Buzz", FizzBuzz.getResult(10));
}
@Test
public void testInputOfEightPrintsTheNumber() {
assertEquals("8", FizzBuzz.getResult(8));
}
@Test
public void testOutputOfProgramIsANonEmptyString() {
String out = FizzBuzz.buildOutput();
assertNotNull(out);
assertNotSame(0, out.length());
}
}
public class FizzBuzz {
private static final int MIN_VALUE = 1;
private static final int MAX_VALUE = 100;
public static String getResult(int input) {
boolean multipleOfThree = ((input % 3) == 0);
boolean multipleOfFive = ((input % 5) == 0);
if (multipleOfThree && multipleOfFive) {
return "FizzBuzz";
}
else if (multipleOfThree) {
return "Fizz";
}
else if (multipleOfFive) {
return "Buzz";
}
return String.valueOf(input);
}
public static String buildOutput() {
StringBuilder output = new StringBuilder();
for (int i = MIN_VALUE; i <= MAX_VALUE; i++) {
output.append(getResult(i));
if (i < MAX_VALUE) {
output.append(", ");
}
}
return output.toString();
}
public static final void main(String[] args) {
System.out.println(buildOutput());
}
}