Skip to content

Commit d828995

Browse files
committed
Update some changes from original repo
1 parent c060788 commit d828995

File tree

1 file changed

+17
-38
lines changed

1 file changed

+17
-38
lines changed

README.md

Lines changed: 17 additions & 38 deletions
Original file line numberDiff line numberDiff line change
@@ -3,7 +3,7 @@ Original Repository: [ryanmcdermott/clean-code-javascript](https://github.com/ry
33
# clean-code-javascript
44

55
## Mục lục
6-
1. [Giới thiệu](#gioi-thieu)
6+
1. [Giới thiệu](#giới-thiệu)
77
2. [Biến](#biến)
88
3. [Hàm](#hàm)
99
4. [Đối tượng và Cấu trúc dữ liệu](#đối-tượng-và-cấu-trúc-dữ-liệu)
@@ -29,17 +29,17 @@ nghiệm thu thập được qua nhiều năm của các tác giả của cuốn
2929

3030
Ngành kỹ thuật phần mềm chỉ phát triển được hơn 50 năm, và chúng ta vẫn
3131
đang học rất nhiều. Một khi kiến trúc phần mềm trở thành phổ biến, có lẽ sau đó
32-
chúng ta sẽ có thêm nhiều luật lệ khó hơn phải tuân theo. Còn giờ đây,
33-
hãy để những hướng dẫn này như là một tiêu chuẩn để đánh giá chất lượng các đoạn
32+
chúng ta sẽ có thêm nhiều luật lệ khó hơn phải tuân theo. Còn giờ đây,
33+
hãy để những hướng dẫn này như là một tiêu chuẩn để đánh giá chất lượng các đoạn
3434
code Javascript mà bạn và team của bạn tạo ra.
3535

36-
Biết những hướng dẫn này thôi sẽ không thể ngay lập tức làm bạn trở thành một
37-
lập trình viên phần mềm tốt hơn được, và làm việc với chúng trong nhiều năm
38-
cũng không có nghĩa bạn sẽ không gặp bất cứ sai lầm nào. Mỗi đoạn code bắt đầu
39-
như một bản thảo đầu tiên, giống như đất sét được nặn nhào và cho tới cuối cùng
36+
Biết những hướng dẫn này thôi sẽ không thể ngay lập tức làm bạn trở thành một
37+
lập trình viên phần mềm tốt hơn được, và làm việc với chúng trong nhiều năm
38+
cũng không có nghĩa bạn sẽ không gặp bất cứ sai lầm nào. Mỗi đoạn code bắt đầu
39+
như một bản thảo đầu tiên, giống như đất sét được nặn nhào và cho tới cuối cùng
4040
thì nó sẽ lộ diện hình hài. Cuối cùng, chúng ta gọt tỉa những khuyết điểm khi
41-
chúng ta xem xét lại nó cùng với các đồng nghiệp.
42-
Đừng để bản thân bạn bị đánh bại bởi những bản thảo đầu tiên,
41+
chúng ta xem xét lại nó cùng với các đồng nghiệp.
42+
Đừng để bản thân bạn bị đánh bại bởi những bản thảo đầu tiên,
4343
thứ mà vẫn cần phải được chỉnh sửa. Thay vào đó hãy đánh bại những dòng code.
4444

4545
## **Biến**
@@ -432,7 +432,7 @@ function showManagerList(managers) {
432432

433433
**Tốt:**
434434
```javascript
435-
function showList(employees) {
435+
function showEmployeeList(employees) {
436436
employees.forEach((employee) => {
437437
const expectedSalary = employee.calculateExpectedSalary();
438438
const experience = employee.getExperience();
@@ -1252,17 +1252,9 @@ class Shape {
12521252
}
12531253

12541254
class Rectangle extends Shape {
1255-
constructor() {
1255+
constructor(width, height) {
12561256
super();
1257-
this.width = 0;
1258-
this.height = 0;
1259-
}
1260-
1261-
setWidth(width) {
12621257
this.width = width;
1263-
}
1264-
1265-
setHeight(height) {
12661258
this.height = height;
12671259
}
12681260

@@ -1272,12 +1264,8 @@ class Rectangle extends Shape {
12721264
}
12731265

12741266
class Square extends Shape {
1275-
constructor() {
1267+
constructor(length) {
12761268
super();
1277-
this.length = 0;
1278-
}
1279-
1280-
setLength(length) {
12811269
this.length = length;
12821270
}
12831271

@@ -1288,21 +1276,12 @@ class Square extends Shape {
12881276

12891277
function renderLargeShapes(shapes) {
12901278
shapes.forEach((shape) => {
1291-
switch (shape.constructor.name) {
1292-
case 'Square':
1293-
shape.setLength(5);
1294-
break;
1295-
case 'Rectangle':
1296-
shape.setWidth(4);
1297-
shape.setHeight(5);
1298-
}
1299-
1300-
const area = shape.getArea();
1301-
shape.render(area);
1302-
});
1303-
}
1279+
const area = shape.getArea();
1280+
shape.render(area);
1281+
});
1282+
}
13041283

1305-
const shapes = [new Rectangle(), new Rectangle(), new Square()];
1284+
const shapes = [new Rectangle(4, 5), new Rectangle(4, 5), new Square(5)];
13061285
renderLargeShapes(shapes);
13071286
```
13081287
**[⬆ back to top](#mục-lục)**

0 commit comments

Comments
 (0)