Jak poprawić ten kod? (Zbyt wielu, jeśli)

Jednym ze sposobów uproszczenia to ... dzwonić, nawet jeśli ich nie potrzebują, ale conditionalise realizację:

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);
}

Następnie w Square(lub cokolwiek):

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

Podobną alternatywą byłoby utrzymanie warunkowego printBorderz pierwotnej funkcji:

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);
}

Nie dbają o IFS. Chciałbym po prostu uczynić go bardziej czytelny:

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);
}

Po pierwsze robisz ok, to jest dokładnie to, co wyraża, nie martwić się o miejsca, którego używasz, większość rozwiązań tu tylko błotnistą wodę.

Jeśli naprawdę chcesz, aby „nie” coś wyglądać, jeśli nie można przenieść parametr granicę do kwadratu. można przesunąć wyściółkę graniczną (10 w przykładzie na placu), ewentualnie również do państwa, które mają być widoczne obramowanie, a potem po prostu zadzwonić square.printBorders (). To zależy wiele od kontekstu, w którym używasz tego.

Państwo nie określić język programowania.

gdyby była jawa, teksty stałe mogą zapewnić dobrą czytelną składnię, bezpieczeństwa typu, a także skorzystać z możliwości wydajnych-bitowych błahy wdrażania EnumSet.

Ewentualnie można również dostarczenie sposobu podpis varargs, ale nie można mieć pewność, że metoda zostanie wywołana z printBorder (N, N), która tak naprawdę nie ma sensu. za pomocą interfejsu EnumSet masz tę gwarancję.

  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..
        }
    }
}

Co powiesz na:

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