java动态选择要执行的类

Question

我很确定这对专家来说是非常基础的东西，但对我来说，作为一个新手，这让我很难过。

我有 3 个解析器，每个都有自己的功能，将来还会有更多的解析器。现在我想要做的是:我希望我的应用程序在运行时根据要出现的页面选择正确的解析器。

为此，我做了以下工作:我有一个接口(interface) (IWebParser):

public interface IWebParser {
    public abstract Object execute(String page, URL url);
    public abstract List<SimpleWebPosting> parse(String page, URL url, List<String> tokens);

    public abstract Boolean canExecute(URL url);
}

我的每个解析器都实现了这个接口(interface)。我有另一个名为 ParserControl 的类，其中有一个方法 submit(String page, URL url) - 这是我的程序总是调用的方法，只要有一个页面要解析。此类 ParserControl 从 xml 文件获取可用的解析器，并尝试(在 while 语句中)是否有任何解析器可以解析有问题的页面。这是通过 canExecute(URL url) 方法完成的。现在，在 canExecute 上收到 true 后，我想执行那个特定的解析器。

我的类 ParserControl 如下所示:

public class ParserControl {
    private static final Logger logger = Logger.getLogger("de.comlineag.snc.parser.ParserControl");
// the list of operational web parser as taken from the properties file is stored within this structure
private static List<IWebParser> webParser; 
// the ParserControl instance - used during instantiation of the class and later to retrieve the list 
private static ParserControl pc = null;

// ParserControl is not to be directly instantiated by other classes
private ParserControl() {
    try {
        webParser = getAllParser();
    } catch (XPathExpressionException | IOException
            | ParserConfigurationException | SAXException e) {
        logger.error("EXCEPTION :: error during parser execution " + e.getMessage());
        e.printStackTrace();
    }
};

// Static 'instance' method - this method is called every time
// the submit method is called but can also be called implicitely to get
// an instance of ParserControl
public static ParserControl getInstance() throws XPathExpressionException, ParserConfigurationException, SAXException, IOException {
    if (pc == null) {pc = new ParserControl();}
    return pc;
}


public static List<SimpleWebPosting> submit(String page, URL url, ArrayList<String> tTerms) throws XPathExpressionException, ParserConfigurationException, SAXException, IOException{
    logger.trace("ParserControl called");
    pc = getInstance();

    while (pc.webParser.iterator().hasNext()) {
        logger.trace("trying parser " + pc.webParser.iterator().getClass().getSimpleName().toString());
        if (((IWebParser) pc.webParser.iterator().getClass().getClassLoader()).canExecute(url)) {
            return ((IWebParser) pc.webParser.iterator().getClass().getClassLoader()).parse(page, url, tTerms);
        } else {
            logger.trace("parser " + pc.webParser.iterator().getClass().getSimpleName().toString() + " returned false to canExecute()" );
        }
    }

    return null;
}


// retrieves all configured parser from the properties file and creates the parser list 
@SuppressWarnings("unchecked")
private <T> ArrayList<T> getAllParser() throws IOException, ParserConfigurationException, SAXException, XPathExpressionException {
    String fileName = "webapp/WEB-INF/properties/webparser.xml";
    ArrayList<T> ar = new ArrayList<T>();

    File file = new File(fileName);
    DocumentBuilderFactory dbf = DocumentBuilderFactory.newInstance();
    DocumentBuilder db = dbf.newDocumentBuilder();
    Document doc = db.parse(file);

    XPathFactory xPathfactory = XPathFactory.newInstance();
    XPath xpath = xPathfactory.newXPath();

    String expression = "//parser[@type='webparser']/value";
    NodeList nodeList= (NodeList) xpath.compile(expression).evaluate(doc, XPathConstants.NODESET);
    for (int i = 0 ; i < nodeList.getLength() ; i++) {
        ar.add((T) nodeList.item(i).getTextContent());
        logger.trace("found parser " + nodeList.item(i).getTextContent().toString() + " in configuration file " + fileName);
    }
    return ar;
}
}

现在，在这个冗长的介绍之后，我的问题是:当执行这个时，我无法实例化解析器类，而是得到一个 NullPointerException。 while 循环中的 logger.trace 返回:

TRACE ParserControl - trying parser Itr   <--- I would expect the class name here!!!
    ERROR SimpleWebCrawler - WEBCRAWLER-Crawler Exception java.lang.NullPointerException

谁能告诉我，我做错了什么？？？

Answer 1

你这里发生了一些奇怪的事情。我看到的问题:

• 您采用了某种单例设计模式，但仍在使用静态变量。
• 你用错了迭代器
• 在不需要泛型的地方使用泛型
• 你的 IWebParser接口(interface)可以返回一个Boolean ，可以为空。它应该能够返回 null 吗？或者它应该是原始类型 ( boolean )。如果它返回 null 并且您将它放在 if 语句中，那么您将获得 NPE。例如:Boolean b=null; if(b) {} // NPE!!

修复:

• 删除static来自 webParser变量
• 摆脱 getAllParser 中的泛型所以构造函数读取 List<IWebParser> getAllParser() (同样如您所见，我将 ArrayList 替换为 List )。
• 修正你的迭代器使用，现在你得到ClassLoader迭代器并尝试将其转换为 IWebParser ，显然不会工作。这是循环的工作版本，如您所见，我在外部声明迭代器并使用 .next()得到下一个IWebParser在循环中。

Iterator<IWebParser> it = pc.webParser.iterator();
while (it.hasNext()) {
    IWebParser parser = it.next();
    logger.trace("trying parser " + parser.getClass().getSimpleName().toString());
    if (parser.canExecute(url)) {
        return parser.parse(page, url, tTerms);
    } else {
        logger.trace("parser " + parser.getClass().getSimpleName().toString() + " returned false to canExecute()" );
    }
}

想象一个 Iterator是一个对象，它的指针位于排序列表中的某个位置。当您调用 webParser.iterator()它构造了一个新的 Iterator即指向列表的开头。现在，如果你试图遍历这些并且你一直调用 webParser.iterator()你总是会得到一个指向第一个元素的迭代器。这就是为什么申报您的 Iterator 很重要在你的循环之外重复使用同一个循环。还值得注意的是，您只调用 .next()当您想将指针移动到下一个索引时在迭代器上，这就是我声明 parser 的原因变量并将其设置为 while 循环中的下一个变量。

---

发表评论

为什么这是单例设计模式？

单例是一种对象，在应用程序中应该只创建一个实例。在 Java 中，这通常是通过使用 private 获得的。具有公共(public)静态方法的构造函数通常名为 getInstance() . getInstance()如果尚未创建和存储一个实例，则该方法将创建自身的实例，或者返回已存储的实例，这通常是通过使用静态变量来存储该类的唯一实例来完成的。

当您使用面向对象编程时，充分利用类和类实例的含义很重要。当您合并静态变量和方法时，您应该始终考虑为什么它们应该是静态的。我认为始终从非静态变量开始是安全的，并且只在需要时才将其设为静态。在这种情况下 List webParser确实属于类实例而不属于每个人，它是在类的构造函数中初始化的，然后仅在类的非静态实例中使用...因此为什么要静态？此外，您使用的是单例模式，这意味着无论如何都只有 1 个实例!

getAllParsers() 错误

我假设您传递了某个解析器的类名以添加到此 ParserControl类(class)。在这种情况下，您可以使用 Class.forName(className).newInstance() .

替换行 r.add((T) nodeList.item(i).getTextContent());
用线 r.add((IWebParser)Class.forName(nodeList.item(i).getTextContent()).newInstance());

您需要传递类的完整路径。例如:com.me.parsers.IFrameParser , 也仅供引用，如果你在一个类中有一个类，你使用 $指定类，例如:com.me.parsers.ParserClass$InnerClassParser