Javaでオブジェクトアレイをバブルソートする

Question

オブジェクトの配列をソートするにはどうすればよいですか？これは配列をソートするための私のコードですが、私は "not a statement"エラーを受け取ります：Movie temp = movies [b]; temp変数を、映画の正確な値/参照を保持するかのように宣言しますか？同じ配列内にある3つの異なるオブジェクト型のいずれかになる可能性がありますか？私はプログラミングに慣れていないので、もし私が無知であると思われるなら、私は謝罪する。私が間違って質問を述べた場合、私を修正したり、質問をしてください。

public static String bubbleSort(Movie[] movies) { 
    for (int a=1; a<movies.length; a++) { 
     for(int b=0; b<movies.length - a; b++) { 
      if (((movies[b].getTitle()).compareTo((movies[b+1].getTitle()))) > 0) 
       //swap movies[b] with movies[b+1] 
       Movie temp = movies[b]; 
      movies[b] = movies[b+1]; 
      movies[b+1] = temp; 
     } 
    } 
} 

Answer 1

配列は、それが唯一のタイプMovieのオブジェクトが含まできMovie[]として定義されます。だからそこにはMovieしか入れることができません。ただし、これを一般化するには、タイプをObject、配列をObject[]と定義する必要があります。

ただし、Movie.getTitle()を使用しているため、コード内に実際にMovieオブジェクトがあると仮定しています。 Objectの参照からアクセスすることはできません。あなたのオブジェクトにComparableを実装させ、配列のタイプと一時変数としてタイプComparableを使用することをお勧めします。

Answer 2

コレクションのAPIを使用して、独自の並べ替えの実装を行うホイールを再作成しないでください。

Answer 3

Movie temp = movies[b];は宣言であり、ステートメントではありません。あなたはこれをしたい：

 if (((movies[b].getTitle()).compareTo((movies[b+1].getTitle()))) > 0) 
     { 
      //swap movies[b] with movies[b+1] 
      Movie temp = movies[b]; 
      movies[b] = movies[b+1]; 
      movies[b+1] = temp; 
     } 

注すべてのIでしたがが宣言を含めることができるブロックにそれを作る、すべてのスワップコードの周りにカッコを追加します。私はこれがあなたが意図したものだと思うが、中括弧は省略した。

Answer 4

あなたは{テストの後に欠けていて、誤って文字列を返すと約束しました。これを試してください。

public static void bubbleSort(Movie[] movies) { 
    for (int a = 1; a < movies.length; a++) { 
     for (int b = 0; b < movies.length - a; b++) { 
      if (((movies[b].getTitle()) 
        .compareTo((movies[b + 1].getTitle()))) > 0) { 
       // swap movies[b] with movies[b+1] 
       Movie temp = movies[b]; 
       movies[b] = movies[b + 1]; 
       movies[b + 1] = temp; 
      } 
     } 
    } 
}