One on one advice is always great, but don't forget to look up tutorials on the subject as well [sometimes tuts imo are more rewarding], but since you are respectably asking for some simple feedback, i would have no problem in doing so.
Procedure DeclarePlayers; Very nice, but it could be better and easier to read if you used a with..do statement instead.
SCAR Code:
Procedure DeclarePlayers;
begin
HowManyPlayers := 1;
NumberOfPlayers(HowManyPlayers);
CurrentPlayer := 0;
with Players[0] do
begin
Name := 'HowDoIChopTree';
Pass := 'butHowDoIChopit';
Nick := 'chop';
Member := False;
Active := True;
end;
end;
As for the Antiban, its fine and you seem to get the concept of it. Great work =].
Procedure MakeCameraNotTerrible; I would change the name of this procedure to something more simple, such as FixCamera, or MakeCamera. I don't know why you want it to press the key up, wait 2 seconds, then press it down =/ but regardless, i think what your doing in this procedure could be summed up with a simple MakeCompass(); and SetAngle(); function.
Procedure ChopTree; It looks alright. You forgot to set an "until" in your repeat .. until loop.
Overall: Theres nothing really to criticize [very small script so far], there are many things that can be added. Im not going to say anything that should be fixed, its extremely novice, and it should be only for you to practice with. If I were you I wouldn't concentrate on making a full script out of this, instead I would use it as a set of notes to learn from. Theres so much more that you can do, just keep learning and learning and practicing, and try not to worry about what others should expect from you, your doing great the way it is =].