really big if

[GodForsakenSoul]

part of my code is a really REALLY big if statement. it has like 16 'and's.

is there a way to short it from like a 3 meter long line of code to something more reasonable?

and yes, i DO need the 16 'and's. i'm comparing something to something else using PixelGetColor().

[SmOke_N]

part of my code is a really REALLY big if statement. it has like 16 'and's.

is there a way to short it from like a 3 meter long line of code to something more reasonable?

and yes, i DO need the 16 'and's. i'm comparing something to something else using PixelGetColor().

If they're mandatory you could do two things.

1. Put the items in an array and do a For/Next loop to check them.

2. Using an underscore is a line continuation.

Without code, there's no real way to show you... but the line continuation could look something like:

If $condition1 = $condition2 AND _
    $conditon3 = $condition4 AND _
    $condition5 = $condition5 AND _
    $ETC = $ETC Then

[martin]

part of my code is a really REALLY big if statement. it has like 16 'and's.

is there a way to short it from like a 3 meter long line of code to something more reasonable?

and yes, i DO need the 16 'and's. i'm comparing something to something else using PixelGetColor().

Another possibility

$result = True
For $n = 1 To 16
    Switch $n
        Case 1
            $result = Codition1
        Case 2
            $result = Codition2
        ;.
        ;.
        ;.
    EndSwitch
    If Not $result Then ExitLoop
    
Next

If $result Then
;code
EndIf

Or if you could put the conditions into an array maybe

If MultiIf() then
;code

endif


Func MultiIf()
For $n = 0 To 15
    If Not $Condition[$n] Then return False 
Next
 Return $True

endfunc

[GodForsakenSoul]

no, no, my code is something like this (though, the example is shorter...)

dim const $target1=123456789
dim const $target2=987654321

if pixelgetcolor(x,y)=$target1 and pixelgetcolor(x,y)=$target1 and pixelgetcolor(x,y)=$target2

except i'm testing ALOT of points to make sure the thing i'm looking for is indeed there.

i want to shorter the large if to something more reasonable.

[Aassdd]

no, no, my code is something like this (though, the example is shorter...)

dim const $target1=123456789
dim const $target2=987654321

if pixelgetcolor(x,y)=$target1 and pixelgetcolor(x,y)=$target1 and pixelgetcolor(x,y)=$target2

except i'm testing ALOT of points to make sure the thing i'm looking for is indeed there.

i want to shorter the large if to something more reasonable.

He just told you.

If they're mandatory you could do two things.

1. Put the items in an array and do a For/Next loop to check them.

2. Using an underscore is a line continuation.

Without code, there's no real way to show you... but the line continuation could look something like:

If $condition1 = $condition2 AND _
    $conditon3 = $condition4 AND _
    $condition5 = $condition5 AND _
    $ETC = $ETC Then

If x's and y's are random positions then you can shorten it.

[Jos]

i want to shorter the large if to something more reasonable.

whats wrong with the multiple line approach as suggested or just use multiple if's

[GodForsakenSoul]

@aassdd: Wtf i didn't see that post earlier ;;

thanks for pointing that out.

[junkew]

no, no, my code is something like this (though, the example is shorter...)

dim const $target1=123456789
dim const $target2=987654321

if pixelgetcolor(x,y)=$target1 and pixelgetcolor(x,y)=$target1 and pixelgetcolor(x,y)=$target2

except i'm testing ALOT of points to make sure the thing i'm looking for is indeed there.

i want to shorter the large if to something more reasonable.

As it seems like you are looking for a certain image on screen check if following threads are helpfull

http://www.autoitscript.com/forum/index.php?showtopic=66545

http://www.autoitscript.com/forum/index.php?showtopic=65748