Exception de pointeur Null, “Tentative de lecture de champ à partir d'une référence d'objet nul”

Je fais une application dans laquelle l'utilisateur tape dans une liste de tâches, et que la liste est enregistrée dans un tableau. Chaque tâche dans le tableau est une instance de la Assignment classe. Cependant, j'ai réalisé qu'en java il n'est pas possible d'ajouter un élément à un tableau d'après le tableau est créé. Donc, ce que j'ai fait, j'ai créé un tableau appelé tasks qui se composait de beaucoup de valeurs nulles: Assignment[]tasks = {null, null, null, null, null, null, null, null, null, null, null, null};. Quand je veux ajouter une tâche au tableau, je viens de remplacer la prochaine valeur null avec l'objet. Cependant, j'ai aussi besoin d'avoir un tableau de simplement des tâches, avec pas de valeurs null. J'ai donc créé un tableau appelé full_tasks pour tous les éléments qui ne sont pas nulles:

for (Assignment task: tasks) {
    if (task != null) {
        realLength += 1;
    }
}

Assignment[] full_tasks = new Assignment[realLength];

for (int i=0; i <= full_tasks.length - 1; i++) {
        full_tasks[i] = new Assignment(tasks[i].name, tasks[i].days_due, tasks[i].time);
}

Alors maintenant, la full_tasks tableau doit être un tableau de toutes les tâches, qui sont nulles, non? Cependant, lorsque je lance l'application, il ne peut pas lancer l'activité, une erreur qu'il dit est causée par une exception de pointeur null:

 Caused by: java.lang.NullPointerException: Attempt to read from field 'java.lang.String com.example.lb.homeworkappv11.Assignment.name' on a null object reference
        at com.example.lb.homeworkappv11.Schedule.sortTasks(Schedule.java:64)

La ligne que l'erreur est:

full_tasks[i] = new Assignment(tasks[i].name, tasks[i].days_due, tasks[i].time);

Je ne suis pas encore totalement sûr de ce qu'un objet nul de référence est, mais je pense que cela signifie que l'un des éléments dans la full_tasks tableau est null. Serait-ce correct? Et si c'est ce que je peux faire pour vous assurer que le full_tasks tableau est seulement la non-null éléments dans le tasks tableau?

Merci beaucoup!

Edit: la fonction constructeur de la classe d'assignation est:

public Assignment(String name, int days, int time) {
    this.name = name;
    this.days_due = days;
    this.time = time;
    this.toSortBy = "nothing";
}
Votre "nom" variable à l'intérieur de la Cession est nulle. Pouvez-vous poster le code du constructeur, vous utilisez?
J'ai ajouté le code ci-dessus
Vérifier ma réponse.

OriginalL'auteur Lucas B | 2015-08-29

  1. 2

    Un null de référence est juste que null. Dans votre code, il est tasks[i].name où vous essayez d'appeler name sur tasks[i] donc tasks[i] est null.

    Il existe un scénario que je pense, où votre code serait certainement jeter un NullPointerException.
    Donc, je vais assumer vos tâches tableau peut regarder ressemble à ceci:

    tasks = [task0, null, task2, task3, null, task5]

    Puis full_tasks aura une taille de 4, mais

    for (int i=0; i <= full_tasks.length - 1; i++) {
        full_tasks[i] = new Assignment(tasks[i].name, tasks[i].days_due, tasks[i].time);
}

    va jeter un NPE dès que i == 1 parce que tasks[1] est null.

    Donc, si vous voulez remplir full_tasks avec seulement non-null tâches assurez-vous que vous avez obtenu le droit indices de tasks.

    Merci beaucoup! Cette réponse a été très utile. J'ai changé mon pour la boucle de sorte qu'il ne prend que la non-null éléments, et maintenant ça fonctionne!

    OriginalL'auteur Sascha Kolberg

  2. 1

    Voici ce que je pense

    Vous êtes juste de trouver le nombre d'éléments qui ne sont pas null. Vous ne savez pas où dans le tableau thest null's sont présents.

    Supposons que SEUL le premier élément est null. Donc realLength sera de 7. Le dernier for boucle s'exécute de i=0 à i=7. Lorsque i=0, tasks[i].name tente d'accéder à la name champ du premier élément; mais votre premier élément se trouve être null`. C'est là que les choses vont mal.

    Solution:

    Il y a un certain nombre de solutions. La plus efficace, je pense, utilise un ArrayList.

    De contourner à l'aide de tableaux, vous devez stocker les indices de tous les éléments qui ne sont pas null. C'est une façon.

    Voici un autre: 

    for (Assignment task: tasks) {
  if (task != null) {
    realLength += 1;
  }
}
Assignment[] full_tasks = new Assignment[realLength];
int count = 0;
for (Assignment task: tasks) {
  if (task != null) {
    full_tasks[count] = new Assignment(task.name, tasks.days_due, task.time);
    count++;
  }
}

    OriginalL'auteur Saud

  3. 0

    Vous pouvez penser à null en tant que quelque chose n'existe pas, par exemple, lorsque vous essayez d'obtenir task.name, où la tâche est null, vous obtenez l'erreur, parce que vous essayez d'obtenir le nom de quelque chose qui n'existe même.

    Réellement ce que votre code est en train de faire maintenant est de vérifier combien de non-nulle d'éléments dans le tableau d'origine, et à essayer d'extraire le supplément n premiers éléments du tableau à un autre tableau.

    c'est à dire si la liste est {null, null, non-null, non-null}, il dispose de 2 non-null article, mais votre code, à tort, des extraits de la liste de la première 2 éléments qui sont {null, null}

    Modifier, de le faire réellement ce que vous voulez:

    ArrayList<Assignment> fullTaskList = new ArrayList<Assignment>();
for (Assignment task: tasks) {
    if (task != null) {
        fullTaskList.add(task);
    }
}
//optional
Assignment[] fullTasks = fullTaskList.toArray(new Assignment[fullTaskList.size()]);

    OriginalL'auteur Derek Fung

  4. 0

    Le problème dans votre code est ici:

    (tasks[i].name, tasks[i].days_due, tasks[i].time).

    Bien que vous faites le décompte de la taille réelle, il pourrait être une valeur null dans entre qui est à l'origine pour être obtenir un objet nul à partir de la liste des tâches[i].

    Une bonne idée pour résoudre ce est en utilisant une Liste au lieu d'un Tableau. Une Liste peut être augmenté ou diminué comme vous le souhaitez, il vous suffit d'ajouter ou de supprimer un élément de votre type d'objet. Par exemple:

    List<Assignment> list = new ArrayList<>();
Assignment assignment1 = new Assignment(etc.);
list.add(assignment1);
list.get(0) //-> returns the Assignment object that you added.

    Ensuite, vous pouvez utiliser list.size() pour obtenir la taille de savoir combien de points sont là, ou à enlever le dernier élément. Et vous n'aurais pas de problèmes pour ajouter de nouveaux éléments à la liste.

    OriginalL'auteur George

  5. 0

    Il semble que vous essayez de copier et compact, le tasks tableau dans la full_tasks tableau (en supprimant les éléments null), mais tout ceci est partiellement erronée, car vous accédez à tasks[i] dans la deuxième boucle sans vérifier si sa valeur est null.

    Au lieu de:

    for (int i=0; i <= full_tasks.length - 1; i++) {
        full_tasks[i] = new Assignment(tasks[i].name, tasks[i].days_due, tasks[i].time);
}

    vous pouvez écrire quelque chose comme ceci:

    for (int i = 0, f = 0; i < tasks.length; i++) {
    if (tasks[i] != null) {
        full_tasks[f] = new Assignment(tasks[i].name, tasks[i].days_due, tasks[i].time);
        f++;
    }
}

    Mais là encore, pour résoudre votre problème d'origine, concernant le fait que vous ne pouvez pas ajouter des éléments à un tableau, je suggère d'utiliser un ArrayList au lieu d'un simple tableau. Cela vous permet d'ajouter des éléments avec ArrayList.add(assignment) et retirez-les de nouveau avec ArrayList.remove(index).

    OriginalL'auteur Floern

  6. 0

    Semble que les "tâches" de la liste est vide. Assurez-vous que le remplis ou mettre un null vérifier
    comme :

    if(tasks[i] !=null) {

full_tasks[i] = new Assignment(tasks[i].name, tasks[i].days_due,  tasks[i].time);

 }

    OriginalL'auteur Rehman