Skip to content
This repository has been archived by the owner on Feb 11, 2023. It is now read-only.

[U] 优化机器人 #24

Merged
merged 8 commits into from
Oct 19, 2019
Merged

[U] 优化机器人 #24

merged 8 commits into from
Oct 19, 2019

Conversation

Taskeren
Copy link
Contributor

No description provided.

@Taskeren
[U] 添加消息组件转换
907f9e6 
新功能允许你将命令参数转换为一个个消息组件，以便调用。

从"nitu2003/Extra"项目中导入
Merged from Project "nitu2003/ExtraBot"
@Taskeren
Merge remote-tracking branch 'origin/master'
99c878f
@Taskeren
[U] 添加两种新的富文本消息并添加对应的解析方法
1556e59
@Taskeren
Copy link
Contributor Author

使用文档

@Taskeren Taskeren changed the title [U] 添加消息组件转换 [U] 优化机器人 Oct 12, 2019
@hykilpikonna
Copy link
Member

PicqBotX 好像有一套 Component 了...
加上你的就是两套 Component 了xx
(cc.moecraft.icq.sender.message.components)

格式问题的话...

  1. 请把 TAB 改成四个空格w (https://javarevisited.blogspot.com/2016/06/how-to-make-eclipse-use-spaces-instead.html)
  2. JavaDocs 介绍后面 @param 前面加个换行w

@Taskeren
Copy link
Contributor Author

Taskeren commented Oct 12, 2019 •
edited
Loading

PicqBotX 好像有一套 Component 了...
加上你的就是两套 Component 了xx
(cc.moecraft.icq.sender.message.components)

格式问题的话...

  1. 请把 TAB 改成四个空格w (https://javarevisited.blogspot.com/2016/06/how-to-make-eclipse-use-spaces-instead.html)
  2. JavaDocs 介绍后面 @param 前面加个换行w
  1. 自带的那套是用来发送消息的，这套是专门用来接收消息的。
  2. 习惯了Tab，改不过来了。

@hykilpikonna
Copy link
Member

可以整合成一套 Component 嘛?

@Taskeren
Copy link
Contributor Author

可以整合成一套 Component 嘛?

感觉会不太方便，因为一部分组件只能接收，不能发送，比如ComponentRich。

@hykilpikonna
Copy link
Member

格式的话... 是看到你有的地方四个空格有的地方 TAB 哇x
image
image
习惯四个空格的话可以全都四个空格嘛w?

@hykilpikonna
Copy link
Member

可以整合成一套 Component 嘛?

感觉会不太方便，因为一部分组件只能接收，不能发送，比如ComponentRich。

那就把 ComponentRich 的公开构造器去掉好了w
然后接收到的时候反射构造好了w

@Taskeren
Copy link
Contributor Author

格式的话... 是看到你有的地方四个空格有的地方 TAB 哇x
image
image
习惯四个空格的话可以全都四个空格嘛w?

上面打错字了，是习惯Tab。

至于这个一会空格，一会Tab，是因为IDEA自己检测的。
在我原创部分是使用的Tab，原来就用的空格的地方它改成空格。（无奈脸）

@Taskeren
Copy link
Contributor Author

可以整合成一套 Component 嘛?

感觉会不太方便，因为一部分组件只能接收，不能发送，比如ComponentRich。

那就把 ComponentRich 的公开构造器去掉好了w
然后接收到的时候反射构造好了w

（反射不是有性能损耗嘛
而且发送归发送，接收归接收不是挺好的，因为本来就不是一个目的而创建的嘛。

@hykilpikonna
Copy link
Member

可以整合成一套 Component 嘛?

感觉会不太方便，因为一部分组件只能接收，不能发送，比如ComponentRich。

那就把 ComponentRich 的公开构造器去掉好了w
然后接收到的时候反射构造好了w

（反射不是有性能损耗嘛
而且发送归发送，接收归接收不是挺好的，因为本来就不是一个目的而创建的嘛。

但是这样改的话大部分 Component 还是重复的嘛w
不反射的话就 package-private 构造器好了,
然后在 package 里面写一个用来创建实例的类w
或者把父类分成 SendableComponent 和普通的 Component 好了x

@Taskeren
Copy link
Contributor Author

可以整合成一套 Component 嘛?

感觉会不太方便，因为一部分组件只能接收，不能发送，比如ComponentRich。

那就把 ComponentRich 的公开构造器去掉好了w
然后接收到的时候反射构造好了w

（反射不是有性能损耗嘛
而且发送归发送，接收归接收不是挺好的，因为本来就不是一个目的而创建的嘛。

但是这样改的话大部分 Component 还是重复的嘛w
不反射的话就 package-private 构造器好了,
然后在 package 里面写一个用来创建实例的类w
或者把父类分成 SendableComponent 和普通的 Component 好了x

有空改（咕

@hykilpikonna hykilpikonna merged commit cf41e5d into HyDevelop:master Oct 19, 2019
@hykilpikonna
Copy link
Member

等等这样不是还是有两套嘛?

image

@hykilpikonna
Copy link
Member

可以放到同一个包里嘛?

@Taskeren
Copy link
Contributor Author

老的可以删了，然后把漏下的俩给补上。
没加ComponentShare和ComponentMusic是因为我打死用不出来

@hykilpikonna
Copy link
Member

旧的那一套不能删呀...
不然就是破坏性更新了...
以前用这个库的人全都要重新改了...
但是我觉得这个功能不值得做一次破坏性更新.
所以只能想办法让以前的接口全都一致的情况下把新的加进去了w

hykilpikonna added a commit that referenced this pull request Feb 11, 2023
@hykilpikonna
Merge pull request #24 from nitu2003/master
d2c4f37 
[U] 优化机器人
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@Taskeren @hykilpikonna