Wie diesen Code besser schreiben? (Zeitformate)

[internet]

Hallo,

ich möchte mir eine Liste mit den verschiedenen Zeitformate füllen.
Wie kann ich diesen Code jedoch besser schreiben?
Beispielsweise eine Schleife?

public List<String> findAllTimeFormat() {

		List<String> allTimeFormatList  = new ArrayList<String>();	
		Date currentTime = new Date();
		
		SimpleDateFormat simpleDateFormat = new SimpleDateFormat("HH:mm");
		String s = simpleDateFormat.format(currentTime);
		allTimeFormatList.add(s);
		
		simpleDateFormat = new SimpleDateFormat("HH.mm");
		s = simpleDateFormat.format(currentTime);
		allTimeFormatList.add(s);
		
		simpleDateFormat = new SimpleDateFormat("h:mm a");
		s = simpleDateFormat.format(currentTime);
		allTimeFormatList.add(s);
		
		return allTimeFormatList;
	}

 

[njans]

private String format(String format, Date date)
{
  SimpleDateFormat simpleDateFormat = new SimpleDateFormat(format);
  return simpleDateFormat.format(date);
}

public List<String> findAllTimeFormat() {
     
            List<String> allTimeFormatList  = new ArrayList<String>(); 
            Date currentTime = new Date();
           
           allTimeFormatList.add(format("HH:mm",currentTime));
           allTimeFormatList.add(format("h:mm a"",currentTime));

            return allTimeFormatList;
        }

So wäre es schöner, wenn du es kompakter willst oder was ist das Problem?

 

[Ruzmanz]

Das ist mMn auch noch Copy & Paste. Evtl. so:

private static final List<String> TIME_FORMATS = Arrays.asList("HH:mm", "h:mm a");

private String format(String format, Date date) {
  SimpleDateFormat simpleDateFormat = new SimpleDateFormat(format);
  return simpleDateFormat.format(date);
}
     
public List<String> findAllTimeFormat() {
  List<String> allTimeFormatList  = new ArrayList<>();
  Date currentTime = new Date();
 
  for(String timeFormat: TIME_FORMATS) {
    allTimeFormatList.add(format(timeFormat,currentTime));
  }

  return allTimeFormatList;
}