質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.50%
Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

Q&A

解決済

3回答

1406閲覧

ローカル変数に値が格納されないエラーの修正方法について

退会済みユーザー

退会済みユーザー

総合スコア0

Java

Javaは、1995年にサン・マイクロシステムズが開発したプログラミング言語です。表記法はC言語に似ていますが、既存のプログラミング言語の短所を踏まえていちから設計されており、最初からオブジェクト指向性を備えてデザインされています。セキュリティ面が強力であることや、ネットワーク環境での利用に向いていることが特徴です。Javaで作られたソフトウェアは基本的にいかなるプラットフォームでも作動します。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

0グッド

0クリップ

投稿2018/02/20 06:43

編集2018/02/20 06:59

前提・実現したいこと

競技プログラミングの問題を解いています。
問題元サイト
事前に以下のように数字がグループ分けされており、
入力される数字xとy(1≤x<y≤12)が同じグループなら「Yes」
異なるグループなら「No」と返すプログラムを書いています。

{1, 3, 5, 7, 8, 10, 12} {4, 6, 9, 11} {2}

発生している問題・エラーメッセージ

以下のように「Yes」か「No」を分岐させるところでエラーが出ています。
ローカル変数としてxp、yp(配列番号)を最初に定義しているのですが、
探索で見つけた後に値を格納することができていないようです。
どのように改変すべきでしょうか。

String xp; String yp = null;

該当のソースコード

java

1package atcoder; 2import java.util.Scanner; 3 4public class sample { 5 public static void main(String[] args) { 6 Scanner scan = new Scanner(System.in); 7 int x = scan.nextInt(); 8 int y = scan.nextInt(); 9 String xp; 10 String yp; 11 12 char[] first ={1, 3, 5, 7, 8, 10, 12}; 13 char[] second ={4, 6, 9, 11}; 14 char[] third ={2}; 15 16 for(int i = 0; i<first.length; i ++){ 17 if(first[i] == x){ 18 xp = "F"; 19 }for(int j = 0; j<first.length; j ++){ 20 if(first[j] == y){ 21 yp = "F"; 22 } 23 if(xp == yp){ 24 System.out.println("Yes"); 25 }else{ 26 System.out.println("No"); 27 } 28 } 29 30 } 31 32 } 33 34} 35 36 37

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

Eclipse Java EE IDE for Web Developers.
Version: Neon.3 Release (4.6.3)

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答3

0

ベストアンサー

文字列(String)の比較は==ではなくequalsメソッドを使ってください。

Java

1if (xp.equals(yp)) { 2 処理 3}

また、グローバル変数という表現は正しくないです。mainメソッド内のローカル変数なんで。
あとは、Eclipseで開発しているならデバッグでxp, ypに値が入っているかを確認してください。

投稿2018/02/20 06:55

unz.hori

総合スコア1057

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2018/02/20 07:06

ご回答いただきましてありがとうございました。
guest

0

int の変数とcharの配列を比較しているので、そもそも一致しないのではないでしょうか。

誤:char[] first ={1, 3, 5, 7, 8, 10, 12};
正:int[] first ={1, 3, 5, 7, 8, 10, 12};

環境が手元にないので試せませんが、これでどうでしょう?

投稿2018/02/20 06:53

rafiene3249

総合スコア53

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

YamakawaJunichi

2018/02/20 06:58

char[] first ={'1', '3', '5', '7', '8', '10', '12'}; との比較だと文字コードが入ってしまってint型との比較はまずいですが、単に整数を入れているのでこれに関しては問題はないと思います。
退会済みユーザー

退会済みユーザー

2018/02/20 07:05

ご回答いただきましてありがとうございました。
guest

0

sample#main() の中に xp,yp がありますが、コレmainの中だけに有効なローカル変数ですよね?
何かの間違いですか?

あとこれは純粋に疑問なんですが、Javaでグローバル変数って定義できるんですか?

投稿2018/02/20 06:51

編集2018/02/20 06:53
episteme

総合スコア16614

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

退会済みユーザー

退会済みユーザー

2018/02/20 06:58

ローカル変数です。 ご指摘いただきましてありがとうございます。
episteme

2018/02/20 07:01

> xp、yp(配列番号)を最初に定義しているのですが 配列"番号"がStringなんですか? xp = "F" は何を意味するんですか?
退会済みユーザー

退会済みユーザー

2018/02/20 07:05

こちらも質問の仕方に問題がございました。 配列番号ではなく、配列グループを意味していました。 xp = "F"は1つめのグループfirstのFです。
episteme

2018/02/20 07:09

あー、xp, yp には F,S,T のいずれかをセットしようと。 だったら char で充分だし、 xp == yp で比較できたのに。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.50%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問