پیشنهاد / نظر /ابهام / سوال / گفتگو

[BaseMax]

سلام

[BaseMax]

templates

بنظرم بهتره جوری بشه که نیاز به پوشه templates نباشه چون الان یک کپی از readme داریم. اینطوری بروزنگه داشتن دو فایل مشکل تر میشه.
یا میشه موقع اجرا یک کپی از readme گرفته بشه اگر واقعا نیازه هرچند بهتره که اینطور نباشد.

[BaseMax]

Max Base, [9/22/2022 3:26 PM]
@app.route("/")
def mainPage():
return render_template("README.html")

Max Base, [9/22/2022 3:26 PM]
میتونید اینجا بروید با پایتون محتوای فایل ریدمی رو بخونید

Max Base, [9/22/2022 3:26 PM]
اینطوری دیگه نیاز به template نیست.

[BaseMax]

lastNames = open("./last-names.txt", mode="r", encoding="utf-8").read().splitlines()
MaleFirstNames = open("./male-first-names.txt", mode="r", encoding="utf-8").read().splitlines()
FemaleFirstNames = open("./female-first-names.txt", mode="r", encoding="utf-8").read().splitlines()

اینجا سه متغییر داریم. دو تا با حرف بزرگ شروع شده اما اولی با حرف کوچک بزرگ شده.
لطفا از یک استاندارد و فرمول برای اسم های متغییر استفاده کنید.

مگر اینکه متغییری ثابت باشد که در اون صورت بهتر است کامل حرف بزرگ باشد. مثل:

const PI = 3.14

[BaseMax]

سلام. چرا با وجود اینکه یک تابعی به اسم getLastName داریم اما از اون تابع برای CreateUser کردن استفاده نکردید و دوباره کد رو تکرار کردید؟

[BaseMax]

while len(users) != count:
    for user in range(count):
        users.add(CreateUser(flag=gender))

متوجه نمیشوم که چه نیازی به ان for داخل حلقه اصلی بوده؟ مطمئن هستید که نیاز بوده؟

[BaseMax]

def getName(flag=0):
name = None

اینجا هم بنظرم از متغییر name هیچ استفاده ای نمی شود. کلا چه نیازی بهش هست؟ اگر نیست می توانید حذف کنید.