2012-03-29 1 views
0

Недавно я опубликовал вопрос о получении NullPointerException всякий раз, когда я вызывал массив объектов. Я проследил проблему до некоторого отключения между основным методом, предоставляющим данные и метод (Team.sortPlayers()), получающий данные.Данные не передаются должным образом между классами; результаты в NullPointerException

public class Project3 { 


public static void main(String[] args) { 
Input3 input = new Input3(); 
Team teams[] = new Team[input.NUMBER_OF_TEAMS]; 
Player players[] = new Player[input.NUMBER_OF_PLAYERS]; 
String playas[] = new String[input.NUMBER_OF_PLAYERS]; 
String temp; 
String name; 

for (int i=0 ; i<input.NUMBER_OF_TEAMS ; i++) { 
    name = input.getNextString(); 
    System.out.println(name); 
    for (int j=0 ; j<input.NUMBER_OF_PLAYERS ; j++) 
    {playas[j] = input.getNextString(); 
     System.out.println(playas[j]);} 
    teams[i] = new Team(name, playas); //THIS LINE SENDS OVER THE DATA TO THE QUESTIONABLE METHOD 
    teams[i].sortPlayers(); 
    System.out.println(teams[i]); 
    } 
} 
} 

//------------------------------ 
// 
//------------------------------ 

class Player { 

public String[] name; 

public Player(String inputname) { 

    name = inputname.split(" "); 

} 
public String[] getName() { 
    return name; 
} 

public String getFirstName() { 
    return name[0]; 
} 

public String getLastName() { 
    String last = name[1]; 
    return last; 
} 
} 

//----------------------- 
// 
//----------------------- 

class Team { 
private String teamname; 
public Player players[]; 
public Player temp; 

public Team(String inputname, String plays[]) { //THIS METHOD RECEIVES A NULL FOR 'INPUTNAME' AND WHAT APPEARS TO BE A JIBBERISH (ex: [email protected], maybe a memory address?) FOR 'PLAYS[]' 

    inputname = teamname; 
    System.out.println(teamname);  
    players = new Player [plays.length]; 
    for(int k=0 ; k<plays.length ; k++) 
    { System.out.println(inputname); 
     this.players[k] = new Player(plays[k]); 
    System.out.println(players[k]); 
    } 

} 

public void sortPlayers() { 
    int n = players.length; 

    for (int pass=1; pass < n; pass++){ 
     for (int i=0; i < n-pass; i++) { 
      String playerName = players[i].getLastName(); 
      String nextPlayerName = players[i+1].getLastName(); 
      if(playerName.compareTo(nextPlayerName) > 0) 
      temp = players[i]; 
     players[i] = players[i+1]; 
     players[i+1] = temp; 
     } 
    } 
} 
} 

Если бы кто-нибудь мог помочь мне понять, что здесь происходит, я был бы очень благодарен! Я отметил две проблемные заявления с комментариями и Pastebin все это можно найти ниже: http://pastebin.com/QGALKbP6

+0

Пожалуйста, разместите stacktrace. – Mohayemin

ответ

2

«Темп» не должен быть переменной членом, он должен быть локальным к, если блок в методе сортировки , и, поскольку, если его расширенная область действия, вы непреднамеренно очищаете некоторые члены вашего игрового массива из-за того, что в вашем блоке «if» отсутствуют некоторые фигурные скобки.

 if(playerName.compareTo(nextPlayerName) > 0) { 
      Player temp = players[i]; 
      players[i] = players[i+1]; 
      players[i+1] = temp; 
     } 

в целом, даже если скобки не являются обязательными в некоторых случаях, вы должны использовать их все время, чтобы избежать тонких ошибок, как это.

+1

Кроме того, у вас есть поле «String» с именем «temp» и поле «Player» с именем «temp». Хотя это ТЕХНОЛОГИЯ приемлемо, это плохая форма. – gobernador

+0

Я чувствую себя настолько глупо, что не замечаю этого ... спасибо, добрый сэр! –

 Смежные вопросы

  • Нет связанных вопросов^_^