2

Собственно, вопрос в том, можно ли реализовать это без использования дополнительных параметров. Если конкретно, в этом коде:

public static bool CheckAndStep(
    ref int x,
    ref int y,
    Direction direction,
    ref VirtualPoint[,] map,
    Point start,
    bool isBack
)
{
    if (
        map[
            x + direction.X,
            y + direction.Y
        ].Flag < 3
        || (
            isBack
            && map[
                x + direction.X,
                y + direction.Y
            ].Flag != 3
        )
    )
    {
        if (start.X != x || start.Y != y)
        {
            if (map[x + direction.X, y + direction.Y].Flag == 4) map[x, y].Flag = 3;
            map[x + direction.X, y + direction.Y].Flag = 4;
        }
        if (!isBack) Logger.Write(
            x
            + ", "
            + y
            + " -> "
            + (x + direction.X)
            + ", "
            + (y + direction.Y)
        );
        else Logger.Write(
            + (x + direction.X)
            + ", "
            + (y + direction.Y)
            + " <- "
            + x
            + ", "
            + y
        );
        x += direction.X;
        y += direction.Y;
        return true;
    }
    else return false;
}

Мне нужно, чтобы Logger.Write() срабатывал только в тесте, но не при обычном выполнении программы.

14
  • 2
    Подсказка: у вас слишком много конкатенаций, это очень медленно работает, попробуйте переписать на StringBuilder.
    – aepot
    23 дек 2020 в 12:26
  • 3
    Мне нужно, чтобы Logger.Write() срабатывал только в тесте Обычно так не делают, а делают так: объявляют в конструкторе зависимость от ILogger, а в тестах подкладывают нужную реализацию. А в продакшене конфигурируют логгер в зависимости от конфиг-файла, например проставляют уровень логирования в verbose/silent и путь в нужное место в %program data%. А так как вы хотите сделать... это придётся класс логгера делать знающим либо специальный параметр, либо знание об окружении (допустим, environment machinename, путей или чего-то ещё). Это хрупко, ломко -- лучше так не делать.
    – A K
    23 дек 2020 в 12:31
  • 1
    Плюс можно писать в консоль теста через Trace.WriteLine
    – A K
    23 дек 2020 в 12:40
  • 2
    Почему не срабатывает - это выясняется на этапе отладки. А тест просто сигнализирует: выполняется код правильно или нет. Если правильно - хорошо. Если неправильно - снова выполняете отладку. 23 дек 2020 в 12:51
  • 3
    @aepot, string.Format внутри себя использует StringBuilder
    – Grundy
    23 дек 2020 в 13:08

1 ответ 1

5

Нет, вам не должно такого хотеться.

Смысл юнит-теста как раз в том и состоит, что он тестирует ваш метод как он есть. Если вы тестируете одно, а в production исполняется другое, зачем тогда был весь тест?

Если вам нужна отладка, то можете добавить любой отладочный код, но обязательно уберите его после того, как ошибка будет найдена. Отладочному коду не место в вашем методе, когда отладка завершится.

Если вам нужно логирование (не только для отладки), тогда имеет смысл абстрагироваться от конкретного логгера, и логировать через интерфейс наподобие ILogger (а уж за поставку конкретной реализации может отвечать, например, dependency injection).

7
  • 1
    еще #if DEBUG можно.
    – aepot
    23 дек 2020 в 13:06
  • 1
    @aepot: Мммм, я как-то сомневаюсь, зачем вообще хранить отладочный код после того, как проблема будет решена? Его ещё и поддерживать придётся. А пока идёт отладка, #if DEBUG тоже ни к чему.
    – VladD
    23 дек 2020 в 13:07
  • 1
    например, если логирование нужно, если это отладочный билд, а не релизный.
    – aepot
    23 дек 2020 в 13:08
  • 1
    @aepot: Если это не отладочный код, а код логирования, то тогда DI нам в помощь, просто заинжектируем пустой логгер в релизном режиме. Иначе у нас есть риск, что в DEBUG и RELEASE-билдах будет выполняться разный код (например, код логирования может ненамеренно создавать побочные эффекты).
    – VladD
    23 дек 2020 в 13:08
  • 3
    Добавлю, что если хочется тестировать часть метода (ну вот хочется), это говорит о том, что метод нужно аккуратно разбить на несколько методов поменьше и проблема уйдет.
    – KoVadim
    23 дек 2020 в 13:36

Ваш ответ

Нажимая на кнопку «Отправить ответ», вы соглашаетесь с нашими условиями использования и подтверждаете, что прочитали и поняли наши политику конфиденциальности и нормы поведения.

Всё ещё ищете ответ? Посмотрите другие вопросы с метками или задайте свой вопрос.