Question

我在节点客户端上使用superagent来解除POST或PATCH这样的请求（这很难看！）：

在if-else中的两个代码块之间，唯一的区别是http verb和API端点，在patch请求的情况下，还附加了/:id网址。

if ( typeof resourceId === 'undefined' ){
  request
    .post('http://localhost:8080/api/resources/')
    .send(JSON.stringify(resource_json))
    .set('Accept', 'application/json')
    .set('Content-Type', 'application/json')
    .set('Authorization', 'Token token=3S4pREbMkMoEGG6zHeUJ7Qtt')
    .end(function(err, res) {
      if (!err && res.ok) {
        console.log(chalk.bold.cyan('Resource created successfully'));
        packageJson.resource_id = res.body.id;
        fs.writeFileSync('package.json', JSON.stringify(packageJson));
        process.exit(0);
      }

      var errorMessage;

      if (res && res.status === 401) {
        errorMessage = "Authentication failed! Bad username/password?";
      } else if (err) {
        errorMessage = err;
      } else {
        errorMessage = res.text;
      }
      console.error(chalk.red(errorMessage));
      process.exit(1);
    });
  } else {
  request
    .patch('http://localhost:8080/api/reources/' + resourceId)
    .send(JSON.stringify(resource_json))
    .set('Accept', 'application/json')
    .set('Content-Type', 'application/json')
    .set('Authorization', 'Token token=3S4pREbMkMoEGG6zHeUJ7Qtt')
    .end(function(err, res) {
      if (!err && res.ok) {
        console.log(chalk.bold.cyan('Resource created successfully'));
        packageJson.book_id = res.body.id;
        fs.writeFileSync('package.json', JSON.stringify(packageJson));
        process.exit(0);
      }

      var errorMessage;

      if (res && res.status === 401) {
        errorMessage = "Authentication failed! Bad username/password?";
      } else if (err) {
        errorMessage = err;
      } else {
        errorMessage = res.text;
      }
      console.error(chalk.red(errorMessage));
      process.exit(1);
    });

  }

我该如何干这个？

Answer 1

可以通过变量访问方法。

const baseUrl = 'http://localhost:8080/api/resources/';
const isNew = undefined === resourceId;
const method = isNew ? 'post' : 'patch';
const url = isNew ? baseUrl : basseUrl + resourceId;

request[method](url).send().end();

或更具可读性。

const baseUrl = 'http://localhost:8080/api/resources/';
const isNew = undefined === resourceId;
let method, url;

if(isNew) {
    method = 'post'
    url = baseUrl;
} else {
    method = 'patch'
    url = baseUrl + resourceId;
}

request[method](url).send().end();

我该如何干掉以下代码？

1 个答案: