Рисование фигур с "+, -, |" в Java

Question

Моя домашняя работа создает фигуры с «+, -, |»

я должен сделать это:

+--+--+--+ 
| | | | 
+--+--+--+ 
| | | | 
+--+--+--+ 
| | | | 
+--+--+--+ 

Но я видел это на выходе:

+--+--+--+ 
| | | | 
+--+--+--+ 
| | | | 
+--+--+--+ 
| | | | 
+--+--+--+ 
| | | | 

Вот мой код:

public class Method { 

     void MakingShapes() 
      { 
       for (int j = 0; j <= 3; j++) { 

        for (int i = 0; i <=9; i++) { 

         if(i%3==0) 
         { 
          System.out.print("+"); 
         } 

         else 
         { 
          System.out.print("-"); 
         } 
        } 

        System.out.print("\n"); 

        for (int i = 0; i <=9; i++) { 

         if(i%3==0) 
         { 
          System.out.print("|"); 

         } 
         else 
         { 
          System.out.print(" "); 
         } 
        } 
        System.out.println(""); 

       } 
      } 
     } 

Я знаю, что моя вина является в цикле, но я не могу найти его, но он выглядит как в последнем цикле. Спасибо.

Answer 1

Я думаю, что вы могли бы сделать это гораздо яснее, используя отдельные методы:

private void verticalLine() { 
    for (int i = 0; i <= 9; i++) { 

     if (i % 3 == 0) { 
     System.out.print("|"); 

     } else { 
     System.out.print(" "); 
     } 
    } 
    System.out.println(""); 
    } 

    private void horizontalLine() { 
    for (int i = 0; i <= 9; i++) { 

     if (i % 3 == 0) { 
     System.out.print("+"); 
     } 

     else { 
     System.out.print("-"); 
     } 
    } 

    System.out.print("\n"); 
    } 

Ваш метод становится:

void MakingShapes() { 
    for (int j = 0; j <= 3; j++) { 
     horizontalLine(); 
     verticalLine(); 
    } 
    } 

и это действительно очевидно, что вам просто нужно дополнительное horizontalLine() после вызовов цикла, чтобы закрыть экран. Как было отмечено в комментарии от @ JamesC.TaylorIV, чтобы получить целевой выход также необходимо уменьшить цикл только работает в три раза:

void MakingShapes() { 
    for (int j = 0; j < 3; j++) { 
     horizontalLine(); 
     verticalLine(); 
    } 
    horizontalLine(); 
    } 

Answer 2

Проблема заключается в том, что ваш внешний контур работает 4 раза. Он будет выполнять внутреннее содержимое 4 раза, что заставляет цикл for печатать символы | в дополнительное время. Попробуйте добавить условие к вам | цикл, который делает, что он работает 1 раз меньше внешнего контура:

for (int i = 0; i <=9 && j < 3; i++) 

Answer 3

Чтобы сделать это лучше код, то первое, что вы должны научиться состоит в том, чтобы сделать это повторно использовать с помощью частных методов: например, вы должны закодировать частный метод для печати последовательности «+ - + - + - +», а другой - для печати «| | | |», последовательность.

Как только вы достигли этого, вам понадобится всего один цикл, без IFs, для получения требуемой формы. И, в конце, вам придется снова вызвать метод «+ - + - + - +».

Попробуйте этот способ и поймите, как вы будете создавать более короткую, легко читаемую и поддерживающую программу.

Answer 4

1. Я рекомендую отформатировать кронштейны по-разному (открывающая фигурная скобка на той же линии). Это экономит некоторое вертикальное пространство, и код становится более разборчивым.

2. Я немного изменил свой код: я добавил методы для улучшения удобочитаемости и изменил инструкции if на встроенные (опять же, чтобы уменьшить требуемое вертикальное пространство и, следовательно, улучшить читаемость).

   private void printHorizontalLine() { 
       for (int i = 0; i <= 9; i++) { 
        System.out.print((i % 3 == 0) ? "+" : "-"); 
       } 
   } 
   
   private void printCells() { 
       for (int i = 0; i <= 9; i++) { 
        System.out.print((i % 3 == 0) ? "|" : " "); 
       } 
   } 
   
   void MakingShapes() { 
       for (int j = 0; j <= 3; j++) { 
        this.printHorizontalLine(); 
        System.out.println(); 
        this.printCells(); 
        System.out.println(); 
       } 
       this.printHorizontalLine(); 
   }