More robust handling of cases where no data files

[scorecard] / src / table-model.cpp

diff --git a/src/table-model.cpp b/src/table-model.cpp
index c0cc2d7..e973bfb 100644 (file)
--- a/src/table-model.cpp
+++ b/src/table-model.cpp
@@ -11,19 +11,28 @@ Qt::ItemFlags ScoreTableModel::flags ( const QModelIndex & index )
 void ScoreTableModel::setScore(QList<Score *> &sList)
 {
   scoreList = sList;
-  score = scoreList.at(currentScore); // NOTE: assumes non-empty list
+  if (scoreList.size() > 0)
+    score = scoreList.at(currentScore); // NOTE: assumes non-empty list
 }
 
 void ScoreTableModel::setClub(QList<Club *> &cList)
 {
   clubList = cList;
-  club = clubList.at(0);
-  course = club->getCourse(0);
+
+  if (clubList.size() > 0)
+    club = clubList.at(0);
+
+  if (club)
+    course = club->getCourse(0);
 }
 
 QString ScoreTableModel::getInfoText()
 {
-  QString str = QString("%1, %2 / [%3/%4]").arg(score->getCourseName()).arg(score->getDate()).arg(currentScore+1).arg(scoreList.count());
+  QString str("");
+
+  if (score)
+    str = QString("%1, %2 / [%3/%4]").arg(score->getCourseName()).arg(score->getDate()).arg(currentScore+1).arg(scoreList.count());
+
   return str;
 }
 
@@ -49,37 +58,45 @@ Course *ScoreTableModel::findCourse(QString &clubName, QString &courseName)
 
 void ScoreTableModel::first()
 {
-  currentScore = 0;
-  score = scoreList.at(currentScore);
-  course = findCourse(score->getClubName(), score->getCourseName());
-  emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+  if (score && course) {
+    currentScore = 0;
+    score = scoreList.at(currentScore);
+    course = findCourse(score->getClubName(), score->getCourseName());
+    emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+  }
 }
 
 void ScoreTableModel::last()
 {
-  currentScore = scoreList.size() - 1;
-  score = scoreList.at(currentScore);
-  course = findCourse(score->getClubName(), score->getCourseName());
-  emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+  if (score && course) {
+    currentScore = scoreList.size() - 1;
+    score = scoreList.at(currentScore);
+    course = findCourse(score->getClubName(), score->getCourseName());
+    emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+  }
 }
 
 void ScoreTableModel::next()
 {
-  if (currentScore < (scoreList.size() - 1)) {
-    currentScore++;
-    score = scoreList.at(currentScore);
-    course = findCourse(score->getClubName(), score->getCourseName());
-    emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+  if (score && course) {
+    if (currentScore < (scoreList.size() - 1)) {
+      currentScore++;
+      score = scoreList.at(currentScore);
+      course = findCourse(score->getClubName(), score->getCourseName());
+      emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+    }
   }
 }
 
 void ScoreTableModel::prev()
 {
-  if (currentScore > 0) {
-    currentScore--;
-    score = scoreList.at(currentScore);
-    course = findCourse(score->getClubName(), score->getCourseName());
-    emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+  if (score && course) {
+    if (currentScore > 0) {
+      currentScore--;
+      score = scoreList.at(currentScore);
+      course = findCourse(score->getClubName(), score->getCourseName());
+      emit dataChanged(createIndex(0, 0), createIndex(ROW_COUNT-1, COL_COUNT-1));
+    }
   }
 }
 
@@ -114,6 +131,9 @@ QVariant ScoreTableModel::data(const QModelIndex &index, int role) const
   if (!index.isValid())
     return QVariant();
 
+  if (!course || !score)
+    return QVariant();
+
   int row = index.row();
   int col = index.column();