Default parameters should be defined at the last JS-0106
Anti-pattern
Minor
3 months ago
—
a year old
Default parameters should be last
82 }
83
84 //X Z = X Y printer space
85 getVoxelSegments(voxelSize, voxelHeight, toolRadius = 0, additive) { 86 let pointArray = new Array();
87 let distance = Vector3.Distance(this.start, this.end);
88 if (distance < 0.1) return;
Description
Putting default parameter at last allows function calls to omit optional tail arguments.
// Correct: optional argument can be omitted
function createUser(id, isAdmin = false) {}
createUser("tabby")
// Incorrect: optional argument can **not** be omitted
function createUser(isAdmin = false, id) {}
createUser(undefined, "tabby")
Bad Practice
function f(a = 0, b) {}
function f(a, b = 0, c) {}
Recommended
function f(a, b = 0) {}