Print

Please login or register.
1 Hour 1 Day 1 Week 1 Month Forever
Login with username, password and session length

[Bauglir]

-snip-

[Virex]

As far as I know you can't really do that. However, may I ask why you're unable to use foo.lenght (with foo the name of the array) as the upper bound of a loop?

[Bauglir]

-snip-

[Virex]

You most definitely can't do that in Java. The names of the different variables (your battleships) have to be known at compile-time, or else your program won't compile.


Also, if you would be some kind as to post the code I'll have a quick look at it. Maybe I can figure out if it's possible to salvage the code, though it might just as well be spaghetti code, knowing the way most profs write example programs for students.
Alternatively, if he's using Swing for the gui, I might be able to salvage it so you can add your own back end, if that'd be easier for you.

[Bauglir]

-snip-

[Virex]

I take it you're bound to the existing code? Because it'd probably help both you and your grip of the language if you'd just rewrite the whole damn thing, keeping only the GUI (and if you really want to I can probably help you rewire that too). I also assume you're pretty much bound to primitive types (no HashMaps, Arraylists and other fancy stuff?)


That looks ugly as hell and unintuitive. I'd just've made a constructor for the battleship that accepts a 1-d array. Then you can call it with

Code: [Select]

ships[i] = new Ship(iArray[i]); //Why not use shipData or another descriptive name instead of iAray here?
That is, assuming it is not possible to make the objects on the fly while parsing the data file, because in that case I would've just parsed one line of the file (probably into a String, because I'm lazy), fed that into the constructor of the ship (which would pick the String apart and set the variables) and then read the next line.

[Shurikane]

Based on what you're trying to do at least as far as I understand, you might be able to do what you seek to do using either of these:

Code: [Select]

List<Something> myThings = new ArrayList<Something>();
This above does pretty much the same job as a basic ol' array but it has nifty tools in it.

Code: [Select]

Map<String, Something> myThings = new HashMap<String, Something>();
Read up on maps a bit and you will find a piece where I think you can supply two keys to lead to one entry (such as X-Y coordinates!)  Otherwise, you could always make up a coordinates convention that fits into a single key variable and use the version above.

In the end, you want your method to return a list or a map and then you can work off that.  The list has the ability to know how many elements it has in it.  The map, I don't remember off the top of my head.

CAUTION: Check with your teacher to make sure you are allowed to use these above tools in the exercise you are doing, if you choose to use them.

[Virex]

Havn't found a way to give HashMap 2 keys yet. You could probably just write a little function to combine the coordinates into a single string though, or just use the Point class (can't remember what you need to include for that, but if you're using netbeans it'll tell you).


Edit: Hashmap also knows it's size.

[Bauglir]

-snip-

[eerr]

If I remember correctly, most array types in java can be extended.