E' buona norma avere un unico punto di ritorno (quando scrivere il metodo così è semplice).

Codice PHP:
private int metodo(){
   
int ret 0
  
// do your stuff here
   
if (condizione) {
      
ret 10
   }else{
      
ret 12
  }
return 
ret

questo è lecito. Come hai fatto tu ho la return dentro l'if (ma come ti ho detto sopra mi piace avere, quando posso, un unico punto di uscita)