新人之Coding Style 的疑惑

RT，TL没有提出空白行的问题，只是我自己写代码的时候，为了增加可读性，一段逻辑结束后加入 空白行，增加段落感。


就比如Leetcode一道题，207. Course Schedule， 这个是我的solution

• 建图
• 初始节点入队列
• BFS kicks off
  

增加空白一行意味着新逻辑的开始。

1. class Solution {
   
2.     public boolean canFinish(int n, int[][] edges) {
   
3.         Map<Integer, Set<Integer>> graph = new HashMap<>();
   
4.         for (int i = 0; i < n; i++) graph.putIfAbsent(i, new HashSet<>());
   
5.         int[] indegree = new int[n];
   
6.         for (int[] edge : edges) {
   
7.             int v = edge[0];
   
8.             int u = edge[1];
   
9.             indegree[v]++;
   
10.             graph.get(u).add(v);
    
11.         }
    
12. 
    
13.         Set<Integer> visited = new HashSet<>();
    
14.         int cnt = 0;
    
15.         Deque<Integer> queue = new ArrayDeque<>();
    
16.         for (int i = 0; i < n; i++) {
    
17.             if (indegree[i] == 0) {
    
18.                 visited.add(i);
    
19.                 queue.offer(i);
    
20.             }
    
21.         }
    
22. 
    
23.         while (!queue.isEmpty()) {
    
24.             int u = queue.poll();
    
25.             for (int v : graph.get(u)) {
    
26.                 if (visited.contains(v)) continue;
    
27.                 indegree[v]--;
    
28.                 if (indegree[v] == 0) {
    
29.                     queue.offer(v);
    
30.                     visited.add(v);
    
31.                 }
    
32.             }
    
33.         }
    
34. 
    
35.         return visited.size() == n;
    
36.     }
    
37. }

复制代码

工作中写到的代码也有这个意思。




另外除了大段逻辑的划分，有一些小的频繁的代码, 也想划分，比如区分前

1. public void test() {
   
2.     Cell idCell = row.createCell(ID_IDX);
   
3.     idCell.setCellStyle(Style.FAILURE);
   
4.     Cell nameCell = row.create(NAME_IDX);
   
5.     nameCell.setCellStyle(Style.SUCCESS);
   
6.     Cell ageCell = row.createCell(AGE_IDX);
   
7.     ageCell.setCellStyle(Style.FAILURE);
   
8. }

复制代码

区分后：

1. public void test() {
   
2.     Cell idCell = row.createCell(ID_IDX);
   
3.     idCell.setCellStyle(Style.FAILURE);
   
4.    
   
5.     Cell nameCell = row.create(NAME_IDX);
   
6.     nameCell.setCellStyle(Style.SUCCESS);
   
7.    
   
8.     Cell ageCell = row.createCell(AGE_IDX);
   
9.     ageCell.setCellStyle(Style.FAILURE);
   
10. }
    

复制代码

PR中reviewer没有提出空白行的问题。



但是去年实习的时候，我的mentor，从🐶家出身，每次PR我的代码，都专门会让我把空白行代码删除掉。他的实力也很强。


今天，我也问了身边SDE好朋友，感觉这个问题没有最终答案，但是我们都比较偏向大段逻辑划分可以加空白。


这个问题是不是仁者见仁，智者见智，opinion-based？


感谢各位指导！

以你们用的linting tool 为主 不要那么迂 不是什么大问题

熟读google coding style

想起实验室同学的导师，有一次到同学的办公桌前听报告，忽然说等一下，然后用同学的鼠标，把同学电脑里windows右下角的网络链接图标拖拽到了右数最后一格。
然后如释重负，若无其事地让同学继续作报告。

不需要空白行，context一起的可以合并增加可读性

ryanzhang0530 发表于 2023-9-20 17:33
以你们用的linting tool 为主 不要那么迂 不是什么大问题

好的，非常感谢！

红A 发表于 2023-9-20 17:34
不需要空白行，context一起的可以合并增加可读性

非常感谢大佬！

匿名用户 发表于 2023-9-20 18:38
熟读google coding style

好的，搜了一下确实是有的，非常感谢

A single blank line may also appear anywhere it improves readability, for example between statements to organize the code into logical subsections. A blank line before the first member or initializer, or after the last member or initializer of the class, is neither encouraged nor discouraged.

每个组都应该有一个coding convention
无所谓对错，但是同一个文件里的格式应该统一

一眼看起来，以为你的代码从花生厂的产品里拷贝粘贴代码出来后\r\n和\n搞砸了。

沒想到1point3acres對code也會打廣告
原本沒看是啥字詞的時候還以為你的code加的不只有空行