Mock Code Review
Code Review
0
const splitProductReviewCMSContent = (htmlString?: string) => {
if (!htmlString) return []
const string = cloneObject(htmlString) as string
const find = /<div class="wp-block-media-text .*?(<\/p><\/div>)/g
const result = string.replace(/(\r|\n)/g, '').match(find)
return result || []
}
/path/to/this/file.ts
- Functionality 1
- Design 1
- Complexity
- Naming 1
- Comments 1
- Styles 1
- Dòng 5: method replace nên được đặt ở phía trên, cùng vị trí với method cloneObject
- Dòng 1: tại phần khai báo function, có thể gán giá trị mặc định cho htmlString là “”, từ đó có thể lượt bở điều kiện !htmlString.
- Dòng 7-9: có thể kết hợp hai dòng này lại, return trực tiếp kết quả từ method .replace() và điều kiện ở phía sau để làm ngắn gọn code.
const splitProductReviewCMSContent = (htmlString?: string) => {
if (!htmlString) return []
const string = cloneObject(htmlString) as string
const find = /<div class="wp-block-media-text .*?(<\/p><\/div>)/g
const result = string.replace(/(\r|\n)/g, '').match(find)
return result || []
}
/path/to/this/file.ts
- Functionality 1
- Design
- Complexity 1
- Naming
- Comments 1
- Styles
- Design 1
- Dòng 5: method replace nên được đặt ở phía trên, cùng vị trí với method cloneObject
- Dòng 1: tại phần khai báo function, có thể gán giá trị mặc định cho htmlString là “”, từ đó có thể lượt bở điều kiện !htmlString.
const splitProductReviewCMSContent = (htmlString?: string) => {
if (!htmlString) return []
const string = cloneObject(htmlString) as string
const find = /<div class="wp-block-media-text .*?(<\/p><\/div>)/g
const result = string.replace(/(\r|\n)/g, '').match(find)
return result || []
}
/path/to/this/file.ts
- Functionality 1
- Design
- Complexity 1
- Naming
- Comments 1
- Styles 1
- Design
- Dòng 5: method replace nên được đặt ở phía trên, cùng vị trí với method cloneObject
- Dòng 1: tại phần khai báo function, có thể gán giá trị mặc định cho htmlString là “”, từ đó có thể lượt bở điều kiện !htmlString.
/path/to/this/file.ts
- Functionality 1
- Design
- Complexity 1
- Naming
- Comments 1
- Styles 1
- Design
- Dòng 5: method replace nên được đặt ở phía trên, cùng vị trí với method cloneObject
- Dòng 1: tại phần khai báo function, có thể gán giá trị mặc định cho htmlString là “”, từ đó có thể lượt bở điều kiện !htmlString.
Related Posts
Leave a Reply Cancel reply
Categories
- Bài viết (1)
- Blog (24)
- Code Review (3)
- Course Introduction (2)
- CSS (4)
- CSS Grid (4)
- Javascript (2)
- Lesson (11)
- Lưu dữ liệu trên trình duyệt (1)
- Network Requests (1)
- OneDirect Projects (1)
- Product Review (3)
- Stage Content (16)
- WCS Courses Content (17)