Here's the final coding:

Code:

Do you think that this will work? If not, please say why.

We don't answer questions like that, in general, because they don't show that the author put any thought into trying to test or debug things himself or herself. Did you test and run your code? Did you find any inexplicable bugs in the code above that you weren't able to track down, and if so, what steps did you already try? Are there specific techniques you used in your code that you're not very familiar with and you feel you might have misused that you'd like us to double-check, like nested loops or something?

Edit: I'll throw you a bone and ask you to think about how to improve this:
Code:
It can be made much, much shorter and less repetitive. How do you think you could do that?

I have tested and run this code. I just wanted to make sure that if I run it many times, it won't crash. And you are far more of an expert than I am, so I ask for help.
And I will think on that.

That's reasonable. Luckily your program is relatively short, but if it was longer, evaluating it purely by inspection would be a fairly thankless task, so we'd generally prefer more specific questions. Something to keep in mind for the future.
Throw out the first thing that comes to your mind. Is there a pattern in that code that could be exploited to make it less repetitive? What are some techniques you're already familiar with to avoid having to repeat a similar command over and over?

Here's a problem that will cause problems after your program has run for a while. It's called a memory leak, and it's caused by Goto'ing out of a While/End, For/End, Repeat/End, If/Then/End, or If/Then/Else/End construct. Luckily, it's trivial to solve here. How can you avoid Goto'ing out of this While loop? Hint: When will this While loop end anyway?
Code:

For the Text( commands, I am trying a For( loop. Goes like this:


Code:


This is not working. It does the B-J sequence many times over, then stops. Have any ideas to make it only run once?

As for the While loops, I just set it to run only once. Problem solved there.

Well, you may want to look more into the sub() command for your entire code. I was able to cut about 80% of your code off by using this function. In your above case, the sub() function will work quite nicely.

What is the sub( command? I haven't ever heard of that. And by the way, I am just about done with the code. But how could I get it so that it doesn't go past the edge of the border of the gameboard? I was thinking of an If-Then-Else-End loop, but while I do have the edge of the gameboard all figure out, I can't figure out how to make the pixels that would normally go over the edge of the gameboard place themselves behind of what has already been placed. For example, let's say that I put the carrier at (J,eight). In this case, three of the the five dots(which I figured out how to make them place themselves horozontally or vertically, depending on what you chose in the menu( command that I wrote) placed horzontally would go over the edge. Any ideas on how I could make those three dots put themselves behind the two already placed?

The sub() command could be very useful in this case. Your code could be shortened to this:


Code:


The way it works is by taking a string and selecting certain parts of it. The 1st argument is the string, the second the offset in the string, and the third the number of characters to select. Hope this helps; it eliminates tons of code...

Considering this is what the original was:


Code:


As for your above question, give me a bit to work it out.

I have tried the sub( command. But for some reason, it now only loops once instead of ten times (five times for each player), and just haves you place just one ship, instead of five. What's wrong?[/code]

Hmm.... This code seems to work just fine for me...

Code:

Of course, it makes you enter in the coordinates for every point of the ship, which is definitely improvable through bounds checking....

I solved the problem with the sub( code. I have now supershortened my program. I am now trying to do that with this bit..

Code:

I have a way to shorten this.
Like this:

Code:

I think this would work, but I don't have any time to test it now. Would anyone be willing to test it for me? What about you, Mateo?

That could work; nice work on coming up with that on your own! An alternative way might be to use 0 and 1 in P, and just make it seem like they are players 1 and 2. That way checking is a lot easier, and switching players can also be easier as well, because you just could not(P). Here's one way that could work:

To clear things up a little, P=0 is player 1, and P=1 is player 2, if that makes sense.

Hope this helps!

Code:

What does the not( command do on my ?

Never mind- I figured what the not( command does on my calc. But there is a problem that I have been trying to solve for about a week now. How can I do the score? I have the score display all set up, but how can I get it so that when M(for player 1) or N(player 2) equals 17, they win. How do I do that?

Also, I have lately been thinking about the ability to save games to continue at a later period. What do you guys think of this idea?