[fix]流程图 #0.2.0 fix 发现的bug

Author: darksheep

about.md

@@ -6,7 +6,7 @@ plugins {
 }
 
 group = "com.darksheep"
-version = "2023-0620-03"
+version = "2025-03-12"
 
 repositories {
     /*上述代码将阿里云Maven仓库设置为优先级最高，

build.gradle.kts

@@ -5,6 +5,7 @@
 ## 功能特点
 
 ### 1. 代码笔记
+
 - 支持在代码中选中文本并添加笔记
 - 笔记自动关联到对应的代码位置
 - 可以快速跳转到笔记关联的代码处
@@ -19,6 +20,7 @@
   - 新笔记自动添加到列表开头并选中
 
 ### 2. 流程图功能
+
 - 支持创建和编辑流程图，可视化代码之间的关系
 - 流程图节点可以关联到具体的代码笔记
 - 节点功能：
@@ -43,13 +45,15 @@
   - 支持删除连接
 
 ### 3. 标签管理
+
 - 支持为笔记添加标签
 - 标签自动关联到流程图
 - 支持通过标签快速查找相关笔记
 - 标签自动去重和格式化
 - 支持批量添加和删除标签
 
 ### 4. 搜索功能
+
 - 支持按标题和内容搜索笔记
 - 实时搜索结果展示
 - 可以直接从搜索结果创建流程图节点
@@ -59,6 +63,7 @@
 ## 技术实现
 
 ### 前端实现
+
 - 使用原生 JavaScript 和 CSS 构建界面
 - 采用 Apple 设计风格：
   - 磨砂玻璃效果（backdrop-filter）
@@ -75,6 +80,7 @@
   - 右键菜单支持
 
 ### 后端实现
+
 - 基于 IntelliJ Platform Plugin SDK
 - 使用 JCEF (Java Chromium Embedded Framework) 实现现代化界面
 - 数据存储：
@@ -87,6 +93,7 @@
   - 异步处理避免阻塞
 
 ### 流程图实现
+
 - 使用 jsPlumb 实现节点连接
 - 自定义布局算法：
   - 力导向图布局
@@ -100,11 +107,13 @@
 ## 使用说明
 
 ### 创建笔记
+
 1. 在代码中选中要做笔记的文本
 2. 使用快捷键或右键菜单创建笔记
 3. 输入笔记内容并保存
 
 ### 使用流程图
+
 1. 点击工具栏的流程图按钮打开流程图界面
 2. 使用"新建节点"按钮创建节点
 3. 可以：
@@ -115,6 +124,7 @@
 5. 使用"加载流程图"按钮查看已保存的流程图
 
 ## 数据存储
+
 - 笔记数据：
   - 标题、内容、代码位置
   - 创建和更新时间
@@ -128,15 +138,29 @@
 - 所有数据存储在本地 SQLite 数据库中
 
 ## 未来计划
+
 ### 笔记列表
-- [ ] 每个笔记可以绑定多个tag,用,分割 (已实现)
-- [ ] 笔记列表增加tags视图,可以查看当前的所有tag,筛选某个tag或者多个tag下的笔记
-- [ ] 笔记搜索功能增强,支持按照笔记代码内容和标题搜索
-- [ ] 修改当前在IDEA编辑器选中代码做笔记的功能,新建笔记时,可以选择一个或多个tag,默认为上次选择的tag,可以下拉搜索或新建tag
+
+- [X] 每个笔记可以绑定多个tag,用,分割 (已实现)
+- [X] 笔记列表增加tags视图,可以查看当前的所有tag,筛选某个tag或者多个tag下的笔记
+- [X] 笔记搜索功能增强,支持按照笔记代码内容和标题搜索
+- [X] 修改当前在IDEA编辑器选中代码做笔记的功能,新建笔记时,可以选择一个或多个tag,默认为上次选择的tag,可以下拉搜索或新建tag
+
 ### 流程图
-- [ ] 按照IOS的设计风格,美化流程图顶部操作栏
-- [ ] 加载流程图时,编辑后再次保存,更新之前的流程图,而非新建一个新的流程图
+
+- [X] 按照IOS的设计风格,美化流程图顶部操作栏
+- [X] 加载流程图时,编辑后再次保存,更新之前的流程图,而非新建一个新的流程图
 - [ ] 流程图节点连接线默认为直线,可以调整为曲线
 - [ ] 流程图节点支持拖拽调整大小
 - [ ] 流程图可以导出为图片
   - [ ] 绘制流程图时,ctrl+s保存流程图
+
+#### bug修复
+
+- [X] 新建流程图时,会覆盖上一个加载的流程图,不确定是id=0导致,还是currentFlowchartId没有reset
+- [X] 新建,保存后再编辑,因为save没有回显id,导致编辑丢失
+- [X] 笔记列表 tag面板应该一行最多显示5个tag,多余5个时换行,
+- [X] 笔记详情 noteLineNumber和path拼在一个div内,导致编辑笔记时,会把path+number的值传给了path
+- [X] 笔记详情 NoteFilePath 不hover时也显示按钮
+- [X] 笔记列表根据tag筛选笔记时,多选tag,应该是并集,而不是交集
+- [X] 当前选择tag可以筛选出笔记,但是点击列表某个笔记时,会展示笔记详情,但是列表会变为所有笔记,丢失了筛选效果

doc/features.md

@@ -29,12 +29,13 @@ public class NoteDataRepository {
     private static void initFlowchartTable() {
         try (Statement stmt = getConnection().createStatement()) {
             stmt.execute(
-                "CREATE TABLE IF NOT EXISTS flowcharts (" +
-                "id INTEGER PRIMARY KEY AUTOINCREMENT," +
-                "name TEXT NOT NULL," +
-                "data TEXT NOT NULL," +
-                "create_time BIGINT NOT NULL" +
-                ")"
+                    "CREATE TABLE IF NOT EXISTS flowcharts (" +
+                            "id INTEGER PRIMARY KEY AUTOINCREMENT," +
+                            "name TEXT NOT NULL," +
+                            "data TEXT NOT NULL," +
+                            "create_time DATETIME DEFAULT CURRENT_TIMESTAMP," +
+                            "update_time DATETIME DEFAULT CURRENT_TIMESTAMP" +
+                            ")"
             );
         } catch (SQLException e) {
             e.printStackTrace();
@@ -138,16 +139,30 @@ public static void deleteNoteData(Integer noteId){
     }
 
     // 流程图相关方法
-    public static void saveFlowchart(FlowchartData flowchart) {
-        String sql = "INSERT OR REPLACE INTO flowcharts (name, data, create_time,id) VALUES (?, ?, ?,?)";
+    public static Integer saveFlowchart(FlowchartData flowchart) {
+        String sql = "INSERT OR REPLACE INTO flowcharts (name, data, update_time,id) VALUES (?, ?,datetime('now'),?)";
         try (PreparedStatement pstmt = getConnection().prepareStatement(sql)) {
             pstmt.setString(1, flowchart.getName());
             pstmt.setString(2, flowchart.getData());
-            pstmt.setLong(3, flowchart.getCreateTime());
-            pstmt.setInt(4, flowchart.getId());
+            if (flowchart.getId() != null) {
+                pstmt.setInt(3, flowchart.getId());
+            } else {
+                pstmt.setNull(3, java.sql.Types.INTEGER);
+            }
             pstmt.executeUpdate();
+            if(flowchart.getId() != null){
+                return flowchart.getId();
+            }
+            try (ResultSet generatedKeys = pstmt.getGeneratedKeys()) {
+                if (generatedKeys.next()) {
+                    return generatedKeys.getInt(1);
+                } else {
+                    throw new SQLException("Creating flowchart failed, no ID obtained.");
+                }
+            }
         } catch (SQLException e) {
             e.printStackTrace();
+            throw new RuntimeException("Failed to save flowchart: " + e.getMessage());
         }
     }
 
@@ -160,34 +175,46 @@ public static List<FlowchartData> getAllFlowcharts() {
 
             while (rs.next()) {
                 FlowchartData flowchart = new FlowchartData();
+                SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
                 flowchart.setId(rs.getInt("id"));
                 flowchart.setName(rs.getString("name"));
                 flowchart.setData(rs.getString("data"));
-                flowchart.setCreateTime(rs.getLong("create_time"));
+                String createTime = rs.getString("create_time");
+                String updateTime = rs.getString("update_time");
+                flowchart.createTime = dateFormat.parse(createTime);
+                flowchart.updateTime = dateFormat.parse(updateTime);
                 flowcharts.add(flowchart);
             }
         } catch (SQLException e) {
             e.printStackTrace();
+        } catch (ParseException e) {
+            throw new RuntimeException(e);
         }
-        
+
         return flowcharts;
     }
 
     public static FlowchartData getFlowchartById(int id) {
         String sql = "SELECT * FROM flowcharts WHERE id = ?";
         try (PreparedStatement pstmt = getConnection().prepareStatement(sql)) {
+            SimpleDateFormat dateFormat = new SimpleDateFormat("yyyy-MM-dd HH:mm:ss");
             pstmt.setInt(1, id);
             ResultSet rs = pstmt.executeQuery();
             FlowchartData flowchart = new FlowchartData();
             if (rs.next()) {
                 flowchart.setId(rs.getInt("id"));
                 flowchart.setName(rs.getString("name"));
                 flowchart.setData(rs.getString("data"));
-                flowchart.setCreateTime(rs.getLong("create_time"));
+                String createTime = rs.getString("create_time");
+                String updateTime = rs.getString("update_time");
+                flowchart.createTime = dateFormat.parse(createTime);
+                flowchart.updateTime = dateFormat.parse(updateTime);
                 return flowchart;
             }
         } catch (SQLException e) {
             e.printStackTrace();
+        } catch (ParseException e) {
+            throw new RuntimeException(e);
         }
         return null;
     }
@@ -301,4 +328,15 @@ public static void updateNote(NoteData noteData) {
             throw new RuntimeException("Failed to update note: " + e.getMessage());
         }
     }
+
+    public static void deleteFlowchart(int id) {
+        String sql = "DELETE FROM flowcharts WHERE id = ?";
+        try (PreparedStatement pstmt = getConnection().prepareStatement(sql)) {
+            pstmt.setInt(1, id);
+            pstmt.executeUpdate();
+        } catch (SQLException e) {
+            e.printStackTrace();
+            throw new RuntimeException("Failed to delete flowchart: " + e.getMessage());
+        }
+    }
 }

src/main/java/com/darksheep/sheepnote/config/NoteDataRepository.java

@@ -2,33 +2,31 @@
 
 import org.jetbrains.annotations.NotNull;
 
+import java.util.Date;
+
 public class FlowchartData {
-    private int id;
+    private Integer id;
     private String name;
     private String data;
-    private long createTime;
+    public Date createTime;
+
+    public Date updateTime;
 
     public FlowchartData(@NotNull String name, @NotNull String data) {
         this.name = name;
         this.data = data;
-        this.createTime = System.currentTimeMillis();
+        this.createTime = new Date();
+        this.updateTime = new Date();
     }
 
     public FlowchartData() {
     }
 
-    public FlowchartData(int id, @NotNull String name, @NotNull String data, long createTime) {
-        this.id = id;
-        this.name = name;
-        this.data = data;
-        this.createTime = createTime;
-    }
-
-    public int getId() {
+    public Integer getId() {
         return id;
     }
 
-    public void setId(int id) {
+    public void setId(Integer id) {
         this.id = id;
     }
 
@@ -50,11 +48,19 @@ public void setData(@NotNull String data) {
         this.data = data;
     }
 
-    public long getCreateTime() {
+    public Date getUpdateTime() {
+        return updateTime;
+    }
+
+    public void setUpdateTime(Date updateTime) {
+        this.updateTime = updateTime;
+    }
+
+    public Date getCreateTime() {
         return createTime;
     }
 
-    public void setCreateTime(long createTime) {
+    public void setCreateTime(Date createTime) {
         this.createTime = createTime;
     }
-} 
+}