2022 Q4修订 #64

otakustay · 2022-11-28T09:34:15Z

No description provided.

PeterPanZH · 2022-12-06T08:53:24Z

typescript.md

+}
+```
+
+### 【强制】要求适当处理类似 Promise 的语句 @佘采华


warn 的规则应该是 [推荐]

PeterPanZH · 2022-12-06T09:00:10Z

html-style-guide.md

 #### [强制] 标签使用必须符合标签嵌套规则。

 解释：

 比如 `div` 不得置于 `p` 中，`tbody` 必须置于 `table` 中。

-详细的标签嵌套规则参见[HTML DTD](http://www.cs.tut.fi/~jkorpela/html5.dtd)中的 `Elements` 定义部分。
-
+详细的标签嵌套规则参见[HTMLf外](https://html.spec.whatwg.org/multipage/)中各元素定义的“Content model”说明。


[HTMLf外]是啥

PeterPanZH · 2022-12-06T09:02:40Z

html-style-guide.md

@@ -837,6 +567,18 @@ viewport meta tag 可以设置可视区域的宽度和初始缩放大小，避
 </table>
 ```

+#### [强制] 新窗口/标签页打开的链接需要按业务需要添加安全相关的 `rel` 属性值 `noopener`。


是不是需要增加 noreferrer？

不，noreferrer是业务诉求，noopener是安全诉求，不考虑在编码规范上去要求业务怎么样

PeterPanZH · 2022-12-06T09:05:12Z

javascript-style-guide.md

+
+对于函数声明（Function Declaration）、类声明（Class Declaration），它们不属于语句（Statement），因此不需要以分号结束。
+
+对于 `export default` 语句，参考其后部分的类型，如果是函数、类声明，同样不需要分号。


所有export都适用，不一定要export default吧

我想named export应该不会有人误解，export default比较特殊些所以特别说明

PeterPanZH · 2022-12-06T09:11:42Z

javascript-style-guide.md


-除去不定长参数以外，函数具备不同逻辑意义的参数建议控制在 `6` 个以内，过多参数会导致维护难度增大。
+##### [建议] 一个函数的参数控制在 **6**** 个以内。


多了两个星号

FlyingFatPenguin · 2022-12-06T11:34:13Z

typescript.md

+}
+```
+
+### [强制]禁止不必要的类型约束


相关讨论：#65

我理解 lint 的职责一方面是统一代码格式，另一方面就是引导使用者写出简洁的代码。
像上面的一系列『禁止不必要……』就很好的体现了这一点。与『不必要的断言』、『不必要的参数』不同的地方是，类型约束在某些场景下是必要的，就像我上面举的例子中，它关乎到能否正确解析代码。即便这个必要是由于语法缺陷导致的，但这确实是一个会遇到的场景。
与它类似的，就像 const a = ({}) 是不必要的圆括号，而 () => ({}) 是有必要的一样的。在 tsx 文件中写带有泛型的箭头函数，这个事情本身没有什么不合理之处。如果强制禁止，会否让开发者太过受限，而背离了写简洁代码的初衷。

参考 typescript-eslint/typescript-eslint#4062 ，官方似乎并没有修正的想法，我们可以考虑ts和tsx使用不同的规则

otakustay added 5 commits November 28, 2022 17:33

更新 JavaScript 规范

b966578

更新HTML规范

923878f

更新React规范

9c4e3dc

更新ESnext规范

d8fee68

增加TypeScript编码规范

3ba34c3

PeterPanZH suggested changes Dec 6, 2022

View reviewed changes

PeterPanZH reviewed Dec 6, 2022

View reviewed changes

FlyingFatPenguin reviewed Dec 6, 2022

View reviewed changes

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

2022 Q4修订 #64

2022 Q4修订 #64

otakustay commented Nov 28, 2022

PeterPanZH Dec 6, 2022

PeterPanZH Dec 6, 2022

PeterPanZH Dec 6, 2022

otakustay Dec 6, 2022

PeterPanZH Dec 6, 2022

otakustay Dec 6, 2022

PeterPanZH Dec 6, 2022

FlyingFatPenguin Dec 6, 2022

otakustay Dec 7, 2022


		对于函数声明（Function Declaration）、类声明（Class Declaration），它们不属于语句（Statement），因此不需要以分号结束。

		对于 `export default` 语句，参考其后部分的类型，如果是函数、类声明，同样不需要分号。


		除去不定长参数以外，函数具备不同逻辑意义的参数建议控制在 `6` 个以内，过多参数会导致维护难度增大。
		##### [建议] 一个函数的参数控制在 6** 个以内。

2022 Q4修订 #64

Are you sure you want to change the base?

2022 Q4修订 #64

Conversation

otakustay commented Nov 28, 2022

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment

Choose a reason for hiding this comment