egghelp/eggheads community

Several conditions ''&&" and '||' in one line

2015-05-16T01:44:59-04:00

Thanks to your advices SpiKe^^ I fixed!

My eternal confusion between string and list.

All variables were string!
I treated all variables as a text string, properly splitting it into a list, and deleted extra operators

Code:

!= -1 } {

I'll put the final solution at the right post.

Statistics: Posted by juanamores — Sat May 16, 2015 1:44 am

Several conditions ''&&" and '||' in one line

2015-05-16T00:48:13-04:00

I see multiple possible reasons why this expression does not resolve as you might be wishing...
None of the reasons rest with the way you are grouping things together.

What exactly is in the variable $text ?
By its name, I have to expect it holds a text string, and not a list.
Sometimes you treat it as text, & others as a list with sublists?

1) In these parts of the expression, you treat $text as a nested list, sort of, and badly (these will probably never resolve to true)...

Code:

([lsearch -exact -ascii -nocase [lindex $text 0 end] [lindex $who1 0 end]] != -1)

2) Right after that, you treat $text as a text string, properly splitting it into a list...

Code:

([llength [split $text]] == 2)

3) And you finish off treating $text as a text string again...

Code:

([string index $text end] == "$whof")

4) Then at the very end of your expression you just throw in an extra bit of code that has no home at all.
Even if the rest of that expression was correct, just throwing in extra operators here and there can't possibly help:)

Code:

 != -1 } {

In order for anyone to try to help with this mess, we would need much more information on what is included in the vars $text and $who1, 2 & 3.

Statistics: Posted by SpiKe^^ — Sat May 16, 2015 12:48 am

Several conditions ''&&" and '||' in one line

2015-05-15T22:59:31-04:00

I need each code segment separated by red brackets, is considered separately from other conditions.

if {(([lsearch -exact -ascii -nocase [lindex $text 0 end] [lindex $who1 0 end]] != -1) && ([llength [split $text]] == 2) \
&& ([string index $text end] == "$whof")) \
|| (([lsearch -exact -ascii -nocase [lindex $text 0 end] [lindex $who2 0 end]] != -1) && ([llength [split $text]] == 3) \
&& ([string index $text end] == "$whof")) \
|| (([lsearch -exact -ascii -nocase [lindex $text 0 end] [lindex $who3 0 end]] != -1) && ([llength [split $text]] == 4) && ([string index $text end] == "$whof")) != -1 } {
more stuff
}

I tried adding other brackets at the beginning of the code and at the end, but it is malfunctioning.

Statistics: Posted by juanamores — Fri May 15, 2015 10:59 pm