设计模式_36:
# 作业
  1. 返回了null，不合理
  2. 直接re-throw
  3. 应该返回NULL值
  4. 抛出一个自定义的异常

# 感想
  异常这种机制的设计，是为了更好地处理真实的异常情况，要合理地使用，不要为了怕麻烦就乱用，制造出一个个排查问题的灾难。

对于 generate() 函数，如果本机名获取失败，函数返回什么？这样的返回值是否合理？
返回null-时间戳-8位随机数字字母符号的字符串.
这样返回合理.因为业务允许一定概率的id重复,并且时间戳-8位随机的数字字母重叠的概率本身就很低,所以代码可以满足业务继续执行,至于最终要不要继续执行可以由上层业务程序控制.

对于 getLastFiledOfHostName() 函数，是否应该将 UnknownHostException 异常在函数内部吞掉（try-catch 并打印日志）？还是应该将异常继续往上抛出？如果往上抛出的话，是直接把 UnknownHostException 异常原封不动地抛出，还是封装成新的异常抛出？
不应该内部吞掉异常,应该直接把异常原封不动地抛出.因为当前是非业务工具类,异常处理该要交由业务程序来处理.

对于 getLastSubstrSplittedByDot(String hostName) 函数，如果 hostName 为 NULL 或者是空字符串，这个函数应该返回什么？
hostName为NULL会抛出空指针异常,这时该要抛出异常由业务程序来处理;hostName为空字符串属于正常业务可以返回空字符串.

对于 generateRandomAlphameric(int length) 函数，如果 length 小于 0 或者等于 0，这个函数应该返回什么？
length小于0时会抛出异常,这时也该要抛出异常由业务程序处理;length等于0时也是正常业务可以返回空字符串.

课堂讨论:



1. 对于 generate() 函数，如果本机名获取失败，函数返回什么？这样的返回值是否合理？

   本机名获取失败时函数将返回null,是不合理的.除非特殊说明,否则上层调用者可能忽视null的存在,从而引发npe.

   

2. 对于 getLastFiledOfHostName() 函数，是否应该将 UnknownHostException 异常在函数内部吞掉（try-catch 并打印日志）？还是应该将异常继续往上抛出？如果往上抛出的话，是直接把 UnknownHostException 异常原封不动地抛出，还是封装成新的异常抛出？

   ​ 首先,本人极端不愿使用Checked-Exception.异常在函数内部吞掉是不好的行为,因为发生异常时,上层调用者并不知道.除非经过妥善的处理,否则会影响业务的正常执行.

   ​ 如果在使用CE的情况下,是一定要向上抛出的.视情况是否包装成新的异常:原始异常准确达意,调用者很容易理解就可以直接抛出,原始异常有歧义或可能导致调用者困惑,就应该封装成新的异常.

   

3. 对于 getLastSubstrSplittedByDot(String hostName) 函数，如果 hostName 为 NULL 或者是空字符串，这个函数应该返回什么？

   先贴原始代码

​ @VisibleForTesting

​        protected String getLastSubstrSplittedByDot(String hostName) {

​         String[] tokens = hostName.split("\\.");

​ String substrOfHostName = tokens[tokens.length - 1];

​ return substrOfHostName;

​ }

​ 原始情况下,会因为hostName为null产生的npe导致程序运行crash.可以为此函数添加一个用于校验 hostName合法的isHostNameValid()函数,在hostName非法时抛出运行时异常(Runtime-Exception),或在非法时返回代表非法的常量值.



4. 对于 generateRandomAlphameric(int length) 函数，如果 length 小于 0 或者等于 0，这个函数应该返回什么？

​ 应该抛出运行时异常或返回代表非法的常量值.

1. 返回”null-1577456311467-3nR3Do45”，不够合理，但勉强能接受
2. 应该吞掉，因为 generate() 并不关心 getLastFiledOfHostName() 抛出的异常
3. 返回 null
4. 返回空字符串

思考是为了下节课吸收更多！

对于今天课堂留的作业，我采用了统一的思路—能用异常解决的都用异常解决。

1.对于 generate() 函数，如果本机名获取失败，函数返回什么？这样的返回值是否合理？
答：抛出异常，这是一个无法恢复的情况，打断正常的处理流程并进入异常逻辑处理模块

2. 对于 getLastFiledOfHostName() 函数，是否应该将 UnknownHostException 异常在函数内部吞掉（try-catch 并打印日志）？还是应该将异常继续往上抛出？如果往上抛出的话，是直接把 UnknownHostException 异常原封不动地抛出，还是封装成新的异常抛出？
答：不应该内部吞掉，应该抛出到上层做统一的异常处理，这里是个单一的模块，不需要再封装

3. 对于 getLastSubstrSplittedByDot(String hostName) 函数，如果 hostName 为 NULL 或者是空字符串，这个函数应该返回什么？
答：抛出异常，异常消息是—hostName为NULL或空字符串

4. 对于 generateRandomAlphameric(int length) 函数，如果 length 小于 0 或者等于 0，这个函数应该返回什么？
答：抛出异常，异常消息是—参数不合法

1. 对于 generate() 函数，如果本机名获取失败，函数返回什么？这样的返回值是否合理？: 抛出异常, 因为获取失败是一个异常情况, 所以需要返回异常
2. 对于 getLastFiledOfHostName() 函数，是否应该将 UnknownHostException 异常在函数内部吞掉（try-catch 并打印日志）？还是应该将异常继续往上抛出？如果往上抛出的话，是直接把 UnknownHostException 异常原封不动地抛出，还是封装成新的异常抛出？: 继续原封不动的往上抛出, 可以在 Controller 层统一处理, 这样子异常的代码会比较统一的处理, 该异常的描述已经很明确了, 可以通过类名知道这个异常是什么.
3. 对于 getLastSubstrSplittedByDot(String hostName) 函数，如果 hostName 为 NULL 或者是空字符串，这个函数应该返回什么？: hostName 为空是一种异常, 应该抛出异常, 因为入参 hostName 不能为空
4. 对于 generateRandomAlphameric(int length) 函数，如果 length 小于 0 或者等于 0，这个函数应该返回什么？: 按照项目组统一规定来, 个人习惯返回 空字符串, 这么返回是方便调用方不用做非空判断

个人见解。

1. 获取本机名失败可以返回一个默认值，比如 "unknown"。因为 Id 只要随机就行，hostName其实不是很重要。不过可以考虑发送一个提醒，告知开发者运行环境有无法获取本机名的问题。
2. 建议吞掉。因为这个问题并不严重，没有必要停止程序。如果要往上抛出，原封不动地抛出，因为这个抛出错误原因很容易理解，不需要进一步包装。
3. 返回空字符吧。进行 NULL 判断可读性不好。
4. 感觉抛出错误比较好，因为这个算是开发者不细心了，让开发者知道错误，然后进行修改。如果不抛出错误，可以考虑返回空字符数组。