factory-pattern 相关问题

我已经创建了Exception xml，并动态创建并抛出异常。 com.package.CheckedException ] >> 嗯，以科学的名义，这是您可以做到的方式。我会建议这样做吗？绝对不是。我自己会做任何远程的事情吗？可能不是。 public class ExceptionFactory { public static void throwException(String className) throws CheckedException, UncheckedException { Class<?> exceptionClass; try { exceptionClass = Class.forName(className); } catch (ClassNotFoundException e) { throw new IllegalArgumentException(e); } try { if (CheckedException.class.isAssignableFrom(exceptionClass)) { throw exceptionClass.asSubclass(CheckedException.class) .newInstance(); } else if (UncheckedException.class .isAssignableFrom(exceptionClass)) { throw exceptionClass.asSubclass(UncheckedException.class) .newInstance(); } else { throw new IllegalArgumentException( "Not a valid exception type: " + exceptionClass.getName()); } } catch (InstantiationException | IllegalAccessException e) { throw new IllegalStateException(e); } } public static void main(String... args) { try { throwException("CheckedException"); } catch (CheckedException e) { System.out.println(e); } catch (UncheckedException e) { System.out.println(e); } } } class CheckedException extends Exception { } class UncheckedException extends Exception { } 我看不出这家工厂的意义。即使您可以使用它（通过将所有异常作为单个祖先类的子类抛出），它的用法也将如下所示： .... if (somethingInWrong) { ExceptionFactory.throwException("SomeKey"); } .... 对于每个键，您仍然必须创建一个异常类以映射到它。可以说SomeKeyException是映射到“ SomeKey”的异常。在那种情况下，简单地写就更安全了： .... if (somethingInWrong) { throw new SomeKeyException(); } .... 这样，编译器会检查您是否正在创建它实际上知道的异常类。如果使用Factory，则可能会使用不是有效密钥的某些String，并且编译器将无法对其进行任何处理。只有在运行时，您的Factory才会找不到映射到无效密钥的异常。一些调整： public static void throwException(final String key) throws Throwable { ExceptionMapping exceptionMapping = exceptionMappings.getExceptionMappings().get(key); if (exceptionMapping != null) { try { Class<Throwable> exceptionClass = (Class<Throwable>)Class.forName(exceptionMapping.getClassName()); try { throw exceptionClass.cast( exceptionClass.newInstance() ); // line X } catch (InstantiationException e) { e.printStackTrace(); } catch (IllegalAccessException e) { e.printStackTrace(); } } catch (ClassNotFoundException e) { e.printStackTrace(); } } } 无需使用反射（正如我在you shouldn't use reflection unless you really have to...上面所说的那样）。您可以将异常类实现为如下形式： class MyExceptions { static void myExceptionsThrower(String key) throws Exception { if("illegalstate".equals(key)) { throw new IllegalStateException("that's my IllegalStateException bro!"); } else if("illegalaccess".equals(key)) { throw new IllegalAccessException("that's my IllegalAccessException bro!"); } // etc... } } 并用于： MyExceptions.myExceptionsThrower(key); 这是我参加这场德比大赛的机会。：-） [其他答案已评论了这是否是合理的设计。为了解决这个问题，我将这些问题放在一边。我的一些烦恼是不必要的警告（即使被压制了），并且没有报告实际出问题的异常。特别是仅打印出堆栈跟踪通常是不够的。是的，这只是测试代码，但是在处理这样的代码（甚至是旨在引发异常的代码）时，人们真的应该考虑如何处理错误。在这种情况下，我选择将这些类型的错误表示为InternalError的实例，因为配置或任何可能以多种方式出错的错误。具体来说：如果找不到该类，则找到该类，但该类不是CheckedException或UncheckedException的子类型（甚至不是普通类），或者没有no-arg构造函数，或者无法访问。某些建议的解决方案的另一个问题是，如果异常类名称为"java.lang.InstantiationException"（或其他内部捕获的异常之一），则可能在内部构造，引发然后捕获该异常类型的实例，从而导致在堆栈跟踪中，但实际上并未引发请求的异常。我通过线性化逻辑而不是嵌套try-catch块来避免这种情况。最后，我将创建异常的代码提取到单独的方法中，以便可以将它用于已检查和未检查的情况。如果重新排列异常层次结构以仅允许单个根异常（我建议不选中）并且具有在Web层处理或扔给调用者的异常子类型，则可以大大简化此过程。 static void throwException(final String exClassName) throws CheckedException, UncheckedException { Class<?> clazz; try { clazz = Class.forName(exClassName); } catch (ClassNotFoundException cnfe) { throw new InternalError(exClassName, cnfe); } if (CheckedException.class.isAssignableFrom(clazz)) { throw newException(clazz.asSubclass(CheckedException.class)); } else if (UncheckedException.class.isAssignableFrom(clazz)) { throw newException(clazz.asSubclass(UncheckedException.class)); } else { throw new InternalError(exClassName + " is not a valid exception"); } } static <X extends Throwable> X newException(Class<X> clazz) { X x; try { x = clazz.newInstance(); } catch (InstantiationException|IllegalAccessException e) { throw new InternalError("creating instance of " + clazz, e); } return x; } 这可能是帮助，以创建自定义先决条件例外，以避免如果条件多则]。在检查null 指针时创建前提条件异常。 class Preconditions { /** * <p> * Checks the value to be null and if null throws a new Exception with the message given. * Used to reduce checking if conditions for complexity. * </p> * @param val - val to check null * @param exceptionClass - exception class to be thrown * @param args - message to be called for throwing exception * @throws Throwable - Common Throwable Exception. */ public static void checkNotNull(final Object val, final Class<?> exceptionClass, final Object ...args) throws Throwable { Class<?>[] argTypes = new Class<?>[args.length]; Arrays.stream(args).map(WithIndex.indexed()).forEach(arg ->argTypes[arg.index()] = arg.value().getClass()); if (null == val) throw (Throwable) exceptionClass.getConstructor(argTypes).newInstance(args); } } 然后您可以在代码中使用它： PreConditionUtil.checkNotNull(objectToCheck, CustomException.class, ErrorCode, "your error message", ...);

java exception-handling factory-pattern

回答 6 投票 8

factory-pattern 相关问题

最新问题