egghelp/eggheads community

alice.tcl 1.4.0

2013-01-22T14:31:47-04:00

Ceasar:
Try the both codes in a logic table.. And read up on De Morgan's law.

Code:

(!expression_A || !expression_B) => !(expression_A && expression_B)

In extension:

Code:

!(!expression_A || !expression_B) => !(!(expression_A && expression_B)) => (expression_A && expression_B)

Code:

/---+---+------------+---------------------\| A | B | !A or !B   | !(A and B)          || 0 | 0 | 1 or 1 = 1 | !(0 and 0) = !0 = 1 || 0 | 1 | 1 or 0 = 1 | !(0 and 1) = !0 = 1 || 1 | 0 | 0 or 1 = 1 | !(1 and 0) = !0 = 1 || 1 | 1 | 0 or 0 = 0 | !(1 and 1) = !1 = 0 |\---+---+------------+---------------------/

Edit: Added logic/truth-table

Statistics: Posted by nml375 — Tue Jan 22, 2013 2:31 pm

alice.tcl 1.4.0

2013-01-22T11:54:04-04:00

I will do some tests on my own cos I still don't get what's wrong in there. Following my logic I'd would have said that:

Code:

      if {![isop $nick $chan] || ![isvoice $nick $chan]} return      # execute code

if any of the two statements (if the user isn't channel operator or he doesn't have voice) returns true it would return, else would execute code, and:

Code:

      if {[isop $nick $chan] || [isvoice $nick $chan]} {        # execute code      }      return 0

if any of the two statements (if the user is channel operator or he has voice) returns true it would execute code, else would return.

Edit: you are right. I don't understand why this double negation expressions with an or operator works like an and expression, not like an or expression as it should.

Statistics: Posted by caesar — Tue Jan 22, 2013 11:54 am

alice.tcl 1.4.0

2013-01-22T11:01:59-04:00

Lets just say I didn't and that's that.

EDIT: Curiosity got the better of me. Yes, I killed the bot entirely and reloaded it. Yes, I made sure 'alter-alice.tcl' was loaded in the .conf after alice.tcl. As you can see below, I was opped and the bot didn't respond. Gave myself a + as well, and it answered. Took the @ away, leaving the +, no response. Took both the @ and + off, no response. It would appear that SpiKe is correct. Sorry caesar! Psst, you did all the hard work and SpiKe only had to change a couple characters! thank you both!

[10:11am] <@Bryan> ANC hi there!
[10:11am] * Bryan sets mode: +v Bryan
[10:11am] <@Bryan> ANC hi there!
[10:12am] <@ANC> Hello .
[10:12am] * Bryan sets mode: -v Bryan
[10:12am] <@Bryan> ANC hi how are you?
[10:13am] * Bryan sets mode: +v-o Bryan Bryan
[10:13am] <+Bryan> ANC whats your name?
[10:14am] * ChanServ sets mode: +o Bryan
[10:14am] * Bryan sets mode: -v Bryan
[10:14am] * Bryan sets mode: -o Bryan
[10:15am] ANC hi there!

Statistics: Posted by bryanwny — Tue Jan 22, 2013 11:01 am

alice.tcl 1.4.0

2013-01-22T02:08:35-04:00

I have my doubts that bryanwny did the things correctly. Anyway.

Statistics: Posted by caesar — Tue Jan 22, 2013 2:08 am

alice.tcl 1.4.0

2013-01-22T02:11:03-04:00

double negative with an or operator works like an and,
whatever... mine worked, yours didn't.... nuff said:)

Statistics: Posted by SpiKe^^ — Tue Jan 22, 2013 2:04 am

alice.tcl 1.4.0

2013-01-22T02:07:29-04:00

Read again what I said above about expressions and look then on:

Code:

if {![isop $nick $chan] || ![isvoice $nick $chan]} return

I don't see any && in there, on ANY of the codes I've posted. I may be dumb but I'm not blind.

Here's something to see better:

% set one "0"; set two "0"
0
% if {$one || $two} { puts "valid" } else { puts "false" }
false
% if {!$one || !$two} { puts "valid" } else { puts "false" }
valid

Statistics: Posted by caesar — Tue Jan 22, 2013 2:01 am

alice.tcl 1.4.0

2013-01-22T01:52:37-04:00

Again... your code required the person to be Both opped And voiced to execute the command.
That is Not what the user was shooting for:)
I am done arguing though on this string.

Statistics: Posted by SpiKe^^ — Tue Jan 22, 2013 1:52 am

alice.tcl 1.4.0

2013-01-22T02:00:05-04:00

Really?

According to the expressions page:

&&
Logical AND. Produces a 1 result if both operands are non-zero, 0 otherwise. Valid for boolean and numeric (integers or floating-point) operands only.

||
Logical OR. Produces a 0 result if both operands are zero, 1 otherwise. Valid for boolean and numeric (integers or floating-point) operands only.

Again, your point?

Code:

if {[isop $nick $chan] || [isvoice $nick $chan]} { execute this code } else { return }

that translates to check if it's op OR if it's voice and if ANY of the statements returns true then executes this code. If none of the statements returns true then it will simply return.

that is the same as:

Code:

if {![isop $nick $chan] || ![isvoice $nick $chan]} { return } else { execute this code }

that translates into if the user isn't operator OR isn't voiced then return, else if ANY of the statements returns true then will execute this code.

The only difference is given by the position of return.

Statistics: Posted by caesar — Tue Jan 22, 2013 1:45 am

alice.tcl 1.4.0

2013-01-22T01:42:08-04:00

not really... you needed more like this for yours to work correctly:)

Code:

if {![isop $nick $chan] && ![isvoice $nick $chan]} return

The or (||) was what made yours not work. That was my point.

Statistics: Posted by SpiKe^^ — Tue Jan 22, 2013 1:42 am

alice.tcl 1.4.0

2013-01-22T01:25:11-04:00

yes, that's the opposite of:

Code:

if {[isop $nick $chan] || [isvoice $nick $chan]} {

but instead to continue with that alice:pubquery proc it returns if any of the checks (op or voice) fails.

Your point?

Statistics: Posted by caesar — Tue Jan 22, 2013 1:25 am

alice.tcl 1.4.0

2013-01-21T12:26:13-04:00

First chunk of code is wrote so it requires both ops and voice:)

if {![isop $nick $chan] || ![isvoice $nick $chan]} return

That will return out, if you either don't have ops |or| don't have voice, double negatives can be very confusing.

Statistics: Posted by SpiKe^^ — Mon Jan 21, 2013 12:26 pm

alice.tcl 1.4.0

2013-01-21T03:30:58-04:00

caesar, I couldn't remember if maybe I didn't kill/reload the bot and just did a .rehash when i tried your 2nd chunk of code, so I just tried it again with a proper shut down/reload -- it still didn't work for me. Why? Beats me! Like you said, at least it's working now though

Statistics: Posted by bryanwny — Mon Jan 21, 2013 3:30 am

alice.tcl 1.4.0

2013-01-21T02:00:54-04:00

I don't get it. The code I've wrote and Spike's are basically doing the same thing, just written in a different manner.

Anyway, it's good if it's finally working.

Statistics: Posted by caesar — Mon Jan 21, 2013 2:00 am

alice.tcl 1.4.0

2013-01-20T23:20:01-04:00

That seems to have done the trick, SpiKe! Thank you everyone for your time and help, much appreciated

Statistics: Posted by bryanwny — Sun Jan 20, 2013 11:20 pm

alice.tcl 1.4.0

2013-01-20T22:46:32-04:00

Try it this way....

Code:

if {[info commands alice:pubquery] eq ""} {    rename alice:pub_query alice:pubquery    proc alice:pub_query {nick uhost hand chan text} {       if {[isop $nick $chan] || [isvoice $nick $chan]} {        alice:pubquery $nick $uhost $hand $chan $text       }      return 0   } }

Statistics: Posted by SpiKe^^ — Sun Jan 20, 2013 10:46 pm