Bánh Mì Chấm Sữa
Junior Member
ý bác là up github lên hay sao áupload code lên xem nào

ý bác là up github lên hay sao áupload code lên xem nào

yep.ý bác là up github lên hay sao á![]()
Để lát e hộp cho bác nhé, github e sợ lộ info mốt ko lên đây nói xấu dcyep.
p/s: theo tôi được biết thì kiểu take home này khá phổ biến nhé, bởi vì người ta cần tuyển là người làm được việc (dù là dùng google) chứ không phải nói xạo.

vl không biết tạo throwaway account à.Để lát e hộp cho bác nhé, github e sợ lộ info mốt ko lên đây nói xấu dc
Trình thì e nghĩ nếu e dỡ thì nó gửi cái mail chửi cũng dc, đằng này im im bực mình quá.
ko bít throwaway account bác ơi, e cũng gà để e mò thửvl không biết tạo throwaway account à.
p/s: không liên quan cơ mà chợt nhớ ra vụ inbox: ai inbox tôi hỏi về cntt tôi nói luôn là tôi không trả lời đâu nhé. nếu muốn thì tạo topic mở, rảnh mọi người vào trả lời, chứ tôi không dành thời gian đi trả lời từng bạn, lãng phí.
nhiều năm sinh hoạt voz qua các nick phiền nhất là vụ này.
tạo một cái account github khác rồi đăng lên thôi có gì mà không biết?ko bít throwaway account bác ơi, e cũng gà để e mò thử
À, hiểu rồi, e nghĩ sâu xa quátạo một cái account github khác rồi đăng lên thôi có gì mà không biết?

Đúng là ko ai chỉ e vụvl không ai dạy bạn là nhét code vào trongsrc/à
mà.gitđâu rồi, không có version control à.
tôi nghĩ đến đây thì người ta đã chả muốn xem tiếp rồi![]()
src/ thật 
.git e nhớ lưu thông tin local ko có tác dụng khi share nên bỏ ra ko gửi, bác cần để làm gì hay sao á 
export function khó viết lắm sao mà toàn export const?
))
e đặt đại thôi bác, có vấn đề gì sao ạmà tôi có hơi tò mò là cái convention idParent, idChild là từ đâu vậy nhỉ?
lần đầu tiên thấy luôn, parentID thì không nói....

Còn chưa tới vòng deal lương bác àmà tôi có hơi tò mò là cái convention idParent, idChild là từ đâu vậy nhỉ?
lần đầu tiên thấy luôn, parentID thì không nói....
export functionkhó viết lắm sao mà toànexport const?
đặt tên function tại sao lại là pascal case? nhái c# à?
async/await có biết dùng không, sao callback kinh thế
readFileSync có vấn đề gì à mà không dùng lại dùng readFile lại bắt nest thêm một level nữa.
thêm combine với cái hungarian notation kiểu M$ nữa tôi đúng là phục luôn))
scoping của variable cũng sai lè, code thì thừa cả đống.
kết: kể ra thì chả đến nỗi nào nhưng tôi đoán là bạn đòi hơi cao cho nên người ta ignore luôn![]()

export function khó viết lắm sao mà toàn export const?
))




Mấy cái code thể dùng tools để phát hiện ngay khi mở code:
1 và 2 thì thấy có xài eslint mà đêr mấy cái lỗi này thì hơi chán.
- Var với let loạn lên.
- Chơi 3 blank line ở đầu files
- Sai chính tả (single, multi)
3 dùng vscode thì nên xài extension code spell checker
GetDeviceToken với GetDeviceTokenByLstParent giống nhau tới 99%, đơn giản thì dùng hàm map để gọi thằng getDevideToken là được rồi.
Thằng GetDeviceTokenOfGroup return 2 kiểu null với array, nên thằng gọi hàm phải check khá mệt, ví dụ bên nó xài filter hoặc map thì khi trả về null bị lỗi, sao không trả về 1 kiểu là array thôi, lúc length bằng 0 thì trả mảng rỗng là dc.
Chút rảnh comment thêm



đi làm lâu chưa, nhìn code hơi lủng củng

1 năm thím à, mà lúc đó e vô thì ứng tuyển C#, xong thử việc thì o team lead nghỉ nên phải tự học rùi làm lun phần nodeJS của ổng song song với cái C#. Làm tự kỷ 1 mình từ đó tới h.đi làm lâu chưa, nhìn code hơi lủng củng![]()


