[TrullSin]

What is the best way to use OR statments?
For Example, I want to check first something and if that thing it is ok it wont check for the second and if the first is not ok it will check for the second, something like that. I don't know if it helps but it is for CS:GO

[Bacardi]

If you like know something about coding, post in right section.

Scripting (17 Viewing) Scripting in SourcePawn

And https://wiki.alliedmods.net/Category...eMod_Scripting





There is... two kind, how you like to use.

PHP Code:

    bool first_something;
    bool second_something;

    second_something = true;


    if( first_something )
    {
        // Use this code block only
    }
    else if( second_something )
    {
        // Use this code block only
    }

..........

    if( first_something || second_something )
    {
        // if one of variable is true, use this code block
    } 


[TrullSin]

Yea my bad, I thought I was posting in the right place, for example I was trying to make something to that prime code (https://forums.alliedmods.net/showpo...29&postcount=3)

PHP Code:

if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820))
    {
        KickClient(client, "");
        return;
    }
    else if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029))
    {
        KickClient(client, "");
        return;
    } 


How could I make the bool and the other stuff with this variables inside if?

[Bacardi]

You not necessary need add variables... ? I'm not sure what you are planning. You could tell more specific.
...
In if() statement you do conditions, check something, is it TRUE or FALSE.
Using

• 7.2 Comparison Operators
• 7.3 Truth Operators

What your code does is compare result to one of enum group values https://github.com/KyleSanderson/Ste...amWorks.inc#L6
It looks like if( 1 == result )

PHP Code:

/* results from UserHasLicenseForApp */
enum EUserHasLicenseForAppResult
{
    k_EUserHasLicenseResultHasLicense = 0,                    // User has a license for specified app
    k_EUserHasLicenseResultDoesNotHaveLicense = 1,            // User does not have a license for the specified app
    k_EUserHasLicenseResultNoAuth = 2,                        // User has not been authenticated
}; 


But if you insist, to use bool variables.

PHP Code:



    bool first = (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820));
    bool second = (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029));

    if(first || second)
    {
        KickClient(client, "");
        return; 
    } 


[Powerlord]

Using bool variables is less efficient than just using the results raw because in SourcePawn, || and && are short-circuited operators.

What does that mean? It means that in certain conditions, the second argument is never evaluated.

• For || the second argument is never evaluated if the first argument is true
• For && the second argument is never evaluated if the first argument is false

So, why is it more efficient to use the results raw? Because of this:

PHP Code:

if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820) ||
    k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029))
{
   // SteamWorks_HasLicenseForApp(client, 54029) is never called if they have a license for app 624820

} 


Edit: If you really want to use a variable, you can still do this:

PHP Code:

bool notLicensed = k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820) ||
    k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029); 


[Bacardi]

What those apps are anyway ? Csgo is one of those, is second TF2?

[brunoronning]

Originally Posted by Bacardi What those apps are anyway ? Csgo is one of those, is second TF2?

https://steamdb.info/sub/54029/

[TrullSin]

This one is not working

PHP Code:

if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820) ||
    k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029))
    {
        KickClient(client, "");
        return;
    } 


And when compiling with this, I get a warning 217: loose indentation in the

PHP Code:

return 


after KickClient.

I have the CS:GO for like a few years, and when I enter, I get kicked.

[Ilusion9]

You should give them a reason for kicking.

PHP Code:


if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820))
{
    KickClient(client, "reason 1");
    return;
}

if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029))
{
    KickClient(client, "reason 2");
    return;
} 


This is how I would write this code.
Also use TAB in code for white spaces to avoid that error.

[TrullSin]

Originally Posted by Ilusion9 You should give them a reason for kicking. PHP Code: if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 624820)) {     KickClient(client, "reason 1");     return; } if (k_EUserHasLicenseResultDoesNotHaveLicense == SteamWorks_HasLicenseForApp(client, 54029)) {     KickClient(client, "reason 2");     return; }  This is how I would write this code. Also use TAB in code for white spaces to avoid that error.

Not working, I get kicked and I have CS:GO for the last 2 years or more, and I got upgraded after the update.. I think that way is checking for the first and then it will check for the second and I dont have the appid 54029 because I didn't buy prime.