Skip to content

Commit 348760e

Browse files
Merge pull request ryanmcdermott#173 from DavidVujic/suggestions
Maker function instead of Function constructor
2 parents eec04a7 + 37d929c commit 348760e

File tree

1 file changed

+7
-5
lines changed

1 file changed

+7
-5
lines changed

README.md

+7-5
Original file line numberDiff line numberDiff line change
@@ -1010,13 +1010,15 @@ console.log(`Employee name: ${employee.getName()}`); // Employee name: undefined
10101010

10111011
**Good:**
10121012
```javascript
1013-
const Employee = function (name) {
1014-
this.getName = function getName() {
1015-
return name;
1013+
function makeEmployee(name) {
1014+
return {
1015+
getName() {
1016+
return name;
1017+
},
10161018
};
1017-
};
1019+
}
10181020

1019-
const employee = new Employee('John Doe');
1021+
const employee = makeEmployee('John Doe');
10201022
console.log(`Employee name: ${employee.getName()}`); // Employee name: John Doe
10211023
delete employee.name;
10221024
console.log(`Employee name: ${employee.getName()}`); // Employee name: John Doe

0 commit comments

Comments
 (0)