Java

[gongbao]

SpaceShip.java, SpaceGame.java里面

 

[gongbao]

Spaceship player = FileIO.loadSpaceship("player.txt");
        //Spaceship player=FileIO.loadSpaceship("enemy.txt");
        setPlayer(player);
        ArrayList<Planet> planets = FileIO.loadPlanets("sol.txt");

        Spaceship.setPlanets(planets);

问题就在这里了，我想你应该实例化两个spaceship，而不是直接用Spaceship.xxx方法，因为你的spaceship class没有用singleton那种xxxclass.getInstance(),所以你不可以直接引用Spaceship.java里面的方法

 

[月盈亏]

Spaceship player = FileIO.loadSpaceship("player.txt");
        //Spaceship player=FileIO.loadSpaceship("enemy.txt");
        setPlayer(player);
        ArrayList<Planet> planets = FileIO.loadPlanets("sol.txt");

        Spaceship.setPlanets(planets);

问题就在这里了，我想你应该实例化两个spaceship，而不是直接用Spaceship.xxx方法，因为你的spaceship class没有用singleton那种xxxclass.getInstance(),所以你不可以直接引用Spaceship.java里面的方法

哇塞，你太厉害，我慢慢看。谢谢

 

[月盈亏]

不客气，我想你应该利用enemy.txt, player.txt读入数据，实例化两个spaceship，再引用.moveTo()方法，然后一切问题都迎刃而解了

怎么实例化啊，给我举个例子呗。

 

[fieldmarshal]

不客气，我想你应该利用enemy.txt, player.txt读入数据，实例化两个spaceship，再引用.moveTo()方法，然后一切问题都迎刃而解了

如果这位不是个女生，你该气死了，哈

 

[gongbao]

可以改嫁

这个……是不是太残忍了哥？

 

[fieldmarshal]

这个……是不是太残忍了哥？

比天天写到两点强

 

[woodenwang]

public class Planet {

public Planet(String name) {
super();
this.name = name;
}

private String name;

public String getName() {
return name;
}

public void setName(String name) {
this.name = name;
}

}

public class SpaceShip {

private String shipName;
private Planet currentPlannt;

public SpaceShip(String shipName) {
super();
this.shipName = shipName;
}
public String getShipName() {
return shipName;
}
public Planet getCurrentPlannt() {
return currentPlannt;
}
public void setShipName(String shipName) {
this.shipName = shipName;
}
public void setCurrentPlannt(Planet currentPlannt) {
this.currentPlannt = currentPlannt;
}

public void moveTo(Planet p){
System.out.println("SpaceShip "+this.getShipName()+" move from "+(this.currentPlannt==null?"Unknown space ":this.currentPlannt.getName())+" To "+p.getName());
this.setCurrentPlannt(p);
}
}

public class SpaceGame {

List<Planet> planets = new ArrayList<Planet>();
SpaceShip shipA = new SpaceShip("A");
SpaceShip shipB = new SpaceShip("B");

public static void main(String[] args){

SpaceGame game = new SpaceGame();
Planet p1 = new Planet("Earth");
Planet p2 = new Planet("Mars");
Planet p3 = new Planet("Jupitor");
Planet p4 = new Planet("Venus");
Planet p5 = new Planet("Saturn");
game.planets.add(p1);
game.planets.add(p2);
game.planets.add(p3);
game.planets.add(p4);
game.planets.add(p5);
game.shipA.moveTo(game.findPlanetByName("Earth"));
game.shipB.moveTo(game.findPlanetByName("Saturn"));
game.shipA.moveTo(game.findPlanetByName("Mars"));
game.shipB.moveTo(game.findPlanetByName("Jupitor"));


}

public List<Planet> getPlanets() {
return planets;
}

public SpaceShip getShipA() {
return shipA;
}

public SpaceShip getShipB() {
return shipB;
}

public void setPlanets(List<Planet> plannets) {
this.planets = plannets;
}

public Planet findPlanetByName(String name){
for (Planet p: planets){
if (p.getName().equals(name))
return p;
}
return null;
}

public void addPlanet(Planet p){
this.planets.add(p);
}

public void setShipA(SpaceShip shipA) {
this.shipA = shipA;
}

public void setShipB(SpaceShip shipB) {
this.shipB = shipB;
}



}

 

[woodenwang]

简单写了一下，参照这个看看，感觉之前你写的版本在OO的基本概念上是混乱的，可以参考这个看看

 

[月盈亏]

public class Planet {

public Planet(String name) {
super();
this.name = name;
}

private String name;

public String getName() {
return name;
}

public void setName(String name) {
this.name = name;
}

}

public class SpaceShip {

private String shipName;
private Planet currentPlannt;

public SpaceShip(String shipName) {
super();
this.shipName = shipName;
}
public String getShipName() {
return shipName;
}
public Planet getCurrentPlannt() {
return currentPlannt;
}
public void setShipName(String shipName) {
this.shipName = shipName;
}
public void setCurrentPlannt(Planet currentPlannt) {
this.currentPlannt = currentPlannt;
}

public void moveTo(Planet p){
System.out.println("SpaceShip "+this.getShipName()+" move from "+(this.currentPlannt==null?"Unknown space ":this.currentPlannt.getName())+" To "+p.getName());
this.setCurrentPlannt(p);
}
}

public class SpaceGame {

List<Planet> planets = new ArrayList<Planet>();
SpaceShip shipA = new SpaceShip("A");
SpaceShip shipB = new SpaceShip("B");

public static void main(String[] args){

SpaceGame game = new SpaceGame();
Planet p1 = new Planet("Earth");
Planet p2 = new Planet("Mars");
Planet p3 = new Planet("Jupitor");
Planet p4 = new Planet("Venus");
Planet p5 = new Planet("Saturn");
game.planets.add(p1);
game.planets.add(p2);
game.planets.add(p3);
game.planets.add(p4);
game.planets.add(p5);
game.shipA.moveTo(game.findPlanetByName("Earth"));
game.shipB.moveTo(game.findPlanetByName("Saturn"));
game.shipA.moveTo(game.findPlanetByName("Mars"));
game.shipB.moveTo(game.findPlanetByName("Jupitor"));


}

public List<Planet> getPlanets() {
return planets;
}

public SpaceShip getShipA() {
return shipA;
}

public SpaceShip getShipB() {
return shipB;
}

public void setPlanets(List<Planet> plannets) {
this.planets = plannets;
}

public Planet findPlanetByName(String name){
for (Planet p: planets){
if (p.getName().equals(name))
return p;
}
return null;
}

public void addPlanet(Planet p){
this.planets.add(p);
}

public void setShipA(SpaceShip shipA) {
this.shipA = shipA;
}

public void setShipB(SpaceShip shipB) {
this.shipB = shipB;
}



}

妈呀，太强了，我得慢慢看。谢谢

 

[woodenwang]

运行结果差不多这样：
SpaceShip A move from Unknown space To Earth
SpaceShip B move from Unknown space To Saturn
SpaceShip A move from Earth To Mars
SpaceShip B move from Saturn To Jupitor

 

[月盈亏]

一行行读.txt，然后根据你spaceship.java的atrributes赋值，用你的constructor建立实例

这工作我做了呀，不过是在game class里实例的

 

[月盈亏]

运行结果差不多这样：
SpaceShip A move from Unknown space To Earth
SpaceShip B move from Unknown space To Saturn
SpaceShip A move from Earth To Mars
SpaceShip B move from Saturn To Jupitor

就要这个结果，太好了，我回家慢慢看。太感谢了

 

[月盈亏]

，几点小建议：
1) 变量名要规范，要遵循Naming Conventions, 我看到你有变量名叫Name,这个在后面有依赖注入的时候，是要出错的；
2）给属性更有意义的名字，我看到你很多属性都叫name,这样程序复杂了，你自己都搞不清楚了
3) 程序里没有注释，自己就很容易混乱，建议关键代码行加注释

一路飙车回来，边开边看，发现了关键问题。
game.shipB.moveTo(game.findPlanetByName("Saturn")

我不知道可以写成shipB.moveTo，我在game class里这样写的
Spaceship s = new Spaceship（）；
s.moveTo("Saturn" );

我以为只有构造函数里的attribute可以跟着实例走，（currenposition不在spaceship的构造函数里）

 

[月盈亏]

好像有点明白了，
Cat.sleep()
dog.sleep()

 

[月盈亏]

把currentpositon作为飞船类中的一个属性

是，我感觉我的问题就在这里。我只认为构造函数中的属性跟class捆绑，其他属性不行。
概念上还是迷糊

 

[xchn818]

类和实例的区别。类方法和对象的区别。也可以考虑使用接口。

 

[月盈亏]

成功了。太谢谢各位大侠了。熬了三天，做梦都在想办法。

 

[月盈亏]

，几点小建议：
1) 变量名要规范，要遵循Naming Conventions, 我看到你有变量名叫Name,这个在后面有依赖注入的时候，是要出错的；
2）给属性更有意义的名字，我看到你很多属性都叫name,这样程序复杂了，你自己都搞不清楚了
3) 程序里没有注释，自己就很容易混乱，建议关键代码行加注释

太感谢你了，能亲你不？

就是shipA.moveTo() 的问题。
我全部都是用spaceship.moveTo(). 所以currePosition没有和A，B联系起来。

 

[月盈亏]

类和实例的区别。类方法和对象的区别。也可以考虑使用接口。

看来我学抽象的东西有障碍，一定要看到实例才能理解。