Question

我有一个带有方法setMapName的类Map，它从Enum mapName中选择它来设置它，在构造函数中有一个规则，根据mapName值命名一个来自数组扇区的单元格Alien或Human，我想要测试单元格的名称是否确实是Alien或Human但我得到null。

public class Map {
    private Name mapName;
    private final Sector [][] sector;
    private int Matrix [][];
    private static final int X=23;
    private static final int Y=14;
    public Map (){
        sector = new Sector[X][Y];
        for (int i=0; i < X; i++){
            for (int j=0; j<Y; j++) {
                sector[i][j] = new Sector (i,j);
            }
        }
        Matrix = new int[23][14];
        if(mapName==Name.FERMI){
            sector[10][8]=new Alien(10,8);
            sector[10][9]=new Human(10,9);
        }
        if(mapName==Name.GALILEI||mapName==Name.GALVANI){
            sector[10][5]=new Alien(10,5);
            sector[10][7]=new Human(10,7);
        }
    }
    public int[][] getMatrix() {
        return Matrix;
    }   
    public void setMatrix(int matrix[][]) {
        Matrix = matrix;
    }
    public Name getMapName() {
        return mapName;
    }
        public void setMapName(Name mapName) {//this is the method i want to use before the constructor
            this.mapName = mapName;
        }
        public Sector[][] getSectors(){
            return sector;
        }
        public void addSectors(){
            Sector.add(sector);
        }
    }

    public enum Name {
    FERMI, GALILEI, GALVANI
    }

    public class MapTest {
        @Test
        public void testMapAlien(){
            Map map = new Map();
            map.setMapName(Name.FERMI);
            assertEquals(Name.Alien, map.getSectors()[10][8].getSectorName());
        }
    }

Answer 1

您的setMapName是Map课程中的非静态成员函数（因为它与java.util.Map发生碰撞而命名不佳）。这意味着可以在Map的现有实例（例如已经构建的Map）上调用它。

您的问题是在调用构造函数后调用setMapName，但您的构造函数需要有效的Name才能正常工作！这是一个经典的鸡和蛋问题。

为什么不直接将MapName传递给构造函数？

public Map (Name mapName){
        sector = new Sector[X][Y];
        for (int i=0; i < X; i++){
            for (int j=0; j<Y; j++) {
                sector[i][j] = new Sector (i,j);
            }
        }
        Matrix = new int[23][14];
        if(mapName==Name.FERMI){
            sector[10][8]=new Alien(10,8);
            sector[10][9]=new Human(10,9);
        }
        if(mapName==Name.GALILEI||mapName==Name.GALVANI){
            sector[10][5]=new Alien(10,5);
            sector[10][7]=new Human(10,7);
        }
        ...
}

Answer 2

您的问题是您没有初始mapName。调用构造函数时，mapName为null。所以......

您可以通过两种方式实现：

//Constructor with argument mapName
public Map (Name mapName){
 setMapName(mapName); //or this.mapName = mapName;
 ...
}

或者：

//Give a value to mapName on the constructor without arguments, but I recommend the first one to your case.
public Map (){
 ...
 mapName = value;
 ...
 setMapName(mapName);
}

Java在构造函数之前调用方法

2 个答案: