android studioでTodoリストの作成

Question

今android studioでTodoリストを作成しています

削除ボタンの増設について、聞きたいことがあります。

以下がTodo.javaです。

package com.example.studytodo;

import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.FragmentManager;

import android.os.Bundle;
import android.util.Log;

public class Todo extends AppCompatActivity {
    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.todo);

        FragmentManager fm = getSupportFragmentManager();
        fm.setFragmentResultListener(TodoFragment.REQUEST_KEY, this, (rkey,result)->{
            // TodoFragment が back したら何する?
            Log.d("MainActivity", "backed todofragment.");
        });

        fm.beginTransaction()
                .replace(R.id.fragment_container_view, new TodoFragment())
                .commit();
    }
}

また、次がTodoFragment.javaです

package com.example.studytodo;

import androidx.annotation.*;
import androidx.fragment.app.*;
import androidx.recyclerview.widget.*;


import android.content.Intent;
import android.os.Bundle;
import android.util.Log;
import android.view.*;
import android.widget.*;

import java.io.Serializable;
import java.time.LocalDateTime;
import java.time.format.DateTimeFormatter;
import java.util.*;

public class TodoFragment extends Fragment {
    static final String REQUEST_KEY = "TodoFragment";
    private List<Task> taskList = new ArrayList<>();

    public TodoFragment() {
        super(R.layout.fragment_todo);
    }

    @Override
    public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
        super.onViewCreated(view, savedInstanceState);

        TaskAdapter taskAdapter = new TaskAdapter();

        RecyclerView recyclerView = view.findViewById(R.id.recyclerViewTasks);
        recyclerView.setAdapter(taskAdapter);

        FragmentManager fm = getChildFragmentManager();
        fm.setFragmentResultListener(AddTaskFragment.REQUEST_KEY, getViewLifecycleOwner(), (rkey, result) -> {
            taskAdapter.addTask((Task) result.getSerializable(AddTaskFragment.RESULTKEY_TASK));
        });

        fm.setFragmentResultListener(AddTaskFragment.REQUEST_KEY, getViewLifecycleOwner(), (rkey, result) -> {
            Task addedTask = (Task) result.getSerializable(AddTaskFragment.RESULTKEY_TASK);
            // TaskAdapterに追加
            taskAdapter.addTask(addedTask);
            // TodoFragment内のリストにも追加
            taskList.add(addedTask);
        });

        ImageButton btBackTodo = view.findViewById(R.id.btBackTodo);
        btBackTodo.setOnClickListener(v -> {
            Toast.makeText(getContext(), "戻るボタンがクリックされました", Toast.LENGTH_SHORT).show();
            getParentFragmentManager().setFragmentResult(REQUEST_KEY, new Bundle()); //(MainActivity に)通知

            Intent intent = new Intent(requireContext(), MainActivity.class);
            startActivity(intent);
            requireActivity().finish();
        });

        ImageButton addButton = view.findViewById(R.id.btAdd);
        addButton.setOnClickListener(v -> {
            Toast.makeText(getContext(), "追加ボタンがクリックされました", Toast.LENGTH_SHORT).show();
            new AddTaskFragment().show(fm, null); //ダイアログ表示
        });
    }
}

class TaskAdapter extends RecyclerView.Adapter<TaskAdapter.ViewHolder> {
    private final List<Task> taskList = new ArrayList<>();

    void addTask(Task task) {
        Log.d("TaskAdded", "Task added: " + task);
        taskList.add(task);
        notifyItemInserted(taskList.size()-1);
    }

    void removeTask(int position) {
        taskList.remove(position);
        notifyItemRemoved(position);
        notifyItemRangeChanged(position, taskList.size());
    }

    @NonNull
    @Override
    public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
        return new ViewHolder(parent);
    }

    @Override
    public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
        Log.d("AdapterDebug", "onBindViewHolder called for position: " + position);
        holder.bind(taskList.get(position));
    }

    @Override
    public int getItemCount() {
        return taskList.size();
    }

    static class ViewHolder extends RecyclerView.ViewHolder {
        private final TextView taskName, dateTime;

        public ViewHolder(@NonNull ViewGroup parent) {
            super(LayoutInflater.from(parent.getContext()).inflate(R.layout.task_item, parent, false));
            taskName = itemView.findViewById(R.id.taskNameTextView);
            dateTime = itemView.findViewById(R.id.dateTimeTextView);
        }
        void bind(Task task) {
            taskName.setText(task.taskName);
            dateTime.setText(task.getDateTime());
        }
    }
}

class Task implements Serializable {
    static final DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy/MM/dd HH:mm");

    final String taskName;
    final LocalDateTime dateTime;

    public Task(String taskName, LocalDateTime dateTime) {
        this.taskName = taskName;
        this.dateTime = dateTime;
    }
    String getDateTime() {
        return formatter.format(dateTime);
    }

    @Override
    public String toString() {
        return taskName + " at " + getDateTime();
    }
}

次がAddTaskFragment.javaです

package com.example.studytodo;

import androidx.annotation.*;
import androidx.appcompat.app.AlertDialog;
import androidx.fragment.app.DialogFragment;

import android.app.*;
import android.os.Bundle;
import android.view.*;
import android.widget.*;

import java.time.*;

public class AddTaskFragment extends DialogFragment {
    static final String REQUEST_KEY = "AddTaskFragment";
    static final String RESULTKEY_TASK = "task";

    private EditText nameView, datetimeView;
    private LocalDateTime datetime;

    @NonNull
    @Override
    public Dialog onCreateDialog(@Nullable Bundle savedInstanceState) {
        View view = LayoutInflater.from(requireContext()).inflate(R.layout.fragment_add_task, null);

        datetime = LocalDateTime.now();

        nameView = view.findViewById(R.id.editTextTaskName);
        datetimeView = view.findViewById(R.id.editTextDateTime);
        datetimeView.setOnClickListener(v -> showDatePickerDialog());

        return new AlertDialog.Builder(requireContext())
                .setTitle("Add Task")
                .setView(view)
                .setPositiveButton("保存", (d, w) -> {
                    String name = nameView.getText().toString();
                    Bundle result = new Bundle();
                    result.putSerializable(RESULTKEY_TASK, new Task(name, datetime));
                    getParentFragmentManager().setFragmentResult(REQUEST_KEY, result);
                    dismiss();
                })
                .setNegativeButton("キャンセル", (d, w) -> dismiss())
                .create();
    }

    private void showDatePickerDialog() {
        DatePickerDialog datePickerDialog = new DatePickerDialog(getContext(), (v, y, m, d) -> { //m: 0-11
            datetime = datetime.with(LocalDate.of(y, m+1, d)); //m: 1-12
            showTimePickerDialog();
        }, datetime.getYear(), datetime.getMonthValue()-1, datetime.getDayOfMonth()); //m: 0-11
        datePickerDialog.show();
    }

    private void showTimePickerDialog() {
        TimePickerDialog timePickerDialog = new TimePickerDialog(getContext(), (v, h, m) -> {
            datetime = datetime.with(LocalTime.of(h, m));
            datetimeView.setText(Task.formatter.format(datetime));
        }, datetime.getHour(), datetime.getMinute(), true);
        timePickerDialog.show();
    }
}

次がTodoViewModel.javaです

package com.example.studytodo;

import androidx.lifecycle.ViewModel;
import androidx.lifecycle.MutableLiveData;

public class TodoViewModel extends ViewModel {
    private MutableLiveData<Task> addedTask = new MutableLiveData<>();

    public void setAddedTask(Task task) {
        addedTask.setValue(task);
    }

    public MutableLiveData<Task> getAddedTask() {
        return addedTask;
    }
}

task_item.xml

<!-- task_item.xml -->
<androidx.constraintlayout.widget.ConstraintLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:app="http://schemas.android.com/apk/res-auto"
    android:layout_width="match_parent"
    android:layout_height="wrap_content"
    android:padding="10dp">

    <TextView
        android:id="@+id/taskNameTextView"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toTopOf="parent" />
    <TextView
        android:id="@+id/dateTimeTextView"
        android:layout_width="0dp"
        android:layout_height="wrap_content"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintStart_toStartOf="parent"
        app:layout_constraintTop_toBottomOf="@id/taskNameTextView" />

    <ImageButton
        android:id="@+id/deleteButton"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        app:layout_constraintEnd_toEndOf="parent"
        app:layout_constraintTop_toBottomOf="@id/dateTimeTextView"
        android:src="@drawable/baseline_clear_24"
        android:background="?android:attr/selectableItemBackgroundBorderless" />

</androidx.constraintlayout.widget.ConstraintLayout>

試したこと

前回回答していただいたときにFragmentを利用したら簡単にできるとコード付きで教えてもらったことで、Todoリストにタスクを追加することはできました。
今現在はタスクを追加した後に削除マークを付け、押すと追加したタスクを削除して画面から消えるようにしたいと考えています。
また、戻ってからまたtodo.xmlの画面に遷移すると追加したはずのタスクが消えてしまいます。。。

Comments

[hoshi-takanori]

// TODO: taskList に追加する処理を実装 のところを実装する必要があるのでは…。

[ottosay]

すみません、具体的にどのようにしたらいいか教えていただけますでしょうか。

[hoshi-takanori]

よく見たら、onActivityResult の方には何か書いてありますが、AddTaskActivity への遷移は addTaskLauncher を使ってるので、onActivityResult は呼ばれないですね。

あと、AddTaskActivity の saveTask メソッドで getParent を使って todoActivity.addTask を呼んでますが、そういうことをしてはいけません。

それから、Todo の setupBackButton メソッドで MainActivity を startActivity してますが、これは元の MainActivity に戻るのではなく、新しく MainActivity を開くことになります。

[ottosay]

詳しく教えていただきありがとうございます。
一度教えてくださった場所を修正してみたいと思います

[ottosay]

新しくMainActivityを開いてしまう件については以下のソースコードで治ったのかなと思います。
private void setupAddButton() {
ImageButton btAdd = findViewById(R.id.btAdd);
btAdd.setOnClickListener(v -> {
// 追加ボタンがクリックされた時の処理
Toast.makeText(Todo.this, "追加ボタンがクリックされました", Toast.LENGTH_SHORT).show();
// アクティビティを終了して前の画面に戻る
finish();
});
}

[ottosay]

何度もすみません、あれから何度か修正し、アプリで実行しているのですがやはりRecyclerViewに追加されません。
こういう風にしたらいいよなど詳しく教えてもらうことは可能でしょうか？

[jimbe]

コードはファイル毎にコードのマークダウン(```だけの行で前後を囲む)を使用してください。
質問の変化については、コメントでなく質問に追加編集してください。

[jimbe]

ここまで作られていて今更と思われるかもしれませんが、 Fragment の利用を考えたほうがよいと思います。
AddTaskActivity なんて DialogFragment で十分でしょう。

[ottosay]

色々と教えて頂きありがとうございます。
修正してみます。

Answer 1

コード量的に書き込めるか確かめるだけのつもりが操作誤って書き込んでしまいました汗
やはり全部は書き込めなかったので java コードだけ載せ、レイアウト等を含めた一通りは github に入れました。
Todo や AddTask をフラグメントにしています。

MainActivity.java

import androidx.appcompat.app.AppCompatActivity;
import androidx.fragment.app.FragmentManager;

import android.os.Bundle;
import android.util.Log;

public class MainActivity extends AppCompatActivity {
  @Override
  protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    FragmentManager fm = getSupportFragmentManager();
    fm.setFragmentResultListener(TodoFragment.REQUEST_KEY, this, (rkey,result)->{
      /*TodoFragment が back したら何する? */
      Log.d("MainActivity", "backed todofragment.");
    });

    fm.beginTransaction()
            .replace(R.id.fragment_container_view, new TodoFragment())
            .commit();
  }
}

TodoFragment.java (TodoAdapter,Task クラスを含む)

import androidx.annotation.*;
import androidx.fragment.app.*;
import androidx.recyclerview.widget.*;

import android.os.Bundle;
import android.util.Log;
import android.view.*;
import android.widget.*;

import java.io.Serializable;
import java.time.LocalDateTime;
import java.time.format.DateTimeFormatter;
import java.util.*;

public class TodoFragment extends Fragment {
  static final String REQUEST_KEY = "TodoFragment";

  public TodoFragment() {
    super(R.layout.fragment_todo);
  }

  @Override
  public void onViewCreated(@NonNull View view, @Nullable Bundle savedInstanceState) {
    super.onViewCreated(view, savedInstanceState);

    TaskAdapter taskAdapter = new TaskAdapter();

    RecyclerView recyclerView = view.findViewById(R.id.recyclerViewTasks);
    recyclerView.setAdapter(taskAdapter);

    FragmentManager fm = getChildFragmentManager();
    fm.setFragmentResultListener(AddTaskFragment.REQUEST_KEY, getViewLifecycleOwner(), (rkey, result) -> {
      taskAdapter.addTask((Task) result.getSerializable(AddTaskFragment.RESULTKEY_TASK));
    });

    Button btBackStudy = view.findViewById(R.id.btBackTodo);
    btBackStudy.setOnClickListener(v -> {
      Toast.makeText(getContext(), "戻るボタンがクリックされました", Toast.LENGTH_SHORT).show();
      getParentFragmentManager().setFragmentResult(REQUEST_KEY, new Bundle()); //(MainActivity に)通知
    });

    Button addButton = view.findViewById(R.id.btAdd);
    addButton.setOnClickListener(v -> {
      Toast.makeText(getContext(), "追加ボタンがクリックされました", Toast.LENGTH_SHORT).show();
      new AddTaskFragment().show(fm, null); //ダイアログ表示
    });
  }
}

class TaskAdapter extends RecyclerView.Adapter<TaskAdapter.ViewHolder> {
  private final List<Task> taskList = new ArrayList<>();

  void addTask(Task task) {
    Log.d("TaskAdded", "Task added: " + task);
    taskList.add(task);
    notifyItemInserted(taskList.size()-1);
  }

  @NonNull
  @Override
  public ViewHolder onCreateViewHolder(@NonNull ViewGroup parent, int viewType) {
    return new ViewHolder(parent);
  }

  @Override
  public void onBindViewHolder(@NonNull ViewHolder holder, int position) {
    Log.d("AdapterDebug", "onBindViewHolder called for position: " + position);
    holder.bind(taskList.get(position));
  }

  @Override
  public int getItemCount() {
    return taskList.size();
  }

  static class ViewHolder extends RecyclerView.ViewHolder {
    private final TextView taskName, dateTime;

    public ViewHolder(@NonNull ViewGroup parent) {
      super(LayoutInflater.from(parent.getContext()).inflate(R.layout.task_item, parent, false));
      taskName = itemView.findViewById(R.id.taskNameTextView);
      dateTime = itemView.findViewById(R.id.dateTimeTextView);
    }
    void bind(Task task) {
      taskName.setText(task.taskName);
      dateTime.setText(task.getDateTime());
    }
  }
}

class Task implements Serializable {
  static final DateTimeFormatter formatter = DateTimeFormatter.ofPattern("yyyy/MM/dd HH:mm");

  final String taskName;
  final LocalDateTime dateTime;

  public Task(String taskName, LocalDateTime dateTime) {
    this.taskName = taskName;
    this.dateTime = dateTime;
  }
  String getDateTime() {
    return formatter.format(dateTime);
  }

  @Override
  public String toString() {
    return taskName + " at " + getDateTime();
  }
}

AddTaskFragment.java

import androidx.annotation.*;
import androidx.appcompat.app.AlertDialog;
import androidx.fragment.app.DialogFragment;

import android.app.*;
import android.os.Bundle;
import android.view.*;
import android.widget.*;

import java.time.*;

public class AddTaskFragment extends DialogFragment {
  static final String REQUEST_KEY = "AddTaskFragment";
  static final String RESULTKEY_TASK = "task";

  private EditText nameView, datetimeView;
  private LocalDateTime datetime;

  @NonNull
  @Override
  public Dialog onCreateDialog(@Nullable Bundle savedInstanceState) {
    View view = LayoutInflater.from(requireContext()).inflate(R.layout.fragment_add_task, null);

    datetime = LocalDateTime.now();

    nameView = view.findViewById(R.id.editTextTaskName);
    datetimeView = view.findViewById(R.id.editTextDateTime);
    datetimeView.setOnClickListener(v -> showDatePickerDialog());

    return new AlertDialog.Builder(requireContext())
            .setTitle("Add Task")
            .setView(view)
            .setPositiveButton("保存", (d, w) -> {
              String name = nameView.getText().toString();
              Bundle result = new Bundle();
              result.putSerializable(RESULTKEY_TASK, new Task(name, datetime));
              getParentFragmentManager().setFragmentResult(REQUEST_KEY, result);
              dismiss();
            })
            .setNegativeButton("キャンセル", (d, w) -> dismiss())
            .create();
  }

  private void showDatePickerDialog() {
    DatePickerDialog datePickerDialog = new DatePickerDialog(getContext(), (v, y, m, d) -> { //m: 0-11
      datetime = datetime.with(LocalDate.of(y, m+1, d)); //m: 1-12
      showTimePickerDialog();
    }, datetime.getYear(), datetime.getMonthValue()-1, datetime.getDayOfMonth()); //m: 0-11
    datePickerDialog.show();
  }

  private void showTimePickerDialog() {
    TimePickerDialog timePickerDialog = new TimePickerDialog(getContext(), (v, h, m) -> {
      datetime = datetime.with(LocalTime.of(h, m));
      datetimeView.setText(Task.formatter.format(datetime));
    }, datetime.getHour(), datetime.getMinute(), true);
    timePickerDialog.show();
  }
}

Comments

[ottosay]

1から10まで全て教えて頂きありがとうございます。
また、調べても分からないことがあれば聞きたいくらい感謝しています！！

[ottosay]

遅くにすみません、質問をすること自体が初めてでわからないのですが、回答者の方に送っていただいたソースコードは利用してもいいのでしょうか？

[jimbe]

誰でも見られるところに書いていますので、基本的には自由に使って頂いて良いと思います。
法的な部分につきましては利用規約
https://teratail.com/legal
の第9条辺りが該当すると思いますが、ホーリツは良く分かりません(*_*)

[ottosay]

教えていただきありがとうございます！

[ottosay]

遅くにすみません。メイン画面に戻るボタンは機能しているのですが、メイン画面に戻ってからTodoの画面に遷移すると追加したはずのタスクが消えてしまいます。。色々とコードを打ってみたのですが、うまくいきません。。。
また、追加したタスクが完了したときに簡単に削除できるように削除ボタンを作成したのですが、これもうまくいきません。。
https://maausa.marurm.com/android-listview/#index_id7
このサイトを参考にしていたのですが、うまくいきません。。
アドバイスを頂けないでしょうか。

[jimbe]

質問を編集して別の質問にはしないでください。
teratail は、 QA の形で情報を蓄積することも目的としています。
もし ottosay さんと同様に RecyclerView の表示で困っている人が teratail に来ても、この質問にその情報があると判断出来無くなります。
同じコードや発展させたコードに対するものだとしても、他に同じ悩みを持って情報を探している人が質問を見つけて解決方法を得ることが出来るようにお考え下さい。
https://teratail.com/help/question-tips

[jimbe]

>～このサイトを参考にして～

参考にされているサイトは ListView で SQLite を使って～という流れのようですが、 ViewModel 等はどう使うおつもりでしょうか。
(質問を再編集して元に戻され、新たに削除ボタンに関する質問を立てて頂ければそちらに移動します。)

[ottosay]

分かりました。
もう一度説明し直したいと思います。

Answer 2

Android StudioならJavaよりKotlinを使った方がいいです。