Dav*_*ers 5 javascript ecmascript-6 eslint
我有以下代码.它完美地工作,并且return this由于链接,调用者需要每个部分:
module.exports = function(res){
return {
success: function(content, contentType, resultCode) {
sendResponse(
res,
content,
validateContentType(contentType),
validateResultCode(resultCode||'ok')
)
return this
},
error: function(resultCode, content, contentType){
sendResponse(
res,
content,
validateContentType(contentType),
validateResultCode(resultCode||'bad_request')
)
return this
},
end: function(callback){
res.end()
callback&&callback()
return this
}
}
}
Run Code Online (Sandbox Code Playgroud)
问题是我需要我的代码来传递一套ESLint规则.上面的代码失败,其中第36行是第一行代码:
36:18 warning Unexpected unnamed function func-names
36:26 error Missing space before function parentheses space-before-function-paren
36:31 error Missing space before opening brace space-before-blocks
38:12 warning Unexpected unnamed method 'success' func-names
38:20 error Missing space before function parentheses space-before-function-paren
43:34 error Infix operators must be spaced space-infix-ops
43:41 error Missing trailing comma comma-dangle
44:5 error Missing semicolon semi
45:15 error Missing semicolon semi
47:10 warning Unexpected unnamed method 'error' func-names
47:18 error Missing space before function parentheses space-before-function-paren
47:52 error Missing space before opening brace space-before-blocks
52:34 error Infix operators must be spaced space-infix-ops
52:50 error Missing trailing comma comma-dangle
53:5 error Missing semicolon semi
54:15 error Missing semicolon semi
56:8 warning Unexpected unnamed method 'end' func-names
56:16 error Missing space before function parentheses space-before-function-paren
56:26 error Missing space before opening brace space-before-blocks
57:13 error Missing semicolon semi
58:4 error Expected an assignment or function call and instead saw an expression no-unused-expressions
58:12 error Infix operators must be spaced space-infix-ops
58:24 error Missing semicolon semi
59:15 error Missing semicolon semi
60:4 error Missing trailing comma comma-dangle
61:3 error Missing semicolon semi
62:2 error Missing semicolon semi
Run Code Online (Sandbox Code Playgroud)
以下代码段传递规则,但不会return this因为规则会产生另一个ESLint错误,no-invalid-this并且严重,不再有效,打破了链接:
module.exports = res => ({
success: (content, contentType, resultCode) => {
sendResponse(
res,
content,
validateContentType(contentType),
validateResultCode(resultCode || 'ok'),
);
},
error: (resultCode, content, contentType) => {
sendResponse(
res,
content,
validateContentType(contentType),
validateResultCode(resultCode || 'bad_request'),
);
},
end: (callback) => {
res.end();
if (callback) {
callback();
return undefined;
}
return undefined;
},
});
Run Code Online (Sandbox Code Playgroud)
我的问题是我如何调整第二个片段(或者第一个片段)在功能上与第一个片段相同但仍然通过ESLint规则?如何正确返回以允许与呼叫者链接?
我的.eslintrc.json:
{
"extends": "airbnb-base",
"rules": {
// use tabs, not spaces, and in switch statements the case statement should indent again (the default is to be level with the switch)
"indent": [ "error", "tab", { "SwitchCase": 1 } ],
// if you want to put a blank line at the beginning or end of a block, knock yourself out
"padded-blocks": [ "off" ],
// i like tabs. besides, we set indent to require them
"no-tabs": [ "off" ],
// seriously, who cares if there's a blank line at the end of the file or not?
"eol-last": [ "off" ],
// sometimes having a long single line makes sense, this also seems buggy and inconsistent, so we ignore it
"max-len": [ "off" ],
// webstorm repeatedly tries to add it for us. it's easier not to fight it, even if it's not required.
"strict": [ "off" ],
// when setting the property of an object, you can specify the name even if it's unnecessary (ie: { foo: foo })
"object-shorthand": [ "off" ],
// unused vars are an error, except for function arguments.
// particularly with callbacks we may not use all the args, but we still like knowing they're available
"no-unused-vars": [ "error", { "vars": "all", "args": "none", "ignoreRestSiblings": true } ],
// you don't have to use operator assignment if you don't want to
"operator-assignment": [ "warn" ],
// we don't want else to be on the same line as the closing } of an if statement
"brace-style": [ "error", "stroustrup" ],
// warn about overly complex code that you may want to refactor
"complexity": [ "warn", 15 ],
// it's possible that implicit coercion is not what you intended. webstorm warns about it, so should we
"no-implicit-coercion": [ "warn" ],
// if you're using 'this' somewhere that isn't a class you're probably doing something wrong
"no-invalid-this": [ "error" ],
// if you're not modifying the variable used in a loop condition, you've probably done something wrong...
"no-unmodified-loop-condition": [ "warn" ],
// don't use .call or .apply when you don't need to
"no-useless-call": [ "warn" ],
// we want to slap you if you don't update your jsdoc, but not necessarily break one of your fingers
"valid-jsdoc": [ "warn" ],
// forgetting to return after calling a callback is an easy mistake to make, so we'll warn you if you are
"callback-return": [ "warn" ]
}
}
Run Code Online (Sandbox Code Playgroud)
正如eslint 规则的文档提到的那样,您需要为匿名函数表达式命名(这样它们就不再是匿名的)。
顺便说一句,这是一个很好的做法,因为您可以更好地调试它,并在需要时将其作为递归函数调用。
那为什么不直接给他们起名字呢?
module.exports = function exports(res) {
return {
success: function success(content, contentType, resultCode) {
sendResponse(
res,
content,
validateContentType(contentType),
validateResultCode(resultCode || 'ok'),
);
return this;
},
error: function error(resultCode, content, contentType) {
sendResponse(
res,
content,
validateContentType(contentType),
validateResultCode(resultCode || 'bad_request'),
);
return this;
},
end: function end(callback) {
res.end();
if (callback) {
callback();
return undefined;
}
return this;
},
}
}
Run Code Online (Sandbox Code Playgroud)
| 归档时间: |
|
| 查看次数: |
279 次 |
| 最近记录: |