It looks really nice for a first script. I don't have an account to test this on or I would. I can try to offer you some help from just looking it over though...
Your buyyews procedure should be BuyYews (just for cosmetics), same with buynats.
Your BuyYews procedure does not have failsafes for the first bit of that. I would recommend that you add some. You have some later on in the procedure.
Quote:
Line 50: [Error] (16256:1): Semicolon (';') expected in script
Can be fixed by adding a semicolon on the line above after your string.
Quote:
Line 182: [Error] (16388:29): Type mismatch in script
AlchItem is a string and as thus does not need IntToStr
After those changes it compiles nicely.
Quote:
Critisizm is HIGHLY welcomed. I see that as a way to make me better.
Awesome attitude. You will make it far here.
No, bad. Make sure you add some fail safes onto that. Basically that is an endless loop. Try setting a constant asking them how many times they want to alch if they don't want to buy. Maybe cut down on some of the constants. Capitalize your variables.
SCAR Code:
MagicLvl=72; //Input magy lvl;
Should be able to be replaced with GetLevel('Magic'...); (don't know the exact way, should be an easy search)
Do all of the GE colors/coords change so much that you have to input them every time? If not try to cut down on those. The chatbox coords shouldn't change much either and so you shouldn't need them as constants.
Again, it looks pretty good. Just some small changes.