Javaのプログラミングで詰まっています

Question

最近、独学でJavaの学習を始めた新参者です。
エクリプスを利用して自分でプログラムを書いているのですが
エラーの出てる箇所の修正方法がわからず詰まっている状態です。
よろしければ、アドバイスのほどよろしくお願いします。

ソースコード
import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;

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

//じゃんけんの定数定義

List<String>player = new ArrayList<>();
player.add("グー");
player.add("チョキ");
player.add("パー");

//入力画面
System.out.println("グー ,チョキ ,パーのいずれかを入力してください");

Scanner scan = new Scanner(System.in);
String str = scan.next();
if (str.equals(player.get(0))) {
System.out.println("あなたがだしたのは" + player.get(0) + "です");
} if (str.equals(player.get(1))) {
System.out.println("あなたがだしたのは" + player.get(1) + "です");
} if (str.equals(player.get(2))) {
System.out.println("あなたがだしたのは" + player.get(2) + "です");
}
else { System.out.println("入力された値が不正です");
return;
}

List<String>enemy = new ArrayList<>();  
enemy.add("グー");
enemy.add("チョキ");
enemy.add("パー");
Collections.shuffle(enemy);

System.out.println("相手がだしたのは" + enemy.get(0) + "です");//繰り返し表示

//じゃんけんの結果
if((player.get(0)) && (enemy.get(1)) || (player.get(1)) && (enemy.get(2)) ||
(player.get(2)) && (enemy.get(0))) {
System.out.println("あなたの勝ちです");
} else if( enemy.get(0) == player.get(0) || enemy.get(1) == player.get(1) ||
enemy.get(2) == player.get(2)){
System.out.println("引き分けです");
} else {
System.out.println("あなたの負けです");
}

}
}
.
.
.
エラーの出る部分
Collections.shuffle(enemy);

if((player.get(0)) && (enemy.get(1)) || (player.get(1)) && (enemy.get(2)) ||
(player.get(2)) && (enemy.get(0))) {
System.out.println("あなたの勝ちです");

上記の２つの部分でエラーが発生しております。
試行錯誤を重ねていますがエラーが消えてくれません。

Comments

[carimatics]

コード部分にはコードブロックを利用してください。コードが確認しやすくなります。詳しくはteratailのヘルプを参照してください。また、エラーが出た場合はエラーメッセージも貼るようにしてください。

Answer 1

player.getはString型ですよね？
if文は評価した結果がboolean型になる必要があります。
このif文は何がしたいんでしょう、説明してください。

Comments

[yamagatamasa]

yonaさん、ご回答ありがとうございます。
String型の比較が目的だったのですが、私が説明足らずなのとif文に対しての理解が不足していたみたいです。
次回、質問させてもらうときはプログラムへの理解を深めつつ、何をしたいか明確に説明できる文章で投稿するように努力します。

Answer 2

//じゃんけんの結果
より下で何がしたいのかというと 恐らくは 例えば
(player.get(0)) && (enemy.get(1))
の部分だと、本当は
str.equals(player.get(0)) && enemy.get(0).equals(player.get(1))
と書きたかったのだと思います
それが端折られて(間違って)いるため目的の結果が出ない状態になっています。
とりあえず修正して完成させたので以下のコードが参考になるのではないかと思います

import java.util.ArrayList;
import java.util.List;
import java.util.Scanner;
import java.util.Collections;

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

		// じゃんけんの定数定義
		int m = 0;
		int t = 0;

		List<String> player = new ArrayList<>();
		player.add("グー");
		player.add("チョキ");
		player.add("パー");

		// 入力画面
		System.out.println("グー ,チョキ ,パーのいずれかを入力してください");

		Scanner scan = new Scanner(System.in);
		String str = scan.next();
		if (str.equals(player.get(0))) {
			System.out.println("あなたがだしたのは" + player.get(0) + "です");
		} else if (str.equals(player.get(1))) {
			m = 1;
			System.out.println("あなたがだしたのは" + player.get(1) + "です");
		} else if (str.equals(player.get(2))) {
			m = 2;
			System.out.println("あなたがだしたのは" + player.get(2) + "です");
		} else {
			System.out.println("入力された値が不正です");

		}

		List<String> enemy = new ArrayList<>();
		enemy.add("グー");
		enemy.add("チョキ");
		enemy.add("パー");
		Collections.shuffle(enemy);
		System.out.println("相手がだしたのは" + enemy.get(0) + "です");// 繰り返し表示

		if (enemy.get(0).equals(player.get(0))) {
		}
		if (enemy.get(0).equals(player.get(1))) {
			t = 1;
		}
		if (enemy.get(0).equals(player.get(2))) {
			t = 2;
		}

		// じゃんけんの結果

		if (m == t) {
			System.out.println("引き分けです");
		}
		if ((m == 0 && t == 1) || (m == 1 && t == 2) || (m == 2 && t == 0)) {
			System.out.println("あなたの勝ちです");
		}
		if ((m == 1 && t == 0) || (m == 2 && t == 1) || (m == 0 && t == 2)) {
			System.out.println("あなたの負けです");
		}

	}
}

Comments

[yamagatamasa]

tetratailさん、説明足らずな質問の意図を汲み取って、修正コードまでつけて回答していただいて本当にありがとうございます！
おかげさまで無事コンパイルできました。
次回質問させていただくときは明確で詳細な文章を心掛けさせていただきます。
もし機会があればよろしくお願い致します。

Answer 3

ひとつ目はCollectionsクラスがインポートされていないのが原因です。
インポート文として以下を追加してください。

import java.util.Collections;

２つ目は括弧の対応が取れていないのが原因です。
if文の書式は基本的には以下のようになります。

if (条件) {
    // 処理
}

論理和や論理積で複数の条件を絡めたチェックをする場合も、ifキーワード直後の括弧内に条件が全て含まれるようにしてください。

if (条件1 && 条件2 || 条件3) {
    // 処理
}

Comments

[yamagatamasa]

Collectionsクラスのインポートは全く気づきませんでした．．．
if文に関しては全く理解が足りてなかったみたいです．．．
皆さんの回答を参考にさせていただき、無事にコンパイルできました！
Carimaticsさん、素早くご回答していただきまして有難うございます！

[yamagatamasa]

追記でお礼申し上げます。
投稿が初めてのものでコードブロックなどの機能をよく理解しておりませんでした。
次回質問させていただくときは機能を利用してわかりやすいものを投稿させていただきます。
ご親切にアドバイスをしていただきまして有難うございます。

[carimatics]

回答が参考になったのなら幸いです。

解決済みの質問も他の方が参考にする可能性があります。
次回からと言わず今回の質問も読みやすく整形していただけると親切かと思います。
ご検討ください。