Wie dieser Code zu verbessern? (Zu viele if)

Eine Möglichkeit zur Vereinfachung es ... Anrufe tätigen, selbst wenn Sie sie nicht brauchen, aber conditionalise die Umsetzung:

printBorder(N, E, S, W){
  square.printBorder(n, 0,0,0,10);
  square.printBorder(e, 0,10,10,10);
  square.printBorder(s, 10,0,10,10);
  square.printBorder(w, 0,0,10,0);
}

Dann in Square(oder was auch immer):

printBorder(condition, top, left, bottom, right) {
  if (!condition) {
    return;
  }
  printBorder(top, left, bottom, right);
}

Eine ähnliche Alternative wäre die bedingten zu halten printBordermit der ursprünglichen Funktion:

printBorder(N, E, S, W){
  printBorder(n, 0,0,0,10);
  printBorder(e, 0,10,10,10);
  printBorder(s, 10,0,10,10);
  printBorder(w, 0,0,10,0);
}

printBorder(condition, top, left, bottom, right) {
  if (!condition) {
    return;
  }
  square.printBorder(top, left, bottom, right);
}

Ich würde über die ifs nicht. Ich würde einfach machen es besser lesbar:

printBorder(N, E, S, W){
  if(N) square.printBorder( 0,  0,  0, 10);
  if(E) square.printBorder( 0, 10, 10, 10);
  if(S) square.printBorder(10,  0, 10, 10);
  if(W) square.printBorder( 0,  0, 10,  0);
}

Zuerst ok tun, das ist genau das, was es ausdrückt, keine Sorge über den Raum, die Sie verwenden, die meisten Lösungen hier nur schlammig das Wasser.

Wenn Sie wirklich wollen, zu ‚tun‘ etwas aus, wenn Sie nicht die Border-Parameter auf den Platz bewegen können. Sie könnte die Grenze Polsterung (10 in Ihrem Beispiel auf den Platz), möglicherweise auch den Staat bewegen, die Grenze angezeigt werden soll, und dann square.printBorders nur () aufrufen. Das hängt viel von dem Kontext, in dem Sie diese verwenden.

Sie haben angeben, welche Programmiersprache nicht.

wenn es Java sind, können Aufzählungen eine gute lesbare Syntax, gibt Sicherheit, sowie die Vorteile der effizienten Bit-Hantieren Fähigkeiten der EnumSet Umsetzung nehmen.

Alternativ können Sie auch eine varargs Methodensignatur, bieten aber dann kann man nicht sicher sein, dass Ihre Methode mit printBorder (N, N) bezeichnet werden, die nicht wirklich Sinn machen. mit der EnumSet Schnittstelle haben Sie diese Garantie.

  public class PrintBorder {

    //this is your method without the if's
    public static void printBorder(EnumSet<Sides> sides) {
        for (Sides side : sides) {
            side.print(square);
        }
    }

    //use it like this
    public static void main(String[] args) {
        printBorder(EnumSet.of(N, E)); //static import here
    }

    //declare an enum for the sides.
    public enum Sides {
        N(0, 0, 0, 10),
        E(0, 10, 10, 10),
        S(10, 0, 10, 10),
        W(0, 0, 10, 0);

        private final int x1;
        private final int y1;
        private final int x2;
        private final int y2;

        Sides(int x1, int y1, int x2, int y2) {
            this.x1 = x1;
            this.y1 = y1;
            this.x2 = x2;
            this.y2 = y2;
        }

        //this method could as well be in the Square class, would be cleaner
        public void print(Square s) {
            s.printBorder(x1, y1, x2, y2);
        }

    }

    //boilerplate here
    private static final Square square = new Square();

    private static class Square {
        public void printBorder(int x1, int y1, int x2, int y2) {
            //do something..
        }
    }
}

Wie wäre es mit:

square.printBorder(N|E|W?0:10, N|S|W?0:10, N?0:10, N|E|S?10:0);