C＃戦艦グリッドとケース実装の問題

Question

現在、戦艦ボーンズモジュールをC＃で作成しようとしていますが、大部分のコードはうまく処理されていますが、ヒットとミスをマークしようとしていますプレイヤーが正しくまたは間違って推測するたびに

私がしようとしているのは、人々が適切なグリッド座標を入力するとケース "H"とケース "M"をアクティブにすることです。

コードはC＃にあります。

基本戦艦を：起こってすることになっているもの

。これはC＃の私の最初の本当のプロジェクトであり、船を隠すことは私にとって重要ではありません。私はちょうど船の座標がケースHとケースMに割り当てられた適切なXとバックグラウンド/フォアグラウンドカラーに置き換えられるのに苦労しています。

何か不足していて追加情報が必要な場合は、私は知っている！

これは、これまで私が持っているコードです：

class Program 
{ 
    static void Main(string[] args) 
    { 
     while (true) 
     { 

      //Basic function that allows the user to enter their 'guesses' for the ship locations. 
      printGrid(Grid); 
      Console.WriteLine("Type 'exit' to exit."); 
      Console.WriteLine("Enter your guess: "); 
      string cons = Console.ReadLine(); 
      cons.Substring(0, 1); 
      cons.Substring(1, 1); 
      cons = cons.ToUpper(); 
      char column = cons[0]; 
      char row = cons[1]; 
      int num = cons[0]; 
      bool result = Int32.TryParse(cons, out num); 

      //Disallows the user from entering anything other than what's valid. Valid characters are, obviously, A - J. 
      string validChars = "ABCDEFGHIJ"; 

      //This allows the user to exit the application by typing "Exit" instead of guessing. 
      if (cons == "EXIT") 
      { 
       return; 
      } 

      //Again, checks to see if the user inputs any numbers higher than what's allotted. 
      else if (!validChars.Contains(column) || (num >= 11)) 
      { 
       Console.WriteLine("Please enter only A-J, and 1-10 e.g 'B5'"); 
      } 
      Console.WriteLine("Column = {1}, Row = {0}", row, column); 

     if (result == false) 
      { 
       Console.WriteLine("Guess was a miss!"); 
      } 
      else 
      { 
       Console.WriteLine("Guess was a hit!"); 
      } 

     } 
    } 
    private static readonly char[,] Grid = new char[,] 
    { 
     {'.', '.', '.', '.', 'S', 'S', 'S', '.', '.', '.'}, 
     {'P', 'P', '.', '.', '.', '.', '.', '.', '.', '.'}, 
     {'.', '.', '.', '.', '.', '.', '.', '.', '.', 'P'}, 
     {'.', '.', '.', '.', '.', '.', '.', '.', '.', 'P'}, 
     {'.', '.', 'A', 'A', 'A', 'A', 'A', '.', '.', '.'}, 
     {'.', '.', '.', '.', '.', '.', '.', 'B', '.', '.'}, 
     {'.', 'S', '.', '.', '.', '.', '.', 'B', '.', '.'}, 
     {'.', 'S', '.', '.', '.', '.', '.', 'B', 'P', 'P'}, 
     {'.', 'S', '.', '.', '.', '.', '.', 'B', '.', '.'}, 
     {'.', '.', '.', '.', '.', '.', '.', '.', '.', '.'}, 
    }; 
    public static void printGrid(Char[,] Grid) 
    { 

     //Let's make the layout pretty. 
     Console.WriteLine("  | A | B | C | D | E | F | G | H | I | J |"); 
     Console.WriteLine("-----#-----#-----#-----#-----#-----#-----#-----#-----#-----#------#"); 

     for (int i = 0; i < 10; i++) 
     { 
      if (i == 9) 
      { 
       Console.Write(" {0} ", i + 1); 
      } 
      else 
       Console.Write(" {0} ", i + 1); 
      for (int j = 0; j < 10; j++) 
      { 
       ShipColors(Grid[i, j]); 
      } 

      //This needs to match the header in order to format properly. 
      Console.Write(" |\r\n"); 
      Console.WriteLine("-----#-----#-----#-----#-----#-----#-----#-----#-----#-----#------#"); 
     } 
    } 

    //Now we're going to declare which colors each ship's cell is going to be. 
    public static void ShipColors(char useThis) 
    { 
     /* 
     * Legend for each ship name: 
     * P = patrol boat 
     * S = Submarine 
     * B = Battleship 
     * A = Aircraft Carrier 
     */ 
     switch(useThis) 
      //Going to use case statements here to automatically color any of the grid's cells with any special characters. 
      //Using cases allows the grid to be re-organized and still be colored properly. 
     { 
      case '.': 
       Console.Write(" | "); 
       //Blank space here needed to be added for formatting the cells properly. 
       Console.Write(" "); 
       break; 
      case 'A': 
       Console.Write(" | "); 
       Console.BackgroundColor = ConsoleColor.Blue; 
       Console.Write(" A "); 
       break; 
       case 'B': 
       Console.Write(" | "); 
       Console.BackgroundColor = ConsoleColor.Green; 
       Console.ForegroundColor = ConsoleColor.Black; 
       Console.Write(" B "); 
       break; 
      case 'P': 
       Console.Write(" | "); 
       Console.BackgroundColor = ConsoleColor.Yellow; 
       Console.ForegroundColor = ConsoleColor.Black; 
       Console.Write(" P "); 
       break; 
      case 'S': 
       Console.Write(" | "); 
       Console.BackgroundColor = ConsoleColor.Red; 
       Console.ForegroundColor = ConsoleColor.Black; 
       Console.Write(" S "); 
       break; 
      case 'H': 
       Console.Write(" | "); 
       Console.ForegroundColor = ConsoleColor.Red; 
       Console.Write(" X "); 
       break; 
      case 'M': 
       Console.Write(" | "); 
       Console.ForegroundColor = ConsoleColor.White; 
       Console.Write(" X "); 
       break; 

     } 
     Console.ResetColor(); 
    } 
} 

Answer 1

あなたは一つの方法で多くのことをやろうとしています。理想的には、入力を検証するメソッド、ヒットを確認するメソッド、グリッドを直前に推測したスポットの指標で再描画するメソッドなど、いくつかのメソッド呼び出しに分解するのが理想的です。

しかし、あなたが入力が正しかったことの検証をやった後に、この特定の問題を支援するために、あなただけのような何か実行する必要があります。

1. インデックスに入力行を変換します。 1で始まる行を表示しますが、配列インデックスは0で始まるので、入力から1を引く必要があります。
2. 入力列をインデックスに変換します。文字Aはint値が65であり、Aが最初の行であることから、インデックスを取得するために列から65を引きます。

そして、それは文字が含まれているかどうかを確認するために、インデックス[newRow, newCol]で私達のGrid配列内のチェックの問題だ：

：ここ

if (char.IsLetter(Grid[rowNumber, colNumber])) 
{ 
    Console.WriteLine("HIT!"); 
} 
else 
{ 
    Console.WriteLine("MISS."); 
} 

はあなたのコードの半クリーンアップバージョンです。

while (true) 
{ 
    Console.Write("Enter your guess: "); 
    string userGuess = Console.ReadLine().ToUpper(); 
    if (userGuess == "EXIT") return; 

    int rowNumber; 

    // Validate input 
    if (userGuess.Length < 2 || userGuess.Length > 3) 
    { 
     Console.WriteLine("Guess must be in the form of ColumnRow, like: B5"); 
    } 
    else if (!"ABCDEFGHIJ".Contains(userGuess[0])) 
    { 
     Console.WriteLine("Guess must begin with a valid column letter (A-J)"); 
    } 
    else if (!int.TryParse(userGuess.Substring(1), out rowNumber) 
      || rowNumber < 1 || rowNumber > 10) 
    { 
     Console.WriteLine("Guess must end with a valid row number (1-10)"); 
    } 
    else 
    { 
     // Valid input! Set actual indexes for the guess 
     int colNumber = userGuess[0] - 65; 
     rowNumber--; 

     if (char.IsLetter(Grid[rowNumber, colNumber])) 
     { 
      Console.WriteLine("HIT!"); 
     } 
     else 
     { 
      Console.WriteLine("MISS."); 
     } 
    } 
}