teratail
回答率
85.48%
質問するログイン新規登録

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

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

新規登録して質問してみよう
ただいま回答率
85.48%
トップ

Q&A

解決済

2回答

1649閲覧

Androidのボタンを押しても何も反応がありません。

Yakusugi
Yakusugi

総合スコア123

0グッド

1クリップ

投稿2022/05/10 12:54

編集2022/05/12 11:24

0

1

Android アプリをJavaで開発しています。
Fragmentに設置したボタンを押下しても、何も反応がありません。
念のため、「Log.d("TAG", "onClick: Alias Button Clicked");」を記述し、
ボタンが押された段階で、ログ上記文字列が表示されるか確認してみたのですが、
何も表示されません。

お手数ではございますが、ご助力頂けますと幸いです。

ご回答の程、よろしくお願いいたします。

HomeFragment.java

package com.myproject.offlinebudgettrackerappproject;

import android.app.DatePickerDialog;
import android.icu.util.Calendar;
import android.os.Bundle;
import android.util.Log;
import android.view.LayoutInflater;
import android.view.View;
import android.view.ViewGroup;
import android.widget.Button;
import android.widget.DatePicker;
import android.widget.EditText;
import android.widget.RadioButton;
import android.widget.RadioGroup;
import android.widget.TextView;

import androidx.fragment.app.Fragment;
import androidx.lifecycle.ViewModelProvider;

import com.myproject.offlinebudgettrackerappproject.databinding.ActivityMainBinding;
import com.myproject.offlinebudgettrackerappproject.model.BudgetTracker;
import com.myproject.offlinebudgettrackerappproject.model.BudgetTrackerAliasViewModel;
import com.myproject.offlinebudgettrackerappproject.model.BudgetTrackerViewModel;

import org.eazegraph.lib.charts.PieChart;

import java.util.List;

/**
 * A simple {@link Fragment} subclass.
 * Use the {@link HomeFragment#newInstance} factory method to
 * create an instance of this fragment.
 */
public class HomeFragment extends Fragment {

    private BudgetTrackerViewModel budgetTrackerViewModel;
    private BudgetTrackerAliasViewModel budgetTrackerAliasViewModel;
    PieChart homePieChart;
    RadioGroup radioHomeGroup;
    RadioButton radioHomeButton;
    EditText radioSearchHomeName;
    EditText radioSearchDateHomeFrom;
    EditText radioSearchDateHomeTo;
    TextView searchCalcResultHomeTxt;
    Button radioSearchHomeBtn;
    ActivityMainBinding activityMainBinding;

    // TODO: Rename parameter arguments, choose names that match
    // the fragment initialization parameters, e.g. ARG_ITEM_NUMBER
    private static final String ARG_PARAM1 = "param1";
    private static final String ARG_PARAM2 = "param2";

    // TODO: Rename and change types of parameters
    private String mParam1;
    private String mParam2;

    public HomeFragment() {
        // Required empty public constructor
    }

    /**
     * Use this factory method to create a new instance of
     * this fragment using the provided parameters.
     *
     * @param param1 Parameter 1.
     * @param param2 Parameter 2.
     * @return A new instance of fragment HomeFragment.
     */
    // TODO: Rename and change types and number of parameters
    public static HomeFragment newInstance(String param1, String param2) {
        HomeFragment fragment = new HomeFragment();
        Bundle args = new Bundle();
        args.putString(ARG_PARAM1, param1);
        args.putString(ARG_PARAM2, param2);
        fragment.setArguments(args);
        return fragment;
    }

    @Override
    public void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        if (getArguments() != null) {
            mParam1 = getArguments().getString(ARG_PARAM1);
            mParam2 = getArguments().getString(ARG_PARAM2);

            budgetTrackerViewModel = new ViewModelProvider.AndroidViewModelFactory(HomeFragment.this
                    .getActivity().getApplication())
                    .create(BudgetTrackerViewModel.class);

//            budgetTrackerViewModel.getAllBudgetTrackerLists().observe(this, budgetTrackers -> {
//                Log.d("TAG", "onCreate: " + budgetTrackers.get(0).getProductName());
//            });
        }
    }

    @Override
    public View onCreateView(LayoutInflater inflater, ViewGroup container,
                             Bundle savedInstanceState)                                                                {
        View view = inflater.inflate(R.layout.fragment_home, container, false);

        homePieChart = (PieChart) view.findViewById(R.id.pie_chart);
        radioHomeGroup = (RadioGroup) view.findViewById(R.id.home_radio_group);
        radioSearchHomeName = (EditText) view.findViewById(R.id.home_radio_search_name);
        radioSearchDateHomeFrom = (EditText) view.findViewById(R.id.home_radio_search_date_from_txt);
        radioSearchDateHomeTo = (EditText) view.findViewById(R.id.home_radio_search_date_to_txt);
        searchCalcResultHomeTxt = (TextView) view.findViewById(R.id.home_radio_search_calc_result_txt);
        radioSearchHomeBtn = (Button) view.findViewById(R.id.home_radio_search_btn);
        radioSearchHomeBtn.setOnClickListener((View.OnClickListener) getActivity());
        activityMainBinding = ActivityMainBinding.inflate(getLayoutInflater());

        Calendar calendar = Calendar.getInstance();
        final int year = calendar.get(Calendar.YEAR);
        final int month = calendar.get(Calendar.MONTH);
        final int day = calendar.get(Calendar.DAY_OF_MONTH);

        radioSearchDateHomeFrom.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                DatePickerDialog datePickerDialog = new DatePickerDialog(
                        getActivity(), new DatePickerDialog.OnDateSetListener() {
                    @Override
                    public void onDateSet(DatePicker datePicker, int year, int month, int dayOfMonth) {
                        month = month + 1;
                        String date = year + "-" + month + "-" + dayOfMonth;
                        radioSearchDateHomeFrom.setText(date);
                    }
                }, year, month, day);
                datePickerDialog.show();
            }
        });

        radioSearchDateHomeTo.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View view) {
                DatePickerDialog datePickerDialog = new DatePickerDialog(
                        getActivity(), new DatePickerDialog.OnDateSetListener() {
                    @Override
                    public void onDateSet(DatePicker datePicker, int year, int month, int dayOfMonth) {
                        month = month + 1;
                        String date = year + "-" + month + "-" + dayOfMonth;
                        radioSearchDateHomeTo.setText(date);
                    }
                }, year, month, day);
                datePickerDialog.show();
            }
        });

        radioHomeGroup.setOnCheckedChangeListener(new RadioGroup.OnCheckedChangeListener() {
            List<BudgetTracker> radioStoreNameLists;
            List<BudgetTracker> radioProductNameLists;
            List<BudgetTracker> radioProductTypeLists;
            String storeName;
            String date1;
            String date2;
            BudgetTracker budgetTracker;
            String calcSumStr;

            @Override
            public void onCheckedChanged(RadioGroup radioHomeGroup, int checkId) {
                switch (checkId) {
                    case R.id.home_radio_store_name:
                        Log.d("TAG", "onCheckedChanged: store chosen");
                        storeName = radioSearchHomeName.getText().toString();
                        date1 = radioSearchDateHomeFrom.getText().toString();
                        date2 = radioSearchDateHomeTo.getText().toString();
                        radioSearchHomeBtn.setOnClickListener(new View.OnClickListener() {
                            @Override
                            public void onClick(View view) {
                                Log.d("TAG", "onClick: Alias Button Clicked");
                                BudgetTrackerAliasViewModel.insert(date1, date2, storeName);

                            }
                        });
                    case R.id.home_radio_product_name:
                        Log.d("TAG", "onCheckedChanged: product name chosen");
                        storeName = radioSearchHomeName.getText().toString();
                        date1 = radioSearchDateHomeFrom.getText().toString();
                        date2 = radioSearchDateHomeTo.getText().toString();
                        budgetTrackerAliasViewModel = new ViewModelProvider(requireActivity()).get(BudgetTrackerAliasViewModel.class);
                        budgetTracker = new BudgetTracker(storeName, date1, date2);
                        radioSearchHomeBtn.setOnClickListener(new View.OnClickListener() {
                            @Override
                            public void onClick(View view) {
                                Log.d("TAG", "onClick: Alias Button Clicked");
                                BudgetTrackerAliasViewModel.insert(date1, date2, storeName);

                            }
                        });
                }
            }
        });


       return view;

    }
}

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

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

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

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

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

hoshi-takanori
hoshi-takanori

2022/05/10 18:50

コードが省略されててよく分かりませんが、onCheckedChanged の中で ViewModel を作ったり、setOnClickListener する意味が分からない…。
jimbe
jimbe

2022/05/10 20:01 編集

一部のコードだけからも、変数名が長い、メソッド分けが(適切に)行われていない等、構造的に酷い状態にあるように思います。 radioSearchHomeBtn.setOnClickListener の箇所を通っているのは確認されたのでしょうか。
guest

回答2件

0

ベストアンサー

不要・使っていない部分等を削除して整理・修正すると次のようになりました。
( BudgetTrackerAliasViewModel の insert が static になっていますが、 ViewModel にそれはおかしいので非 static と仮定しています。)
ログ上は動作しました。

HomeFragment.java

java
1package com.teratail.q_4svhzp3oouavma;
2
3import android.app.DatePickerDialog;
4import android.icu.util.Calendar;
5import android.os.*;
6import android.util.Log;
7import android.view.View;
8import android.widget.*;
9
10import androidx.annotation.*;
11import androidx.fragment.app.Fragment;
12import androidx.lifecycle.ViewModelProvider;
13
14import com.teratail.q_4svhzp3oouavma.model.*;
15
16public class HomeFragment extends Fragment {
17  private final String TAG = "HomeFragment";
18
19  public HomeFragment() {
20    super(R.layout.fragment_home);
21  }
22
23  @Override
24  public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
25    super.onViewCreated(view, savedInstanceState);
26
27    BudgetTrackerAliasViewModel model = new ViewModelProvider(requireActivity()).get(BudgetTrackerAliasViewModel.class);
28    Calendar calendar = Calendar.getInstance();
29
30    Button searchButton = view.findViewById(R.id.home_radio_search_btn);
31    EditText nameView = view.findViewById(R.id.home_radio_search_name);
32
33    EditText dateFromView = view.findViewById(R.id.home_radio_search_date_from_txt);
34    dateFromView.setOnClickListener(v -> selectDate((EditText)v, calendar));
35
36    EditText dateToView = view.findViewById(R.id.home_radio_search_date_to_txt);
37    dateToView.setOnClickListener(v -> selectDate((EditText)v, calendar));
38
39    RadioGroup modeGroupView = view.findViewById(R.id.home_radio_group);
40    modeGroupView.setOnCheckedChangeListener((group, checkedId) -> {
41      switch(checkedId) {
42        case R.id.home_radio_store_name:
43          Log.d(TAG, "store mode");
44          searchButton.setOnClickListener(v -> {
45            Log.d(TAG, "store insert");
46            String name = nameView.getText().toString();
47            String dateFrom = dateFromView.getText().toString();
48            String dateTo = dateToView.getText().toString();
49            model.insert(dateFrom, dateTo, name);
50          });
51          break;
52        case R.id.home_radio_product_name:
53          Log.d(TAG, "product mode");
54          searchButton.setOnClickListener(v -> {
55            Log.d(TAG, "product insert");
56            String name = nameView.getText().toString();
57            String dateFrom = dateFromView.getText().toString();
58            String dateTo = dateToView.getText().toString();
59            model.insert(dateFrom, dateTo, name);
60          });
61          break;
62      }
63    });
64    modeGroupView.check(R.id.home_radio_store_name);
65  }
66
67  private void selectDate(TextView textView, Calendar calendar) {
68    final int year = calendar.get(Calendar.YEAR);
69    final int month = calendar.get(Calendar.MONTH);
70    final int day = calendar.get(Calendar.DAY_OF_MONTH);
71    new DatePickerDialog(getActivity(), (datePicker, y, m, d) -> {
72      textView.setText(y + "-" + (m+1) + "-" + d);
73    }, year, month, day).show();
74  }
75}
plain
1D/HomeFragment: store mode
2D/HomeFragment: store mode
3D/HomeFragment: store insert
4D/HomeFragment: product mode
5D/HomeFragment: product insert

(起動直後に "store mode" が 2 回も出るのは何故か分かりません。)

投稿2022/05/12 05:19

編集2022/05/12 05:27
jimbe
jimbe

総合スコア12648

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

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

Yakusugi
Yakusugi

2022/05/12 11:25

ご回答頂きありがとうございました。 また、上記コードも編集しましたが、 206行目をreturn view;に変えた所、こちらでも動く事確認出来ました。
jimbe
jimbe

2022/05/12 13:55

Fragment.onCreateView() のオーバーライド時はレイアウトのインフレートのみとし、その他の処理は onViewCreated で行うことが勧められています。 https://developer.android.com/reference/androidx/fragment/app/Fragment#onCreateView(android.view.LayoutInflater,android.view.ViewGroup,android.os.Bundle) > It is recommended to only inflate the layout in this method and move logic that operates on the returned View to onViewCreated. 「このメソッドのレイアウトのみを拡張し、返されたビューで動作するロジックをonViewCreatedに移動することをお勧めします。」(by Google翻訳) また、Fragment にレイアウト ID をパラメータとするコンストラクタが追加され、それを用いると Fragment.onCreateView() がインフレートしてくれるようになりましたので、結果 onCreateView() はオーバーライドしなくなっています。 https://developer.android.com/reference/androidx/fragment/app/Fragment#Fragment(int)
guest

0

こちらも諸々省略しますが、ボタンについてのみ記述します。
oncheckedchanged の中にonClickが入っているのがちょっとよくわからないです。
oncheckedchanged 内でviewの定義を行っているのも良くないです。
ライフサイクルを考えると、onCreateで定義しないと動かなくないでしょうか？

ラジオボタン変更時とボタン押下時はそれぞれメソッドを分けて考えるべきかと思います。

@Override
protected void onCreate(Bundle savedInstanceState) {
super.onCreate(savedInstanceState);
setContentView(R.layout.activity_main);
Button radioSearchHomeBtn;
radioSearchHomeBtn = (Button) findViewById(R.id.home_radio_search_btn);
radioSearchHomeBtn.setOnClickListener(this);
}
public void onClick(View v) {
switch(v.getId()){
case R.id.home_radio_search_btn:
Log.d("TAG", "onClick: Alias Button Clicked");
break;
}
}

投稿2022/05/11 05:21

編集2022/05/11 05:22
tomoka_murata
tomoka_murata

総合スコア28

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

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

Yakusugi
Yakusugi

2022/05/12 01:11 編集

ご回答ありがとうございます。 現状を共有するため、HomeFragment.javaを全文掲載しました。 私に方で確認した所、今回の場合、ボタン事態の定義には問題は無く、 ラジオグループに問題があるようです。 今回ラジオグループをcase文で分けている理由としては、 個々のラジオボタンが選択された場合に、それぞれで違う動きをさせたい為です。 また、ご回答の中で指摘して頂いた「radioSearchHomeBtn.setOnClickListener(this);」ですが、 今回Fragmentを使っているため、「radioSearchHomeBtn.setOnClickListener((View.OnClickListener) getActivity());」 としたところ、下記エラーが出てしまいました。 2022-05-11 21:25:07.604 5235-5235/com.myproject.offlinebudgettrackerappproject E/AndroidRuntime: FATAL EXCEPTION: main Process: com.myproject.offlinebudgettrackerappproject, PID: 5235 java.lang.ClassCastException: com.myproject.offlinebudgettrackerappproject.MainActivity cannot be cast to android.view.View$OnClickListener at com.myproject.offlinebudgettrackerappproject.HomeFragment.onCreateView(HomeFragment.java:108) 上記回答の中で、onCreateの中でボタン押下時の処理を定義しておりますが、 選択したラジオボタンごとに、ボタン押下時の処理を変えたい為、 ボタン押下時の処理を別途分けると言うの難しいのですが、 この観点では、解決策などがありましたが、ご提示頂く事は可能でしょうか。 下記エラー全文です。 2022-05-11 21:25:07.604 5235-5235/com.myproject.offlinebudgettrackerappproject E/AndroidRuntime: FATAL EXCEPTION: main Process: com.myproject.offlinebudgettrackerappproject, PID: 5235 java.lang.ClassCastException: com.myproject.offlinebudgettrackerappproject.MainActivity cannot be cast to android.view.View$OnClickListener at com.myproject.offlinebudgettrackerappproject.HomeFragment.onCreateView(HomeFragment.java:108) at androidx.fragment.app.Fragment.performCreateView(Fragment.java:2963) at androidx.fragment.app.FragmentStateManager.createView(FragmentStateManager.java:518) at androidx.fragment.app.FragmentStateManager.moveToExpectedState(FragmentStateManager.java:282) at androidx.fragment.app.FragmentManager.executeOpsTogether(FragmentManager.java:2189) at androidx.fragment.app.FragmentManager.removeRedundantOperationsAndExecute(FragmentManager.java:2100) at androidx.fragment.app.FragmentManager.execPendingActions(FragmentManager.java:2002) at androidx.fragment.app.FragmentManager.dispatchStateChange(FragmentManager.java:3138) at androidx.fragment.app.FragmentManager.dispatchActivityCreated(FragmentManager.java:3072) at androidx.fragment.app.FragmentController.dispatchActivityCreated(FragmentController.java:251) at androidx.fragment.app.FragmentActivity.onStart(FragmentActivity.java:502) at androidx.appcompat.app.AppCompatActivity.onStart(AppCompatActivity.java:246) at android.app.Instrumentation.callActivityOnStart(Instrumentation.java:1432) at android.app.Activity.performStart(Activity.java:7848) at android.app.ActivityThread.handleStartActivity(ActivityThread.java:3294) at android.app.servertransaction.TransactionExecutor.performLifecycleSequence(TransactionExecutor.java:221) at android.app.servertransaction.TransactionExecutor.cycleToPath(TransactionExecutor.java:201) at android.app.servertransaction.TransactionExecutor.executeLifecycleState(TransactionExecutor.java:173) at android.app.servertransaction.TransactionExecutor.execute(TransactionExecutor.java:97) at android.app.ActivityThread$H.handleMessage(ActivityThread.java:2016) at android.os.Handler.dispatchMessage(Handler.java:107) at android.os.Looper.loop(Looper.java:214) at android.app.ActivityThread.main(ActivityThread.java:7356) at java.lang.reflect.Method.invoke(Native Method) at com.android.internal.os.RuntimeInit$MethodAndArgsCaller.run(RuntimeInit.java:492) at com.android.internal.os.ZygoteInit.main(ZygoteInit.java:930)
jimbe
jimbe

2022/05/12 03:45

> 今回ラジオグループをcase文で分けている理由としては、 >個々のラジオボタンが選択された場合に、それぞれで違う動きをさせたい為です。 現状は全く同じ処理をしているようですが、これはこれから変更ということでしょうか。
Yakusugi
Yakusugi

2022/05/12 04:50

はい、その通りです。 現状は、まだプロトタイプの状態なので、 今後、処理を分けていくつもりです。
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

質問の解決につながる回答をしましょう。 サンプルコードなど、より具体的な説明があると質問者の理解の助けになります。 また、読む側のことを考えた、分かりやすい文章を心がけましょう。

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

ただいまの回答率
85.48%

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

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

質問する

関連した質問

トップ

Androidのボタンを押しても何も反応がありません。