diff --git a/BC-Free/app/src/main/java/ca/josephroque/bowlingcompanion/fragment/StatsListFragment.java b/BC-Free/app/src/main/java/ca/josephroque/bowlingcompanion/fragment/StatsListFragment.java index 8cd2c1e16..1e30ca774 100644 --- a/BC-Free/app/src/main/java/ca/josephroque/bowlingcompanion/fragment/StatsListFragment.java +++ b/BC-Free/app/src/main/java/ca/josephroque/bowlingcompanion/fragment/StatsListFragment.java @@ -496,9 +496,10 @@ protected List[] doInBackground(Byte... statsToLoad) { } } + short gameScore = cursor.getShort(cursor.getColumnIndex(Contract.GameEntry.COLUMN_SCORE)); boolean gameIsManual = (cursor.getInt(cursor.getColumnIndex( Contract.GameEntry.COLUMN_IS_MANUAL)) == 1); - if (gameIsManual) { + if (gameIsManual || gameScore == 0) { cursor.moveToNext(); continue; } @@ -731,13 +732,15 @@ private void setGeneralAndDetailedStatValues( int totalMatchPlayGames = 0; for (int stat : statValues[mStatsMatch]) totalMatchPlayGames += stat; - for (byte i = 0; i < statValues[mStatsMatch].length; i++) - listStatNamesAndValues.get(mStatsMatch).set(i, Pair.create( - listStatNamesAndValues.get(mStatsMatch).get(i).first, - decimalFormat.format( - statValues[mStatsMatch][i] / (double) totalMatchPlayGames * 100) - + "% [" + statValues[mStatsMatch][i] + "/" + totalMatchPlayGames - + "]")); + if (totalMatchPlayGames > 0) { + for (byte i = 0; i < statValues[mStatsMatch].length; i++) + listStatNamesAndValues.get(mStatsMatch).set(i, Pair.create( + listStatNamesAndValues.get(mStatsMatch).get(i).first, + decimalFormat.format( + statValues[mStatsMatch][i] / (double) totalMatchPlayGames * 100) + + "% [" + statValues[mStatsMatch][i] + "/" + totalMatchPlayGames + + "]")); + } for (byte i = 0; i < statValues[mStatsOverall].length; i++) listStatNamesAndValues.get(mStatsOverall).set(i, Pair.create(