Require template literals instead of string concatenation JS-0246
Anti-pattern
Minor
8 months ago
—
3 years old
Unexpected string concatenation
1537
1538 Tutorial.sequelize.fn(
1539 "ST_GeomFromGeoJSON",
1540 '{"type":"Polygon","coordinates":[[' + geojson + "]]}"1541 ),
1542 Tutorial.sequelize.col("location")
1543 ),Unexpected string concatenation
1471 })
1472 .catch((err) => {
1473 res.status(500).send({
1474 message: "Could not delete Tutorial with id=" + id,1475 });
1476 });
1477};Unexpected string concatenation
1446 })
1447 .catch((err) => {
1448 res.status(500).send({
1449 message: "Error updating Tutorial with id=" + id,1450 });
1451 });
1452};Unexpected string concatenation
1314 })
1315 .catch((err) => {
1316 res.status(500).send({
1317 message: "Error retrieving Tutorial with id=" + id,1318 });
1319 });
1320};Unexpected string concatenation
968 "ST_Contains",
969 Tutorial.sequelize.fn(
970 "ST_GeomFromGeoJSON",
971 '{"type":"Polygon","coordinates":[[' + geojson + "]]}" 972 ),
973 Tutorial.sequelize.col("location")
974 ),
Description
In ES2015 (ES6), we can use template literals instead of string concatenation.
var str = `Hello, ${name}!`;
Bad Practice
var str = "Hello, " + name + "!";
var str = "Time: " + (12 * 60 * 60 * 1000);
Recommended
var str = "Hello World!";
var str = `Hello, ${name}!`;
var str = `Time: ${12 * 60 * 60 * 1000}`;
var str = "Hello, " + "World!";