Java Why do I get this error? Exception in thread "main" java.lang.NullPointerException

Question

This is the code that it is giving me the error for on the line with >>>>>>> I've already looked at this thread Exceptions but I still do not understand what I need to change :( I am a total beginner to programming.

import java.io.FileInputStream;
import java.io.FileNotFoundException;
import java.util.Scanner;

public class manyVowels {

  public static final String wordList = "words.txt";

  public static void main(String[] args) {
        Scanner fileIn = null;
        try {
            //locate and open file
            fileIn = new Scanner(new FileInputStream("words.txt"));
        } catch (FileNotFoundException e) {
            //if the file cannot be found, the program prints the message and quits
            System.out.println("File not found. ");
            System.exit(0);
        }
        String word = null;
        if (fileIn.hasNext()) //if there is another word continue
        {

            String finalWord = null; // defines the word with most consecutive vowels
            int maxVowels = 0;//sets initial value to 0
            while (fileIn.hasNext()) {
                // for each word in the file
                int vowels = 0;
/*error here-->*/ for (int i = 0; i < word.length() && i < word.length() - maxVowels + vowels; i++) {
                    // for each character in the word, and exit early if the word is not long enough to beat maxVowels
                    if (hasVowels(word.charAt(i))) {
                        // consonants reset this to 0
                        vowels++;
                    } else {
                        // reached the end of the word so check if the count is higher than maxVowels
                        if (vowels > maxVowels) {
                            maxVowels = vowels;
                            finalWord = word;
                        }
                        vowels = 0;
                    }
                }
                // comparing vowels to maxVowels
                if (vowels > maxVowels) {
                    maxVowels = vowels;
                    finalWord = word;
                }
            }

            //seek vowels
            word = fileIn.next();
            for (int i = 0; i < word.length(); i++) {
                if
                        ((word.charAt(i) == 'A')
                        || (word.charAt(i) == 'E')
                        || (word.charAt(i) == 'I')
                        || (word.charAt(i) == 'O')
                        || (word.charAt(i) == 'U')
                        || (word.charAt(i) == 'a')
                        || (word.charAt(i) == 'e')
                        || (word.charAt(i) == 'i')
                        || (word.charAt(i) == 'o')
                        || (word.charAt(i) == 'u')) {
                    //prints the final word with the most consecutive vowels
                    System.out.println("The word with the most consecutive vowels is: " + word);
                    System.exit(0);
                }

            }
        }
    }

    private static boolean hasVowels(char charAt) {
        throw new UnsupportedOperationException("Inserted by template."); //NetBeans generated method
    }
}

See stackoverflow.com/questions/218384/… ; you have String word = null; and then later: word.length() - which means you are calling the length() method on a variable that has a null value. — Brad Peabody
– Brad Peabody, Commented Sep 20, 2014 at 1:59
When you get the exception there is a stack trace that tells you exactly where the exception is. That statement has a null pointer somewhere in it. This is an incredibly easy problem to debug. — Hot Licks
– Hot Licks, Commented Sep 20, 2014 at 2:07
thank you. I've looked at this already, but since it was a little different from what I am trying to accomplish (reading words from a file) I am finding it difficult to directly apply the corrections to my file. I am confused :( — spoilmealwayz
– spoilmealwayz, Commented Sep 20, 2014 at 2:07
@HotLicks it showed me where the problem lies. It is because I asked for length after setting the word value to null. the problem is, I need the file to read words from a text file, so I'm not sure what value to put on "word." I just tried "" but the program does not stop running — spoilmealwayz
– spoilmealwayz, Commented Sep 20, 2014 at 2:09
Hey are you in the same class as the person in this question? stackoverflow.com/questions/25858770/…. I gave a full code response, maybe take a look at it. Edit: upon looking further (esp the comments) you are using my code — nmore
– nmore, Commented Sep 20, 2014 at 4:14

salezica · Accepted Answer · 2014-09-20 01:58:58Z

1

Following the logic, you initialize the String word to null, and then proceed to call word.length().

word, being null, has no length(). Thus, the NullPointerException.

Assign a string to word before attempting to measure its length.

String word = "Hello!";

answered Sep 20, 2014 at 1:58

salezica

77.5k29 gold badges113 silver badges168 bronze badges

Sign up to request clarification or add additional context in comments.

3 Comments

spoilmealwayz Over a year ago

Now i understand...the reason why I set it to null was because it will be reading a list of words from a file, so I could not assign it a specific word :-/ Instead, I took out null and made it into "" but the program is still running...not sure.

salezica Over a year ago

You should rethink the flow so that by the time word.length() is called, word already has a meaningful value. Read from the file first, ask questions later

MarGar Over a year ago

Assign something to it before you enter your for loop?

Eduardo Javier Huerta Yero · Accepted Answer · 2014-09-20 02:04:00Z

1

The variable word seems to be null, I think you skipped an instruction in which you fill it with a word read from fileIn. Your code should read something like this:

     while (fileIn.hasNext()) {
            // for each word in the file
            word = fileIn.next();
            int vowels = 0;
            for (int i = 0; i < word.length() && i < word.length() - maxVowels + vowels; i++) {
               ...

answered Sep 20, 2014 at 2:04

Eduardo Javier Huerta Yero

3234 silver badges11 bronze badges

Comments

Moni · Accepted Answer · 2014-09-20 04:09:48Z

1

you defined word as a null. When you say word.length() it means you are saying null.length() that's why you are getting null pointer exception.

You should initialize String variable "word" before doing any operation( calling any string method by using '.' )

If you have any predefined value, initialize with that else initialize with empty string.

String word = "xyz" ; String word = "" ;

answered Sep 20, 2014 at 4:09

Moni

4333 silver badges9 bronze badges

Collectives™ on Stack Overflow

Java Why do I get this error? Exception in thread "main" java.lang.NullPointerException

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Linked

Hot Network Questions

Collectives™ on Stack Overflow

3 Answers 3

3 Comments

Comments

Comments

Your Answer

Sign up or log in

Post as a guest

Linked

Related