Post reply

Warning: this topic has not been posted in for at least 120 days.
Unless you're sure you want to reply, please consider starting a new topic.
Name:
Email:
Subject:
Message icon:

shortcuts: hit alt+s to submit/post or alt+p to preview


Topic Summary

Posted by: riffruff
« on: December 10, 2008, 01:32:59 PM »

Most other scripts don't create their own anti-flood.  Why do you think yours needs to?
Posted by: ArticWolve
« on: December 10, 2008, 01:31:35 PM »

you're talking about the Beta.... 2.6R3's Queue SUCKS.
Posted by: riffruff
« on: December 10, 2008, 01:23:15 PM »

Quote from: Shadow
This would only return the ping of the USER who performed it. And  besides, yours only matched the left four, which would conflict when  someone says "Pingme" (Navy), whereas mine matches the left five on  "Ping " (Red). Yours also does not have a bypass to prevent  spamming/flooding out, whereas mine does (Green). Your is for the beta  and has no way to separate the "Ping" from the "Username" of the person  a USER wanted to ping (Indigo), whereas mine keeps the message INTACT  and then uses the "/say /" command to process it (At least the one I  am/was using) (Orange).

And what do you know, the ONLY thing we have in common with our scripts is this:
The Sub...End Sub Declaration and ONE End If.

PS: Vector told me it has to be 2.6 R3  compatible to wind up on his server of plugins.
Sigh -- I don't think you're understanding...

First, as Vector said, yours won't work.  Second, it's not hard to add the ping for a user -- it adds what, 3 more lines?  Third, Stealthbot has a built in queue -- sorry, I'm not going to create another one.  Forth, for ping, if I wanted backwards compatibility, I wouldn't pass it to the CP, I would just do addq ping.
Posted by: Vector
« on: December 10, 2008, 12:12:19 PM »

This won't work:

Code: [Select]
              AddQ "//" & Message
AddQ() sends text to battle.net WITHOUT going through the bot's internal command processor. It doesn't even work via scripting. Use Command() instead.
Posted by: ArticWolve
« on: December 10, 2008, 07:40:24 AM »

Then allow me to explain it so you can understand:
 
Code: [Select]
Sub NTPR_Event_Usertalk(Username, Flags, Message, Ping)  The Start of the Sub
 
 
Code: [Select]
  If LCase(Message) = LCase("Pingme") Then
     NTPRVar01 = Ping
     NTPRVar02 = Username
       If GetSetting("NTPR", "Pingme") Then
         SetSetting "NTPR", "Pingme", False, "", True
         AddQ "/w " & NTPRVar02 & " Your ping is currently: " & NTPRVar01 & " ms."
         TimerEnabled "NTPR", "PingmeReset", True
       Else
         Exit Sub
       End If
 The PINGME No Trigger Command
  I am using Variables instead to name PING and USERNAME.
  I am using a Setting for a timer so the bot doesn't flood out.
 
Code: [Select]
  ElseIf Left(LCase(Message), 5) = "ping " Then
       If GetSetting("NTPR", "Ping") Then
          SetSetting "NTPR", "Ping", False, "", True
            If InStr(Lcase(GetBotVersion), "beta") Then
              AddQ "//" & Message
              TimerEnabled "NTPR", "PingReset", True
            Else
              AddQ "/say /" & Message
              TimerEnabled "NTPR", "PingReset", True
            End If
        End If
     End If
 The PING No Trigger Command.
  I look for "Ping ", and if that message IS "Ping " then it executes a Beta Version Check.
  If Beta, it uses "//", but if not. it uses "/say /" to send the command to battle net.
  Then enables a Timer to prevent the bot from flooding out as fast.
 
 
Code: [Select]
End Sub  The Sub is ended.
 
  Now can you follow my code?
 
  Riffruff's code:
 
 
Quote
Sub Event_UserTalk(Username, Flags, Message, Ping)
    if left(lcase(message), 4) = "ping" then
      Command BotVars.Username, "//ping " & Username, True
    end if
  end sub
 My Code (From 1.22)
   
Quote
If Left(LCase(Message), 5) = "ping " Then
       If GetSetting("NTPR", "Ping") Then
         SetSetting "NTPR", "Ping", False, "", True

               AddQ "/say /" & Message
             TimerEnabled "NTPR", "PingReset", True
           End If

         End If
 
  This would only return the ping of the USER who performed it. And  besides, yours only matched the left four, which would conflict when  someone says "Pingme" (Navy), whereas mine matches the left five on  "Ping " (Red). Yours also does not have a bypass to prevent  spamming/flooding out, whereas mine does (Green). Your is for the beta  and has no way to separate the "Ping" from the "Username" of the person  a USER wanted to ping (Indigo), whereas mine keeps the message INTACT  and then uses the "/say /" command to process it (At least the one I  am/was using) (Orange).

And what do you know, the ONLY thing we have in common with our scripts is this:
The Sub...End Sub Declaration and ONE End If.

PS: Vector told me it has to be 2.6 R3  compatible to wind up on his server of plugins.
Posted by: Hero
« on: December 10, 2008, 06:00:14 AM »

Your stuff looks a lot more complicated then what riffruff posted...
Posted by: ArticWolve
« on: December 09, 2008, 07:15:54 PM »

But I did that before you replied.
Posted by: riffruff
« on: December 09, 2008, 06:52:13 PM »

How would it be public?

With what hero posted it would, but with what I posted, it wouldn't.
Posted by: ArticWolve
« on: December 09, 2008, 06:47:51 PM »

Because then "Every Command" would be public. I don't want that.
Posted by: riffruff
« on: December 09, 2008, 06:35:52 PM »

Quote from: Hero
My question is why are you making them, when they can be accomplished  by 3 lines. What are you wanting  different?
I don't understand why either...You kinda said the same thing as me in your first post in this topic

Quote from: Shadow
Because then all of them would be passed to the Command processor, which I don't want.
Why?  Your just making your own command processor...Why waste time making your own, when you can use a built in one that is more efficient?
Posted by: ArticWolve
« on: December 09, 2008, 01:52:25 PM »

Because then all of them would be passed to the Command processor, which I don't want.
Posted by: Hero
« on: December 09, 2008, 01:28:58 PM »

My question is why are you making them, when they can be accomplished  by 3 lines. What are you wanting  different?
Posted by: ArticWolve
« on: December 09, 2008, 01:22:53 PM »

Basically, each one is a separate plugin for Stealth Bot, yet at the same time, can be implemented into one another. Even though these are standalone on the site, I have them all in one plugin for "Organizational" reasons.
Posted by: Hero
« on: December 09, 2008, 01:18:29 PM »

I still don't  get  the point .
Posted by: ArticWolve
« on: December 09, 2008, 06:48:44 AM »

Because each one is "standalone" and shouldn't all be included in the main script until I am finished.