Can't find java.lang.NullPointerException

Question

0.00/5 (No votes)

See more:

I have coded this program

This is sparse matrix class

Java

public class MatrixTerm {
 public int row,col,value;   
}

class SparsMatrix {
  private int Row,Col,Term;
 public  MatrixTerm SmArray[];
  int count=0;
  
  
  public SparsMatrix(int r,int c,int t){
      Row=r;
      Col=c;
      Term=t;
       SmArray =new MatrixTerm [20];
        }
  
  
  
  public void ReadSparse(){
        Scanner input= new Scanner(System.in);
      System.out.println("lotfan satr va  soton matrix vorodi ra vared konid ");
      Row=input.nextInt();
      Col=input.nextInt();
        int[][] matrix = new int [Row][Col];    
               for (int i=0;i<row;++i){>
                   for(int j=0;j<col;++j){>
                  System.out.println("lotfan "+"("+(i+1)+","+(j+1)+")"+" ra vared konid");     
                   matrix[i][j]=input.nextInt();
                   }}
               
              
                for (int i=0;i<Row;++i){>
                   for(int j=0;j<Col;++j){>
                       if(matrix[i][j]!=0){
                 SmArray[i].row=i;/// <big>HERE SAY java.lang.NullPointerException</big>
                 SmArray[i].col=j;
                 SmArray[i].value=matrix[i][j];
                           ++count;
                       }
                       
                   }
               }
  }
               
      public void show(){
          for(int i=0;i<count;++i){>
              System.out.println(SmArray[i]);
          }
    
}
               
               
  }

when i test the program in line 36 it say java.lang.NullPointerException
plz help me

Posted 31-Oct-11 2:08am

DominoBoy

Updated 31-Oct-11 7:42am

Chris Maunder774.3K

v2

Add a Solution

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

E.F. Nijboer · Accepted Answer · 2011-10-31T02:26:00

Check the value of i with the debugger. The Row value is dynamic but the declaration of the SmArray is hardcoded 20. SmArray =new MatrixTerm [20];

First of all, SparsMatrix is never called so the array isn't initialized. Second, you might consider using a list instead of an array because a list will expand as needed and therefor more efficient.

Add these imports:

C#

import java.util.Arrays;
import java.util.List;
import java.util.Iterator;

Change the array declaration to a generic list:

List smArray<matrixterm> = new ArrayList<matrixterm>();</matrixterm></matrixterm>

Change the MatrixTerm to add a constructor:

Java

public class MatrixTerm {
  public int row,col,value;
 
  public MatrixTerm(int row, int col, int value) {
    this.row = row;
    this.col = col;
    this.value = value;
  }
}

Add values to smArray like this:

smArray.add(new MatrixTerm(row, col, value));

Good luck!