Usage of comma operators should be avoided JS-0090
Anti-pattern
Minor
5 months ago
—
6 months old
Unexpected use of comma operator
383 t.zoomedHd.removeAttribute("sizes"),
384 t.zoomedHd.removeAttribute("loading"),
385 (g = t.zoomedHd.addEventListener("load", function () {
386 t.zoomedHd.removeEventListener("load", g), 387 t.zoomedHd.classList.add("medium-zoom-image--opened"),
388 t.zoomedHd.addEventListener("click", r),
389 document.body.appendChild(t.zoomedHd),Unexpected use of comma operator
51 { isDarkTheme: t, themeMode: n }
52 );
53 }
54 (t = !0),55 (n = !1),
56 (window.wc = { darkLightEnabled: t, isSiteThemeDark: n }),
57 window.netlifyIdentity &&Unexpected use of comma operator
4261 (i.default.fn.toast = _e._jQueryInterface),
4262 (i.default.fn.toast.Constructor = _e),
4263 (i.default.fn.toast.noConflict = function () {
4264 return (i.default.fn.toast = me), _e._jQueryInterface;4265 }),
4266 (t.Alert = d),
4267 (t.Button = h),Unexpected use of comma operator
4258 t
4259 );
4260 })();
4261 (i.default.fn.toast = _e._jQueryInterface),4262 (i.default.fn.toast.Constructor = _e),
4263 (i.default.fn.toast.noConflict = function () {
4264 return (i.default.fn.toast = me), _e._jQueryInterface;Unexpected use of comma operator
4217 } else e();
4218 }),
4219 (e._clearTimeout = function () {
4220 clearTimeout(this._timeout), (this._timeout = null);4221 }),
4222 (t._jQueryInterface = function (e) {
4223 return this.each(function () {
Description
The comma operator includes multiple expressions where only one is expected. It evaluates each operand from left to right and returns the value of the last operand. However, this frequently obscures side effects, and its use is often an accident. Here are some examples of sequences:
Bad Practice
let a = (3, 5); // a = 5
a = b += 5, a + b;
while (a = next(), a && a.length);
(0, eval)("doSomething();");
res = doSomething(), val;
0, eval("doSomething();");
do {} while (doSomething(), !!test);
for (; doSomething(), !!test; );
if (doSomething(), !!test);
switch (val = func(), val) {}
while (val = func(), val < 42);
with (doSomething(), val) {}
Recommended
res = (doSomething(), val);
(0, eval)("doSomething();");
do {} while ((doSomething(), !!test));
for (i = 0, j = 10; i < j; i++, j--);
if ((doSomething(), !!test));
switch ((val = func(), val)) {}
while ((val = func(), val < 42));
with ((doSomething(), val)) {}