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

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

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

解決済

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

Yakusugi
Yakusugi

総合スコア117

2回答

0グッド

1クリップ

883閲覧

投稿2022/05/10 12:54

編集2022/05/12 11:24

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; } }

以下のような質問にはグッドを送りましょう

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

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

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

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

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

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

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

適切な質問に修正を依頼しましょう。

hoshi-takanori

2022/05/10 18:50

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

2022/05/10 20:01 編集

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

回答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

総合スコア10741

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

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

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

このような回答には修正を依頼しましょう。

回答へのコメント

Yakusugi

2022/05/12 11:25

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

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

総合スコア28

良いと思った回答にはグッドを送りましょう。
グッドが多くついた回答ほどページの上位に表示されるので、他の人が素晴らしい回答を見つけやすくなります。

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

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

このような回答には修正を依頼しましょう。

回答へのコメント

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

2022/05/12 03:45

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

2022/05/12 04:50

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

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

ただいまの回答率
86.12%

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

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

質問する

関連した質問

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