为什么这被认为是不安全的？

Question

最近，我用 java MVC 框架编写了一个简单的游戏，一位高级程序员在审查后告诉我，得分是不安全的，其 getter 和 setter 在玩家模型本身中定义。不幸的是，我无法从他们那里获得更多关于如何以及为何如此的信息，而且我无法弄清楚为什么。有人可以回顾一下这些类并指出为什么这种架构不可取吗？播放器型号：public class player implements Comparable<player>{    private int score;    private int consecutive_misses;    private String name;    private int foul_count;    private int turn;    private boolean win;    public player()    {        this.score=0;        this.consecutive_misses=0;        this.foul_count=0;        this.win=false;    }    public void setWin(boolean win)    {        this.win=win;    }    public boolean getWin()    {        return this.win;    }    public void setPlayerName(String name)    {        this.name=name;    }    public void setTurn(int turn)    {         this.turn=turn;    }    public int getTurn()    {        return this.turn;    }    public void addScore(int score)    {        this.score+=score;    }    public String getName()    {        return this.name;    }    public int getScore(){        return this.score;    }     public void resetFoulCount() {            this.foul_count = 0;    }     public void incrementFoulCount() {            this.foul_count++;    }     public int getFoulCount()     {         return this.foul_count;     }     public void resetConsecutiveMisses() {            this.consecutive_misses = 0;    }     public void incrementConsecutiveMisses() {            this.consecutive_misses++;    }      public int getConsecutiveMisses()      {          return this.consecutive_misses;      }      public int compareTo(player comparePlayer)      {          int compareScore=((player) comparePlayer).getScore();          return compareScore-this.score;      }}

Answer 1

类不应该暴露它的内部结构。通过暴露其内部结构，它违反了许多法律，例如德米特法则，其他类别可以利用这种情况并设置他们想要的任何分数。