eslint-plugin-better-codes 1.0.2 → 1.0.4

package/README.md

@@ -55,7 +55,12 @@ module.exports = {
 - 箭头函数
 - 方法函数
 
-除去React组件函数
+不校验的情况：
+- React组件函数
+- 函数体只有一行代码的函数
+- 用于map/forEach中返回元素/组件用于渲染的箭头函数
+
+<!-- TODO:待解决的问题：针对返回结果为true/false的函数，使用try...catch进入catch分支之后改返回false还是true还是不返回(undefined)，因为按照之前的逻辑报错之后后续的代码不应再执行的，但是现在会直接继续走false/true的逻辑，造成逻辑错误 -->
 
 ## 许可证
 

package/libs/function-with-try-catch-code.js

@@ -136,7 +136,53 @@ function isReactComponentFunction(node) {
   }
 }
 
-export default {
+// 判断函数体是否只有一行
+function isOnlyOneLine(body) {
+  try {
+    if (!body || body.type !== "BlockStatement") return false;
+    const stmts = body.body || [];
+    return stmts.length === 1 && stmts[0].type === "ExpressionStatement";
+  } catch (err) {
+    return false;
+  }
+}
+
+// 判断函数是否作为 array methods 的回调（如 map/forEach）直接返回用于渲染的元素/组件
+function isArrayMethodCallback(node) {
+  try {
+    if (!node || !node.parent) return false;
+    // 父节点应为 CallExpression，且当前函数是该调用的一个参数
+    const p = node.parent;
+    if (p.type !== "CallExpression") return false;
+    const args = p.arguments || [];
+    if (!args.includes(node)) return false;
+    const callee = p.callee;
+    if (!callee) return false;
+    // 支持形如 data.map(...) 或 data?.map(...)
+    if (callee.type === "MemberExpression") {
+      const prop = callee.property;
+      const name = prop && (prop.name || (prop.value && String(prop.value)));
+      // 判断返回语句
+      const returnNode = node.body?.body?.at(-1);
+      if (
+        name &&
+        /^(map|forEach)$/i.test(name) &&
+        returnNode &&
+        returnNode.type === "ReturnStatement" &&
+        JSON.stringify(returnNode.argument?.raw || "").startsWith("<") &&
+        JSON.stringify(returnNode.argument?.raw || "").endsWith(">")
+      ) {
+        return true;
+      }
+    }
+
+    return false;
+  } catch (err) {
+    console.log(err);
+  }
+}
+
+module.exports =  {
   meta: {
     type: "suggestion",
     docs: {
@@ -154,7 +200,12 @@ export default {
           (node.type === "ArrowFunctionExpression" &&
             node.body &&
             node.body.type !== "BlockStatement") ||
-          isReactComponentFunction(node)
+          // 作为 React 组件/组件函数跳过
+          isReactComponentFunction(node) ||
+          // 只有一行表达式的函数跳过
+          isOnlyOneLine(node.body) ||
+          // 作为 array 方法回调（如 map/forEach）用于渲染时跳过
+          isArrayMethodCallback(node)
         ) {
           return;
         }

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "eslint-plugin-better-codes",
-  "version": "1.0.2",
+  "version": "1.0.4",
   "main": "index.js",
   "scripts": {
     "test": "npx eslint ./test.js"