08 | 写好声明的“八项纪律”

代码图片建议用 carbon 美化下，可读性会更好：） https://carbon.now.sh/

第28，29，39，40，44行相对应带上大括号会更好 第39，40，44行属于一行表示多重行为，不便于观察和理解，换行会更好 第28，31，36，42，47行的代码与前面的代码没有很好的分块，构成不了视觉上面清晰的代码块，在前面加上空行会更好 第35行命名不够清晰，修改为Map<Character, Character> charMapping = new HashMap<>(); 第29行，就像前面 @背着吉他的大漠狼 同学说的，if 里面的 match 函数其实表示的意义不太明确，读者需要去看 match 函数的大致实现，如果在此之前定义一个boolean isMatched = match(word, pattern); 再将 isMatched 放到 if 里面会好很多；而且把函数直接放在 if 语句里面其实也违反了我们之前提到的单行单一行为的原则，因为运行函数返回结果和表达式条件判断在概念上本就是两个完全不同的行为 能看到的就这些了，不足之处还请老师和大神们补充

细节无小事，高手不是掌握了降龙十八掌，而是简单的事也能做到极致，学习了。

length方法直接写在for循环语句中会不会不是很好呢？因为这样每次循环都会调用它去获取word字符的长度，所以我觉得可以在for循环直接声明一个变量，值就是word的字符长度： int wordLength = word.length(); for (int i = 0; i < wordLength; i++) { …… }

``` import java.util.ArrayList; import java.util.HashMap; import java.util.List; import java.util.Map; class Solution { /** * Return a list of the words in words that match the given pattern. * * A word matches the pattern if there exists a permutation of * letters p so that after replacing every letter x in the pattern * with p(x), we get the desired word. * * Example: * Input: words = ["abc","deq","mee","aqq","dkd","ccc"], * pattern = "abb" * Output: ["mee","aqq"] * * Explanation: "mee" matches the pattern because there is * a permutation {a -> m, b -> e, ...}. * * "ccc" does not match the pattern because * {a -> c, b -> c, ...} is not a permutation, since a * and b map to the same letter. */ public List<String> findAndReplacePattern(String[] words, String pattern) { List<String> ans = new ArrayList(); for (String word : words) { if (match(word, pattern)) { ans.add(word); } } return ans; } public boolean match(String word, String pattern) { Map<Character, Character> M = new HashMap(); for (int i = 0; i < word.length(); ++i) { char w = word.charAt(i); char p = pattern.charAt(i); if (!M.containsKey(w)) { M.put(w, p); } if (M.get(w) != p) { return false; } } boolean[] seen = new boolean[26]; for (char p : M.values()) { if (seen[p - 'a']) { return false; } seen[p - 'a'] = true; } return true; } } ``` 改成我喜欢的格式,虽然不一定准确,但是自己看上去舒服很多

还想问个问题就是c++得成员变量初始化可以再声明的时候初始化也可以在构造函数初始化，那哪种比较好呢

1，if语句尽量不要涉及复杂的表达式，可以用局部变量申明出来 2，例子中有些许的不同语义的卸载了同一行，写没很好使用花括号 3，方法体内可以将申明部分与逻辑部分使用空行加强阅读理解

老师，声明时就初始化，好像我定义那些实体Bean时候，并没有这么做，这个也要看情况吧？

老师，请教您一个问题： Java中类变量的初始化不是都放在构造函数或者初始化函数中吗？您的示例中是声明的时候直接初始化的，是不是有复杂逻辑（需要通过调用方法赋值）的才需要放在初始化函数中？

emptyMethod的方法体里还需要加一行blank的注释吗？