AndroidStudioでのカレンダー作成で取得ずれが起きる

Question

以下のようなコードでカレンダーをGridViewにて作成しているのですが、今月のクリックイベントのときはちゃんとArrayListに値が入っていて正常に日付を取り出せるのですが、Prevボタンを押して前の月に移動したときや次の月の、クリックイベントではArrayListがずれており(前の月の日付列が入っていなければいけないのに、一番最初に保持した値が残っている）、カレンダーの表記上はちゃんと出ているのですが、正常にクリックした日付を取得できません。調べても解決できなそうなのでよろしければご助言お願いいたします。

Calendar Class

private List<Date> dateArray = new ArrayList();
    private Button prevButton, nextButton;
    private CalendarAdapter mCalendarAdapter;
    private GridView calendarGridView;
    private DateManager mDateManager;
    private KakeiboDatabase kDB;
    int foodEx,dailyNe,eduEx,lesson,entEx,other;
    int total;
    float foodExPar, dailyNePar, eduExPar, lessonPar, entExPar, otherPar;
    float oneHan = 100;


    @Override
    public void onItemClick(AdapterView<?> parent, View view, int position, long id) {
        mDateManager = new DateManager();
        dateArray = mDateManager.getDays();
        String date = titleText.getText().toString().substring(0,7);
        date = date + "-" + dateArray.get(position).toString().substring(8,10); //ここでポジションがあっているのにdateArrayがずれていて前月や次の月の日付取得ができません
        String judge = readData(date);
        if(judge!=null) {
            Intent intent = new Intent(getApplication(), DailyData.class);
            intent.putExtra("CalendarID", date);
            startActivity(intent);
        }
    }


    private static Calendar instance = null;

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

      

        instance = this;

        titleText = findViewById(R.id.titleText);
        prevButton = findViewById(R.id.prevButton);
        prevButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mCalendarAdapter.prevMonth();
                titleText.setText(mCalendarAdapter.getTitle());
                listenerSet();
            }
        });
        nextButton = findViewById(R.id.nextButton);
        nextButton.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                mCalendarAdapter.nextMonth();
                titleText.setText(mCalendarAdapter.getTitle());
                listenerSet();
            }
        });
        calendarGridView = findViewById(R.id.calendarGridView);
        mCalendarAdapter = new CalendarAdapter(this);
        calendarGridView.setAdapter(mCalendarAdapter);
        titleText.setText(mCalendarAdapter.getTitle());


        calendarGridView.setOnItemClickListener(this);
    }
 public void listenerSet(){
        calendarGridView.setOnItemClickListener(this);
    }
}

alendarAdapter Class

public class CalendarAdapter extends BaseAdapter {
    private List<Date> dateArray = new ArrayList();
    private Context mContext;
    private DateManager mDateManager;
    private LayoutInflater mLayoutInflater;
    private String spendPrice;
    private String incomePrice;

    //カスタムセルを拡張したらここでWigetを定義
    private static class ViewHolder {
        public TextView dateText;
    }

    public CalendarAdapter(Context context){
        mContext = context;
        mLayoutInflater = LayoutInflater.from(mContext);
        mDateManager = new DateManager();
        dateArray = mDateManager.getDays();
    }

    @Override
    public int getCount() {
        return dateArray.size();
    }

    @Override
    public View getView(int position, View convertView, ViewGroup parent) {
        ViewHolder holder;
        if (convertView == null) {
            convertView = mLayoutInflater.inflate(R.layout.calendar_cell, null);
            holder = new ViewHolder();
            holder.dateText = convertView.findViewById(R.id.dateText);
            convertView.setTag(holder);
        } else {
            holder = (ViewHolder)convertView.getTag();
        }

        //セルのサイズを指定
        float dp = mContext.getResources().getDisplayMetrics().density;
        AbsListView.LayoutParams params = new AbsListView.LayoutParams(parent.getWidth()/7 - (int)dp, (parent.getHeight() - (int)dp * mDateManager.getWeeks() ) / mDateManager.getWeeks());
        convertView.setLayoutParams(params);

        //その月日の金額をまとめる
        Calendar cal = (Calendar)mContext;
        SimpleDateFormat format = new SimpleDateFormat("yyyy-MM-dd", Locale.JAPAN);
        String sDate = format.format(dateArray.get(position));

        String getSpend = cal.readData(sDate);
        String getIncome = cal.incomeData(sDate);

        spendPrice = "<font color=\"red\">" + getSpend + "円</font>";
        incomePrice = "<font color=\"blue\">" + getIncome + "円</font>";

        String price;

        //日付とデータベースの金額を表示させる
        SimpleDateFormat dateFormat = new SimpleDateFormat("d", Locale.JAPAN);
        if((getSpend==null) && (getIncome == null)){
            holder.dateText.setText(dateFormat.format(dateArray.get(position)));
        } else if((getSpend==null) && (getIncome!=null)){
            holder.dateText.setText(Html.fromHtml(dateFormat.format(dateArray.get(position)) + "<br>" + incomePrice));
        } else if((getSpend!=null) && (getIncome==null)) {
            holder.dateText.setText(Html.fromHtml(dateFormat.format(dateArray.get(position)) + "<br>" + spendPrice));
        } else {
            holder.dateText.setText(Html.fromHtml(dateFormat.format(dateArray.get(position)) + "<br>" + incomePrice + "<br>" + spendPrice));
        }

        //当月以外のセルをグレーアウト
        if (mDateManager.isCurrentMonth(dateArray.get(position))){
            convertView.setBackgroundColor(Color.WHITE);
        }else {
            convertView.setBackgroundColor(Color.LTGRAY);
        }

        //日曜日を赤、土曜日を青に
        int colorId;
        switch (mDateManager.getDayOfWeek(dateArray.get(position))){
            case 1:
                colorId = Color.RED;
                break;
            case 7:
                colorId = Color.BLUE;
                break;

            default:
                colorId = Color.BLACK;
                break;
        }
        holder.dateText.setTextColor(colorId);

        return convertView;
    }

    @Override
    public long getItemId(int position) {
        return 0;
    }

    @Override
    public Object getItem(int position) {
        return null;
    }

    //表示月を取得
    public String getTitle(){
        SimpleDateFormat format = new SimpleDateFormat("yyyy-MM", Locale.JAPAN);
        return format.format(mDateManager.mCalendar.getTime());
    }

    //翌月表示
    public void nextMonth(){
        mDateManager.nextMonth();
        dateArray = mDateManager.getDays();
        this.notifyDataSetChanged();
    }

    //前月表示
    public void prevMonth(){
        mDateManager.prevMonth();
        dateArray = mDateManager.getDays();
        this.notifyDataSetChanged();
    }
    static Date truncateTime(Date d) {
        Instant instant = d.toInstant();
        ZonedDateTime zonedDateTime = instant.atZone(ZoneId.of("Asia/Tokyo", ZoneId.SHORT_IDS));
        ZonedDateTime truncated = zonedDateTime.truncatedTo(ChronoUnit.DAYS);
        return Date.from(truncated.toInstant());
    }
}

DateManager Class

public class DateManager {
    Calendar mCalendar;

    public DateManager(){
        mCalendar = Calendar.getInstance();
    }

    //当月の要素を取得
    public List<Date> getDays(){
        //現在の状態を保持
        Date startDate = mCalendar.getTime();

        //GridViewに表示するマスの合計を計算
        int count = getWeeks() * 7 ;

        //当月のカレンダーに表示される前月分の日数を計算
        mCalendar.set(Calendar.DATE, 1);
        int dayOfWeek = mCalendar.get(Calendar.DAY_OF_WEEK) - 1;
        mCalendar.add(Calendar.DATE, -dayOfWeek);

        List<Date> days = new ArrayList<>();

        for (int i = 0; i < count; i ++){
            days.add(mCalendar.getTime());
            mCalendar.add(Calendar.DATE, 1);
        }

        //状態を復元
        mCalendar.setTime(startDate);

        return days;
    }

    //当月かどうか確認
    public boolean isCurrentMonth(Date date){
        SimpleDateFormat format = new SimpleDateFormat("yyyy.MM", Locale.JAPAN);
        String currentMonth = format.format(mCalendar.getTime());
        if (currentMonth.equals(format.format(date))){
            return true;
        }else {
            return false;
        }
    }

    //週数を取得
    public int getWeeks(){
        return mCalendar.getActualMaximum(Calendar.WEEK_OF_MONTH);
    }

    //曜日を取得
    public int getDayOfWeek(Date date) {
        Calendar calendar = Calendar.getInstance();
        calendar.setTime(date);
        return calendar.get(Calendar.DAY_OF_WEEK);
    }

    //翌月へ
    public void nextMonth(){
        mCalendar.add(Calendar.MONTH, 1);
    }

    //前月へ
    public void prevMonth(){
        mCalendar.add(Calendar.MONTH, -1);
    }
}

Answer 1

ざっと見た感じですが, Calendar クラスの onItemClick で, dateArray を作成する DateManager が作られていますが,

mDateManager = new DateManager(); // <- これです
dateArray = mDateManager.getDays();
String date = titleText.getText().toString().substring(0,7);
date = date + "-" + dateArray.get(position).toString().substring(8,10)

DateManager は「何月の」dateArray を生成していますか?
画面の表示が前月や次月に移動した場合に, それに追従した dateArray が生成されるようになっていますでしょうか.

Comments

[Aies]

回答ありがとうございます。
前の月表示前である当月のdateArrayが生成されているように思えます。
DateManagerのコンストラクタにあるインスタンス設定で最初の月の値がまた入っているのでしょうか？

[jimbe]

どのようにするとどのようなインスタンスが作成されるのかは, 作者の Aies さんがご理解頂いていなくてはなりません.

DateManager はどのような存在でしょうか.
DateManager は CalendarAdapter クラスでも使われていますが, こちらは前月・次月ボタンの操作によって, メソッドを呼び出して自身の内容を更新しているようです.
つまり DateManager は, 現在表示中の月の各日付を管理しているモノと考えられます.
ですから, このアプリの起動時に１つだけ生成して使うことを想定したモノで, 何度も作られるモノでは無いと思われます.

CalendarAdapter 内の DateManager のインスタンス mDateManager を使っては如何でしょう.
CalenderAdapter のインスタンスは Calender クラスが mCalendarAdapter として持っています.
CalendarAdapter には, 本来こういった場合に使う getItem(int position) がありますが, 現在は null を返していて使えません.
mDateManager.getDays().get(position) を返すようにすることで, 問題の

date = date + "-" + dateArray.get(position).toString().substring(8,10);

という行が

date = date + "-" + mCalendarAdapter.getItem(position).toString().substring(8,10);

と書けるようになります.

[jimbe]

なお, toString は本来デバッグ用のもので, ～.toString().substring(8,10) のように解析して使用することを想定されていないものだということを念のためお伝えしておきます.
toString の返す文字列は, 実装やバージョン, 時にはオブジェクトの内容毎にでも異なって良いことになっています. それを知らずにいますと, 後になって「何もしていないのに動かなくなった」となる場合があります.
文字列にして一部を取り出すのでは無く, Date 型から月・日など必要なデータを一つずつ取り出して文字列化(フォーマット)するようにしたほうが良いと思います.

[Aies]

なるほど。ご指摘の通り、一つずつコメントをつけるなどして理解するようにしたら、解決し、理解も深まりました。ありがとうございました。

結果おっしゃっていた通り、return nullをかえ、受け取るようにしたらできました。