我正在尝试改变这一点:

function twist() {
  this.settings = null;
  delete this.settings;
  this.whatever = null;
  this.something['hello'] = null;
  this.hello = "test";
}


到这个:

function twist() {
  delete this.settings;
  delete this.settings;
  delete this.whatever;
  delete this.something['hello'];
  this.hello = "test";
}


所以我为jscodeshift编写了以下codemod:

export default function transformer(file, api) {
  const j = api.jscodeshift;

  return j(file.source)
    .find(j.ExpressionStatement, {expression: j.BinaryExpression})
    .filter(p => p.value.expression.operator === "=")
    .filter(p => p.value.expression.right.type === "Literal" && p.value.expression.right.raw === "null")
    .filter(p => p.value.expression.left.type === "MemberExpression")
    .replaceWith(path => j.unaryExpression("delete", path.value.expression.left))
    //.filter(p => { console.log(p.value); return true;})
    .toSource();
}


但是我得到了错误:

{operator: delete, argument: [object Object], prefix: true, loc: null, type: UnaryExpression, comments: null} does not match type string

最佳答案

您的代码在做什么,就是用ExpressionStatement替换UnaryExpression。用Statements替换Expressions可能很麻烦。

它还寻找BinaryExpression,但是BinaryExpression没有=作为运算符。

实际上,您的过滤器应该是AssignmentExpression,因为您实际要做的是用AssignmentExpression替换ExpressionStatement内部的UnaryExpression,从而用另一个替换一个Expression

export default function transformer(file, api) {
  const j = api.jscodeshift;

  return j(file.source)
    .find(j.AssignmentExpression, {
      operator: "=",
      right: {
        type: "Literal",
        value: null
      }
    })
    .replaceWith(({ node: { left } }) => j.unaryExpression("delete", left))
    .toSource();
}


这是一个ASTExplorer示例:https://astexplorer.net/#/gist/afb441a9709f82cd6fc3ba2860c98823/6f21cdb12a5a43aa1ca460aaa40d9ef63e1ef4bc

09-25 16:37