Как исправить нарушение принципа DRY?

Question

[IRuuy]

public Token match(ITokenType expectedToken, int tokenNum) {
        if(hasToken(_currentTokenNum))
            if(expectedToken == getToken(tokenNum).get_type())
                return getToken(tokenNum);
        return null;
    }

public Token match(Class<?> clazz, int tokenNum) {
        if(hasToken(_currentTokenNum))
            if(clazz.isInstance(getToken(tokenNum).get_type()))
                return getToken(tokenNum);
        return null;
}

Comments

[Vindicar]

Сколько всего таких методов, и как часто их приходится писать?
Можно изобразить обобщённый метод, который во втором if вызывает переданную лямбду, но есть ли смысл?

Answer 1

[IRuuy]

Прикрепил решение

public Token match(ITokenType expectedToken, int tokenNum) {
    return match(t -> expectedToken == t.get_type(), tokenNum);
}

public Token match(Class<?> clazz, int tokenNum) {
    return match(t -> clazz.isInstance(t.get_type()), tokenNum);
}

private Token match(Predicate<Token> predicate, int tokenNum) {
    if(hasToken(_currentTokenNum))
        if (predicate.test(getToken(tokenNum)))
            return getToken(tokenNum);

    return null;
}

Answer 2

[Alexandroppolus]

Первым параметром передать проверялку для ITokenType.
Например, лямбду boolean checker(ITokenType tokenType)

внутри match вызвать с параметром getToken(tokenNum).get_type()

Answer 3

[Steel_Balls]

ChatGpt помог:

Comments

[IRuuy]

Стоит отметить, что в вашем решении в одном условии обрабатывается, как существование токена, так условие с его вызовом. Но как вызвать условие для токена, которого не существует?
Правильнее будет разбить на два:

private Token matchInternal(Suplier<Boolean> condition, int tokenNum) {
    if(hasToken(_currentTokenNum))
        if (condition.get())
            return getToken(tokenNum);
}

[mayton2019]

Самая первая версия кода была вполне читаемая. Я-бы ее оставил. Игры с функциями высших порядков
как-то очень странно смотрятся в аргументах методов класса.

Вобщем я-бы не упарывался такими достижениями.

[Dmitry Roo]

Там даже кнопка есть "copy code"
Почему вы ей не воспользовались?!

[IRuuy]

mayton2019, Но если понадобится добавить ещё один метод который сопоставляет, например, два поля из класса, в этом случае снова копипастить один и тот же метод, меняя только условие?