Solved Reseting Players in File

Hey Guys, I'm trying to create an Lobby-System which should read out Data out of my "Database" (and File which I created).
When the Server stops, all Players should be removed so I also have to update my File. For this I'm using this Function:

Show Spoiler Hide Spoiler

Code:java

public static boolean resetAllPlayers()
    {
        checkFile();
        File tempFile = new File("plugins/Lobbys","reset.tmp");
        try {
            reader = new FileReader(file);
            writer = new FileWriter(tempFile);
            BufferedReader br = new BufferedReader(reader);
            BufferedWriter bw = new BufferedWriter(writer);
            LineNumberReader lnr = new LineNumberReader(reader);
            lnr.skip(Long.MAX_VALUE);
            for (int i=0; i <= lnr.getLineNumber(); i++)
            {
                br.skip(i);
                if (br.readLine() == null) continue;
                String [] data = br.readLine().split(",");
                bw.write(data[0] + "," + data[1] + "," + data[2] + "," + data[3] + "," + 0);
            }
            lnr.close();
            br.close();
            bw.close();
            writer.close();
            reader.close();
            file.delete();
            tempFile.renameTo(file);
            return true;
        } catch (Exception e) {
            e.printStackTrace();
            return false;
        }
    }

Not the problem is, that it won't update the Players (last number in line) in the File, it erases the entire Line. I didn't try it out with more Lines yet (maybe it's deleting the hole File).

Could someone help me? Thanks in advance.

 

So do you want the players inventory to be cleared when the join or do you want the player data to be removed?


Edit:
So what is the reason that you need to remove all the player data on shutdown?

 

It's because otherwise the the Data would say (example): 20 Players are in Lobby1. Max Players are 24. Only 4 can join now > Server has started and there are just 5 Players on.
So I'ld like to remove them when the Server shutsdown so when the Server starts again, all Lobbys are joinable.
But instead of removing/replacing the Players in the File, it's removing the entire Line.

 

I think the problem is the way you read the file. According to the Java documentation BufferedReader.skip(Integer) skips that number of Charachters, not lines. Also, as far as I know BufferedReader.readLine() returns the line the reader is on and move on to the next automatically, so you might want to try something like:

Code:java

 String readLine = br.readLine();
 
while(readLine != null) {
 
    String [] data = readLine.split(",");
 
    bw.write(data[0] + "," + data[1] + "," + data[2] + "," + data[3] + "," + 0);
 
    readLine = br.readLine();
 
}

http://docs.oracle.com/javase/6/docs/api/java/io/BufferedReader.html

Also I don't quite get why you don't delete the file if it only holds the players you want to reset anyway?

 

igwb Oh sorry I didn't update the Code yet :/
I allready cleared that out, new Code:

Show Spoiler Hide Spoiler

Code:java

 public static boolean resetAllPlayers()
{
checkFile();
File tempFile = new File("plugins/Lobbys","reset.tmp");
try {
reader = new FileReader(file);
writer = new FileWriter(tempFile);
BufferedReader br = new BufferedReader(reader);
BufferedWriter bw = new BufferedWriter(writer);
for (int i=0; br.ready(); i++)
{
if (br.readLine() == null) continue;
String [] data = br.readLine().split(",");
bw.write(data[0] + "," + data[1] + "," + data[2] + "," + data[3] + "," + 0);
}
br.close();
bw.close();
writer.close();
reader.close();
file.delete();
tempFile.renameTo(file);
return true;
} catch (Exception e) {
e.printStackTrace();
return false;
}
}

I guess I allready have the problem, 1 mom

 

PreFiXAUT Could you perhaps post the file you are trying to read and the output file?

 

igwb The output file is a temp file created at the beginning of the function. The original File is deleted and the tempfile is renamed to the original File.
file:

Code:

-108,81,32,24,10

I have no Idea what I've changed but now it's working xD
Code I'm using now:

Show Spoiler Hide Spoiler

Code:java

public static boolean resetAllPlayers()
{
checkFile();
File tempFile = new File("plugins/Lobbys","reset.tmp");
try {
reader = new FileReader(file);
writer = new FileWriter(tempFile);
BufferedReader br = new BufferedReader(reader);
BufferedWriter bw = new BufferedWriter(writer);
String temp;
for (int i=0; (temp = br.readLine()) != null ; i++)
{
String [] data = temp.split(",");
bw.write(data[0] + "," + data[1] + "," + data[2] + "," + data[3] + "," + 0);
}
br.close();
bw.close();
writer.close();
reader.close();
file.delete();
tempFile.renameTo(file);
return true;
} catch (Exception e) {
e.printStackTrace();
return false;
}
}

EDIT by Moderator: merged posts, please use the edit button instead of double posting.