Viele if else-Anweisungen in einen saubereren Ansatz umwandeln

Meinen code hier erkennt, ob die mimeType ist gleich um einige MIME geben, wenn es ist, es wird eine gewisse Umstellung

public void convertToMp3(File src, File target,String mimeType){
    if(mimeType.equals("audio/mpeg")){
        ...
    }else if(mimeType.equals("audio/wav")){
        mp3ToWav();
    }else if(mimeType.equals("audio/ogg")){
        ...
    }else if(...){
    ... //More if and else here
}

Habe ich verkürzt mein code, weil es eine Menge von else if Aussagen, Was design pattern ist geeignet zum entfernen von vielen wenn und sonst oder else if Aussagen?

InformationsquelleAutor der Frage user962206 | 2013-01-03

  1. 191

    Haben, könnten Sie einen Converter - Schnittstelle. Dann könnte man eine Klasse erstellen, die für jeden MIME-Typ, wie:

    public interface Converter {

    public void convertToMp3();
    public void convertToOgg();

}

public class MpegConverter implements Converter {

    public void convertToMp3() {
        //Code here
    }

    public void convertToOgg() {
        //Code here
    }

}

    Bräuchte man eine Klasse wie diese für jeden Konverter. Dann könnte man die Einrichtung eine Karte wie diese:

    Map<String, Converter> mimeTypeMap = new HashMap<String, Converter>();

mimeTypeMap.put("audio/mpeg", new MpegConverter());

    Dann Ihre convertToMp3 Methode, so wird:

    Converter converter = mimeTypeMap.get(mimeType);
converter.convertToMp3();

    Mit diesem Ansatz könnte man leicht hinzufügen, verschiedene Konverter in die Zukunft.

    Alle ungetestet, wahrscheinlich nicht kompilieren, aber Sie bekommen die Idee

    InformationsquelleAutor der Antwort cowls

  2. 23

    Wenn Sie pre-JDK7, Sie können hinzufügen, ein enum für alle MIME Typen:

      public static enum MimeTypes {
      MP3, WAV, OGG
  }

  public class Stuff {
      ...
      switch (MimeTypes.valueOf(mimeType)) {
          case MP3: handleMP3(); break;
          case WAV: handleWAV(); break;
          case OGG: handleOGG(); break;
      }
  }

    Und haben einen Blick auf den Stack Overflow Frage Java - Konvertieren von String zu enumwie konvertieren von Strings, enums.

    InformationsquelleAutor der Antwort pgras

  3. 15

    Empfiehlt sich die Verwendung des Strategy design pattern und ein Map zu versenden, um die geeignete Strategie. Besonders nützlich, wenn Sie zusätzliche Funktionen benötigen, zusätzlich zu einer Umwandlung für einen bestimmten mimeType oder die Wandler sind große und komplizierte code-und würden Sie wollen, um jede arbeitet in seinem eigenen .java Datei.

     interface Convertor {
    void convert(File src, File target);
 }

 private static void convertWav(File src, File target) {
    ...
 }

 ...

 private static final Map< String, Convertor > convertors = new ...;
 static {
    convertors.put("audio/wav", new Convertor {
       void convert(File src, File target) {
          convertWav(src, target);
       }
    });
    convertors.put("audio/ogg", new Convertor {
       void convert(File src, File target) {
          convertOgg(src, target);
       }
    });
    ...
 }

 public void convertToMp3(File src, File target, String mimeType){
     final Convertor convertor = convertors.get(mimeType);
     if (convertor == null ) {
        ...
     } else {
        convertor.convert(src, target);
     }
 }

    InformationsquelleAutor der Antwort Raedwald

  4. 3

    Wenn Sie die gleichen Methoden für jeden Fall sollten Sie überprüfen, State-Muster

    InformationsquelleAutor der Antwort Marcin Szymczak

  5. 2

    Wenn Sie JDK 7 verwenden, können Sie switch-case konstruieren:

    Finden Sie unter: Warum kann ich nicht Umschalten auf einen String?

    Vorherige Versionen if-else ist die einzige Wahl.

    InformationsquelleAutor der Antwort Azodious

  6. 2

    Es ist definitiv eine Strategie-Entwurfsmuster. Aber Sie haben ein großes problem in Ihrem Allgemeinen design. Es ist nicht eine gute Programmierung Gewohnheit, die Verwendung von String zum identifizieren eines Typs. Ganz einfach, weil es leicht bearbeitbar und Sie können einen Grammatik Fehler und verbringen den ganzen Nachmittag auf der Suche für ein Programmier-Fehler. Sie können vermeiden Sie die Verwendung von map<>.

    Ich Folgendes vorschlagen:

    1. Verlängern class-Datei. Die neue Klasse wird ein neues Attribut FileType und eine neue Methode convertTo(FileType) der Klasse File. Dieses Attribut hält seine Art: "audio" "wav"... und wieder nicht die Verwendung von String, Enum Verwenden. In diesem Fall nannte ich es FileType. Erweitern Sie die Datei so viel wie Sie wollen: WavFile, AudioFile...
    2. Eine Strategie, die dp-erstellen Sie Ihre Konverter.
    3. Verwenden Sie ein Werk dp initialisiert den Konverter.
    4. Da jede Datei kennt seinen eigenen Typ und der Zieltyp (verwenden Sie convertTo () - Methode geben Sie die Ziel-Typ), wird er rufen Sie die Fabrik, um die richtigen Konverter automatisch!!!

    Dieses design ist skalierbar, und Sie können hinzufügen, so viel wie Sie brauchen, Dateityp und-Wandler.
    Die Antwort Ihr wählt ist irreführend!!!!
    Es gibt einen großen Unterschied zwischen Programmieren und hacken.

    InformationsquelleAutor der Antwort george

  7. 0

    Wenn Sie sich nicht mit Java 7 könnten Sie eine enum und verwenden Sie diesen Wert mit einem switch Fall. Sie müssen dann nur übergeben Sie den enum-Wert (statt einer Datei, ich weiß nicht, warum Sie das tun). Es sähe ordentlicher zu.

    Diese sollten mit dem, was Sie tun möchten:

     [Java Enum Examples][1] - 
 [Java Switch Case examples][2]

    InformationsquelleAutor der Antwort Skepi

Schreibe einen Kommentar