Cum de a îmbunătăți acest cod? (Prea multe dacă)

O modalitate de a simplifica ... efectua apeluri, chiar dacă nu aveți nevoie de ele, dar conditionalise punerea în aplicare:

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

Apoi , în Square(sau orice altceva):

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

O alternativă similară ar fi de a menține condițional printBordercu funcția inițială:

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

Nu aș pasă de FI. Mi-ar face doar mai ușor de citit:

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

Mai întâi faci bine, acest lucru face exact ceea ce exprimă, nu vă faceți griji cu privire la spațiul pe care îl utilizați, cele mai multe dintre soluțiile de aici doar plin de noroi apa.

Dacă doriți într-adevăr să „fac“ ceva uita, dacă nu puteți muta parametrul de frontieră în piață. ai putea muta capitonarea de frontieră (10 în exemplul dvs. în pătrat), eventual, de asemenea, statul care ar trebui să fie prezentat la frontieră, și apoi apel doar square.printBorders (). Asta depinde foarte mult de contextul în care sunt utilizați acest lucru.

nu ați specificat ce limbaj de programare.

dacă ar fi java, enums poate oferi bună sintaxă ușor de citit, de siguranță de tip, precum și să profite de capacitățile eficiente-jonglarea biți ale implementării EnumSet.

Alternativ, ar putea oferi, de asemenea, o metodă de semnătură varargs, dar atunci nu poate fi sigur că metoda va fi numit cu printBorder (N, N), care nu prea are sens. folosind interfața EnumSet aveți această garanție.

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

Ce zici de:

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