Too many arguments passed to function call JS-W1038
Bug risk
Minor
5 months ago
—
6 months old
N takes no parameters, but 1 was given
11087 q(function () {
11088 q(n.s, n.c).ajaxForm(e);
11089 }))
11090 : N(11091 "terminating; zero elements found by selector" +11092 (q.isReady ? "" : " (DOM not ready)"),11093 ),11094 this
11095 );
11096 }),N takes no parameters, but 1 was given
11083 var n = { s: this.selector, c: this.context };
11084 return (
11085 !q.isReady && n.s
11086 ? (N("DOM not ready, queuing ajaxForm"),11087 q(function () {
11088 q(n.s, n.c).ajaxForm(e);
11089 }))N takes no parameters, but 1 was given
10991 } catch (e) {
10992 N("error caught: ", e), (r = "error"), (p.error = t = e || r);
10993 }
10994 p.aborted && (N("upload aborted"), (r = null)),10995 p.status &&
10996 (r =
10997 (200 <= p.status && p.status < 300) || 304 === p.statusN takes no parameters, but 2 were given
10989 (r = "parsererror"), (p.error = t = e || r);
10990 }
10991 } catch (e) {
10992 N("error caught: ", e), (r = "error"), (p.error = t = e || r);10993 }
10994 p.aborted && (N("upload aborted"), (r = null)),
10995 p.status &&N takes no parameters, but 1 was given
10943 --D)
10944 )
10945 return (
10946 N("requeing onLoad callback, DOM not available"),10947 void setTimeout(A, 250)
10948 );
10949 var n = S.body ? S.body : S.documentElement;
Description
JavaScript allows passing more arguments to a function than needed. However, doing so makes your code less readable, and may not produce the result you would expect from the function call.
Bad Practice
function mult(a, b, c) {
return a * b * c;
}
mult(1, 2, 3, 4); // '4' will be ignored, and the result will be '6'.
mult(...nums); // this is OK.
Recommended
function mult(a, b, c) {
return a * b * c;
}
mult(1, 2, 3);
mult(...nums); // this is OK.