andbin ha scritto:
saraciao ha scritto:
cosi:
No.
public boolean numeroCivicoObbligatorio(String numeroCivico, String msgMancante, String msgInvalido) {
if (numeroCivico == null || numeroCivico.isEmpty()) {
messaggiErrore.add(msgMancante);
return false;
}
boolean ok = true;
try {
if (Integer.parseInt(numeroCivico) <= 0) {
ok = false;
}
} catch (NumberFormatException e) {
ok = false;
}
if (!ok) {
messaggiErrore.add(msgInvalido);
}
return ok;
}
Così è più lungo. Si può ridurre senza la variabile ok, accettando di ripetere 2 volte il add + return.
saraciao ha scritto:
Pero la seconda parte secondo me non va bene ma c'ho provato. Poi nel controller per visualizzaAvvisi ho questo che viene usato anche in altri metodi
public void VisualizzazioneAvvisi(String string)
Se cambio con VisualizzazioneAvvisi(ListList<String> erroreStrighe) poi per gli altri metodi non mi va più bene..come devo fare?
Puoi fare comunque entrambe/più cose. Avevo già detto prima di usare il costruttore di VisualizzazioneAvvisi solo per creare la dialog. E poi di fare dei metodi (almeno uno) per impostare gli errori. E qui puoi fare tutte le varianti che vuoi:
- con solo 1 String
- con un varargs String...
- con un array String[]
- con un List<String>
ok pero poi:
validazioni.NumeroCivicoObbligatorio(Integer.toString(numeroCivico), "Inserisci il numero civico", "Il numero Civico inserita non è valida");
È corretto? Se faccio così è corretto?
public boolean NumeroCivicoObbligatorio(String numeroCivico, String msgMancante, String msgInvalido) {
if (numeroCivico == null || numeroCivico.isEmpty()) {
messaggiErrore.add(msgMancante);
return false;
}
try {
if (Integer.parseInt(numeroCivico) <= 0) {
messaggiErrore.add(msgInvalido);
}
} catch (NumberFormatException e) {
messaggiErrore.add(msgInvalido);
}
return true;
}
Nel controller invece faccio:
public void VisualizzazioneAvvisi(String string)
{
if(dialogErrore == null) {
dialogErrore = new VisualizzazioneAvvisi(this,string);
dialogErrore.setVisible(true);
dialogErrore.setBounds(0, 0,360, 150);
dialogErrore.setLocationRelativeTo(menu);
}
else {
// cosa devo inserire??
}
}
ed il metodo in visualizzaAvvisi come deve crearlo?