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]);
}
}
}
}
答案 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
是静态的,所以对于班级的所有实例它总是一样的。和你的其他班级成员一样。