-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
For reviewing my code #1
Comments
واقعا ممنونم ازتون که وقت گذاشتید و کد رو نگاه کردید، خیلییییی لطف کردید❤
حتما رعایت میکنم اینهایی که گفتید رو
…On Thu, Aug 5, 2021 at 12:06 AM Ahmad ***@***.***> wrote:
من بصورت اجمالی نگاه کردم و با اجازه، چند تا نکته به نظرم رسید:
1- سعی کنید app.js همیشه خلوت باشه. مثلا خط های 25-18 و 66-43 میتونه توی
یه فایل جدا باشه و در app.js فقط import کنید.
2-از naming convention <https://github.com/felixge/node-style-guide> های
nodejs تبعیت بشه بهتره.(نام فایل ها و فولدرها بعضی با حروف بزرگ شروع شدن و
بعضی با حروف کوچک )
3- validation های سمت سرور حیاتی هستن و حتما باید باشن. (مثلا
express-validator <https://www.npmjs.com/package/express-validator> )
یکی از نقاط قوت کدها استفاده از async و await هست.
موفق باشی
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#1 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/AL5OTIZCLY3GSFLBYUA4YHLT3GJDHANCNFSM5BPIGC4Q>
.
|
من بصورت اجمالی نگاه کردم و با اجازه، چند تا نکته به نظرم رسید: یکی از نقاط قوت کدها استفاده از async و await هست. |
If there was anything to edit or adding to the project, you can use this issue to do that
The text was updated successfully, but these errors were encountered: