search

KARPED1EM / TownOfNext

An AmongUs mod with learning & love
https://tonx.cc
GNU General Public License v3.0
58 stars 32 forks source link

Special Message Blocking #164

Closed Slok7565 closed 9 months ago

Slok7565 commented 9 months ago

A Feature Just Like "TryHideMsg" But More Powerful

KARPED1EM commented 9 months ago

问题

Role Base 调用的 OnSendMessage() 方法似乎被删除了，并且没有被放到任何新的代码中 此外，赌怪和审判的代码就是在这类方法中，也就是说，Chat Manager 里的代码会被重复运作

建议

基本上就是说，鉴于新Base下赌怪、法官、黑手党等指令系职业的代码被写在Base中，因此需要对代码进行调整。 这个Pr的主意是不错的，并且在TOHE时期已经有规划制作（因为一些原因被废稿了） 如果您比较繁忙没有时间对代码进行修改，请关闭这个Pr，我可能会在后续开发中加入这个功能。 我会等待您关闭Pr或修改后重新考虑合并。

Slok7565 commented 9 months ago

我再改一下吧，话说我想问下为什么TrialMsg写成动态不是静态？

Slok7565 commented 9 months ago

ok了，测试过没问题

Slok7565 commented 9 months ago

咔哥看看a

KARPED1EM commented 9 months ago

你直接给法官新建一个对象？∑(;°Д°) 我不用测试我就知道法官现在有无限审判的bug了

Slok7565 commented 9 months ago

之前实例化的时候习惯了，想都没想就这么整了)

Slok7565 commented 9 months ago

好了

KARPED1EM commented 9 months ago

如果说在使用职业类的时候直接创建一个的话，那么这跟静态类没有区别了。 如果你不知道我在说啥，你测试下看看会不会有无限审判的Bug，等到没有之后再说吧

Slok7565 commented 9 months ago

意思是明白，新base的这个也确实不会解决，至少在旧Base的TOHEX没问题，我等下个版本看看你怎么解决没有OnSendMessage的情况我再加吧 除了这个其他都没什么问题，我先关了吧

Slok7565 commented 9 months ago

意思是明白，技术力不够所以新base的这个也确实不会解决，至少在旧Base的TOHEX没问题，我等下个版本借鉴下你怎么解决没有OnSendMessage的情况我再加吧 除了这个其他都没什么问题，我先关了吧

KARPED1EM commented 9 months ago

dev版本正在做chatmanager了，到时候你看看