인덱스를 음수로 반환하는 방법

Question

플레이어 클래스를 만들었고 플레이어를 만들려고하는 메뉴 기반 플레이어 시스템에서 해당 클래스의 배열을 만들었습니다. 내 ProcessCreate 메서드는 InsertPlayer에서 반환 된 삽입 인덱스를 확인하고 -1과 같지 않은 경우 "성공적으로 만든"메시지를 표시하지만 -1을 반환하는 GetInsertIndex 메서드를 얻는 방법을 찾는 데 어려움을 겪고 있습니다. InsertPlayer의 범위 예외는 사용자가 음수를 입력 또는 채팅 여기서

그것으로 확인 내 ProcessCreate 방법의 일부 이해 될 것이다

빈 문자열을 가지는 경우 경우 밖으로 인덱스 버리지 -1

playerindex = InsertPlayer(number, firstName, lastName, goals, assists, players, ref playerCount); 


        if (playerindex != -1 || playerCount > 0) 
        { 


          Console.WriteLine("\n{0,7} {1,-20}{2, -20}{3,8}{4,8}{5,8}\n", "Number", "First Name", "Last Name", "Goals", " Assists", "Points"); 
          Console.WriteLine("{0,7} {1,-20}{2, -20}{3,8}{4,8}{5,8}", 
          players[playerindex].Number, players[playerindex].FirstName, players[playerindex].LastName, 
          players[playerindex].Goals, players[playerindex].Assists, players[playerindex].Points()); 
          Console.WriteLine("Sucessfully created!"); 
          Console.WriteLine(); 


        }  

      } 
      else 
       Console.WriteLine("\nCreate Player: the player number already exists"); 
     } 
     else 
      Console.WriteLine("\nCreate Player: the player roster is already full"); 

    } 

당신은 당신이 말한대로 -1 인덱스를 반환에 대해 -1 그나마 걱정을 반환하려면 여기 내 InsertPlayer 및 getInsertIndex를 방법

static Int32 InsertPlayer(Int32 number, String firstName, String lastName, Int32 goals, 
    Int32 assists, Player[] players, ref Int32 playerCount) 
{ 

     Int32 insertIndex, shiftCount; 

     insertIndex = GetInsertIndex(number, players, ref playerCount); 
     for (shiftCount = playerCount; shiftCount > insertIndex; shiftCount--) 
      players[shiftCount] = players[shiftCount - 1]; 


      try 
      { 
       players[insertIndex] = new Player(number, firstName, lastName, goals, assists); 

      } 
      catch(Exception e) 
      { 
       Console.WriteLine(e.Message); 
      } 
      if (insertIndex != -1) 
      { 
       playerCount++; 
      } 

      return insertIndex; 
} 

    static Int32 GetInsertIndex(Int32 number, Player[] players, 
ref Int32 playerCount) 
{ 
    Int32 index = 0; 
    bool found = false; 
    while (index < playerCount && found == false) 

     if (players[index].Number > number || playerCount <= 0) 
     { 
      found = true; 
      index = -1; 
     } 
     else 
      index++; 
    return index; 
} 

Answer 1

이다. 그냥 -1을 반환; 예외 안에

catch(Exception e) 
{ 
    Console.WriteLine(e.Message); 
    insertIndex = -1; 
    // or 
    return -1;  
} 

문자열이 비어 있는지 확인하고 오류를 직접 잡아서 -1을 반환합니다. 그것은 것

return Array.IndexOf(players,player); 

을 : 반환 다음

  Player player = new Player(); 
      try 
      { 
       player = new Player(number, firstName, lastName, goals, assists); 
       //or assign individually instead of using the constructor 
       //if your player class allows it 
       players[insertIndex] = player; 

      } 
      catch(Exception e) 
      { 
       Console.WriteLine(e.Message); 
      } 

:

Answer 2

는 전체 코드를 읽어 보지 않았하지만 플레이어 목록이 실제 배열에 있기 때문에, 당신은 뭔가를 할 수 없습니다 인덱스가 없으면 -1을 반환합니다.

Answer 3

목록은 많은 수의 fu를 제공하므로 플레이어 배열을 List<Player> 대신 사용하는 것이 좋습니다. 당신이 필요로하는 단점. 이 같은 InsertPlayer를 다시 작성하는 경우

당신은 GetInsertIndex을 제거 할 수 :

static int InsertPlayer(int number, string firstName, string lastName, 
    int goals, int assists, List<Player> players) 
{ 
    var index = -1; 
    if (!players.Any(p => p.Number == number)) 
    { 
     var player = new Player(number, firstName, lastName, goals, assists); 
     index = players.FindLastIndex(x => x.Number < number) + 1; 
     players.Insert(index, player); 
    } 
    return index; 
} 

선수 번호가 이미 목록에 존재하고, 그렇지 않으면 올바른 위치에 새로운 선수를 추가하고 다음을 반환하는 경우이 코드는 -1 반환 새로운 색인.

catch (Exception e)을 사용하지 마십시오. 단지 디버깅을 매우 어렵게 만들고 코드를 향상시키지 않습니다.

또한 전염병처럼 ref을 사용하지 마십시오. 그들은 거의 요구되지 않습니다. 당신이 그것을 지나가고 있다는 사실은 큰 숫자의 players 배열을 미리 할당했기 때문에 배열의 크기가 변경되는 것을 피하도록 말합니다. 글쎄 List<Player>은 모든 크기 조정을 처리하며 언제든지 players.Count을 사용할 수 있습니다.