NullpointerExeption при получении элемента из ArrayList

Question

Я просто хочу получить первые несколько элементов из списка. Но когда я делаю это, есть нулевой указатель в строке

returnList.add(returnList1.get(i)) 

Я тестировал returnList1. Он имеет значения.

ArrayList<someObject> returnList1 = new ArrayList<someObject>(); 
ArrayList<someObject> returnList = new ArrayList<someObject>(); 
returnList1=bunruiSummaryService.getFromDB(param); 
int j = 10; 
if(returnList1.size() < j){ 
    j = returnList1.size(); 
} 
for(int i = 0; i<j; i++){ 
    returnList.add(returnList1.get(i)); 
} 

Answer 1

Есть слишком много проблем с этой небольшой кусок кода. Позвольте мне показать вам, как я хотел бы это увидеть:

//somewhere else defined 
private static final int MAX_SIZE_OF_SOME_LIST = 10; //instead of j=10, you simply have to know what the number 10 represents! 

List<SomeObject> resultList = new ArrayList<someObject>(); 
List<SomeObject> returnList = bunruiSummaryService.getFromDB(param); //at this point, make sure to return an empty list instead of null 
int maxSize = MAX_SIZE_OF_SOME_LIST; 
if(returnList.size() < MAX_SIZE_OF_SOME_LIST){ 
    maxSize = returnList.size(); //if you ensured the returned list is not null, if the returned list is empty, you get 0 here 
} 
//even so 
int maxSize = returnList.size() < MAX_SIZE_OF_SOME_LIST ? returnList.size() : MAX_SIZE_OF_SOME_LIST; 

for(int i = 0; i<maxSize; i++){ 
    resultList.add(returnList.get(i)); 
} 

//or you could use subList: 
resultList.addAll(returnList.subList(0, maxSize)); 

Answer 2

Прежде всего, в вашем коде есть много сквозных отверстий.

Далее в NPE,

Либо returnList1=bunruiSummaryService.getFromDB(param); возвращение null

возможность вы не intialised returnList

returnList.add(returnList1.get(i)) 

Answer 3

Try поставить нулевую проверку перед использованием и обрабатывать соответствующим образом:

ArrayList<someObject> returnList1 = new ArrayList<someObject>(); 
    returnList1=bunruiSummaryService.getFromDB(param); 
    if(returnList1 == null) 
    { 
     System.out.println("ERROR: List is null"); 
     //handle as per your need 
    } 

Вам также может потребоваться проверить, как вы инициализировали свой returnList. Я думаю, там есть какая-то проблема.

Answer 4

Вы создаете ArrayList для returnList1, затем вы перезаписываете его данными из базы данных, но вы не создаете ArrayList для returnList. Вы должны заменить:

ArrayList<someObject> returnList1 = new ArrayList<someObject>(); 

с

ArrayList<someObject> returnList = new ArrayList<someObject>();