É uma prática ruim usar o retorno dentro de um método vazio?

Imagine o seguinte código:

void DoThis()
{
    if (!isValid) return;

    DoThat();
}

void DoThat() {
    Console.WriteLine("DoThat()");
}

Há algum problema em usar um retorno dentro de um método vazio? Tem alguma penalidade de desempenho? Ou seria melhor escrever um código como este:

void DoThis()
{
    if (isValid)
    {
        DoThat();
    }
}

questionAnswers(10)

yourAnswerToTheQuestion