论一段简单的代码的更改

以下代码有何问题

    public static String decodeBuffer(String str ,String charset){
        if (str == null) {
            return str = StringUtils.EMPTY;
        }
         try {
            byte[] byteStr = new BASE64Decoder().decodeBuffer(str.trim());
            return new String(byteStr,charset);
        } catch (UnsupportedEncodingException e) {
            return new String();
        } catch (IOException e) {
            return new String();
        }
    }

改成这样会不会好一些？（更改对空串的处理，如果为空串就直接返回空串）

    public static String decodeBuffer(String str, String charset) {
        String decodeStr = StringUtils.EMPTY;
        if (!StringUtils.isBlank(str)) {
            try {
                byte[] byteStr = new BASE64Decoder().decodeBuffer(str);
                return new String(byteStr, charset);
            } catch (UnsupportedEncodingException e) {
                return new String(); // 异常处理有问题
            } catch (IOException e) {
                return new String(); // 异常处理需要特殊处理
            }
        }
        return decodeStr;
    }

 有时候在想写程序应该写到什么地步？精益求精，还是适可而止，而作为程序员，有时候真应该需要一点执着精神，不求最好，但求更好……

再改一改（添加对异常日志记录，保证方法只有一个return）：

    public static String decodeBuffer(String str, String charset) {
        String decodeStr = StringUtils.EMPTY;
        if (!StringUtils.isBlank(str)) {
            try {
                byte[] byteStr = new BASE64Decoder().decodeBuffer(str);
                decodeStr = new String(byteStr, charset);
            } catch (UnsupportedEncodingException e) {
                log.debug("UnsupportedEncodingException happened:" + e.getMessage()); // 记录异常日志
            } catch (IOException e) {
                log.debug("IOException happened:" + e.getMessage()); // 记录异常日志
            }
        }
        return decodeStr;
    }