Come rendere accettabile il metodo Elenco che contiene oggetti di qualsiasi tipo di dati

Question

Il mio obiettivo è controllare il tipo di messages e quindi convertirli in stringhe e aggiungerli. Come posso raggiungere questo obiettivo?

public void addMessages(List<?> messages) { 
    if (messages != null) { 
     if (messages instanceof String) { 
      for (String message : messages) { 
       this.messages.add(message); 
      } 
     } 
    } 

    else { 
     for (Object message:messages) { 
      this.messages.add(String.valueOf(message)); 
     } 
    } 
} 

Answer 1

Si può solo passare un elenco di oggetti - non è nemmeno necessario il if/else perché si può sempre chiamare semplicemente "toString()" o "String.valueOf" sull'oggetto messaggio:

public void addMessages(List<Object> messages) { 
    if (!CollectionUtils.isEmpty(messages)) { 
     for (Object message : messages) { 
      this.messages.add(String.valueOf(message)); 
     } 
    } 
} 

Nota a margine: Potrebbero sorgere problemi con l'inserimento di elementi nulli nell'elenco dei messaggi, quindi è consigliabile verificarne la presenza nel ciclo. Altre insidie ​​potenziali sono:

• this.messages non è inizializzata e l'aggiunta di messaggi getta un NullPointerException
• se questo è un metodo di un Singleton (ad esempio, un servizio di primavera) che lavorano con le variabili di istanza dovrebbe essere evitato

Answer 2

public void addMessages(List<Object> messages) { 

che è appena sufficiente dove messages tenere tutti i tipi di oggetti.