Primeiro é uma boa prática fazer o que é correto e não porque algo é boa prática. E o que é correto depende da situação. O que as pessoas costumam chamar de boa prática é o que costuma ser correto na maioria das situações. O problema é que as pessoas sempre acabam achando que algo deve ser usado sempre.
Se é um exercício e ele pede fazer fazer algo de uma forma, seria bom fazer do jeito que ele pede. Deve ter um motivo para ele pedir assim. Ou ele é um exercício ruim. Ok, você quer ir além?
Indo no ponto que você quer. Lançar uma exceção no primeiro caso, até onde puder entender, é uma abuso do recurso.
Exceções devem ser usadas para situações excepcionais. Este é um caso que você está usando para controle de fluxo.
Este método claramente deve dizer se a operação foi bem sucedida ou não. Não há problema de race condition. Não há porque preferir o uso da exceção.
Se a operação do método foi mal sucedida, você usa essa informação para emitir a mensagem normalmente. Usa um if
normalmente.
Claro que pode ser correto usar uma exceção, mas no que você explicou não parece ser o caso. Se explicar de outra forma, se arrumar um argumento melhor para preferir a exceção, aí eu posso sugerir outra coisa. Por isso iniciei com a questão da boa prática.
No segundo caso fica claro pra mim que o método deve devolver o número do assento livre para que esse número possa ser utilizado em algum lugar. Pode não ser isso porque não estou com todo enunciado.
Você tem três opções para informar que não há assento livre.
- Escolher um número que indique isso, zero ou um número negativo por exemplo. Esta forma não é bem vista porque confunde o que está fazendo. Está usando o mesmo retorno para duas coisas. Mas não deixa de ser uma opção (ruim).
- Já que o método não pede para retornar nada, talvez tenha especificado em algum lugar que este valor está disponível de outra forma, então retorne um
bool
informando se tem assento livre ou não. Talvez o exercício deixou isso com você para te testar se iria tomar a decisão certa. É exatamente igual ao primeiro caso. - Usar um duplo retorno também pode ser a solução. Ou seja o retorno de fato é um
bool
e você passar uma variável como referência (out
é suficiente em C# se é a linguagem que você está usando) para pegar o número do assento. Veja o TryParse como exemplo. É um método que foi criado nos mesmo moldes justamente para resolver esse exato problema. E note que oTryParse
porque os desenvolvedores do .Net perceberam depois que o Parse lançava um exceção para controlar fluxo, para reger uma regra de negócio.
Mas posso te garantir que a InvalidOperationException
não foi criada para isso. É um exceção de mecanismo e não de regra de negócio.