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

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

ただいまの
回答率

90.09%

switch caseで一個ずつ処理していく方法がわからない

解決済

回答 3

投稿

  • 評価
  • クリップ 0
  • VIEW 1,588

NB16

score 9

前提・実現したいこと

初心者です。
androidstudioでswitchボタンをONにすると設定されていません。から設定されています。にかわるシステムを作っています。
それで、条件分岐であるswitch文を使ってみたのですが、実行時、case1のswitchボタンをONにするとcase1とcase2が同時に設定されています。に変わってしまいます。
僕はそれをcase1の部分だけを設定されています。に変わるようにしたいです。
説明が下手ですいません。

該当のソースコード

import android.app.Activity;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.widget.CompoundButton;
import android.widget.CompoundButton.OnCheckedChangeListener;
import android.widget.Switch;
import android.widget.TextView;
import android.widget.Toast;

import static to.msn.wings.boshitetyo.R.id.textView1;

public class MainActivity extends Activity implements OnCheckedChangeListener  {
private TextView textView[] = new TextView[5];

@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);

textView[0] = (TextView) findViewById(textView1);
Switch switch1 = (Switch) findViewById(R.id.switch1);
switch1.setOnCheckedChangeListener(this);

textView[1] = (TextView) findViewById(R.id.textView2);
Switch switch2 = (Switch) findViewById(R.id.switch2);
switch2.setOnCheckedChangeListener(this);
}
@Override
public void onCheckedChanged(CompoundButton buttonView, boolean isChecked){
for(int textViewNum = 1; textViewNum < 6; textViewNum++){

switch (textViewNum) {
case 1:
if (isChecked) {
textView[0].setText("設定中です。");
Toast.makeText(MainActivity.this, "ONにしました。", Toast.LENGTH_SHORT).show();

} else {
textView[0].setText("設定されていません。");
Toast.makeText(MainActivity.this, "OFFにしました。", Toast.LENGTH_SHORT).show();

}
break;
case 2:
if (isChecked) {
textView[1].setText("設定中です。");
Toast.makeText(MainActivity.this, "ONにしました。", Toast.LENGTH_SHORT).show();

} else {
textView[1].setText("設定されていません。");
Toast.makeText(MainActivity.this, "OFFにしました。", Toast.LENGTH_SHORT).show();

}

break;

}
}

}

}

試したこと

switch文のほかにif文などを使ってみたのですが、やっぱりだめでした。

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

一番新しいバージョンを使っています。

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

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

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

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

質問への追記・修正、ベストアンサー選択の依頼

  • swordone

    2017/07/18 00:02

    コードはマークダウンしてください。質問編集画面でコードを反転選択し、画面上部にある<code>ボタンを押してください。また、「一番新しいバージョン」は閲覧するタイミングによって変わるので、具体的なバージョンを記載してください。

    キャンセル

回答 3

checkベストアンサー

0

下記のリンク先は CheckBox の実装例ですが、 Switch と同じ CompoundButton のサブクラスなので、読み替えることができます。
現状のまとめて処理するのではなく、 Switch ごとに設定状態の変更を処理してあげればわかりやすいと思います。

CheckedChange イベントを処理する

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/07/18 23:50

    返信遅くなりました。
    回答ありがとうございます。
    とても参考になりました。

    キャンセル

0

onCheckedChangedメソッドの仮引数isCheckedは、
チェック状態が変化したものの変化後のチェック状態が渡されます。
リスナを登録したいずれかのスイッチをONに変化させたのなら
isCheckedはtrueになるためすべてのif文が通ります。

Switchを配列にして、各Switchに対してisChecked()を使って状態を確認するほかありません。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/07/18 23:54

    返信遅くなりました。
    回答ありがとうございます!
    とても参考になりました。

    キャンセル

-1

自分の書いたコードをよく見ましょう。
for文でtextViewNumを1から5まで増やしています。
switch文のcaseには1と2があるので、当然両方とも実行されます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

  • 2017/07/18 00:50

    その意図は分かりますが、そもそもこのコードを改善してそのようにできると思えなかったので、現状の現象が起きる原因だけを回答しました。

    また、質問者が1つのスイッチのオンオフで全てのスイッチのオンオフを更新したいかどうかがわからないため、とりあえずの回答になってしまいました。
    ごめんなさい。

    キャンセル

  • 2017/07/18 23:52

    回答ありがとうございます!
    質問が大変わかりにくくてすいません。

    キャンセル

  • 2017/07/18 23:53

    後、返信が遅くなり申し訳ございません。

    キャンセル

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

  • ただいまの回答率 90.09%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

同じタグがついた質問を見る