index out of bounds (Nie potrafię naprawić)

L 07/05/2012 - 21:34:21: [AMXX] Run time error 4: index out of bounds 
L 07/05/2012 - 21:34:21: [AMXX]    [0] Naruto.sma::Zakoncz_Tworzenie (line 1011)
L 07/05/2012 - 21:34:21: [AMXX]    [1] Naruto.sma::Stworz_Postac_Handler (line 9

Mam takie błędy jeżeli dobrze pamiętam przy takim błędzie powinno się dodać czy gracz jest na serwerze i czy nie jest botem niestety nie zadziałało.

KOD

public Stworz_Postac_Handler(id, menu, item)
{
if(!is_user_alive(id)||is_user_bot(id)) 
return PLUGIN_CONTINUE;

if(item == MENU_EXIT)
{
menu_destroy(menu);
return PLUGIN_CONTINUE;
}
switch(item)
{
case 0:
{
console_cmd(id, "messagemode Imie");

}
case 1:
{
Wybierz_Wioske(id)
}
case 2:
{
Zakoncz_Tworzenie(id)
}
}
return PLUGIN_CONTINUE;
}

public Zakoncz_Tworzenie(id)
{
if(!is_user_alive(id)||is_user_bot(id)) 
return PLUGIN_CONTINUE;

for(new i=0;i<4;i++) {if(strlen(postac[id][i])==0) postacie++;}
new idpostaci = 1+postacie;
formatex(postac[id][idpostaci],31,"%s",imiebo[id]);
wioska[id][idpostaci] = wioskabo[id];
ColorChat(id,GREEN,"[%s]^x01 Stworzyles bohatera^x03 %s^x01 pochodzacego z^x03 %s!",prefix,imiebo[id],wioskabo[id]);
menu_uzytkownika(id)

return PLUGIN_CONTINUE;

}

Linijka 1011

formatex(postac[id][idpostaci],31,"%s",imiebo[id])

Nie mogę się z tym uproać proszę o pomoc.

pokaż deklaracje zmiennych
postac
postacie
imiebo

i wytłumacz za co one odpowiadają ( domyślam się ale ty wiesz lepiej co miałeś na myśli deklarując je )

new imiebo[33], wioskabo[33];
new postac[33][4][32];
new postacie;

imiebo ma przetrzymać Imie.
postac jest to imię już stworzonej postaci posiada nadane wolne id
postacie służy do zliczania posiadanych postaci (potrzebne do nadania id nowo powstałej postaci)

Kod zagmatwany dało by się to zrobić lepiej

w ZakonczTworzenie nie widzę zerowania
postacie

new idpostaci = 1+postacie;
numerowanie w tablicy zaczyna się od 0 więc po co dodawanie 1 ?

for(new i=0;i<4;i++) {if(strlen(postac[id][i])==0) postacie++;}
new idpostaci = 1+postacie;

w najgorszym przypadku idpostaci przyjmie wartość 5 więc będzie poza zakresem

formatex(postac[id][idpostaci],31,"%s",imiebo[id]);

chodzi ci raczej o

formatex(postac[id][idpostaci],31,"%s",imiebo);

( możesz to zrobić też poprzez copy )

ColorChat(id,GREEN,"[%s]^x01 Stworzyles bohatera^x03 %s^x01 pochodzacego z^x03 %s!",prefix,imiebo[id],wioskabo[id]);

tak samo tutaj imiebo i woskabo też jako całą tablice a nie pojedynczą komórke

Teraz mój kod wygląda następująco

public Zakoncz_Tworzenie(id)
{
if(!is_user_alive(id)||is_user_bot(id))
return PLUGIN_CONTINUE;

for(new i=0;i<4;i++) {if(strlen(postac[id][i])==0) postacie++;}
new idpostaci = postacie;
formatex(postac[id][idpostaci],31,"%s",imiebo);
wioska[id][idpostaci] = wioskabo[id];
ColorChat(id,GREEN,"[%s]^x01 Stworzyles bohatera^x03 %s^x01 pochodzacego z^x03 %s!",prefix,imiebo,wioskabo);
menu_uzytkownika(id)
postacie = 0;
return PLUGIN_HANDLED;

}

Błąd występuje taki

L 07/05/2012 - 22:57:25: [AMXX] Run time error 4: index out of bounds 
L 07/05/2012 - 22:57:25: [AMXX]    [0] Naruto.sma::Zakoncz_Tworzenie (line 1023)
L 07/05/2012 - 22:57:25: [AMXX]    [1] Naruto.sma::Stworz_Postac_Handler (line 9

Warunki czy gracz żyje i czy nie jest botem daj w tworzeniu menu. Także po wyborze najlepiej niszczyć menu.

public Stworz_Postac_Handler(id, menu, item)
{
	if(item == MENU_EXIT)
	{
		menu_destroy(menu);
		return PLUGIN_HANDLED;
	}

	switch(item)
	{
		case 0:
		{
			console_cmd(id, "messagemode Imie");

		}
		case 1:
		{
			Wybierz_Wioske(id)
		}
		case 2:
		{
			Zakoncz_Tworzenie(id)
		}
	}
	menu_destroy(menu);
	return PLUGIN_HANDLED;
}

Nie działa.

Do Zamknięcia Zapałka pomógł mi rozwiązać problem.