Review the Services

[wewoor]

我觉得这个讨论是有提醒到我的，#176 (comment)

togglePanelMaximized 本身更倾向是对 事件（Action） 本身对一种响应, Service 里面还是偏主动操作的抽象比较好；

我看了下，其他 Service 也有这种命名。后面统一再 Review 下 Service吧

Originally posted by @wewoor in #176 (comment)

[wewoor]

Services

Useless methods or the naming refactoring:

IActivityBarService #289

• addBar -> add (支持单个/数组类型)
• toggleContextMenuCheckStatus

ISidebarService

• addPane -> add

IExplorerService #291

• editPanel -> updatePanels

IFolderTreeService

• getFileIconByExtensionName (useless?)
• getRootFolderByRootId 如果是明确的 getRootFolder 接口，应该是不需要后面这段话的
• onUpdateFileContent (useless?)
• initTree (useless?)

IMenuBarService

• addRootMenu 和 initiMenu 重叠，且命名 initiMenu -> initiMenus

INotificationService

• showHideNotifications -> toggleNotification

IProblemsService

showHideProblems -> toggleProblems
移除常规增删改查方法的 Problems 后缀

ISearchService

• 声明 Properties 成方法（Function)
• 方法名重构

IStatusBarService

• 重构 append 方法 （隐藏 left/right 逻辑）
• 重构 remove 逻辑
• updateItem -> update

参考

• 命名：https://github.com/DTStack/molecule/blob/main/CONTRIBUTING.md

[mortalYoung]

IExplorerService

1. deletePanel -> removePanel
2. updateRender 不确定这个需不需要

IMenuBarService

有一个多余的 getState 方法声明

INotificationService

这个不考虑改成 add remove update 吗