Как заставить одно из моих слов печатать вертикально?

Question

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

У меня правильное количество раз, но слова неправильно форматируются.

Это должно выглядеть так:

 b
lottery
 a
 t

  b
  o
  a
lottery

   b
   o
   a
lottery

Но мои печатают так:

b
lotto
    a
    t
                b
lotto
    a
    t
                b
lotto
    a
    t

Мой код отображается ниже, что не так с моим методом печати, который вызывает это?

 public class Assg23
{
    public static void main(String[] args)
    {

            String w1 = args[0];
            String w2 = args[1];

            int numberOfCrosses = 0;

            int pos1=0;
            int pos2=0;


            for(int i=0; i < w1.length(); i++)
            {
                for(int j=0; j < w2.length(); j++)
                {   
                    if(w1.charAt(i) == w2.charAt(j))
                    {
                        numberOfCrosses++;
                        crossesAt(w1, pos1, w2, pos2);
                        printCross(w1, pos1, w2, pos2);
                    }
                }
            }

        if(numberOfCrosses == 0)
        {
            System.out.println("Words do not cross");
        }
    }

    private static boolean crossesAt(String w1, int pos1, String w2, int pos2)
    {
        if(w1.charAt(pos1) == w2.charAt(pos2))
        {
            return true;
        }
        else
        {
            return false;
        }

    }

    private static void printCross(String w1, int pos1, String w2, int pos2)
    {
        for(int i=0; i < w1.length(); i++)
        {
            for(int j = 0; j < w2.length(); j++)
            {
                if((j== pos1) && i<w2.length())
                {
                    System.out.println(w2.charAt(i));
                }
                if((i == pos2) && j<w1.length())
                {   
                    System.out.print(w1.charAt(j));
                }
                else
                {
                    System.out.print(" ");
                }   

            }
        }



    }

Answer 1

Вот правильная версия вашего main() метод. Вы заметите, что я удалил crossesAt() метод, потому что все, что он делает, это сравнивает два символа, что можно сделать менее чем за одну строку кода.

public static void main(String[] args) {
    String w1 = args[0];
    String w2 = args[1];

    int numberOfCrosses = 0;

    for (int i=0; i < w1.length(); i++) {
        for (int j=0; j < w2.length(); j++) {
            if (w1.charAt(i) == w2.charAt(j)) {
                numberOfCrosses++;
                printCross(w1, i, w2, j);
            }
        }
    }

    if (numberOfCrosses == 0) {
        System.out.println("Words do not cross");
    }
}

А вот правильно работающая реализация вашего printCross() метод:

private static void printCross(String w1, int pos1, String w2, int pos2) {
    // you can replace this for-loop with the line that follows it
    // if you don't get compiler errors
    String spaces = "";
    for (int i=0; i < pos1; ++i) spaces += " ";
    //String spaces = String.format(String.format("%%0%dd", pos1), 0).replace("0", " ");

    for (int i=0; i < w2.length(); ++i) {
        if (i == pos2) {
            System.out.println(w1);
        }
        else {
            System.out.println(spaces + w2.charAt(i));
        }
    }
}

Вот пример printCross() использовался:

printCross("boat", 1, "lottery", 1);
printCross("coffee", 4, "beverage", 3);

Выход:

 l
boat
 t
 t
 e
 r
 y
    b
    e
    v
coffee
    r
    a
    g
    e

Answer 2

Используя решение, предложенное Тимом (ему достается кредит), вот полный код:

public static void main(String[] args) {
        String w1 = "boat";
        String w2 = "lottery";

        int numberOfCrosses = 0;

        int pos1=0;
        int pos2=0;
        int spaces = 0;

        for(int i=0; i < w1.length(); i++)
        {
            for(int j=0; j < w2.length(); j++)
            {
                spaces = 0;
                if(w1.charAt(i) == w2.charAt(j))
                {
                    numberOfCrosses++;
                    printCross(w1, i, w2);
                } else spaces++;
            }
        }

        if(numberOfCrosses == 0)
        {
            System.out.println("Words do not cross");
        }
    }

    private static void printCross(String w1, int pos, String w2) {
        String spaces = String.format(String.format("%%0%dd", pos1), 0).replace("0", " ");
        int lengthToTraverse = (w1.length() > w2.length()) ? (w1.length()) : (w2.length());
        for (int i=0; i < lengthToTraverse; ++i) {
            if (i == pos) {
                System.out.println(w2);
            }
            else {
                if(i<w1.length()) {
                    System.out.println(spaces + w1.charAt(i));
                }
            }
        }
    }

Я добавил чек на i и удалили один дополнительный параметр в методе. Вывод это:

b lottery a t b o a lottery b o a lottery