-
Notifications
You must be signed in to change notification settings - Fork 4
Description
现在本项目中存在大量代码样式问题,这会降低外部开发者对本项目的评价,甚至对本项目的可靠性产生怀疑。
我建议对本项目的代码样式进行一次全面的规范化,添加 EditorConfig 文件,并通过 GitHub Action 自动执行 checkstyle 检查,以确保新增代码的样式与其他部分一致。
这是我粗略翻阅部分源码文件后注意到的样式问题:
- 代码中存在很多连续的空行,请考虑删除冗余空行
- import 语句格式混乱
- 部分文件会在导入标准库和导入其他类的 import 语句之间加入空行,但还有很多文件没有这样做,请将它们统一
- 错误分组:
Lines 13 to 16 in d0a8878
import org.ruyisdk.devices.services.DeviceService; import java.util.List; import org.eclipse.jface.window.Window; ruyisdk-eclipse-plugins/plugins/org.ruyisdk.packages/src/org/ruyisdk/packages/JsonParser.java
Lines 11 to 15 in 5605eb3
import javax.json.*; import org.ruyisdk.ruyi.util.RuyiFileUtils; import java.io.StringReader;
- 错误缩进:
Line 471 in a27e0a3
class OutputDialog extends Dialog {
- 遗漏空行:
ruyisdk-eclipse-plugins/plugins/org.ruyisdk.core/src/org/ruyisdk/core/ruyi/model/RuyiVersion.java
Lines 15 to 16 in d0a8878
} public static RuyiVersion parse(String versionStr) { ruyisdk-eclipse-plugins/plugins/org.ruyisdk.packages/src/org/ruyisdk/packages/JsonParser.java
Lines 1 to 2 in 5605eb3
package org.ruyisdk.packages; import java.util.List;
- 遗漏空格:
ruyisdk-eclipse-plugins/plugins/org.ruyisdk.ruyi/src/org/ruyisdk/ruyi/util/RuyiFileUtils.java
Line 13 in 9f20f66
public static Path getDefaultInstallPath(){ Line 124 in 32f447a
public int initTelemetryStatus(){ ruyisdk-eclipse-plugins/plugins/org.ruyisdk.ruyi/src/org/ruyisdk/ruyi/util/RuyiFileUtils.java
Line 13 in 9f20f66
public static Path getDefaultInstallPath(){
这只是我粗略扫过一遍源码后注意到的样式问题,实际还有更多需要修正的样式问题。请使用自动化工具(如 checkstyle)检查并规范化代码样式,后续开发中请多使用 IDE 的代码格式化功能。