Πώς να βελτιώσει αυτόν τον κωδικό; (Πάρα πολλά αν)

Ένας τρόπος για την απλοποίηση το ... πραγματοποιείτε κλήσεις, ακόμη και αν δεν τα χρειάζεστε, αλλά conditionalise της εφαρμογής:

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

Στη συνέχεια, Square(ή οτιδήποτε άλλο):

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

Μια παρόμοια εναλλακτική λύση θα ήταν να κρατήσει την υπό όρους printBorderμε την αρχική λειτουργία:

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

Εγώ δεν θα νοιάζονται για το IFS. Θα ήθελα απλά να είναι πιο ευανάγνωστο:

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

Πρώτα κάνετε εντάξει, αυτό κάνει ακριβώς αυτό που εκφράζει, μην ανησυχείτε για το χώρο που χρησιμοποιείτε, οι περισσότερες από τις λύσεις εδώ μόνο λάσπη το νερό.

Αν θέλετε πραγματικά να «κάνει» κάτι εξετάσουμε αν δεν μπορείτε να μετακινήσετε την παράμετρο των συνόρων στην πλατεία. μπορείτε να μετακινήσετε τα σύνορα padding (10 στο παράδειγμά σας στην πλατεία), ενδεχομένως και το μέλος που θα πρέπει να εμφανίζονται στα σύνορα, και στη συνέχεια απλά καλέστε square.printBorders (). Αυτό εξαρτάται πολύ από το πλαίσιο στο οποίο θα χρησιμοποιείτε αυτό.

δεν διευκρίνισε ποια γλώσσα προγραμματισμού.

αν ήταν java, enums μπορεί να προσφέρει καλό αναγνώσιμη σύνταξη, την ασφάλεια του τύπου, καθώς και να επωφεληθούν από τις δυνατότητες αποδοτικής λίγο-κόλπα της εφαρμογής EnumSet.

Εναλλακτικά, θα μπορούσατε επίσης να παρέχει μια μέθοδο varargs υπογραφή, αλλά τότε δεν μπορείτε να είστε σίγουροι ότι η μέθοδος θα κληθεί με printBorder (Ν, Ν), η οποία δεν έχει νόημα. χρησιμοποιώντας το περιβάλλον EnumSet που έχουν αυτή την εγγύηση.

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

Τι λέτε για:

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