Question

我写了以下代码。一切正常。但是，我被告知它是以不正确的格式编写的。我不确定正确的编码方式是什么。任何帮助或方向将非常感激。

package com.mygdx.gameworld;

import com.badlogic.gdx.Gdx;
import com.mygdx.gameobjects.Egg;
import com.mygdx.gameobjects.Ground;
import com.mygdx.gameobjects.Hero;

public class GameWorld {

Hero hero = new Hero(); //This seems to the bit that causes some amusement.
Egg egg = new Egg(); //This seems to the bit that causes some amusement.
Ground ground = new Ground(); //This seems to the bit that causes some amusement.


public void update(float delta) {
    Gdx.app.log("GameWorld", "update");
    egg.update(delta);
    hero.update(delta);
    ground.update(delta);

}

public Hero getHero() {
    return hero;
}

public Egg getEgg() {
    return egg;
}

public Ground getGround() {
    return ground;
}

}

Answer 1

这些注释变量是您班级的成员。它们没有修饰符，这意味着它们仅在类内部或同一包中的类中可见。如果您想在测试时模拟它们，这绝对有意义。在其他情况下，我不建议使用它们，而是将它们设为私有。

有关修饰符和类成员的更多信息：http://docs.oracle.com/javase/tutorial/java/javaOO/accesscontrol.html

Answer 2

代码基本上没问题，但应该做一些改动：

Eclipse 格式化（ Ctrl + Shift + F ， Alt + Netbeans 中的 Shift + F 。
由于您有getter，所有字段都应该是私有的（否则它们在 com.mygdx.gameworld 包中可见）。
在我看来，将英雄，蛋，地面实例移动到构造函数会更加清晰。
除非有一些非常复杂的逻辑，否则您的字段/方法不应包含任何注释。尝试将您的代码视为文档。

我真的建议： Clean Code by Robert C. Martin

Libgdx保持代码整洁

2 个答案: