定向爬虫 瓦力

有些代码写得可有点那个哦。

首先看看PropertiesFactory类的代码，并且这个类也不能这样叫，最好只能叫ProperiesUtil类吧。StringFactory类也一样，最多也只能叫StringUtil类吧。

public static Map loadPropertiesFile(Map map, String path)
  {
    Properties prop = new Properties();
    FileInputStream fin = null;
    try {
      fin = new FileInputStream(path);
      prop.load(fin);
      map.putAll(prop);
    }
    catch (FileNotFoundException e) {
      e.printStackTrace();
      try
      {
        fin.close();
      }
      catch (IOException e) {
        e.printStackTrace();
      }
    }
    catch (IOException e)
    {
      e.printStackTrace();
      try
      {
        fin.close();
      }
      catch (IOException e) {
        e.printStackTrace();
      }
    }
    finally
    {
      try
      {
        fin.close();
      }
      catch (IOException e) {
        e.printStackTrace();
      }
    }
    return map;
  }

 一个流被反反复复关了这么多次，虽然说可以重复关闭，但是有一个finally块不就可以了吗，并且你也应该判断一个fin是否为null，否则从finally块的抛出的空指针异常就不太好了。

我们再看看StringFactory类的方法。

public static boolean chk(String regex, String s)
  {
    if ((s == null) || (s.trim().length() < 0))
      return false;

    Pattern p = Pattern.compile(regex);
    Matcher m = p.matcher(s.trim());
    return m.find();
  }

 首先chk的命名就不太规范了，chk和check差几个字母呢，有必要这样省不？

还有，s.trim().length()有可能会小于0吗？最少的情况下也就是空字符串也就是等于0吧。

其他的类也有问题，如果楼主说这个东西是商业项目的话，我就觉得有点那个了。

public static Map loadPropertiesFile(Map map, String path)
  {
    Properties prop = new Properties();
    FileInputStream fin = null;
    try {
      fin = new FileInputStream(path);
      prop.load(fin);
      map.putAll(prop);
    }
    catch (FileNotFoundException e) {
      e.printStackTrace();
      try
      {
        fin.close();
      }
      catch (IOException e) {
        e.printStackTrace();
      }
    }
    catch (IOException e)
    {
      e.printStackTrace();
      try
      {
        fin.close();
      }
      catch (IOException e) {
        e.printStackTrace();
      }
    }
    finally
    {
      try
      {
        fin.close();
      }
      catch (IOException e) {
        e.printStackTrace();
      }
    }
    return map;
  }

public static boolean chk(String regex, String s)
  {
    if ((s == null) || (s.trim().length() < 0))
      return false;

    Pattern p = Pattern.compile(regex);
    Matcher m = p.matcher(s.trim());
    return m.find();
  }

谢谢 指点