Ad:

Java Discussion Thread
Unsolved
Views: 1095

Jul 2nd, 2009

null pointer exception error while finding longest word in line

Expand Post »

i am getting run time error as provided in title
my code is

Java Syntax (Toggle Plain Text)
class maxstring
{
public static void main(String args[]) 
{
String s1="nikhil is a good boy";
int d;int x=0;
d=s1.length();
String[] s2=new String[d];
char ch;
for(int i=0;i<=d-1;i++)
{
ch=s1.charAt(i);
if(ch==' ')
{
s2[x]=s1.substring(x,i-1);
x=i+1;
}
}
String max=s2[0];
for(int i=0;i<=d-1;i++)
{
if(s2[i].compareTo(max)>0)
max=new String(s2[i]);
}
System.out.println(max);
}
}
class maxstring
{
public static void main(String args[]) 
{
String s1="nikhil is a good boy";
int d;int x=0;
d=s1.length();
String[] s2=new String[d];
char ch;
for(int i=0;i<=d-1;i++)
{
ch=s1.charAt(i);
if(ch==' ')
{
s2[x]=s1.substring(x,i-1);
x=i+1;
}
}
String max=s2[0];
for(int i=0;i<=d-1;i++)
{
if(s2[i].compareTo(max)>0)
max=new String(s2[i]);
}
System.out.println(max);
}
}

Last edited by Tekmaven; Jul 2nd, 2009 at 9:06 pm. Reason: Code Tags

Similar Threads

Getting a null pointer exception when saving image to file. in Java
Null Pointer Exception problem in Java
Null Pointer Exception....... in Java
Null pointer exception while getting other applet in the same page in Java
null pointer exception --- urgent in Java

akulkarni

Reputation Points: 11

Solved Threads: 4

Junior Poster

Offline

111 posts
since Jun 2009

Permalink

Jul 2nd, 2009

Re: null pointer exception error while finding longest word in line

Hi,
when you create your array you give it the length of all the characters in the string you're analyzing, but you're not initiaing it, so it contains only null (s2[0] = null, s2[1] = null etc) since string is an object. In the last for-loop you're calling method compareTo() for a null object. You can add a null check in the if statement to avoid this problem (if(s2[i] != null && the rest)

di2daer

Reputation Points: 22

Solved Threads: 13

Junior Poster in Training

Offline

66 posts
since Sep 2008

Permalink

Jul 2nd, 2009

Re: null pointer exception error while finding longest word in line

Where are you "getting" it? The exception provides a line number, you know.

Also, why do for(int i=0;i<=d-1;i++) when you can do for(int i=0;i<d;i++).

I believe, however, that you problem is here

Java Syntax (Toggle Plain Text)
String[] s2=new String[d];
String[] s2=new String[d];

Doing that creates an array of reference variables that must point to a String, however, initially all the elements contain a null pointer. You need to make sure that every element actually contains a reference to a String or the line if(s2[i].compareTo(max)>0) will cause an NPE on any element where you have not explicitly initiated a String reference.

You can probably solve it by changing ....

Edit:

Scratch that, I'm not sure what you're trying to do with that "logic" so I'll simply second that "add a null check".

Last edited by masijade; Jul 2nd, 2009 at 3:49 pm.

masijade

Moderator

Reputation Points: 1471

Solved Threads: 490

Industrious Poster

Offline

4,043 posts
since Feb 2006

Permalink

Jul 2nd, 2009

Re: null pointer exception error while finding longest word in line

[code]
i am trying to create an array of strings with String [] s2=new String[25]
i will store every word in that array my line is "nikhil is a good boy"(s1) i am using substring to separate those words removing the spaces and put it into s2 array i want s2[0]=nikhil
s2[1]=is s2[2]=a s2[3]=good s[4]=boy i wish to print the longest word in the string s1
please help

akulkarni

Reputation Points: 11

Solved Threads: 4

Junior Poster

Offline

111 posts
since Jun 2009

Permalink

Jul 2nd, 2009

Re: null pointer exception error while finding longest word in line

Java Syntax (Toggle Plain Text)
 i commented the tocompare part and changed max=new s2...line in it to max=s2 but same error.It is not printing the words separately remooving spaces. i got the program with stringtokenizer but i want to create it on my own
 i commented the tocompare part and changed max=new s2...line in it to max=s2 but same error.It is not printing the words separately remooving spaces. i got the program with stringtokenizer but i want to create it on my own

akulkarni

Reputation Points: 11

Solved Threads: 4

Junior Poster

Offline

111 posts
since Jun 2009

Permalink

Jul 3rd, 2009

Re: null pointer exception error while finding longest word in line

akulkarni,

14th post and you don't know, how to and where to use code tag?
Read this How to use code tags.

Try to include a complete description of your problem at your first post of this thread.

What masijade said:

Quote ...

Scratch that, I'm not sure what you're trying to do with that "logic" so I'll simply second that "add a null check".

Here is my suggestion :
Use split() method of String class.

Java Syntax (Toggle Plain Text)
    String s1="nikhil is a good boy";
   String []p=s1.split(" ");
    String s1="nikhil is a good boy";
   String []p=s1.split(" ");

adatapost

Moderator

Reputation Points: 2136

Solved Threads: 1228

Posting Genius

Offline

6,527 posts
since Oct 2008

Permalink

Jul 3rd, 2009

Re: null pointer exception error while finding longest word in line

Click to Expand / Collapse Quote originally posted by akulkarni ...

[code]
i am trying to create an array of strings with String [] s2=new String[25]
i will store every word in that array my line is "nikhil is a good boy"(s1) i am using substring to separate those words removing the spaces and put it into s2 array i want s2[0]=nikhil
s2[1]=is s2[2]=a s2[3]=good s[4]=boy i wish to print the longest word in the string s1
please help

Okay, so let's go through a bit of what you are actually doing.

Java Syntax (Toggle Plain Text)
class maxstring {
    public static void main(String args[])  {
        String s1="nikhil is a good boy";
        int d;int x=0;
        d=s1.length();
        /* With this next statement you now have the following array
         * 0 --> null
         * 1 --> null
         * 2 --> null
         * ...
         * 17 --> null
         * 18 --> null
         * 19 --> null
         */
        String[] s2=new String[d];
        char ch;
        for(int i=0;i<=d-1;i++) {
            ch=s1.charAt(i);
            if(ch==' ') {
                s2[x]=s1.substring(x,i-1);
                x=i+1;  // x is 0,7,10,12,17
            }
        }
        /* all elements, other than the following are still null
         * 0 --> nikhil
         * 7 --> is
         * 10 --> a
         * 12 --> good
         * 17 --> boy
         */
        String max=s2[0];
        /* in the following loop all elements except those listed above
         * will throw an NPE, since those elements are null.
         */
        for(int i=0;i<=d-1;i++) {
            /* This statement compares the Strings alphabetically
             * I thought you wanted the longest String?  Shouldn't you
             * be comparing their "length()"s?
             */
            if(s2[i].compareTo(max)>0)
                max=new String(s2[i]);
        }
        System.out.println(max);
    }
}
class maxstring {
    public static void main(String args[])  {
        String s1="nikhil is a good boy";
        int d;int x=0;
        d=s1.length();
        /* With this next statement you now have the following array
         * 0 --> null
         * 1 --> null
         * 2 --> null
         * ...
         * 17 --> null
         * 18 --> null
         * 19 --> null
         */
        String[] s2=new String[d];
        char ch;
        for(int i=0;i<=d-1;i++) {
            ch=s1.charAt(i);
            if(ch==' ') {
                s2[x]=s1.substring(x,i-1);
                x=i+1;  // x is 0,7,10,12,17
            }
        }
        /* all elements, other than the following are still null
         * 0 --> nikhil
         * 7 --> is
         * 10 --> a
         * 12 --> good
         * 17 --> boy
         */
        String max=s2[0];
        /* in the following loop all elements except those listed above
         * will throw an NPE, since those elements are null.
         */
        for(int i=0;i<=d-1;i++) {
            /* This statement compares the Strings alphabetically
             * I thought you wanted the longest String?  Shouldn't you
             * be comparing their "length()"s?
             */
            if(s2[i].compareTo(max)>0)
                max=new String(s2[i]);
        }
        System.out.println(max);
    }
}

A split() will give you an array of Strings, but that is not exactly "doing it yourself" as you said you wanted to. If you really want to parse the String yourself to create an array of Strings, then either use an ArrayList or handle your index better, or simply add the null check (which is the least effecient variant), or simply get the lengths of the Strings as you "find" them and compare them to the longest up to that point.

Last edited by masijade; Jul 3rd, 2009 at 3:30 am.

masijade

Moderator

Reputation Points: 1471

Solved Threads: 490

Industrious Poster

Offline

4,043 posts
since Feb 2006

Permalink

Jul 4th, 2009

Re: null pointer exception error while finding longest word in line

Java Syntax (Toggle Plain Text)
 
 
Dear Masijade
i tried with finding length and then comparing but found it impossible. can u help me about that.Also how do i do the Null check .thanks for your explaination i will give more time to think over it cause right now i am not able to visualise that NPE error falling short of grasping the concept.

Dear Masijade
i tried with finding length and then comparing but found it impossible. can u help me about that.Also how do i do the Null check .thanks for your explaination i will give more time to think over it cause right now i am not able to visualise that NPE error falling short of grasping the concept.

akulkarni

Reputation Points: 11

Solved Threads: 4

Junior Poster

Offline

111 posts
since Jun 2009

This thread is more than three months old

No one has posted to this discussion for at least three months. Please let old threads die and do not reply to them unless you feel you have something new and valuable to contribute that absolutely must be added to make the discussion complete. Otherwise, please start a new thread in this forum instead.

Previous Thread in Java Forum Timeline: Need help with character movement on slanted surfaces

Next Thread in Java Forum Timeline: doubt on object reference

DaniWeb Message

Cancel Changes

User Name
Password