JAVAでじゃんけんあいこしか出ず。

Question

```### 前提・実現したいこと
java初心者です初めての課題として
javaで基礎的なじゃんけんのゲームを作っています。
ですがグー、チョキ、パーのいずれを入力してもあいこになってしまいます。
それと別問題かもしれませんがインスタンスの生成時にstaticとうたないとエラーが出るので仕方なく打っているのですがそこについてもお聞きしたいです。

ここに質問の内容を詳しく書いてください。
（例）PHP(CakePHP)で●●なシステムを作っています。
■■な機能を実装中に以下のエラーメッセージが発生しました。

### 発生している問題・エラーメッセージ
必ずあいこになる
```


```JAVA
package myfirstone; 
import java.util.Random;
import java.util.Scanner;

public class Myfirstclass {

	public static void main(String[] args) {
     Scanner scanner = new Scanner(System.in);
     
    System.out.println("グー、チョキ、パーのいずれかを入力してください");
    String pon = scanner.nextLine();
    scanner.close();
    System.out.println("あなたは" + pon + "を出しました。");
    System.out.println("-----------------------");
    Random rand = new Random();
    int teki = rand.nextInt(3);
    switch (teki) {
    case 0:
    	System.out.println("相手はグーを出しました");
    	break;
    case 1:
    	System.out.println("相手はチョキを出しました");
    	break;
    case 2:
    	System.out.println("相手はパーを出しました");
    	break;	
    }
    System.out.println("-----------------------");
    paa.janken(teki);
    
	}

}
```
```
```JAVA
package myfirstone;

public class paa {
	public  static String pon;
	public  static int teki;

	public static void janken(int teki) {

		
		
 if(pon=="グー" && teki==1 ||pon=="チョキ" && teki==2 || pon=="パー" && teki==0) {
			System.out.println("あなたの勝ちです");
		}
	else if(pon=="グー" && teki==2|| pon=="チョキ" && teki==0||pon=="パー" && teki==1) {
		System.out.println("あなたの負けです");
	}
	else {
		System.out.println("あいこです");
	} 
	
	}

  }

```


### 試したこと

入力をグー、チョキ、パーではなく数字に変えるとできたが、グー、チョキ、パーの入力で行いたい。

### 補足情報（FW/ツールのバージョンなど）

ここにより詳細な情報を記載してください。

Accepted Answer

Javaでの文字列比較は、"==演算子" ではなく、equalsメソッドを使った方が安全のようです。

ヒント: [参考1](https://qiita.com/awesam86/items/5d3461ecd4af30d88d71)

> 上記コードのif文の条件がtrueとなるので、値の比較ができているように錯覚してしまうかもしれませんが、これは参照先(アドレス値)の比較をして、String Constant Poolの再利用によって参照先(アドレス値)が同じだったためtrueとなるのです。なので、下記のような場合には成立しません。

とあるので、場合によってはうまくいかない可能性もありますね。

**文字列を比較するときはequalsメソッドを使いましょう。**

それに他の方も仰っているように、「フィールドpon が初期化も代入されていない」ですね。

--------------

[追記1]

> それと別問題かもしれませんがインスタンスの生成時にstaticとうたないとエラーが出るので仕方なく打っているのですがそこについてもお聞きしたいです。

今回のメインとは別枠ですが、一応書いておきます。

まず、この問題をどうにかする前に、「**オブジェクト指向とは何か**」を理解する必要があります。

オブジェクト指向(以降 OOP) とは、**データ(= フィールド)と処理(= メソッド)をひとまとめにしたオブジェクト**を中心に見る発想法です。

C言語とかのような言語では、データと処理(関数)は別々でした。
でも、この関係するものをひとまとめにしたオブジェクトにすることで、運用しやすくしています。

「**対象データとその処理方法はオブジェクトだけが知っている**」状態にできるので、
「**オブジェクトに管理や処理を任せる**」ということが出来ます。

そう、C#で言えば Stringとかみたいなものですね。

質問者さんはこのStringに入っているデータの状態、完全に把握しますか?

そりゃ、もちろん、「今はディレクトリパスが入っている」とかみたいなものは把握するとは思いますが、
事細かに把握しますか?

しませんね。

文字列分割とかも、「String君、文字列を分割したものをくれないか?」とオブジェクトに依頼したりするだけですよね。

呼び出し側は実装がどうなっているのかは興味なく、単に特定の処理(文字列分割, 文字列置換 etc.) だけが興味あるのですよね。

~~そう、C言語の関数の範囲がオブジェクトレベルにまで広がった感じです。~~

ですが、勝手にオブジェクトを作って……とやると、コンパイラとかが理解できません。

いきなり、「これ、田中さんに渡して」とだけ言われるようなものです。
(いや、田中さんって誰よ???)

どの人を言っているのかを教えてくれないとわかりませんね。

そう、そういう情報を教えてあげないといけません。

で、その役割が**クラス**です。

「こういうデータ構造があるんで、よろぴく~」とコンパイラに教えてあげるのです。
(言語によってはインタプリタだったりするが)

で、普通、クラスのメンバ( フィールドやメソッド ) には staticは付けません。

```C#
public class ClassA{
    public ClassA( int n ){
        this.N = n;
        this.name = "test";
    }
    public void add( int a ){
        this.N += a;
    }
    
    // 他にもメソッド等がある
    
    public int N{ get; set; }
    
    private string name;
}
```

みたいに。(上記サンプルは特に意味のないものですが)

そして、コンパイラとかに「このクラスでオブジェクトを作りますね~」と教えてあげる必要があります。

```C#
ClassA obj = new ClassA( 100 );
```

これによって、obj は N というデータや nameというデータをもったもので、
それに対する処理も完備しています。

これによって、N や name を管理したり処理したりするのは obj になります。

もちろん、一つのクラスから複数個のオブジェクトを作る事もできます。

とにかく、オブジェクトに管理を任せるのです。

でも、どうしても一つのメソッドで完結するような処理があります。

そういう場合は**苦肉の策として**、staticなメソッド等を使います。

ですが、これは オブジェクトに結びついたものではなく、クラスレベルでついています。

(イメージ的にはC言語の関数. 実際には別物だが, あくまでイメージとして)

なので、staticなメソッド内で自分クラスのメンバを使いたい場合でもインスタンス化(オブジェクトを生成すること)が必要です。

今回のように、paaクラスのjankenメソッドはstaticになっています。
そのため、ある意味、別のクラスからアクセスしているようなものとなり、
「おいおい、staticなやつから非staticなやつにアクセスできんぞ」と言っているのです。

クラスを組んだからOOP……ではありません。

**フィールドとメソッドをひとまとめにして、そのオブジェクトに管理や処理を任せるための発想法がOOPです**。

Answer

ベストでてますが
この質問の問題は文字列比較でなく、paa.pon が未設定なことです。
※＝＝でなくequalsを使うべきというのを否定するものではないです

なので
```java
    String pon = scanner.nextLine();
    scanner.close();
    System.out.println("あなたは" + pon + "を出しました。");
```
の部分を
```java
    paa.pon = scanner.nextLine().intern();
    scanner.close();
    System.out.println("あなたは" + paa.pon + "を出しました。");
```

とすると、「String Constant Poolの再利用」するので、＝＝でも動作します。

Answer

すべて解決したわけではないですが、行き詰っていたので皆さんの助言を一つ一つ紐解いて学習していきたいと思います。丁寧な回答ありがとうございました！！

投稿2021/09/11 06:26

pirikinoko

総合スコア2

Answer

> インスタンスの生成時にstaticとうたないとエラーが出るので仕方なく打っているのですが

このコードの中で「インスタンスの生成」をしているのは Scanner と Random だけです。(どちらも "new クラス名(～)" という文があります。)
メソッドやクラス内に定義した変数に static を付けることは「インスタンスの生成時」と表現される事象では無く、単にそれらを「static とする」と宣言しているだけです。

static が必要とされているのは、janken メソッドを(paa クラスのインスタンスを経由せずに) paa.janken として呼び出している為でしょう。
"クラス名.メソッド名" という呼び出し方は(インスタンスを必要としない)クラスメソッドに可能なことで、クラスメソッドはインスタンスメソッドに static を付けた見た目になります。
~~paa.pon や~~ paa.teki は現状使われていないようですので、static は有っても無くても構わない状態かと思います。

Answer

> pon=="グー"

これでは文字列の比較はできません

って、そもそもこの変数に代入してるところが見当たりませんが。。

試したこと

補足情報（FW/ツールのバージョンなど）

関連した質問