[dataviruset]

Hello, I am a bit tired and I don't care whatever you say, but, yeah.
No matter how important it is, I want to optimize this stupid bit of code. I just wonder, haha.

Which alternative would be the best?

PHP Code:

// alternative 1
SetEntityGravity(LRprogressplayer1, GetConVarFloat(sm_hosties_lr_db_gravity));
SetEntityGravity(LRprogressplayer2, GetConVarFloat(sm_hosties_lr_db_gravity));

// alternative 2
new Float:gravity = GetConVarFloat(sm_hosties_lr_db_gravity);
SetEntityGravity(LRprogressplayer1, gravity);
SetEntityGravity(LRprogressplayer2, gravity); 


[retsam]

lol.. Id doubt that makes any difference...

Obviously the less getconvarint's,floats,etc you do, the better....so the second example would be better....

Personally, if it were me, I might not even do either of those ways.

If you have to get that convar many times, Id make a global and set the value of the cvar to the global OnConfigsExecuted. Then you never have to getconvarfloat on a bunch of stuff(assuming you had a bunch of stuff) when running the code..... I see many people making those mistakes a lot and have a billion getconvar**** for stuff thats used repeatedly.

If you only use it once in that instance, the second example is fine. Its common sense coding really, if you getconvarfloat once, why get it again, or a third time, or forth time, etc...

[BAILOPAN]

Write code that is simpler and easier to maintain. I think the bottom one fits the bill.