|
That's a good start. 
OK, you have a board class, what do you put on the board?
In the real game, you don't add "single square" ships: you add 4 x Destroyer (two squares), 3 x submarines (three), 2 x Battleships (four) and 1 x Carrier (five).
So why not create an abstract base class Ship, which is inherited by the concrete classes Destroyer, Submarine, Battleship, and Carrier?
The abstract class has properties Length and Name which the concrete classes implement. The Name is so you can say "you sank my Destroyer!" without knowing which concrete class it is, and the Length says how many squares it takes.
Add a Add method to your board which accepts a Ship, a top left coordinate, and an orientation (NSEW) and modify your Hit method to see if it's sunk.
This way, the board doesn't need to know what ship it is, and it enforces rules like "you can't put it there because it hang over the side" and such like.
Have a think about it - I know it sounds complicated, but it's really pretty easy when you get the hang of it and it really does make your code easier to write and modify!
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
I've added the abstract Ship class within my Gameboard class below
Does it seem like it's headed in the right direction?
abstract class Ship
{
public string Name;
public int Length;
}
class Destroyer: Ship
{
}
class Submarine : Ship
{
}
class Cruiser : Ship
{
}
class Battleship : Ship
{
}
class Carrier : Ship
{
}
|
|
|
|
|
OK, those aren't properties: they are fields and it's considered bad practice to expose fields directly.
And the problems with fields are that they can be changed at any time, and your derived classes don't have to do anything with them - like giving them a value!
Creating abstract properties gets round both of those: the derived class must implement the property - which means it must return a relevant value - and it can be read only.
Here's my version of your code:
public abstract class Ship
{
public abstract string Name { get; }
public abstract int Length { get; }
}
public class Destroyer : Ship
{
public override string Name { get => "Destroyer"; }
public override int Length { get => 2; }
}
Because the properties are marked abstract, every time your derive a class from Ship it must implement it or your code won't compile!
Because I don't specify a setter in the abstract class property definition you can't add a setter to the derived class either, so the property cannot be changed by the outside world.
The derived class Destroyer "knows" how long it should be, and what the type of ship is called, so it implements that in the getter - the outside world doesn't need to know anything about it, the Destroyer class provides it as needed.
If you haven't seen this syntax before:
public override string Name { get => "Destroyer"; }
It's just a "short form" of this:
public override string Name
{
get
{
return "Destroyer";
}
}
Each of your derived classes implements it's own version, and the system uses the right one later:
private void MyButton_Click(object sender, EventArgs e)
{
Destroyer d1 = new Destroyer();
Cruiser c1 = new Cruiser();
ShowMe(d1);
ShowMe(c1);
}
private void ShowMe(Ship s)
{
Console.WriteLine($"A {s.Name} that is {s.Length} long.");
}
Prints:
A Destroyer that is 2 long.
A Cruiser that is 3 long.
You are letting Ships handle themselves instead of needing information to set be in your main code.
For example, if you wanted to add a helicopter class:
*
***
* You don't have to change your main code at all: it doesn't "need to know" that it's a weird shape, the new Helicopter class would deal with it, and it's just another Ship as far as your code is concerned:
public class Helicopter: Ship
{
public override string Name { get => "Helicopter"; }
public override int Length { get => 5; }
}
Does that make sense?
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Yes that makes sense I can wrap my head around the concepts
I went through a couple of tutorials on abstract classes and getters/setters
along with applying them in Visual Studio
With your explanation of how the derived class(unique ships)
knows what propeties to take on without leaking the data
to the outside world it really simulates the blindness of
the physical game.
In the previous message you said:
"Add a Add method to your board which accepts a Ship, a top left coordinate, and an orientation (NSEW) and modify your Hit method to see if it's sunk."
From my understanding I'll be implementing something like this below:
playerA.PlaceShip(playerACruiser, int x,int y, Orientation.Vertical)
And then I will be implementing logic inside of the PlaceShip Method
to set them on the board?
You're helping me grow a ton and I really appreciate it
|
|
|
|
|
That's about it, but I'd probably simply it a bit.
There is a .NET struct that is really useful, called Point - which describes a ... well, a location in 2D space!
So the PlaceShip method signature would be
public Ship PlaceShip( Ship ship, Point loc, Orientation wayUp)
{
...
return ship;
}And I'd call it like this:
playerA.PlaceShip(new Cruiser(), new Point(x, y), Orientation.Vertical);
...
playerA.PlaceShip(new Destroyer(), new Point(x, y), Orientation.Vertical);
You seem to be doing pretty well! 
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
With the current setup to add the ship(s) with this method
public Ship PlaceShip(Ship ship, Point loc, Orientation direction)
{
for (int i = 0; i < ship.Length; i++)
{
}
Console.WriteLine(direction);
loc.x = 5;
loc.y = 5;
Console.WriteLine($"The {ship.Name} has a length of {ship.Length} squares");
return ship;
}
I understand how the ships will be created and passed in and I have
created the struct Point but don't quite know how I am going to
access values inside of it and pass them through as arguments
when PlaceShip is called. This is what I have so far and how I am
approaching it
public struct Point
{
public int x;
public int y;
public Coordinate(int x, int y)
{
this.x = x;
this.y = y;
}
}
Also I have created a class for Orientation and know that I need to
pass in either Horizontal or Vertical as an argument
public static int[] Vertical()
{
Console.WriteLine("I am Vertical");
return new int[5];
}
public static int[] Horizontal()
{
Console.WriteLine("I am Horizontal");
return new int[5];
}
|
|
|
|
|
You don't need to construct a Point class / struct - it's already in .NET: Point Struct (System.Drawing) | Microsoft Learn[^]
And why would you need two arrays? Does it matter once the ship is in place? Or is a Ship a collection of Point values that need to be hit?
Simplest solution is to use an Orientation enum and pass a value of that to your PlaceShip method:
private Random rand = new random();
...
Point loc = new Point (rand.Next(0, 10), rand.Next(0, 10));
Orientation orient = Enum.GetValues(typeof (Orientation))
.OfType<Orientation>()
.ElementAt(rand.Next(0, ));
Ship s = PlaceShip (new BattleShip, loc, orient);The code for a random enum value may be unfamiliar to you: it uses generics (which you may not have met yet) and Linq methods (which you may not have met either) to do the work:
Gets an array of all possible values of the enum
Cast the array to a more useful type - a collection of Orientation values.
Select a random element.
I could have written it like this:
orient = (Orientation)rand.Next(0, 4); Which gives the same result - but it assumes that the orientation values are sequential: if you want to add diagonals later there are good reasons for making your enum values non-sequential:
public enum Orientation
{
Up = 1,
Down = 2,
Left = 4,
Right = 8,
}Because now you can combine values to form "Up and Right" without changing working code.
orient = Orientation.Up | Orientation.Right; Does that make sense?
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
I understand the creation and passing of the Enum for the orientation
From what I have searched and found for the Point struct/class:
I was not able to instantiate/access this from the foundational .NET suite
So I looked up how to create my own Point struct(Struct in C#[^]),
and tried to access it like that
Maybe there is a better way to access the Point struct/class?
Also I am having a difficult time accessing classes, properties,
and methods across my two files GameBoard.cs and Program.cs
Program.cs
namespace ConsoleBattle
{
public class Program
{
public static void Main(string[] args)
{
GameBoard playerA = new GameBoard();
GameBoard playerAGuesses = new GameBoard();
bool sunk = false;
string message;
Welcome();
Console.WriteLine("X: 0 - 9");
Console.WriteLine("Y: 0 - 9");
while (true)
{
playerA.PlaceShip(new Submarine(),new Point(x,y), Orientation.Down);
while (!sunk)
{
try
{
Console.WriteLine("Enter your guess: X, Y");
string guess = Console.ReadLine();
int xPos = Convert.ToInt32(guess.Split(',')[0]) - 1;
int yPos = Convert.ToInt32(guess.Split(',')[1]) - 1;
if (xPos > 9 || yPos > 9)
{
Console.WriteLine(message = "You are off the board, try again!");
}
playerA.CheckGuess(xPos, yPos);
}
catch
{
Console.WriteLine(message = "Unable to process coordinates");
}
Console.ReadLine();
}
Console.WriteLine(message = "You sunk the ship!");
Console.WriteLine("Play Again? [Y or N]");
string answer = Console.ReadLine().ToUpper();
if (answer == "Y")
{
continue;
}
else if (answer == "N")
{
return;
}
else
{
return;
}
}
}
private static void Welcome()
{
Console.WriteLine("Welcome to Console Battle!");
Console.WriteLine("Enter username: ");
string username = Console.ReadLine();
Console.WriteLine($"\nLet's begin {username} press Enter!");
Console.ReadLine();
}
}
}
|
|
|
|
|
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Threading.Tasks;
namespace ConsoleBattle
{
public class GameBoard
{
public enum Square
{
Water,
Miss,
Ship,
Hit,
}
private Square[,] TheBoard = new Square[10, 10];
public Square[,] CheckGuess(int x, int y)
{
if (TheBoard[x, y] == Square.Ship)
{
Console.WriteLine(TheBoard[x, y] = Square.Hit);
}
else if (TheBoard[x, y] == Square.Water)
{
Console.WriteLine(TheBoard[x, y] = Square.Miss);
}
else if (TheBoard[x, y] == Square.Miss)
{
string message1 = "you already missed here";
Console.WriteLine(message1);
}
else
{
string message2 = "this spot was hit already";
Console.WriteLine(message2);
}
return TheBoard;
}
public Ship PlaceShip(Ship ship, Point loc, Orientation wayUp)
{
Console.WriteLine(ship.Name);
Console.WriteLine(wayUp);
Console.WriteLine(loc);
return ship;
}
}
public enum Orientation
{
Up = 1,
Down = 2,
Left = 4,
Right = 8,
}
public struct Point
{
public int x;
public int y;
public Point(int x, int y)
{
this.x = x;
this.y = y;
}
}
public abstract class Ship
{
public abstract string Name { get; }
public abstract int Length { get; }
}
public class Destroyer : Ship
{
public override string Name { get => "Destroyer"; }
public override int Length { get => 2; }
}
public class Submarine : Ship
{
public override string Name { get => "Submarine"; }
public override int Length { get => 3; }
}
public class Cruiser : Ship
{
public override string Name { get => "Cruiser"; }
public override int Length { get => 3; }
}
public class Battleship : Ship
{
public override string Name { get => "Battleship"; }
public override int Length { get => 4; }
}
public class Carrier : Ship
{
public override string Name { get => "Carrier"; }
public override int Length { get => 5; }
}
}
It has gotten pretty messy and I am feeling a little frustrated 
Any suggestions on what I can do to clean it up?
This must be one of the longest threads ever
|
|
|
|
|
Hi Griff,
Hope you are doing well
I am up to speed with everything you mentioned on this post
I figured out I needed the using directive System.Drawing in order
to access the built in .NET Point struct
With that said I successfully implemented the randomization of
Point loc, and the Orienation orient (represented as Enums)
I am able to call the PlaceShip method on the object playerA
and create the 1 ship, 2 location, and 3 orientation as follows:
playerA.PlaceShip(new Submarine(), loc, orient);
...
public Ship PlaceShip(Ship ship, Point loc, Orientation wayUp)
{
loc = (x, y);
string direction;
int x;
int y;
if (wayUp == Orientation.Right)
{
direction = "right";
}
else if (wayUp == Orientation.Left)
{
direction = "left";
}
else
{
direction = "left,right";
}
if (wayUp == Orientation.Down)
{
direction += "down";
}
else if (wayUp == Orientation.Up)
{
direction += "up";
}
else
{
direction += "up,down";
}
if (direction == "up")
{
for (int i = y; i < y - 4; i--)
{
TheBoard[i, y] = Square.Ship;
}
}
else if (direction == "down")
{
for (int i = y; i < y + 4; i++)
{
TheBoard[i, y] = Square.Ship;
}
}
else if (direction == "left")
{
for (int i = x; i < x - 4; i--)
{
TheBoard[x, i] = Square.Ship;
}
}
else
{
for (int i = x; i < x + 4; i++)
{
TheBoard[x, i] = Square.Ship;
}
}
Console.WriteLine(ship.Name);
Console.WriteLine(wayUp);
Console.WriteLine(loc);
return ship;
}
In the console when I run the program with PlaceShip() above I get back the following:
Submarine
Left
{X=7, Y=1}
What I am working on now is placement of the Ship(s)
inside the PlaceShip Method:
I am trying to acces the x, y coordinates that come from Point loc
within the PlaceShip Method, and them use them in the assignment
logic with the looping and if/else statements in order to populate
the board with the Enum Ship
I know that I need to take the location and orientation and loop
over them to populate TheBoard with the Enum Ship, but having
a hard implementing that functionality.
|
|
|
|
|
Hi! I haven't forgotten you - I've just been really, really busy for the last week, and it doesn't look like it'll improve till the weekend at least.
I will get back to you, but I don't want to hurry what I say and either send you off on a wrong tangent or confuse you - and that takes time to review the whole conversation we've had and plan out how best to help you next!
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Okay no worries at all I completely understand and appreciate the valuable time you spend while helping me.
I'll continue to try and move forward, and watch/code C# tutorials since there is what seems like a plethora of concepts to cover in .NET
|
|
|
|
|
Hope all is well
So I am able to pass in all three arguments to the PlaceShip method
And I am copying the Point loc inside of of the method and assigning them values of loc.X, loc.Y,
so I can use them in my loops in my conditional statements (not sure if that's the best way)
I am trying to place and populate the ship with the Enum Ship
Sometimes the values get filled, but other times I am off the board or the logic won't execute. I can see
the values get populated on TheBoard when I step through it while debugging.
This is what it looks like currently:
public Ship PlaceShip(Ship ship, Point loc, Orientation direction)
{
loc = new Point(loc.X, loc.Y);
Console.WriteLine(loc);
if (direction == Orientation.Up)
{
for (int i = loc.Y; i < loc.Y - 5; i--)
{
TheBoard[loc.X, i] = Square.Ship;
}
}
else if (direction == Orientation.Down)
{
for (int i = loc.Y; i < loc.Y + 5; i++)
{
TheBoard[loc.X, i] = Square.Ship;
}
}
else if (direction == Orientation.Left)
{
for (int i = loc.X; i < loc.X - 5; i--)
{
TheBoard[i, loc.Y] = Square.Ship;
}
}
else
{
for (int i = loc.X; i < loc.X + 5; i++)
{
TheBoard[i, loc.Y] = Square.Ship;
}
}
Console.WriteLine(ship.Name);
Console.WriteLine(direction);
return ship;
}
}
|
|
|
|
|
In terms of randomly populating, you'd have to tally the rows and columns that are available "before" each "chance" throw; whose range would depends on the # of available "slots".
A "5 cell" ship on a 10 x 10 grid would obviously only fit in a row or column that had 5 "contiguous" empty cells; something that could be figured out by iterating over the array before each throw. Use the "id #" of the ship to populate the grid cells and identify what got hit. Another throw could determine the offset within the column / row if more than 5 cells are available for "slotting".
"Before entering on an understanding, I have meditated for a long time, and have foreseen what might happen. It is not genius which reveals to me suddenly, secretly, what I have to say or to do in a circumstance unexpected by other people; it is reflection, it is meditation." - Napoleon I
|
|
|
|
|
Hi. I'm working with Visual Studio on a application that changes a windows registry. There is a small forms that requests an address in a text box. The contents of the text box wil be writen is the registry. I need the key to be LocalMachine and here is a snippet of this code:
private void button1_Click(object sender, EventArgs e)
{
RegistryKey chaveBase, chave;
chaveBase = RegistryKey.OpenBaseKey(RegistryHive.LocalMachine, RegistryView.Registry64);
chave = chaveBase.CreateSubKey("Software\\Antares", true);
chave.SetValue("Endereço", textBox1.Text);
}
The problem is that I receive the message: "System.UnauthorizedAccessException: 'Access to the registry key 'HKEY_LOCAL_MACHINE\Software\Antares' is denied".
If I change LocalMachine for CurrentUser, it works fine. What should I do to access LocalMachine?
|
|
|
|
|
This is because you have to be running the code as an administrator to make any changes to anything under HKEY_LOCAL_MACHINE.
Hold down a Shift key, right-click your executable and click "Run as administrator". You'll, of course, have to supply admin credentials, or if you're already using an admin account, you'll have to OK using the admin part of your account to run the app. Then it'll work.
|
|
|
|
|
Thank you Dave, but code should run without user intervention.
|
|
|
|
|
Yeah, ... and?
You said you created a WINDOWS FORMS APP, which is an app that requires user interaction.
If this is a test app for what you're really writing, like a Windows Service app, then you can run that code under the Local System account, which will have admin permissions.
|
|
|
|
|
To add to what Dave has - rightly - said ...
The problem is due to a layer of security added after the registry became too bloated and vulnerable: it was filled with rubbish, and fully available to code to view or modify.
In more modern OSes (think Vista and later) it became much harder to access the registry, and Admin access was required for most of the content. This included the entire LocalMachine branch. Do not expect it to get easier to access, if anything it will get harder as storing app settings there is now discouraged to reduce the bloat.
So unless you have a very, very good reason to use the registry you shouldn't do it: use an app config file, or store data in a User data folder instead. It's easier to do, and a lot more future proof!
This may help: SWhere should I store my data?[^]
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Hi, OriginalGriff.
I read your article which you suggested and I'll probably change my storage place. On question: your article is 10 years old. Are those information still valid?
Thank you.
|
|
|
|
|
Yes - the locations are still the same!
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Hi, OriginalGriff.
I used the code you wrote in your article, but I received an error message. First see a snipet of my code:
public static Guid AppGuid
{
get
{
Assembly asm = Assembly.GetEntryAssembly();
object[] attr = (asm.GetCustomAttributes(typeof(GuidAttribute), true));
return new Guid((attr[0] as GuidAttribute).Value);
}
}
This is in fact your code. The error is on the line of the return:
"System.IndexOutOfRangeException: 'Index was outside the bounds of the array.'"
Do you know what causes this error?
Thanks.
|
|
|
|
|
That's because your app doesn't have a GuidAttribute.
Go into your project Properties, click on the Application tab, then Assembly Information button. You'll find a GUID box in the dialog that pops up. That box is empty. That's where you have to put your GUID.
|
|
|
|
|
Normally, that is filled in by VS when it creates the project template. Dunno what he used to do that ... 
"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!
|
|
|
|
|
Under the .NET Framework projects (4.8 at least), it's filled in by the Windows Forms template, but not the WPF template. I haven't tried any of the others.
On .NET Core projects, the "Assembly Information" option doesn't even exist. The assembly info is automatically generated at compile time, but is controllable in the .csproj file.
I haven't spent very much time looking at this because I don't expose much to COM, so YMMV.
|
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
