Каскадный · [ Стандартный ] · Линейный РПГ текстовый файтинг, Закрытый бета тест

[Alexander87]

Пройдите по ссылке, нажмите кнопку Run
Вытяните экран мышкой снизу вверх и наслаждайтесь.

Игра на двоих.
1) Введите имена игроков.
2) Подберите класс, оружие и броню.
3) Выберете рандомные доп. характеристики.
4) Вступайте в бою друг с другом.



https://onlinegdb.com/Hy9iKBP27

Категорически прошу всячески указывать на все кривости, ничтожности и косяки игры.
Так-же прошу Вас подбросить идей по поводу классовых умений и умений за манну. Каких умений не хватает.
Подскажите если баланс кривой.
Без Вас мне не справиться!!!

Сообщение отредактировал Alexander87 - Среда, 31 октября 2018, 18:26

[Oreanor]

на этом месте я сдался )


ps: "не" с глаголами пишется раздельно

Сообщение отредактировал Oreanor - Среда, 31 октября 2018, 14:45

[Alexander87]

на этом месте я сдался )

Заменю на просто Enter
надо 1 нажать просто.

[Nik1]

Alexander87, сорян. Дропнул на выборе брони. Ты хоть картинки туда вставь что ли. Пока совсем не весело(.

[Ol_]

Дропнул на выборе брони.

Зря, там на втором уровне фаталити делать можно

[Oreanor]

Зря, там на втором уровне фаталити делать можно

а аскишный мультик в конце есть?

[Alexander87]

Alexander87, сорян. Дропнул на выборе брони. Ты хоть картинки туда вставь что ли. Пока совсем не весело(.

Так текстовая игра ведь))

[Nik1]

Так текстовая игра ведь))

Ну набей картинку символами тогда.

[Ol_]

а аскишный мультик в конце есть?

Не знаю, я еще до конца не дошел. На 4-м уровне вообще жесть начинается

[Landis]

Как вы в это играете, вообще ?))

[Ol_]

Как вы в это играете, вообще ?))

Не играл, но осуждаю

[Landis]

Не играл, но осуждаю

Так я не осуждаю. Я справшиваю-играете как.

[Nik1]

Так я не осуждаю. Я справшиваю-играете как.

Игра на выносливость. Кто дольше в ней пробыл, тот победил.

[Alexander87]

Я понимаю что игра кривая, не красивая и ужасно оптимизированна, я просто пытался хоть-что-то написать, я старался но первый блин ... комом. Я понимаю что Вы пытаетесь обосрать игру и это заслуженно.
Но прошу хотя бы указывать на конкретные косяки.
2 месяца назад я впервые начал серьезно увлекаться программированием ... и это плоды моего познания. Они ничтожны, но я стараюсь.
Возможно я когда нибуть и смогу создать игру от которой Вы не захотите блевать, но это не сейчас.
Поэтому постарайтесь хотя-бы попробовать и немного уделить времени моему детещу. И указать на явные и самые жесткие недостатки.
А программистов прошу указать где код слишком кривой, ведь даже Вы когда-то были очень слабы в програмировании.

[Nik1]

Я понимаю что игра кривая, не красивая и ужасно оптимизированна, я просто пытался хоть-что-то написать, я старался но первый блин ... комом. Я понимаю что Вы пытаетесь обосрать игру и это заслуженно.
Но прошу хотя бы указывать на конкретные косяки.
2 месяца назад я впервые начал серьезно увлекаться программированием ... и это плоды моего познания. Они ничтожны, но я стараюсь.
Возможно я когда нибуть и смогу создать игру от которой Вы не захотите блевать, но это не сейчас.
Поэтому постарайтесь хотя-бы попробовать и немного уделить времени моему детещу. И указать на явные и самые жесткие недостатки.
А программистов прошу указать где код слишком кривой, ведь даже Вы когда-то были очень слабы в програмировании.

Не хотел тебя обидеть или обосрать твое детище. Просто ты попросил указать на косяки, я так и сделал. Пока это скучно. Добавь описательной части что ли, как в текстовый квестах. Не просто выберите оружие, а вы спускаетесь по скрипучим ступеням в оружейную и берете оружие... Ну хоть как то развеселить, если нет визуала.
Технически циферки нажимаются нормально. Ты молодец.

Сообщение отредактировал Nik1 - Среда, 31 октября 2018, 18:24

[Alexander87]

Не хотел тебя обидеть или обосрать твое детище. Просто ты попросил указать на косяки, я так и сделал. Пока это скучно. Добавь описательной части что ли, как в текстовый квестах. Не просто выберите оружие, а вы спускаетесь по скрипучим ступеням в оружейную и берете оружие... Ну хоть как то развеселить, если нет визуала.
Технически циферки нажимаются нормально. Ты молодец.

Спасибо!, хорошая идея!

[Oreanor]

Не отчаивайся, первые блины они всегда такие.

[УльтраБлокС]

Огромная гора вложенных списков, то, что можно было вынести в классы почему-то на функциях сделано. Выглядит код как какой-то монструозный код на C, только на 3-ю минуту я понял, что это таки Python. Не надо так.

[Akell]

касательно кода. слишком он спутанный и непонятный. это плохо и лишние сложности.

предлагаю:
0) pep8
1) почитать про enum и вдобавок придумать там нормальные структуры данных, чтобы с ними было легко и приятно работать. а то у тебя всё через хардкоженные индексы в двумерных массивах.
2) т.к. тебе нужно обрабатывать ввод, то тебе нужны методы (функции или процедуры, как угодно эт называй) для обработки ввода и проверки его на ошибки. очень стоит вычленить это в отдельное что-то. например в модуль назвать это utils и туда пихать.
3) тоже самое что с п2, только касательно вывода информации.
4) очень много фактически одинаковых строк, прям 1 к 1. почитай про DRY (don't repeat yourself) и постарайся применять.
5) касательно общей архитектуры и структуры проекта и вообще. начинай думать и двигаться в направлении вычленения модулей, сущностей (классов/обьектов) и их аспектов/свойств и прочего, ищи что там у них есть общего и что можно вытащить в отдельные сущности. и переделай всё в направлении обьектно-ориентированного подхода. это позволит проще вносить изменения, улучшать, исправлять, тестировать код. просто сейчас у тебя всё в кашу смешано.
поясню о чем речь. в идеале как должно быть: ты хочешь добавить какую-то новую возможность. например в нового игребального персонажа или механику. и ты сам себя спрашиваешь, в скольких местах нужно переделывать? в идеале таких мест должно быть меньше чем пальцев на руке. (в самом недостижимом идеале - в 1одном месте)
у тебя просто сейчас очень сильно связанный код, с ним сложно работать и непродуктивно. от этого надо избавляться.
6) после того как появится куча модулей и прочего, можно приделать например юнит-тестики, и прочие тесты. это позволит иметь хоть какую-то уверенность в констистенстности вносимых изменений в код.
7) и замечинание про чрезмерное увлечение операторов ветвление (условным оператором). я понимаю что ты хочешь сделать, но вввидимо ввиду недостаточного опыта и навыков, ты используешь не те инструменты.
тебе бы очень подошло обьектно-ориентированное что-то.
тогда бы вместо перебирания вручную всех вариаций в if-else ты бы просто писал Player.takes_damage_from(Another_player) или Player.attacks(Another_player) и всё. Не нужно было бы для каждого отдельного вида бойца перебирать всё это через if. Вся конкретика происходящего была бы внутри бойца встроена уже. Сейчас фактически ты на себя взял всю ту низкоуровневую работу и те нюансы работы кода, которые бы не пришлось вручную делать при ООП. Т.е. ты вручную некие аналоги виртуальных фукнций к неким обьектам маппишь. Т.е. ты изобретаешь некое отдаленое подобие ООП с кучей лишних недостатков, вместо напрямую применения ООП. Что тебе мешает делать это явно? ты просто делаешь лишнюю рутинную работу, в которой легко наделать ошибок.
8) Не привязывайся к коду (в смысле симпатий), будь готов выкинуть его и переделать в любой момент.

знаешь что тебе бы очень подошло, какой-нибудь github или что-то такое, где можно и документацию вести и обссуждение и прочее. Тут неочень подходяще и просто неудобно это.

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

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

если конкретика интересует, то например вот: 734-738

CODE

       player_inoy = warrior[(warrior.index(player))-1]
       if player_inoy[7][0]<player_inoy[7][2]:
           player_inoy[7][0]=player_inoy[7][0]+player_inoy[7][1]
           if player_inoy[7][0]>player_inoy[7][2]:
               player_inoy[7][0]=player_inoy[7][2]

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

ты в целом используешь подход: очень простые структуры данных, но очень сложный код для работы с ними. а надо наоборот. "умные" и хитрые структуры, но работать с ними максимально простым и понятным кодом. чтобы не было непонятного битов, индексов и прочего туда-сюда.

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

таких аналогичных проблем ещё много в коде.

можно ещё долго и упорно искать недостатки, в целом, ещё кроме наименования переменных и общего стиля особых дефектов нет (если принять во внимание уровень твоих навыков).

по итогам: код конечно не production ready, требуется значительная проработка архитектуры и структуры и дизайна проекта.

[Alexander87]

касательно кода. слишком он спутанный и непонятный. это плохо и лишние сложности.

предлагаю:
0) pep8
1) почитать про enum и вдобавок придумать там нормальные структуры данных, чтобы с ними было легко и приятно работать. а то у тебя всё через хардкоженные индексы в двумерных массивах.
2) т.к. тебе нужно обрабатывать ввод, то тебе нужны методы (функции или процедуры, как угодно эт называй) для обработки ввода и проверки его на ошибки. очень стоит вычленить это в отдельное что-то. например в модуль назвать это utils и туда пихать.
3) тоже самое что с п2, только касательно вывода информации.
4) очень много фактически одинаковых строк, прям 1 к 1. почитай про DRY (don't repeat yourself) и постарайся применять.
5) касательно общей архитектуры и структуры проекта и вообще. начинай думать и двигаться в направлении вычленения модулей, сущностей (классов/обьектов) и их аспектов/свойств и прочего, ищи что там у них есть общего и что можно вытащить в отдельные сущности. и переделай всё в направлении обьектно-ориентированного подхода. это позволит проще вносить изменения, улучшать, исправлять, тестировать код. просто сейчас у тебя всё в кашу смешано.
поясню о чем речь. в идеале как должно быть: ты хочешь добавить какую-то новую возможность. например в нового игребального персонажа или механику. и ты сам себя спрашиваешь, в скольких местах нужно переделывать? в идеале таких мест должно быть меньше чем пальцев на руке. (в самом недостижимом идеале - в 1одном месте)
у тебя просто сейчас очень сильно связанный код, с ним сложно работать и непродуктивно. от этого надо избавляться.
6) после того как появится куча модулей и прочего, можно приделать например юнит-тестики, и прочие тесты. это позволит иметь хоть какую-то уверенность в констистенстности вносимых изменений в код.
7) и замечинание про чрезмерное увлечение операторов ветвление (условным оператором). я понимаю что ты хочешь сделать, но вввидимо ввиду недостаточного опыта и навыков, ты используешь не те инструменты.
тебе бы очень подошло обьектно-ориентированное что-то.
тогда бы вместо перебирания вручную всех вариаций в if-else ты бы просто писал Player.takes_damage_from(Another_player) или Player.attacks(Another_player) и всё. Не нужно было бы для каждого отдельного вида бойца перебирать всё это через if. Вся конкретика происходящего была бы внутри бойца встроена уже. Сейчас фактически ты на себя взял всю ту низкоуровневую работу и те нюансы работы кода, которые бы не пришлось вручную делать при ООП. Т.е. ты вручную некие аналоги виртуальных фукнций к неким обьектам маппишь. Т.е. ты изобретаешь некое отдаленое подобие ООП с кучей лишних недостатков, вместо напрямую применения ООП. Что тебе мешает делать это явно? ты просто делаешь лишнюю рутинную работу, в которой легко наделать ошибок.
8) Не привязывайся к коду (в смысле симпатий), будь готов выкинуть его и переделать в любой момент.

знаешь что тебе бы очень подошло, какой-нибудь github или что-то такое, где можно и документацию вести и обссуждение и прочее. Тут неочень подходяще и просто неудобно это.

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

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

если конкретика интересует, то например вот: 734-738

CODE

       player_inoy = warrior[(warrior.index(player))-1]
       if player_inoy[7][0]<player_inoy[7][2]:
           player_inoy[7][0]=player_inoy[7][0]+player_inoy[7][1]
           if player_inoy[7][0]>player_inoy[7][2]:
               player_inoy[7][0]=player_inoy[7][2]

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

ты в целом используешь подход: очень простые структуры данных, но очень сложный код для работы с ними. а надо наоборот. "умные" и хитрые структуры, но работать с ними максимально простым и понятным кодом. чтобы не было непонятного битов, индексов и прочего туда-сюда.

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

таких аналогичных проблем ещё много в коде.

можно ещё долго и упорно искать недостатки, в целом, ещё кроме наименования переменных и общего стиля особых дефектов нет (если принять во внимание уровень твоих навыков).

по итогам: код конечно не production ready, требуется значительная проработка архитектуры и структуры и дизайна проекта.

Спасибо большое!!! Вы указали куда двигаться и что для этого использовать. Вы мне очень помогли. Буду искать и учить новые методы реализации. Вы верно подметили что я использовал самые базовые неэффективные способы для реализации задуманного. Буду учить ООП и новые библиотеки что могут мне помочь в упращении кода. Завтра разберусь подробно в Вашей оценке и начну изучать неведомые мне вещи, что были Вами описаны.