consiglio sull'acquisizione in input

[abbobba]

Salve
dovrei acquisire in input dei valori per forza mediante lo scanner(System.in).
Dovrei però verificare che il numero inserito o la stinga inserita non superino 4 numeri/caratteri.
Avevo pensato ad un controllo sulla variabile che riceve l'input. Per i numeri il controllo sarebbe
Es. n < 10000
per i caratteri
Es. stringa.length < 5
queste condizioni inserite in un ciclo magari con un booleano per uscire solo se si rispettano le condizioni.


Secondo questo è l'unico modo o si può ottimizzare qualcosa?

[andbin]

Originariamente inviato da abbobba
Secondo questo è l'unico modo o si può ottimizzare qualcosa?

Innanzitutto se usi nextInt() (o i nextXyz degli altri primitivi es. nextLong ecc...) di Scanner, bisogna sempre prestare attenzione ai token malformati. Se il token non è un valore primitivo in formato corretto es. "12a", i nextXyz lanciano InputMismatchException ma il token rimane nel buffer interno e quindi bisognerebbe toglierlo.
Un esempio di metodo basilare, con un loop per ripetere l'input se malformato, è questo (presuppone un 'scanner' come variabile di istanza):

public int readIntInput(String prompt) {
    while (true) {
        try {
            System.out.print(prompt);
            return scanner.nextInt();
        } catch (InputMismatchException e) {
            scanner.next();    // butta via il token
        }
    }
}

Oltre a questo, puoi chiaramente "farcire" il metodo come vuoi, es. mettere un messaggio di errore o, come hai appena chiesto, fare una ulteriore validazione sul range del valore, che sarebbe da fare poco prima di far ritornare il valore.

Ma quanto ho mostrato è già abbastanza al minimo (se vuoi fare un loop con gestione dell'input malformato) ... nel senso che meno di questo non avrei idea di come si possa fare.

[abbobba]

ti ringrazio della risposta però sto preparando l'esame di fondamenti di programmazione dove il blocco try-catch non è stato spiegato... mi salvo questo tuo codice per il futuro

il controllo che ho detto prima l'ho implementato in questo modo

                do {
		System.out.println("Inserire numero di conto (max 4 cifre): ");
		nConto1 = sc.nextInt();
		}
		while (nConto1 > MAX_LENGHT ); 
		
		do {
		System.out.println("Inserire nome (max 5 caratteri): ");
		nome1 = sc.next();
		}
		while (nome1.length() > MAX_LENGHT2);

può andare o è troppo rozzo?


PS ma nel tuo codice il while non cicla all'infinito?

[Alex'87]

Originariamente inviato da abbobba
PS ma nel tuo codice il while non cicla all'infinito?

C'è un return, guarda bene

[andbin]

Originariamente inviato da abbobba
può andare o è troppo rozzo?

Certo, può andare (con le premesse che hai fatto). A parte il fatto che non gestendo InputMismatchException lasci aperta questa questione. Ma potrebbe anche essere una strategia nel senso: se sbuca fuori una eccezione come questa, accetti che magari il programma termini immediatamente.
Comunque va bene, non preoccuparti, non avendo ancora visto try-catch sei in un certo senso "giustificato".