[SamuraiBarbi]

Heya,
I've been having difficulties with keyvalues ( still trying to learn how to use them in SourceMod ). What I'm trying to do is write each new player that joins the game during a map to a file, so I have a list of players and the needed relevant info for those players in a keyvalue file. The problem I'm experiencing is this:

1). It's not writing into a subgroup. Instead it's just writing to the root.

This is what I want it to do

"roundPlayers"
{
    "apocalyptica:24"
    {
        "IsInfected"        "0"
    }
    "apocalyptica:25"
    {
        "IsInfected"        "0"
    }
    "apocalyptica:26"
    {
        "IsInfected"        "0"
    }
}

and this is what it's doing

"apocalyptica:26"
{
    "IsInfected"        "0"
}

2). It's overwriting each previous player with the new player. Example, there's 5 players that joined and the keyvalues file is saying the following.

"apocalyptica:26"
{
    "IsInfected"        "0"
}

This is my code. I just have it hooking player_activate right now.

public Action:HookPlayerActivate(Handle:event, const String:name[], bool:dontBroadcast)
{
    new userID = GetEventInt(event,"userid");

    
    decl String:userString[64];
    Format(userString,sizeof(userString),"apocalyptica:%i",userID);
    
    
    new Handle:roundPlayersConfig = CreateKeyValues("roundPlayers");
    decl String:path[PLATFORM_MAX_PATH];
    BuildPath(Path_SM,path,PLATFORM_MAX_PATH,"configs/km/es_round_players.txt");
    FileToKeyValues(roundPlayersConfig,path);
    
    KvGotoNextKey(roundPlayersConfig);
    if(KvJumpToKey(roundPlayersConfig,userString,false))
        return;
    
    KvJumpToKey(roundPlayersConfig,userString,true);
    KvSetNum(roundPlayersConfig,"IsInfected",0);
    KeyValuesToFile(roundPlayersConfig,path); 
    CloseHandle(roundPlayersConfig);
}

Anywho I was hoping someone could please tell me what I'm doing wrong.
Thanks for your time.

[dalto]

So, your problems are here.

if(KvJumpToKey(roundPlayersConfig,userString,false))
        return;

This says if the key does not exist than exit. There are two problems with this. First, if the key does not exist then you will exit the function and never get to the code that creates it. Second, you are not closing the roundPlayersConfig handle in this case and will leak handles.

Just remove the above code. It is not needed since the next line

KvJumpToKey(roundPlayersConfig,userString,true);

Does exactly what you want. It goes to the key if it exists and creates it if it doesn't.

Also, one a side note, do you want to store steam id's instead of userid's?

[SamuraiBarbi]

Oooo, ok dalto, I'll give that a shot. Woot! For this particular list I cannot use steamids because it must keep track of the bot players too and the bot's steam id's are all BOT. I'll let ya know how this turns out. =D

[SamuraiBarbi]

Alrighty, I made the changes you suggested and while they solved the problems you mentioned, I'm still experiencing the problems in my original post.

Here's the updated code:

public Action:HookPlayerActivate(Handle:event, const String:name[], bool:dontBroadcast)
{
    new userID = GetEventInt(event,"userid");

    
    decl String:userString[64];
    Format(userString,sizeof(userString),"apocalyptica:%i",userID);
    
    
    new Handle:roundPlayersConfig = CreateKeyValues("roundPlayers");
    decl String:path[PLATFORM_MAX_PATH];
    BuildPath(Path_SM,path,PLATFORM_MAX_PATH,"configs/km/es_round_players.txt");
    FileToKeyValues(roundPlayersConfig,path);
    
    KvGotoNextKey(roundPlayersConfig);
    
    KvJumpToKey(roundPlayersConfig,userString,true);
    KvSetNum(roundPlayersConfig,"IsInfected",0);
    KeyValuesToFile(roundPlayersConfig,path); 
    CloseHandle(roundPlayersConfig);
}

Any ideas?

[dalto]

Sorry, I misread your post. Unload the plugin, delete the keyvalues file and remove this line:

KvGotoNextKey(roundPlayersConfig);

Also, just a thought, but do you really want to load the file into memory and then put it back to disk everytime that event is called? I would consider just loading it OnPluginStart() and saving it to disk periodically.

[BAILOPAN]

player_activate isn't too bad of an event as long as they file isn't very big.

[SamuraiBarbi]

i don't mean to be a pain guys but i still can't get rid of the problems i stated in my first post. is there something that got overlooked?

[SamuraiBarbi]

I'm still having the same problem from the original post. Is there anyone that could please try the code out and see if they know why it's still being a bugger?

[Nican]

http://wiki.alliedmods.net/KeyValues...d_Scripting%29

PHP Code:

public Action:HookPlayerActivate(Handle:event, const String:name[], bool:dontBroadcast){
    new userID = GetEventInt(event,"userid");

    decl String:userString[64];
    Format(userString,sizeof(userString),"apocalyptica:%i",userID);
    
    
    new Handle:roundPlayersConfig = CreateKeyValues("roundPlayers");
    decl String:path[PLATFORM_MAX_PATH];
    BuildPath(Path_SM,path,PLATFORM_MAX_PATH,"configs/km/es_round_players.txt");
    FileToKeyValues(roundPlayersConfig,path);

    if(!KvJumpToKey(roundPlayersConfig,userString)){
        CloseHandle(roundPlayersConfig);
        return;
    }

    KvSetNum(roundPlayersConfig,"IsInfected",0);
    KeyValuesToFile(roundPlayersConfig,path); 
    CloseHandle(roundPlayersConfig);
} 


2. I HIGHLY recommend you not doing that, just use a normal array instead:

PHP Code:

new bool:IsInfected[MAXPLAYERS + 1];

public Action:HookPlayerActivate(Handle:event, const String:name[], bool:dontBroadcast){
    new userID = GetEventInt(event,"userid");
    
    IsInfected[ userID ] = true;
} 


[dalto]

They can't easily be stored in array because they are userid's NOT client indexes so they grow over time and are not limited to MAXPLAYERS