Skip to content

Commit 9ebaa2e

Browse files
authored
Merge pull request #4 from Broote/patch-4
Update README.md
2 parents 79a353a + f6c3bda commit 9ebaa2e

File tree

1 file changed

+19
-16
lines changed

1 file changed

+19
-16
lines changed

README.md

Lines changed: 19 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -868,6 +868,23 @@ account.setBalance(100);
868868

869869
**Плохо:**
870870
```javascript
871+
872+
const Employee = function(name) {
873+
this.name = name;
874+
};
875+
876+
Employee.prototype.getName = function getName() {
877+
return this.name;
878+
};
879+
880+
const employee = new Employee('John Doe');
881+
console.log(`Employee name: ${employee.getName()}`); // Employee name: John Doe
882+
delete employee.name;
883+
console.log(`Employee name: ${employee.getName()}`); // Employee name: undefined
884+
```
885+
886+
**Хорошо:**
887+
```javascript
871888
function makeEmployee(name) {
872889
return {
873890
getName() {
@@ -877,23 +894,9 @@ function makeEmployee(name) {
877894
}
878895

879896
const employee = makeEmployee('John Doe');
880-
console.log(`Employee name: ${employee.getName()}`); // Имя сотрудника: John Doe
881-
delete employee.name;
882-
console.log(`Employee name: ${employee.getName()}`); // Имя сотрудника: undefined
883-
```
884-
885-
**Хорошо:**
886-
```javascript
887-
const Employee = function (name) {
888-
this.getName = function getName() {
889-
return name;
890-
};
891-
};
892-
893-
const employee = new Employee('John Doe');
894-
console.log(`Employee name: ${employee.getName()}`); // Имя сотрудника: John Doe
897+
console.log(`Employee name: ${employee.getName()}`); // Employee name: John Doe
895898
delete employee.name;
896-
console.log(`Employee name: ${employee.getName()}`); // Имя сотрудника: John Doe
899+
console.log(`Employee name: ${employee.getName()}`); // Employee name: John Doe
897900
```
898901
**[⬆ Назад к Содержанию](https://github.com/maksugr/clean-code-javascript#Содержание)**
899902

0 commit comments

Comments
 (0)