would like to have only specific users to use the script

[Cyborg5000]

Tried with

Global $user = 'user1' And 'user2' And 'user3' And 'user4'

Also

Global $user = 'user1' Or'user2' Or'user3' Or 'user4'

    Global $userid = GUICtrlRead($USERIDINPUT)
    If    Not $user ==  $userid Then

        MsgBox(0, "Temp", "You can run the script")

    Else
        MsgBox(0, "Temp", "You cannot run the script")
        Exit
    EndIf

[Cyborg5000]

Also tried

 

Global $user = 'user1' Or'user2' Or'user3' Or 'user4'

    Global $userid = GUICtrlRead($USERIDINPUT)
    If    $user =  $userid Then

OR

$userid = $user Then

        MsgBox(0, "Temp", "You can run the script")

    Else
        MsgBox(0, "Temp", "You cannot run the script")
        Exit
    EndIf

[JLogan3o13]

Moved to the correct forum, as the DEV forum very clearly states:

  Quote

Do not create AutoIt-related topics here, use AutoIt General Help and Support...

Expand  

 

[Jos]

  On 4/30/2018 at 4:45 PM, Cyborg5000 said:

Global $user = 'user1' And 'user2' And 'user3' And 'user4'

Also

Global $user = 'user1' Or'user2' Or'user3' Or 'user4'

Expand  

I am wondering why you think this should work at all?
You are creating an boolean result to which $User will be set.

One way is to do it like this:

Global $user = '|user1|user2|user3|user4|'
If StringInStr($user, "|" & $userid & "|") Then
    ;
EndIf

also for the records: these kind of tests will always fail:

  On 4/30/2018 at 4:45 PM, Cyborg5000 said:

If  Not $user ==  $userid Then

Expand  

as Not $user will be evaluated as first step!

Jos

[water]

You could have a look at Xprotec in the example scripts section to limit access to your script.
But be aware that you can not get 100% protection - search the forum for explanations why

[ViciousXUSMC]

If you are having them log into the computer @UserName is great as it authenticates against who they are logged in as, nobody can simply just type somebody elses name into a field.  You can then check against a AD security group with other AD integration UDF that we have.

At the most basic level, I would put my names in an .ini file or in the script as an array and check it in a loop with stringinstr() or stringregexp() depending how secure you want it and how easy to modify it needs to be.

Edited April 30, 2018 by ViciousXUSMC