É melhor chamar uma função que não tenha um efeito nesse ponto, se ela melhorar a clareza do código?

Você tem uma invariante:

Only a single view (out of 3) is ever active (and visible).

Em seguida, sugiro que você forneça uma função para alternar a atividade e a visibilidade de todas as visualizações de uma só vez:

[setActiveView viewID:2]

Esta função irá:

• verifique se a visualização já está ativa, evitando trabalho desnecessário
• define a exibição como ativa e visível
• defina as outras duas visualizações como inativas e invisíveis

Tem várias vantagens sobre uma chamada bruta para setVisibility :

• amigável: chamá-lo desnecessariamente não cria um problema de desempenho
• defensiva: seu parâmetro único é muito mais difícil de ser alterado, enquanto para setVisibility é mais difícil lembrar que o intervalo de valores é 0.0f - 1.0f e que apenas um deve ser definido como 1.0f
• resiliente: o próximo cara não pode esquecer acidentalmente uma das visualizações
• adaptável: adicionar / remover uma visualização não exige examinar minuciosamente todo o código do aplicativo para descobrir onde estão os comutadores, uma única função (esta) precisa ser atualizada

Idealmente, para ajudar a impor a invariante, nenhuma outra função deve ser capaz de atrapalhar essa configuração ...

Idéia alternativa: se seu objetivo é evitar que bugs ocorram porque as pessoas esquecem que há três visualizações e fazem algo com apenas duas delas, o que elas realmente deveriam fazer com todas elas, faça uma função que torne <<> impossível esquecer:

setViewVisibilities(0.0f, 0.0f, 1.0f)

Agora você tem algo muito mais poderoso - tempo de compilação garante que você não esqueceu . Se você esquecer um parâmetro, o compilador gritará com você. Isso é muito mais útil do que comentários ou código desnecessário, pois cria um protocolo nomeado restrito que impõe a propriedade que você gosta.

Para o caso em que view3 não precisa de visibilidade, você pode adicionar um comportamento em que passar um valor especial como -1.0 ou nil ou algo assim significa "não altere a visibilidade da exibição em absoluto". Isso contorna o problema de configurar visibilidades desnecessariamente.

Acredito que adicionar um comentário explicando que a chamada é desnecessária (e por que) é melhor.

^{(talvez, o fato de uma chamada ser desnecessária, ou que você precise de um comentário sobre isso, possa ser um cheiro de código)}

Neste caso particular, @Mattieu M. tem a solução certa.

No caso mais geral, onde não há transformação similar, você tem que se perguntar: Existe qualquer chance de que um futuro programador possa bagunçar isso?

A resposta geralmente é sim. O que significa, sim, você deve adicionar a chamada. Talvez alguma versão futura do framework comece com todas as visualizações OFF em vez de ON.