5 replies to this topic

[palarie]

Witam.

Chciałbym się dowiedzieć czy da radę zapisać poniższy kod w prostszy sposób (z cod moda):

if (poziom_gracza[attacker] <=50)
	{
	new Float: dmg = ( 25.0+float(inteligencja_gracza[attacker]))-redukcja_obrazen_gracza_rakieta[pid];
	if( dmg <= 1.0 )
	dmg = 2.0;
	ExecuteHam(Ham_TakeDamage, pid, ent, attacker, dmg , 1);
        }
else if (poziom_gracza[attacker] <=200)
	{
	new Float: dmg = ( 30.0+float(inteligencja_gracza[attacker])/2)-(redukcja_obrazen_gracza_rakieta[pid])/2;
	if( dmg <= 1.0 )
	dmg = 2.0;
				
	ExecuteHam(Ham_TakeDamage, pid, ent, attacker, dmg , 1);
	}
	else if (poziom_gracza[attacker] <=500)
	{
	new Float: dmg = ( 45.0+float(inteligencja_gracza[attacker])/3)-(redukcja_obrazen_gracza_rakieta[pid])/3;
	if( dmg <= 1.0 )
	dmg = 2.0;
	ExecuteHam(Ham_TakeDamage, pid, ent, attacker, dmg , 1);
	}

Chodzi głównie o to, aby za każdym razem nie powtarzać tego:

if( dmg <= 1.0 )
    dmg = 2.0;

[Vasto_Lorde]

if(dmg <= 1.0)
     dmg=2.0;

To możesz zamienić na to:

dmg=dmg<=1.0 ? 2.0 : dmg

Efekt jest ten same ale nie wiem czy jest on szybszy.

 

Albo (teraz mnie oświeciło) możesz zrobić tak:

if(poziom_gracza[attacker] <= 50)
     new Float: dmg=(25.0+float(inteligencja_gracza[attacker]))-redukcja_obrazen_gracza_rakieta[pid];
else if(poziom_gracza[attacker]<=200)
     new Float: dmg=(30.0+float(inteligencja_gracza[attacker])/2)-(redukcja_obrazen_gracza_rakieta[pid])/2;
else if(poziom_gracza[attacker]<=500)
     new Float: dmg=(45.0+float(inteligencja_gracza[attacker])/3)-(redukcja_obrazen_gracza_rakieta[pid])/3;
 
if(dmg<=1.0)
     dmg=2.0;
ExecuteHam(Ham_TakeDamage, pid, ent, attacker, dmg , 1);

Ewentualnie zamienić ostatniego ifa na to co podałem wyżej.

[Seba]

	new Float:dmgbase, div
	
	if(poziom_gracza[attacker] <= 50)
	{
		dmgbase = 25.0
		div = 1
	}
	else if(poziom_gracza[attacker] <= 200)
	{
		dmgbase = 30.0
		div = 2
	}
	else if(poziom_gracza[attacker] <= 500)
	{
		dmgbase = 45.0
		div = 3
	}	
	
	new Float:dmg = ( dmgbase+float(inteligencja_gracza[attacker])/div)-redukcja_obrazen_gracza_rakieta[pid]/div;	
	ExecuteHam(Ham_TakeDamage, pid, ent, attacker, (dmg <= 1.0) ? 2.0 : dmg, 1);

Edited by Seba, 30.10.2013 23:23.

[Misiaczek ;c]

	new Float:dmgbase, div
	switch (poziom_gracza[attacker]) {
		case 1 .. 50: {
			dmgbase = 25.0
			div = 1
		}
		case 51 .. 200 {
			dmgbase = 30.0
			div = 2
		}
		case 201 .. 500: {
			dmgbase = 45.0
			div = 3
		}
	}
	new Float:dmg = ( dmgbase+float(inteligencja_gracza[attacker])/div)-redukcja_obrazen_gracza_rakieta[pid]/div;	
	ExecuteHam(Ham_TakeDamage, pid, ent, attacker, (dmg <= 1.0) ? 2.0 : dmg, 1);

[Seba]

O ile dobrze pamietam to switch w pawnie powoduje wiecej problemow niz korzysci. Krzykaczom z gory od razu proponuje skompilowac np. to

new a 
switch (a) {
case 1 .. 10000: {
 
}
case 10001 .. 100000: {
 
}
}

;]

Edited by Seba, 31.10.2013 14:36.

[palarie]

Dzięki za pomoc, rozwiązanie Seby najbardziej mi przypadło.