[Dúvida] fiz o exibe menu da seguinte forma porem nao sei se seria uma boa opcao e gostaria de otmizar mais meus codigos

public class Principal {
    private ConsumirApi consumirApi = new ConsumirApi();
    private ConverterDados converterDados = new ConverterDados();

    private String automovel;

    private Scanner leitura = new Scanner(System.in);

    private final String URL = "https://parallelum.com.br/fipe/api/v1/";

    private String endereco;

    public void exibeMenu(){
        var menu= """   
                Busque o preço de Automóveis pela tabela Fipe
                Carros 1
                Motos 2
                Caminhões 3"
                Escolha uma das opções digitando o número correspondente:
                """;
        System.out.println(menu);
        var opcao = leitura.nextInt();
        if (opcao == 1){
            automovel = "carros";
        } else if (opcao == 2) {
            automovel = "motos";
        } else if (opcao == 3) {
            automovel = "caminhoes";
        }else {
            System.out.println("Opção invalida ");
        }
        
        endereco = URL + automovel + "/marcas";
        var json = consumirApi.obterDados(endereco);
        System.out.println(json);