Eroare Cod

[LondoN eXtream]

| Afiseaza codul

GetTeroPlayers(const Type)
{
	new Count = 0, i;
	static g_max_players;

	if(!g_max_players)
		g_max_players = get_maxplayers();

	switch(Type)
	{
		case 1: // All Players From Team
		{
			for(i = 1; i <= g_max_players(); i++)
			{
				if(cs_get_user_team(i) == CS_TEAM_T)
					Count++;
			}
		}

		case 2: // Alive Only
		{
			for(i = 1; i <= g_max_players(); i++)
			{
				if(cs_get_user_team(i) == CS_TEAM_T && is_user_alive(i))
					Count++;
			}
		}
	}

	return Count;
}

In sursa il folosesc altfel:

new PlayersNum = GetTeroPlayers(1) sau GetTeroPlayers(2)

si nu functioneaza, oare de ce?

[H2K// oDoamne]

n-am testat csf

| Afiseaza codul

function ListTerrorists(const Type){

	new iPlayers[32], iTAlive, iAllT, MessageCell[32]; 

	switch(Type){

		case 1: 
		{

			get_players(iPlayers, iAllT, "e", "TERRORIST"); 

			client_print(0, print_chat, "[AMXX] Total terrorists in the team is %d", iAllT)

		}

		case 2: 
		{

			get_players(iPlayers, iTAlive, "ae", "TERRORIST"); 

			client_print(0, print_chat, "[AMXX] All alive terrorists in the team is %d", iTAlive)
		}


	}

	return PLUGIN_HANDLED;

}

[Nubo]

La case 1 vei primi erori. Verifica daca i e conectat pana a verifica echipa.
La case 2 pune is_alive primul.
In for g_max_players e scris ca functie, fiind variabila.

[Fuffy]

https://forums.alliedmods.net/showpost. ... ostcount=7

[HamletEagle]

Poti sa faci ceva de genul:

Code: Select all

new Players[32], NumCT, NumT
get_players(Players, NumT, "e", "TERRORIST")
get_players(Players, NumCT, "e", "CT")

Adaugi flagul "a" pe langa e daca vrei sa ii numeri doar pe cei care sunt in viata. Cand vrei sa procesezi toti playerii, nu mai face loop de la 1 la maxplayers, foloseste get_players. Problema e ca tu procesezi fiecare index care e posibil sa corespunda unui player, nu iei playeri valizi neaparat(ai avea nevoie de verificari suplimentare). get_players iti returneaza un array cu playeri valizi si poti filtra dupa cum iti doresti.

[H2K// oDoamne]

Poti sa faci ceva de genul:

Code: Select all

new Players[32], NumCT, NumT
get_players(Players, NumT, "e", "TERRORIST")
get_players(Players, NumCT, "e", "CT")

Adaugi flagul "a" pe langa e daca vrei sa ii numeri doar pe cei care sunt in viata. Cand vrei sa procesezi toti playerii, nu mai face loop de la 1 la maxplayers, foloseste get_players. Problema e ca tu procesezi fiecare index care e posibil sa corespunda unui player, nu iei playeri valizi neaparat(ai avea nevoie de verificari suplimentare). get_players iti returneaza un array cu playeri valizi si poti filtra dupa cum iti doresti.

Inca n-au invatat sa optimizeze codul, ci sa adauge cat mai mult.Cantitate si nu calitate, ceea ce este total gresit.

[LondoN eXtream]

Poti sa faci ceva de genul:

Code: Select all

new Players[32], NumCT, NumT
get_players(Players, NumT, "e", "TERRORIST")
get_players(Players, NumCT, "e", "CT")

Adaugi flagul "a" pe langa e daca vrei sa ii numeri doar pe cei care sunt in viata. Cand vrei sa procesezi toti playerii, nu mai face loop de la 1 la maxplayers, foloseste get_players. Problema e ca tu procesezi fiecare index care e posibil sa corespunda unui player, nu iei playeri valizi neaparat(ai avea nevoie de verificari suplimentare). get_players iti returneaza un array cu playeri valizi si poti filtra dupa cum iti doresti.

Inca n-au invatat sa optimizeze codul, ci sa adauge cat mai mult.Cantitate si nu calitate, ceea ce este total gresit.

pe tine te-a bagat cineva in seama?
nu mai spun ca ai lasat un cod de nota 999999, ca mna, esti cel mai bun scripter. :troll

[HamletEagle]

Poti sa faci ceva de genul:

Code: Select all

new Players[32], NumCT, NumT
get_players(Players, NumT, "e", "TERRORIST")
get_players(Players, NumCT, "e", "CT")

Adaugi flagul "a" pe langa e daca vrei sa ii numeri doar pe cei care sunt in viata. Cand vrei sa procesezi toti playerii, nu mai face loop de la 1 la maxplayers, foloseste get_players. Problema e ca tu procesezi fiecare index care e posibil sa corespunda unui player, nu iei playeri valizi neaparat(ai avea nevoie de verificari suplimentare). get_players iti returneaza un array cu playeri valizi si poti filtra dupa cum iti doresti.

Inca n-au invatat sa optimizeze codul, ci sa adauge cat mai mult.Cantitate si nu calitate, ceea ce este total gresit.

pe tine te-a bagat cineva in seama?
nu mai spun ca ai lasat un cod de nota 999999, ca mna, esti cel mai bun scripter. :troll

A lasat un cod corect, care-i problema? Si nu vad sa fi zis el ca e cel mai bun scripter sau ceva. Mda, iar atitudinea asta.. vine un om, te ajuta si tu in loc sa-i multumesti, te iei de el.

[Lux0R^]

Inca n-au invatat sa optimizeze codul, ci sa adauge cat mai mult.Cantitate si nu calitate, ceea ce este total gresit.

pe tine te-a bagat cineva in seama?
nu mai spun ca ai lasat un cod de nota 999999, ca mna, esti cel mai bun scripter. :troll

A lasat un cod corect, care-i problema? Si nu vad sa fi zis el ca e cel mai bun scripter sau ceva. Mda, iar atitudinea asta.. vine un om, te ajuta si tu in loc sa-i multumesti, te iei de el.

Legat de cod corect, am luat asta din primul cod desi n-am rabdare sa vad la ce cod te-ai referit :
for(i = 1; i <= g_max_players(); i++)
Hint : i <=

[FaTzZu]

| Afiseaza codul

GetTeroPlayers(const Type)
{
	new Count = 0, i;
	static g_max_players;

	if(!g_max_players)
		g_max_players = get_maxplayers();

	switch(Type)
	{
		case 1: // All Players From Team
		{
			for(i = 1; i <= g_max_players(); i++)
			{
				if(cs_get_user_team(i) == CS_TEAM_T)
					Count++;
			}
		}

		case 2: // Alive Only
		{
			for(i = 1; i <= g_max_players(); i++)
			{
				if(cs_get_user_team(i) == CS_TEAM_T && is_user_alive(i))
					Count++;
			}
		}
	}

	return Count;
}

In sursa il folosesc altfel:

new PlayersNum = GetTeroPlayers(1) sau GetTeroPlayers(2)

si nu functioneaza, oare de ce?

Alt hint : dacă folosesti "i" doar in loop initializeaza-l acolo ( for(new i ...)

[LondoN eXtream]

new | Afiseaza codul

GetTeroPlayers(const Type) {
	new Count = 0;

	static g_max_players;
	
	if(!g_max_players)
		g_max_players = get_maxplayers();

	switch(Type) {
		case 1: {
			for(new i = 1; i <= g_max_players; i++)	{
				if(is_user_connected(i) && cs_get_user_team(i) == CS_TEAM_T)
					Count++;		
			}
		}

		case 2: {
			for(new i = 1; i <= g_max_players; i++) {
				if(is_user_connected(i) && is_user_alive(i) && cs_get_user_team(i) == CS_TEAM_T)
					Count++;
			}
		}
	}


	return Count;
}

[HamletEagle]

@London, tu nu vrei sa inveti nimic? Ti-am explicat ca nu e ok sa faci loop de la 1 la maxplayers, foloseste ce ti-am aratat mai sus. Tot ce faci tu acolo e facut automat de get_players.

@Lux0R^, vorbeam de codul lui H2K// oDoamne.