ID:146961
 
Thanks people, I have fixed my problem.
Put "#define DEBUG" somewhere in your coding.
It'll give you a line, and add "//Line here> to the line in your coding(edit your first post).

:o Then it'll be MUCH easier to help.
Two problems:

  • You should not be using goto here. You can replace the trader1 part with a simple do-while() loop, and the trader2 label is completely redundant. Never ever ever use goto unless you're in a loop structure so complex there's no other reasonable way around it.
  • You never checked to see if M has a client. If it doesn't have one, you can't use it as a target for alert() or input().

    I'd also subtract tradelist from display when offering choices of what to trade next.

    Lummox JR