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

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

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

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

Android Studio

Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

Q&A

解決済

2回答

5130閲覧

ボタンを押したとき乱数によって文字表示

huton

総合スコア30

Java

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

Android Studio

Android Studioは、 Google社によって開発された、 Androidのネイティブアプリケーション開発に特化した統合開発ツールです。

0グッド

0クリップ

投稿2019/07/02 06:39

前提・実現したいこと

Android Studioでボタンを押したときに乱数を発生させ、その乱数によって異なる文字を表示させた後に乱数を配列に入れるプログラムを作成しています。
以下のようなプログラムを書き実行したところボタンに関するworningはあったのですがエラーは発生しませんでした。しかしアプリを開くと問題を起こして停止してしまいます。

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

以下のものはボタンに関してのworningです。 Hardcoded string "開始", should use @string resource Hardcoding text attributes directly in layout files is bad for several reasons: * When creating configuration variations (for example for landscape or portrait) you have to repeat the actual text (and keep it up to date when making changes)

該当のソースコー

java

1import java.util.Random; 2 3public class MainActivity extends AppCompatActivity { 4 5 int i=0; 6 int x[] = new int[0]; 7 TextView textView = (TextView) findViewById(R.id.text); 8 9 @Override 10 protected void onCreate(Bundle savedInstanceState) { 11 super.onCreate(savedInstanceState); 12 setContentView(R.layout.activity_main); 13 14 Button button_sta=(Button)findViewById(R.id.button_sta); 15 16 button_sta.setOnClickListener(new View.OnClickListener(){ 17 @Override 18 public void onClick(View v){ 19 Random rand=new Random(); 20 int num=rand.nextInt(4); 21 if(num==0) { 22 textView.setText("←"); 23 x[i] =num; 24 } 25 26 else if(num==1) { 27 textView.setText("↑"); 28 x[i] = num; 29 } 30 31 else if(num==2) { 32 textView.setText("→"); 33 x[i] = num; 34 } 35 else if(num==3) { 36 textView.setText("↓"); 37 x[i] = num; 38 } 39 40 } 41 }); 42 43 44

試したこと

if文やfor文を消してボタンを押すと乱数を表示するようなプログラムを書いてみても実行に問題はないのですが、アプリを開くと停止してしまいました。

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

ここにより詳細な情報を記載してください。

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

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

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

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

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

m.ts10806

2019/07/02 06:50

>該当のソースコー 細かくて申し訳ないですが、途中で切れているのでなおしてもらえたらと・・・
huton

2019/07/02 10:56

ややこしくてすみません。 このあとはonCreateとMainActivityの分の}が2つあるだけです。
guest

回答2

0

ベストアンサー

IndexOutOfBoundsException などは発生していないでしょうか? 以下の部分を,

int x[] = new int[0];

次のように変更してみてください.

int x[] = new int[4];

投稿2019/07/02 11:49

ytk8

総合スコア57

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

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

huton

2019/07/02 13:27

ボタンを押したら動くようになりました! ありがとうございました!
swordone

2019/07/02 13:48 編集

これ、配列である必要性がない気が…
huton

2019/07/02 14:01

このあとにいろいろ加えようとして先に書いちゃったんですよね....すみません
guest

0

推測ですが、落ちたときの例外はNullPointerExceptionではないでしょうか?

textViewがActivityのフィールド変数となっており、onCreateの前で初期化が行われています。この場合、**findViewByIdがnullを返してしまいます。それに対してonClickでsetTextしようとしたため、**落ちると思います。

投稿2019/07/02 06:54

swordone

総合スコア20649

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

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

huton

2019/07/02 11:19

参考にさせていただき、setContentView(R.layout.activity_main);の下にtextViewを初期化する文を移動したのですがこのときTextViewの前にfinalがついても大丈夫なのでしょうか。 またこの状態で実行するとアプリを開くことはできましたが、ボタンを押すとやはり落ちてしまいます。 おこがましいとは思いますがもしよろしければご助言いただけないでしょうか。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

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

ただいまの回答率
85.50%

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

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

質問する

関連した質問