VS Code的产品做的很优秀,其源码也质量颇高,清晰、整洁、富有美感。
下面是 src\vs\workbench\common\notifications.ts 文件中的两段代码,大家感受一下:
get sticky(): boolean {
if (this._sticky) {
return true; // explicitly sticky
}
const hasActions = this.hasActions;
if (
(hasActions && this._severity === Severity.Error) || // notification errors with actions are sticky
(!hasActions && this._expanded) || // notifications that got expanded are sticky
(this._progress && !this._progress.state.done) // notifications with running progress are sticky
) {
return true;
}
return false; // not sticky
}
是不是感觉命名准确、层次分明、逻辑清晰,看起来一目了然。
逻辑块之间用空行分隔,多个判断条件的时候,每个条件单独成行。
变量定义紧跟变量使用,放到一个逻辑块里面。
不太能直接看出含义的地方加注释说明。
还有这段:
private get hasActions(): boolean {
if (!this._actions) {
return false;
}
if (!this._actions.primary) {
return false;
}
return this._actions.primary.length > 0;
}
大家注意看它的判断语句的写法,这两条判断其实可以写成一个,但是作者没有这么做,而是分开来写,这样更加清晰,易于阅读和理解。
另外,先判错,再判对,避免代码嵌套,也是个常用的最佳实践。
像这样的代码,在VS Code源码中比比皆是,常看看有助于提高代码品味和编程水平。
欢迎关注公众号:清晰编程,获取更多精彩内容
标签:Code,return,hasActions,之美,actions,sticky,源码,._ From: https://blog.csdn.net/wannianchuan/article/details/139539839