2

このコードの単語は、共通の文字で交差し、単語が互いに交差する回数だけ出力されることになっています。

私は正しい回数を持っていますが、単語が正しい方法でフォーマットされていません。

次のようになります。

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

            }
        }



    }
4

2 に答える 2

3

これがあなたのmain()メソッドの正しいバージョンです。このメソッドを削除したことにお気付きでしょう。このcrossesAt()メソッドは 2 つの文字を比較するだけなので、1 行未満のコードで実行できます。

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
于 2015-10-19T05:38:43.313 に答える
0

Tim によって提案された解決策を使用して(クレジットは彼に与えられます)、完全なコードは次のとおりです。

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し、メソッドの余分なパラメーターを 1 つ削除しました。それを出力します：

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

于 2015-10-19T05:50:32.453 に答える