search

Gandi-IDE / custom-extension

GNU Lesser General Public License v2.1
43 stars 58 forks source link

提交 更多运算（numpro.js） 拓展 #197

Closed MoreBugOfDog closed 3 months ago

MoreBugOfDog commented 6 months ago

修复了所有review出的问题 请审核 20240406

Arkos123 commented 6 months ago

那个设置哪些积木显示的窗口怎么没了，我很喜欢那个功能（）

Arkos123 commented 6 months ago

VUW}ZEAWZ7YH7PBZOU~TWFL 这些积木感觉都可以删除吧（因为已经有字符串扩展了）

Arkos123 commented 6 months ago

image 这些感觉也可以不要了，有进制扩展了 image

fath11 commented 6 months ago

Here are my suggestions image image image image

everything else can stay

MoreBugOfDog commented 5 months ago

zh-cn: 与 bilioicik 开发完成未PR的拓展合并，已经经过 bilioicik 的同意。 修复了review出的问题。

en: Merged with bilioicik to finish development of un-PR'd extensions, with bilioicik's permission. Fixed problems with review. @fath11 @sylarhcn @FurryR

MoreBugOfDog commented 5 months ago

关于以小驼峰来命名函数这点实在改不动，已经在ccw上跟HGN商量过了，HGN表示能接受。下次命名一定符合规范。

× HGN √HCN

bilioicik commented 5 months ago

关于以小驼峰来命名函数这点实在改不动，已经在ccw上跟HCN商量过了，HCN表示能接受。下次命名一定符合规范。

fath11 commented 5 months ago

image this will 100% trigger a lot of people, it will be better if you use neutral example inputs that have no chance of making people mad.

fath11 commented 5 months ago

My problems with this extension still stand. All of these functionalities can already be done with existing extensions. You should focus more on #195

MoreBugOfDog commented 5 months ago

我对这个扩展的问题仍然存在。所有这些功能都已经可以通过现有的扩展来完成。你应该更多地关注#195

Hello fath11! We will be further culling or hiding duplicate blocks for this extension. Thank you for your review. For PR https://github.com/Gandi-IDE/custom-extension/pull/195 , we are making changes to him, thank you for your attention.

bilioicik commented 5 months ago

最新更改： numpro重复的if更改为了简洁的switch和case； numpro与已有扩展雷同的关于大于小于判断功能被删除； 字符互换积木中的参数默认值分别更改为了ccw、c、w； 新出了四个关于数组中元素大小的精简化积木； 改为使用 IIFE 格式。

MoreBugOfDog commented 5 months ago

All issues reviewed have been resolved, please review again. 已经解决所有审核出的问题，请再次审查。 @sylarhcn @fath11 @FurryR

bilioicik commented 5 months ago

额，我还是更喜欢原来的descp。首先这个扩展还涉及字符串数组对象，跟这个描述的“数学”本身关系并不多；其次这个描述既长又太夸张了。我选择的“Green for Go”出自Thomas Greenberg的专辑《Something Simple》的第17首；PR#195的“Moving Onwards”也是出自《Something Simple Acoustic》；封面更是印证这点，算是一个彩蛋了。

bilioicik commented 5 months ago

我对这个扩展的问题仍然存在。所有这些功能都已经可以通过现有的扩展来完成。你应该更多地关注#195

我刚改正没按小驼峰命名的错误，计划编写使角色按照函数图像轨迹运动的积木，积木类型conditional，这样它运动过程中可以运行别的积木。

MoreBugOfDog commented 5 months ago

@fath11 @sylarhcn @FurryR please review