Как улучшить мое решение FizzBuzz для роли TDD? [закрыто]

В настоящее время этот вопрос не подходит для нашего формата вопросов и ответов. Мы ожидаем, что ответы будут подкреплены фактами, ссылками или опытом, но этот вопрос, скорее всего, вызовет дебаты, аргументы, опросы или расширенное обсуждение. Если вы считаете, что этот вопрос можно улучшить и, возможно, снова открыть, посетите справочный центрдля получения рекомендаций.

Закрыт 7 лет назад.

Недавно у меня было интервью, где меня попросили создать традиционное решение FizzBuzz:

Вывести список чисел от 1 до 100.

Для всех чисел, кратных 3 и 5, число заменяется на «FizzBuzz».

Для всех оставшихся чисел, кратных 3, число заменяется на "Fizz"

Для всех остальных чисел, кратных 5, число заменяется на "Buzz"

Мое решение было написано на 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 items = new ArrayList();

        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 отделена от генерация выходных данных, чтобы сделать решение более масштабируемым
Только одно утверждение на тест, чтобы упростить их
В каждом случае проверяется только самая базовая единица логики
Также проверяется финальный тест для подтверждения построения строки

Код:

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());
    }
}

java tdd fizzbuzz

задан seanhodges 6 March 2012 в 14:43

0 ответов

Другие вопросы по тегам:

java tdd fizzbuzz

Как улучшить мое решение FizzBuzz для роли TDD? [закрыто]

0 ответов

Похожие вопросы: