View Single Post
  #4  
Old 05-17-2011, 02:49 AM
Astram Astram is offline
Era iPhone PR
Astram's Avatar
Join Date: Aug 2010
Posts: 324
Astram can only hope to improve
Send a message via AIM to Astram
Quote:
Originally Posted by cbk1994 View Post
Please don't release scripts like this into the code gallery. I really do appreciate your effort and that you're trying to help others, but releasing broken model scripts isn't the way to do it.

A couple things to work on:
  • Your styling is very inconsistent. This is the most crucial issue with your script.
  • The minimum clientside timeout is 0.05 seconds, not 0.0005 seconds.
  • You don't need to check for q and Q. String comparison is not case-sensitive.
  • Try to use meaningful object names rather than _Button1.
  • I would recommend using clientWidth and clientHeight instead of clientExtent.
  • Generally use foreach loops when looping through arrays like you did for displaying weapons.

Again, thanks for trying to help . Let me know if anything I said doesn't make sense.
I used the RC Graal Gui Editor. Because I needed to set it up quick. xD I was in a hurry. I put the script through the Java beatuifier soooo... Also, the timer works perfectly. Thanks, I just got mixed up with the clientWidth. So it would be like.
PHP Code:
clientWidth clientHeight 10
?_?
__________________
-Toad
The worlds biggest Toad fan...
Era iPhone FTW!


Reply With Quote