Description
getAllPlayerStatistics() returns an empty map on SQLException, making DB errors indistinguishable from "no data found". This can cause data loss if a connection failure during player load results in populateFromEntries({}) followed by markClean().
Solution
Either throw a checked exception, return a result wrapper, or use a flag to distinguish error from empty.
Files
Priority: Medium
Description
getAllPlayerStatistics()returns an empty map onSQLException, making DB errors indistinguishable from "no data found". This can cause data loss if a connection failure during player load results inpopulateFromEntries({})followed bymarkClean().Solution
Either throw a checked exception, return a result wrapper, or use a flag to distinguish error from empty.
Files
PlayerStatisticDAO.javaPriority: Medium