Dobrze zrobiłem plugin ?

Witam, przed chwilą zrobiłem jeden plugin i się chcę was spytać czy jest on dobrze zrobiony, kompilatorem dobrze mi go skompilowało ale że serwer ma dużo graczy to wolę się spytać czy go wgrać na serwer czy nie będzie żadnego crasha:

/* Plugin generated by AMXX-Studio */

#include <amxmodx>
#include <amxmisc>
#include <hamsandwich>
#include <cstrike>

#define PLUGIN "Model klawisza"
#define VERSION "1.2"
#define AUTHOR "iplaycs"


public plugin_init() {
    register_plugin(PLUGIN, VERSION, AUTHOR)
    RegisterHam(Ham_Spawn, "player", "runda", 1);        
}
public runda(id){
    if(cs_get_user_team(id) == CS_TEAM_CT) {            
    if(is_user_connected(id))                
    cs_set_user_model(id, "klawisz")
}
}
public plugin_precache(){    
    precache_model("models/player/klawiszfpg/klawisz.mdl")    
}

Dobrze jest!

raczej błędu nie widzę.

Oj panowie !

precache_model("models/player/klawiszfpg/klawisz.mdl")

na

precache_model("models/player/klawisz/klawisz.mdl")

ewentualnie na

precache_model("models/player/klawiszfpg/klawiszfpg.mdl")      

Crash będzie jak tego nie zrobisz.

Użytkownik kapi10072 edytował ten post 29.07.2011 18:01

a ta funkcja, żeby działała powinna wyglądać raczej tak

public runda(id){

    if(cs_get_user_team(id) == CS_TEAM_CT && is_user_connected(id)) {            

    cs_set_user_model(id, "klawisz")

}

}

Ciekawe...
Jeżeli chodzi o sam crash:

[ 39] Model klawisza 1.2 iplaycs test.amxx running

Wrzuciłem plugin i jakoś działa. Magia.

Wbijam na serwer i nie ma crasha

Jednak nazywanie tak samo katalogu i pliku oczywiście jest zalecane

Jest zalecane, musi być takie samo.

dodatkowo przed użyciem

cs_get_user_team

najlepiej sprawdź czy gracz żyje bo może walić error logami

czyli tak

#include <amxmodx>

#include <hamsandwich>

#include <cstrike>



#define PLUGIN "Model klawisza"

#define VERSION "1.2"

#define AUTHOR "iplaycs"





public plugin_init()

{

	register_plugin(PLUGIN, VERSION, AUTHOR)

	RegisterHam(Ham_Spawn, "player", "runda", 1);    	

}

public runda(id)

{

	if ( !is_user_alive(id) ) return HAM_IGNORED;

	

	if(cs_get_user_team(id) == CS_TEAM_CT)

	{        	

		if(is_user_connected(id))            	

			cs_set_user_model(id, "klawiszfpg")

	}

}

public plugin_precache()

{	

	precache_model("models/player/klawiszfpg/klawiszfpg.mdl")	

}

Yyy a nie powinno być if(cs_get_user_team(id) == 2)?

kisiel96, przeczytaj to: Troche o enum

A czy funkcja runda nie powinna coś zwracać? Dodaj na sam jej koniec:

return HAM_SUPERCEDE;

Nomaf, niekoniecznie, po prostu będzie warning bez tego