🎯 このPRの本質(コードから判断)

「学習時間の持ち方を変更したPR」

🧠 Before → After(設計の変化)


❌ Before(問題のある設計)


🟥 ① ログから計算していた(本来は正しいが)

// タスクに紐づくログを取得
const filteredLogs = studyLogs.filter(
  (log) => log.taskId === task.id
);

// 合計時間を計算
const totalMinutes = filteredLogs.reduce(
  (sum, log) => sum + log.minutes,
  0
);

🟥 ② でも更新は「直接上書き」

updateTask(index, {
  ...task,
  totalMinutes: Number(minutes), // ← 上書き
});

💥 問題

・ログで計算してるのに
・タスクでも持ってる

👉 二重管理

❌ ③ ログ追加(完全上書き)

if (minutes) {
  addStudyLog(task.id, Number(minutes));
}