Author Topic: Combine ulx slap and ulx whip into one command  (Read 3307 times)

0 Members and 1 Guest are viewing this topic.

Offline atomicspark

  • Full Member
  • ***
  • Posts: 196
  • Karma: 12
Combine ulx slap and ulx whip into one command
« on: July 21, 2007, 12:57:28 PM »
It does seem to be a bit redundant to have two commands that basicaly do the same thing. Suggestion? Make it into one command!

Example, I want to slap megiddo once.

Code: [Select]
ulx slap megiddo 100
Now, I want to slap megiddo 10 times.

Code: [Select]
ulx slap megiddo 100 10 1
Note that the command has been switched around for compatibility. It's now slap megiddo for 100 points, 10 times, every 1 second. The old way was slap megiddo 10 times, every 1 sec, for 100 damage. This new way enables the last two values to be purely optional with default of 1 (just like damages default value is 0, if left off).

Why bother you ask? Ulysses motto is "Striving for Simplicity".

Offline JamminR

  • Ulysses Team Member
  • Hero Member
  • *****
  • Posts: 8096
  • Karma: 390
  • Sertafide Ulysses Jenius
    • Team Ulysses [ULib/ULX, other fine releases]
Re: Combine ulx slap and ulx whip into one command
« Reply #1 on: July 21, 2007, 05:04:52 PM »
I understand and somewhat agree with your point of view atomicspark.
Majority of the slap access userbase on the 3 servers I admin on have no idea that slap even has inputs, let alone how to use them.
They get all excited when I use a whip.
Though the recode wouldn't be too difficult, I like the idea of keeping them separate more. I've seen many admins repeatedly screw up the whip command and end up whipping themself (or a poor abused player) for a long time. At least with a slap, they know its once.
"Though a program be but three lines long, someday it will have to be maintained." -- The Tao of Programming