java我的代码中出现了一个空指针异常，因为hashmap没有填充，或者可能没有填充

Question

我得到了这个练习： 实现以下类，该类加载并打印一组数据值

import java.util.Iterator;
public class MyLoader {
    public void loadAndPrintValues(Iterator<String> keysToLoad, Data data, Printer printer) {

        // Load data values like this:
        // String value = data.loadValue(key);

        // Print loaded data value like this:
        // printer.printEntry(key, value);
    }
}

然而，当我做这个练习时，我得到了一个NullPointerException，可能来自while (keysToLoad.hasNext())或key = keysToLoad.next();。我想我得到这个异常是因为“数据”没有被填充，但我不知道怎么做。这是我的代码和错误消息

public interface Data{
    //I made this method
     public void makeEntry(String key, String value);
    //given
     public String loadValue(String key);
}

public interface Printer {
     public void displayEntry(String key, String value);
}

import java.util.Iterator;
import java.util.HashMap;

public class MyLoader implements Data, Printer {
    Data data; // = new MyLoader();
    Printer printer; // = new MyLoader();
    Iterator<String> iter; // = new MyLoader();

      String key = "";
String value = "";

HashMap<String, String> ht = new HashMap<String, String>();

public MyLoader(){
//  this.database = null;
//  this.key = null;
//  this.value = null;
//  this.ht = null;
//  this.iter = null;
    System.out.println("now in the constructor");
}

public MyLoader(Iterator<String> iter, Data data, Printer printer){
    this.data = data;
    this.printer = printer;
    this.iter = iter;
}

public void loadAndPrintValues(Iterator<String> keysToLoad, Data data, Printer printer) {
    try {
        if (ht.isEmpty()){
            System.out.println("ht is empty");
            throw new NullPointerException("Database is empty."); 
        }else {
            this.data = data;
        }
           while (keysToLoad.hasNext()){               
           // Load data values like this:
           key = keysToLoad.next();
           value = data.loadValue(key);
           // Print loaded data value like this:
           printer.printEntry(key, value);
       }
    }catch (NullPointerException npe){
        System.out.println("caught null pointer ");
        System.out.println(npe.getMessage());
    }
}

@Override
public void makeEntry(String key, String value){
    ht.put(key, value);
}

@Override
public void printEntry(String key, String value) {  
        System.out.println("[" + key + " : " + value + "]");
}

@Override
public String loadValue(String key) {
    System.out.println("loadValue:" + key);
    if(this.ht.containsKey(key))            
        return this.ht.get(key);
    else {
        System.out.println("No key in database.");
        throw new NullPointerException("No key in data."); 
    }
}

}

public class test {
public static void main(String[] args) {
    try {
    MyLoader mdbl = new MyLoader();

    mdbl.makeEntry("0", "zero");
    mdbl.makeEntry("1", "One");
    mdbl.makeEntry("2", "Two");
    mdbl.makeEntry("3", "Three");

    mdbl.loadAndPrintValues(mdbl.iter, mdbl.data, mdbl.printer);

    }
    catch(NullPointerException e){
        System.out.println(e.getMessage());
    }
}
}

Answer 1

永远不要捕捉NullPointerException。移除带有NullPointerException的catch块，运行它，然后发布堆栈跟踪

Answer 2

您正在为方法的迭代器、数据和打印机参数传递空值loadAndPrintValues（），这将导致NullPointerException

Answer 3

在这一行 mdbl。加载和打印值（mdbl.iter、mdbl.data、mdbl.printer）； 你没有初始化mdbl。数据和mdbl。打印机

Answer 4

mdbl.iter永远不会被赋值，即是null（默认情况下，因为您使用的MyLoader构造函数没有参数，其中iter没有赋值）。因此，当您将其传递给试图对其执行操作的方法时，自然会得到一个NullPointerException（不是错误）

你不应该

try{ /* ... */ } catch (NullPointerException npe){ /* ... */ }

阻塞，因为任何一个像样的IDE都允许你点击一次就到达抛出异常的那一行，如果你捕捉到异常并简单地将一条消息打印到System.out，情况就不是这样了