Comment améliorer ce code? (Trop si)

Une façon de simplifier ... faire des appels, même si vous ne les avez pas besoin, mais conditionalise la mise en œuvre:

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

Puis , en Square(ou autre):

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

Une alternative similaire serait de garder le conditionnel printBorderavec la fonction d' origine:

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

Je ne se soucient pas du ifs. Je venais de le rendre plus lisible:

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

D'abord, vous faites bien, cela fait exactement ce qu'il exprime, ne vous inquiétez pas l'espace que vous utilisez, la plupart des solutions ici juste l'eau boueuse.

Si vous voulez vraiment « faire » quelque chose regarder si vous ne pouvez pas déplacer le paramètre frontière sur la place. vous pouvez déplacer le rembourrage de bordure (10 dans votre exemple sur la place), peut-être aussi l'Etat qui devrait être montré frontière, puis il suffit d'appeler square.printBorders (). Cela dépend beaucoup du contexte dans lequel vous utilisez ce.

vous n'avez pas spécifié quel langage de programmation.

si elle était java, énumérations peut fournir une bonne syntaxe lisible, la sécurité de type, ainsi que de tirer parti des capacités de bit-tripoter efficace de la mise en œuvre EnumSet.

Sinon, vous pouvez également fournir une signature de méthode varargs, mais vous ne pouvez pas être sûr que votre méthode sera appelée avec printBorder (N, N), qui ne fait pas vraiment de sens. en utilisant l'interface EnumSet vous avez cette 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..
        }
    }
}

Que diriez-vous:

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