Почему в этом Java-коде есть исключение нулевого указателя? - PullRequest
1 голос
/ 20 марта 2011

Этот код принимает пользователей и фильмы из двух отдельных файлов и вычисляет оценку пользователя для фильма.Когда я запускаю код, я получаю следующую ошибку:

Exception in thread "main" java.lang.NullPointerException
    at RecommenderSystem.makeRecommendation(RecommenderSystem.java:75)
    at RecommenderSystem.main(RecommenderSystem.java:24)

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

import java.io.*;
import java.lang.Math;

public class RecommenderSystem
{
    private Movie[] m_movies;
    private User[] m_users;

    /** Parse the movies and users files, and then run queries against them.
     */
    public static void main(String[] argv)
                       throws FileNotFoundException, ParseError, RecommendationError
    {
        FileReader movies_fr = new FileReader("C:\\workspace\\Recommender\\src\\IMDBTop10.txt");
        FileReader users_fr = new FileReader("C:\\workspace\\Recommender\\src\\IMDBTop10-users.txt");
        MovieParser mp = new MovieParser(movies_fr);
        UserParser up = new UserParser(users_fr);

        Movie[] movies = mp.getMovies();
        User[] users = up.getUsers();

        RecommenderSystem rs = new RecommenderSystem(movies, users);
        System.out.println("Alice would rate \"The Shawshank Redemption\" with at least a "
                           + rs.makeRecommendation("The Shawshank Redemption", "asmith"));
        System.out.println("Carol would rate \"The Dark Knight\" with at least a "
                           + rs.makeRecommendation("The Dark Knight", "cd0"));
    }

    /** Instantiate a recommender system.
     *
     * @param movies    An array of Movie that will be copied into m_movies.
     * @param users     An array of User that will be copied into m_users.
     */
    public RecommenderSystem(Movie[] movies, User[] users)
           throws RecommendationError
    {
        m_movies = movies;
        m_users = users;
    }

    /** Suggest what the user with "username" would rate "movieTitle".
     *
     * @param movieTitle    The movie for which a recommendation is made.
     * @param username      The user for whom the recommendation is made.
     */
    public double makeRecommendation(String movieTitle, String username)
                  throws RecommendationError
    {
        int userNumber;
        int movieNumber;
        int j=0;
        double weightAvNum =0;
        double weightAvDen=0;

        for (userNumber = 0; userNumber < m_users.length; ++userNumber)
        {
            if (m_users[userNumber].getUsername().equals(username))
            {
                break;
            }
        }

        for (movieNumber = 0; movieNumber < m_movies.length; ++movieNumber)
        {
            if (m_movies[movieNumber].getTitle().equals(movieTitle))
            {
                break;
            }
        }

        // Use the weighted average algorithm here (don't forget to check for
        // errors).
        while(j<m_users.length){
            if(j!=userNumber){
            weightAvNum = weightAvNum + (m_users[j].getRating(movieNumber)- m_users[j].getAverageRating())*(m_users[userNumber].similarityTo(m_users[j]));
            weightAvDen = weightAvDen + (m_users[userNumber].similarityTo(m_users[j]));
            }
            j++;
        }
        return (m_users[userNumber].getAverageRating()+ (weightAvNum/weightAvDen));

    }
}

class RecommendationError extends Exception
{
    /** An error for when something goes wrong in the recommendation process.
     *
     * @param s     A string describing the error.
     */
    public RecommendationError(String s)
    {
        super(s);
    }
}

1 Ответ

1 голос
/ 20 марта 2011

Если файл, который вы разместили, не изменился из файла, который сгенерировал трассировку стека, которую вы разместили, то исключение нулевого указателя в строке 75 находится где-то в этом коде:

weightAvNum = weightAvNum + (m_users[j].getRating(movieNumber)- m_users[j].getAverageRating())*(m_users[userNumber].similarityTo(m_users[j]));

Таким образом, поскольку m_users не равно нулю (в противном случае он бы потерпел крах раньше), либо m_users[j], либо m_users[userNumber] равно нулю, то есть в массиве m_users есть некоторый нулевой элемент.

Добро пожаловать на сайт PullRequest, где вы можете задавать вопросы и получать ответы от других членов сообщества.
...