Hvordan forbedre denne koden? (For mange hvis)

En måte å forenkle det ... ringe selv om du ikke trenger dem, men conditionalise gjennomføringen:

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

Deretter i Square(eller hva):

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

En lignende alternativ ville være å holde betinget printBordermed den opprinnelige funksjon:

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

Jeg ville ikke bryr seg om ifs. Jeg vil bare gjøre den mer 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);
}

Først du gjør ok, dette gjør akkurat hva det uttrykker, ikke bry deg om den plassen du bruker, de fleste av løsningene her bare gjørmete vannet.

Hvis du virkelig ønsker å 'gjøre' noe ut hvis du ikke kan flytte Border parameter inn på torget. du kan flytte grensen padding (10 i ditt eksempel inn på torget), muligens også staten som grenser skal vises, og deretter bare ringe square.printBorders (). Det avhenger mye av konteksten der du bruker dette.

du ikke spesifisere hvilke programmeringsspråk.

om det var java, kan enums gi god lesbar syntaks, type sikkerhet, samt dra nytte av de effektive bit-fikling mulighetene i EnumSet gjennomføring.

Alternativt kan du også gi en varargs metode signatur, men da kan du ikke være sikker på at metoden vil bli kalt med printBorder (N, N), som egentlig ikke gir mening. bruker EnumSet grensesnittet du har denne garantien.

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

Hva med:

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