How to enclose "or" conditions into one condition?

[daigo]

This is the line I'm having trouble with:

Code: Select all

if {[string match -nocase "abc" $txt] || [string match -nocase "def" $txt] || [string match -nocase "ghi" $txt] || [string match -nocase "jkl" $txt] && $nick eq "daigo"}

Whenever someone says "abc" "def" or "ghi" the script responds to them, even though I stated that I want only the nickname "daigo" to be responded to. I assume this is because I need to enclose all 4 of the || conditions together, but I tried brackets [], {} and parentheses () but none of them work.

[willyw]

...
and parentheses () but none of them work.

I think parenthesis work.

Show us the line of code, exactly as you used it, that utilizes parenthesis.

[Get_A_Fix]

The parenthesis are necessary when you're using "".

For example, if I needed two conditions in an IF statement, ending in "", I would then use ( ) to keep it closed.

Code: Select all

if {[lindex [split $arg] 0] ne ""} {
if {[string match -nocase "*matchthis*" $arg]} {

would then become

Code: Select all

if {([lindex [split $arg] 0] ne "") && ([string match -nocase "*matchthis*" $arg])} {

This would fix your issue with the last condition of your statement; $nick eq "daigo"

[SpiKe^^]

Try it like this...

Code: Select all

if {([string match -nocase "abc" $txt] || [string match -nocase "def" $txt] || [string match -nocase "ghi" $txt] || [string match -nocase "jkl" $txt]) && $nick eq "daigo"}

[daigo]

Oh, this works..I thought I tried this already, but I must've made a typo somewhere.

[willyw]

...
I thought I tried this already, but I must've made a typo somewhere.

That's why I asked for your exact code, as you used it.