Quote:
Originally Posted by Emera
Perhaps a little support? Not all of us take your crap kindly. I think its really good astram. +rep
|
Jer is one of the people on this forum who is constantly giving scripting advice. His "crap" includes a fairly complete introductory guide, numerous specific guides, and tons of helpful posts.
The Code Gallery
should be reserved for high-quality scripts. There is a forum where he can post scripts that are incomplete to get feedback on.
My reply to another of Astram's code gallery threads:
Quote:
Originally Posted by cbk1994
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. |
Clearly all that support worked pretty well since he ignored almost every point I made.
Once again, Astram, thanks for trying to help. I'm glad that new scripters have an interest in improving the learning resources available for GScript. This script shows that you are making good progress toward learning GScript.
However, posting scripts like this one in the Code Gallery does more damage than it does good. I've recommended this before and I'll recommend it again. Post your scripts in the scripting forum first so an experienced scripter can help you polish it, then post the final version in the Code Gallery.