《代码整洁之道》 之第七章 错误处理

要点简介

  • 为什么要使用异常而不是返回码(if-else)
  • 主流语言对待受检异常和非受检异常态度和处理方式
  • 抛出异常的注意事项
  • 如果不想使用异常(不打断流程)应该怎么处理
  • null值的处理

一、判断和异常的取舍问题

刚写Java代码时,我也有一个疑问,就是为什么要有异常这个东西。比如要访问一个文件,如果文件不存在完全可以通过if-else进行判断。
现在,其实有了更深的理解。就是简单的if-else其实也可以。但是在复杂情况下,如:代码段7.1,有很多状态需要判断的时候,代码就会显得杂乱无章。
正常情况下,if-else应该用于表现程序逻辑,业务代码。但是按照下面的写法,业务代码和错误判断(包含很多前置检查)就有可能混为一谈。不好区分。

代码段7.1
public class DeviceController {
    public void sendShutDown() {
        DeviceHandle handle = getHandle(DEVl);
        // Check the state of the device
        if (handle != DeviceHandle.INVALID) {
            // Save the device status to the record field
            retrieveDeviceRecord(handle);
            // If not suspended, shut down
            if (record.getStatus != DEVICE.SUSPENDED) {
                pauseDevice(handle);
            } else {
                logger.log("Device suspended.Unable_to_shut_down" );
            }
        } else {
            logger.log("Invalid handle for: " + DEVI.toStringO);
        }
    }
}

所以,下面 代码段7.2 所示就会更简洁和优雅.异常检查和业务逻辑相互独立.

代码段7.2
public class DeviceController {
    public void sendShutDown() {
        try {
            tryToShutDown();
            /*以及很多业务代码*/
        } catch (DeviceShutDownError e) {
            logger.log(e);
        } catch (OtherError e) {
            logger.log(e);
        }
    }
}

二、推荐使用不可控异常(unchecked Exception)

目前,C#、C++、Python、Ruby都不再(或从未)支持或受检异常。因为其违反了开放\闭合原则,即如果在某层次方法里面抛出了受检异常,那么其上层除非catch住,否则都要重重抛出此异常。完全的耦合了。
所以现在的建议是:
一般应用开发仅考虑抛出unchecked Exception,关键代码库,则可考虑checked Exception

三、抛出异常的注意事项

你抛出的每个异常,都应当提供足够的环境说明,以便判断错误的来源和处所。在Java中,你可以从任何异常里得到堆栈踪迹(stacktrace);然而,堆栈踪迹却无法告诉你该失败操作的初衷。应创建信息充分的错误消息,并和异常一起传递出去。在消息中,包括失败的操作和失败类型。如果你的应用程序有日志系统,传递足够的信息给catch块,并记录下来。

另外,补充一点,抛出异常之前,应该记录日志,否则你可能无法找到异常的抛出地点。如代码段7.3

代码段7.3
if (record.getStatus != DEVICE.SUSPENDED) {
        log.error("原因-异常");
        throw new BusinessException("message","Detail");
    } 

四、可不可以不使用异常呢

五、不要返回和传递null值

5.1不要返回null值

不要返回null值的原因:
(1)返回null值会导致程序中充满对null的判断;
(2)如果不注意对null的判断,则会导致很不友好的空指针异常;
针对这个问题也搜了一下,辩证的看到处理方法:
(1)对于集合和数组作为返回值,使用长度为零的数组或者空集合,而不是null;
(2)字符串作为返回值,使用空字符串来代替null;
(3)当空对象与其他返回对象有一样的行为和意义时,使用空对象;
关于这一条的理解目前,我还没理解到,这里暂时做一个标记,期望后续可以理解

例如,有一个方法返回一个迭代器。一个空的迭代器可以定义为NullIterator并实现Iterator接口,他的next方法永远返回null,他的hasNext方法永远返回false。这样,使用这个方法返回值的代码就不要进行null判断,因为NullIterator的行为与其他迭代器一样。

(4)针对这个规则的例外任何在逻辑上表示查找(search或者get)的意思时,应该返回null;代码段7.4 说明了为什么应该例外处理;

代码段7.4
User getUserById(String id){
   User user = dao.getUserById(id);
   if(user == null)
      return new User();
   return user;
}

5.2不要传递null值

不传递null值的意思,即不要在方法调用时,将参数设置为null。这将会带来更不可预料的后果。
但是如果程序中如果却需要对null值进行判断,可以抛出InvalidArgumentException或者使用断言来处理(而不是仅仅忽略null值)。

代码段7.5
    public double xProjection(Point p1, Point p2) {
        if (p1 == null || p2 == null) {
            throw InvalidArgumentException ("Invalid argument for MetricsCalculator.xProjection");
            }
            return (p2.x - pl.x) * 1.5;
        }
    }

你可能感兴趣的:(《代码整洁之道》 之第七章 错误处理)