为数组中的对象赋值

时间:2012-05-24 20:07:34

标签: java arrays bufferedreader

    package macroreader;

    import java.io.BufferedReader;
    import java.io.FileReader;
    import java.io.IOException;

    public class MacroReader {

        public static Macro[] macroArray = new Macro[20];

        public static int macroID;

        public static BufferedReader br;

        public static void main(String[] args) throws IOException {
            br = new BufferedReader(new FileReader("Macros.txt"));
            String currentLine;
            while((currentLine = br.readLine()) != null) {
                if(currentLine.equalsIgnoreCase("#newmacro")) {
                    br.mark(1000);
                    createMacro();
                    br.reset();
                }
            }
            if (br != null) {
                br.close();
            }
        }

    public static void createMacro() throws IOException {
        String currentLine;
        macroID = getEmptyMacro();
        while((currentLine = br.readLine()) != null && !currentLine.equalsIgnoreCase("#newmacro")) {
            macroArray[macroID].readMacro(currentLine);
        }
        macroArray[macroID].inUse = true;
        macroArray[macroID].printData();
    }

    public static int getEmptyMacro() {
        for(int i = 0; i < macroArray.length; i++) {
            if(!macroArray[i].inUse) {
                return i;
            }
        }
        return 0;
    }

}

而不是将从文件阅读器读取的值分配给数组中的指定对象,在本例中为“macroID”,它将值分配给数组中的所有对象

现在只在整个文件中编辑,但问题出在createMacro()void

这是我的Macro类

package macroreader;

public class Macro {

    public static String key;
    public static String[] commands = new String[20];
    public static boolean inUse;

    public static void readMacro(String input) {
        if (!input.equals("")) {
            if (input.startsWith("key = ")) {
                key = input.substring(6);
                System.out.println("Key Value for Macro set to " + key);
            } else {
                for (int i = 0; i < commands.length; i++) {
                    if (commands[i] == null) {
                        commands[i] = input;
                        System.out.println("Command [" + input + "] assigned");
                        break;
                    }
                }
            }
        }
    }

    public static void printData() {
        System.out.println("Macro Key: " + key);
        for(int i = 0; i < commands.length; i++) {
            if(commands[i] != null) {
                System.out.println(commands[i]);
            }
        }
    }

}

2 个答案:

答案 0 :(得分:2)

改变数组中所有内容的经典原因&#34;如果您实际上已将相同的对象分配给数组中的每个元素。我们无法判断您是否正在执行此操作,因为您没有向我们展示macroArray的初始化,但它可能是这样的:

Macro m = new Macro();
for (int i = 0; i < macroArray.length; i++) {
  macroArray[i] = m;
}

这将导致您描述的结果。要修复它,请为数组的每个元素创建一个单独的对象:

for (int i = 0; i < macroArray.length; i++) {
  macroArray[i] = new Macro();
}

答案 1 :(得分:2)

我怀疑 - 你的inUse是静态的,所以对于班级的所有实例它总是一样的。和你的其他班级成员一样。