Не могли бы вы оценить мой код на Python?

Question

[Никита Гитович]

Итак мои знания это: База
Вот решил написать "Типо" игру (На самом деле это даже не игра, а отсылка)
Ну так вот к чему я это клоню, хочу попросить вас оценить мой код: Пойдёт ли он для новичка?, и есть ли у меня вообще будущее)(По возможности оценивайте по 10-ти бальной шкале)

аvailable_locations = ["Chaes", "Kordon"]    #Доступные локации
monolit = []                          #Списки в которых скоро будут персонажи
stalker = []
mutant = []

print(f"Welcome to колхозный S.T.A.L.K.E.R ёпт\nДоступные локации:{аvailable_locations}\n")       #Выводит список доступных локаций
a = input("Введите локацию: Пример(Kordon)\n>")
territory = {a: "Территория обитания меченого"}                  #Словарь в котором записана локация она понадобится для улсовных операторов
print("==============================================================")

if "Chaes" in territory: 
    for i in range(20):      
        mon = {"Name": "Monolit", "HP": 120, "Rang": "Master"}     #Создаю характеристику Монолита...
        monolit.append(mon)                   #Добавляю монолитовцев в выше указанный список
    for m in monolit:          #Вывожу название и характеристику монолитов
        print(m)
    print("Уууууу ты нарвался на монолитов")

elif "Kordon" in territory:
    for i in range(10):
        odinochki = {"Name": "Stalker", "HP": 100, "Rang": "Noob"}        #Создаю характеристику Сталкеров
        stalker.append(odinochki)       #Добавляю сталкеров в вышеуказанный список
    for o in stalker:             #Вывожу название и характеристику
        print(o)

    for x in range(5):            
        mut = {"Name": "Psevdosobaki", "HP": 90, "Rang": "None"}   
        mutant.append(mut)          #Добавляю мутантов в вышеуказанный список
    print("\n")
    for p in mutant:          #Вывожу Название и характеристику
        print(p)
    print("Не мельтеши особо и трогать тебя не будут ")

print("==============================================================")

pl = input()     #Это для того чтоб после завершения программы консоль сразу не выкидывала

Comments

[Jacen11]

и это все? что тут оценивать то? консольная недоприложенька. Такое в школе обычно пишут.

[Jacen11]

pl = input() #Это для того чтоб после завершения программы консоль сразу не выкидывала

в вечный цикл нужно добавить весь код с ожиданием кода и тогда можно будет "ходить" вечно

нейминг у переменных плохой, как вообще можно понять из "stalker" что это список сталкеров?

ооп никак не используется

ну как первая программа после пары дней изучения программирования на 8 потянет, для человека уже умеющего прогать, но не на питоне около 2, если бы синеор это написал 0,00001 из 10

[Никита Гитович]

Jacen11, Хах, если бы в школах вообще изучали пайтон.
А то у нас этот паскаль с лазарусом.

[Никита Гитович]

Jacen11, Как я писал GNUBack-у я не разбираюсь (пока-что) в функиях и ооп но в скором времени постараюсь разузнать.

[GavriKos]

Никита Гитович, ну так а что плохого в паскале с лазарусом? Язык как язык

[Jacen11]

Никита Гитович, я в школе учил qBasic и мои игры были посложнее, с большей логикой, недографикой, анимацией и музыкой. Ох, как же много там было goto)
язык особого значения при старте изучения роли не играет, тут же простое консольное приложение, любой язык общего назначения годится для этого

Answer 1

[bituke]

LOCATIONS = ["Chaes", "Kordon"]

def print_available_locations(locations):
    print(f"Welcome to колхозный S.T.A.L.K.E.R ёпт\nДоступные локации:{locations}\n")

def create_monolit(n=20):
    return [{"Name": "Monolit", "HP": 120, "Rang": "Master"} for i in range(n)]

def create_stalker(n=10):
    return [{"Name": "Stalker", "HP": 100, "Rang": "Noob"} for i in range(n)]

def create_mutant(n=5):
    return [{"Name": "Psevdosobaki", "HP": 90, "Rang": "None"} for i in range(n)]

def handle_location(location):
    if location == "Chaes":
        monolit = create_monolit()
        for mon in monolit:
            print(mon)
        print("Уууууу ты нарвался на монолитов")
    elif location == "Kordon":
        stalker = create_stalker()
        for st in stalker:
            print(st)
        mutant = create_mutant()
        for mu in mutant:
            print(mu)
        print("Не мельтеши особо и трогать тебя не будут ")
    else:
        print("Некорректное название локации")

def main():
    print_available_locations(LOCATIONS)
    location = input("Введите локацию: Пример(Kordon)\n>")
    print("==============================================================")
    handle_location(location)
    print("==============================================================")
    input()

if __name__ == '__main__':
    main()

Comments

[bituke]

а лучше так:

class Location:
    def __init__(self, name, description):
        self.name = name
        self.description = description

    def __str__(self):
        return f"{self.name}: {self.description}"

class Character:
    def __init__(self, name, hp, rank):
        self.name = name
        self.hp = hp
        self.rank = rank

    def __str__(self):
        return f"{self.rank} {self.name} (HP: {self.hp})"

class Game:
    def __init__(self):
        self.locations = [Location("Chaes", "Территория монолитов"), 
                          Location("Kordon", "Передовая сталкеров")]
        self.characters = []

    def start(self):
        print("Welcome to колхозный S.T.A.L.K.E.R ёпт")
        print("Доступные локации:")
        for location in self.locations:
            print(location)
        print("")

        location_name = input("Введите локацию (например, Kordon): ")
        location = None
        for l in self.locations:
            if l.name == location_name:
                location = l
                break

        if location is None:
            print("Неизвестная локация!")
            return

        print("==============================================================")

        if location.name == "Chaes":
            for i in range(20):
                monolit = Character("Monolit", 120, "Master")
                self.characters.append(monolit)

            print("Уууууу ты нарвался на монолитов:")
            for c in self.characters:
                print(c)

        elif location.name == "Kordon":
            for i in range(10):
                stalker = Character("Stalker", 100, "Noob")
                self.characters.append(stalker)

            for i in range(5):
                mutant = Character("Psevdosobaki", 90, "None")
                self.characters.append(mutant)

            print("Не мельтеши особо и трогать тебя не будут:")
            for c in self.characters:
                print(c)

        print("==============================================================")

        input("Нажмите Enter, чтобы выйти")

if __name__ == "__main__":
    game = Game()
    game.start()

[bituke]

а чтобы разобраться в коде копай интернет и доки

[Hemul GM]

bituke, локации тоже нужно обернуть в классы. Если локация подразумевает конкретный сценарий, то его там и описать.
Регистрировать классы локаций. В рабочих алгоритмах не должно быть вообще строк

Answer 2

[Bright144]

class Pers:

    def __init__(self, Name: str, HP: int, Rang: str) -> None:
        self.Name = Name
        self.HP = HP
        self.Rang = Rang

    def print_stat(self):
        print(self.__dict__)

аvailable_locations = ["Chaes", "Kordon"]    #Доступные локации
print(f"Welcome to колхозный S.T.A.L.K.E.R ёпт\nДоступные локации:{аvailable_locations}\n")       #Выводит список доступных локаций
a = input("Введите локацию: Пример(Kordon)\n>")
territory = {a: "Территория обитания меченого"}                  #Словарь в котором записана локация она понадобится для улсовных операторов
print("==============================================================")

if "Chaes" in territory:
    monolit = [Pers("Monolit", 120, "Master") for i in range(20)]
    [i.print_stat() for i in monolit]
    print("Уууууу ты нарвался на монолитов")
elif "Kordon" in territory:
    odinochki = [Pers("Stalker", 100, "Noob") for i in range(10)]
    [i.print_stat() for i in odinochki]
    print("\n")
    mut = [Pers("Psevdosobaki", 90, "None") for i in range(5)]
    [i.print_stat() for i in mut]
    print()
    print("Не мельтеши особо и трогать тебя не будут ")
print("==============================================================")
pl = input()     #Это для того чтоб после завершения программы консоль сразу не выкидывала

персонажов лучше сделать объектами, потом с легкости можешь добавить функционал или статы на классе Pers. На пример стат брони, урона, метод нанесение и получение урона.