diff --git a/app/src/main/java/io/legado/app/data/dao/BookGroupDao.kt b/app/src/main/java/io/legado/app/data/dao/BookGroupDao.kt
index 51068b149..6162ffc5a 100644
--- a/app/src/main/java/io/legado/app/data/dao/BookGroupDao.kt
+++ b/app/src/main/java/io/legado/app/data/dao/BookGroupDao.kt
@@ -1,10 +1,7 @@
 package io.legado.app.data.dao
 
 import androidx.lifecycle.LiveData
-import androidx.room.Dao
-import androidx.room.Insert
-import androidx.room.OnConflictStrategy
-import androidx.room.Query
+import androidx.room.*
 import io.legado.app.data.entities.BookGroup
 
 @Dao
@@ -17,5 +14,11 @@ interface BookGroupDao {
     val maxId: Int
 
     @Insert(onConflict = OnConflictStrategy.REPLACE)
-    fun insert(bookGroup: BookGroup)
+    fun insert(vararg bookGroup: BookGroup)
+
+    @Update
+    fun update(vararg bookGroup: BookGroup)
+
+    @Delete
+    fun delete(vararg bookGroup: BookGroup)
 }
\ No newline at end of file
diff --git a/app/src/main/java/io/legado/app/ui/main/bookshelf/BookshelfViewModel.kt b/app/src/main/java/io/legado/app/ui/main/bookshelf/BookshelfViewModel.kt
index a56a08871..f4664fa22 100644
--- a/app/src/main/java/io/legado/app/ui/main/bookshelf/BookshelfViewModel.kt
+++ b/app/src/main/java/io/legado/app/ui/main/bookshelf/BookshelfViewModel.kt
@@ -7,28 +7,28 @@ import io.legado.app.data.entities.BookGroup
 
 class BookshelfViewModel(application: Application) : BaseViewModel(application) {
 
-    fun saveBookGroup(group: String?) {
-        if (!group.isNullOrBlank()) {
-            execute {
-                App.db.bookGroupDao().insert(
-                    BookGroup(
-                        App.db.bookGroupDao().maxId + 1,
-                        group
-                    )
-                )
-            }
-        }
-    }
 
     fun addGroup(groupName: String) {
-
+        execute {
+            val maxId = App.db.bookGroupDao().maxId
+            val bookGroup = BookGroup(
+                groupId = maxId.plus(1),
+                groupName = groupName,
+                order = maxId.plus(1)
+            )
+            App.db.bookGroupDao().insert(bookGroup)
+        }
     }
 
     fun upGroup(bookGroup: BookGroup) {
-
+        execute {
+            App.db.bookGroupDao().update(bookGroup)
+        }
     }
 
     fun delGroup(bookGroup: BookGroup) {
-
+        execute {
+            App.db.bookGroupDao().delete(bookGroup)
+        }
     }
 }
diff --git a/app/src/main/java/io/legado/app/ui/main/bookshelf/GroupManageDialog.kt b/app/src/main/java/io/legado/app/ui/main/bookshelf/GroupManageDialog.kt
index d33a62476..7db85906e 100644
--- a/app/src/main/java/io/legado/app/ui/main/bookshelf/GroupManageDialog.kt
+++ b/app/src/main/java/io/legado/app/ui/main/bookshelf/GroupManageDialog.kt
@@ -59,7 +59,7 @@ class GroupManageDialog : DialogFragment(), Toolbar.OnMenuItemClickListener {
 
     private fun initData() {
         tool_bar.title = getString(R.string.group_manage)
-        tool_bar.inflateMenu(R.menu.group_manage)
+        tool_bar.inflateMenu(R.menu.book_group_manage)
         tool_bar.menu.applyTint(requireContext(), false)
         tool_bar.setOnMenuItemClickListener(this)
         adapter = GroupAdapter(requireContext())
diff --git a/app/src/main/res/menu/book_group_manage.xml b/app/src/main/res/menu/book_group_manage.xml
new file mode 100644
index 000000000..4c887905d
--- /dev/null
+++ b/app/src/main/res/menu/book_group_manage.xml
@@ -0,0 +1,27 @@
+<?xml version="1.0" encoding="utf-8"?>
+<menu xmlns:tools="http://schemas.android.com/tools"
+    xmlns:app="http://schemas.android.com/apk/res-auto"
+    xmlns:android="http://schemas.android.com/apk/res/android">
+
+    <item
+        android:id="@+id/menu_add"
+        android:icon="@drawable/ic_add"
+        android:title="@string/add_group"
+        app:showAsAction="always"
+        tools:ignore="AlwaysShowAction" />
+
+    <item
+        android:id="@+id/menu_group_local"
+        android:title="@string/local"
+        android:checkable="true"
+        android:checked="true"
+        app:showAsAction="never" />
+
+    <item
+        android:id="@+id/menu_group_audio"
+        android:title="@string/audio"
+        android:checkable="true"
+        android:checked="true"
+        app:showAsAction="never" />
+
+</menu>
\ No newline at end of file