Processor#run() 方法的改进的一个建议

[ykgarfield]

Processor#run() 有如下的一个代码片段:

SelectionKey key = null;
try {
	key = iter.next();
	iter.remove();
	if (key.isReadable()) {
		read(key);
	} else if (key.isWritable()) {
		write(key);
	} else if (!key.isValid()) {
		close(key);
	} else {
		throw new IllegalStateException("Unrecognized key state for processor thread.");
	}
} 

这里是先判断了 isReadable()、isWritable(), 最后判断的 key.isValid(), 这里最好应该先判断 key.isValid(). 在判断 isReadable()、isWritable().因为在判断 isReadable()、isWritable() 之前 key 已经无效了, 所以可能会产生 CancelledKeyException.虽然在 catch 中最终会捕获到这个异常, 但是按照比较好的方式做的话, 应该还是先判断 key.isValid() 会更好一点.