this post was submitted on 30 Nov 2024
16 points (100.0% liked)

Python

6628 readers
54 users here now

Welcome to the Python community on the programming.dev Lemmy instance!

๐Ÿ“… Events

PastNovember 2023

October 2023

July 2023

August 2023

September 2023

๐Ÿ Python project:
๐Ÿ’“ Python Community:
โœจ Python Ecosystem:
๐ŸŒŒ Fediverse
Communities
Projects
Feeds

founded 2 years ago
MODERATORS
 

My third blog post. I added variation in the tense of the generated text as well as another possible action for the character.

top 6 comments
sorted by: hot top controversial new old
[โ€“] 0101100101@programming.dev 1 points 3 days ago* (last edited 3 days ago) (1 children)

I've only glanced down your code and am not familiar with your previous efforts. Combine insulting and stirred-up to one class. "CharacterTraits" or so. This then makes it easier to add more traits like happiness, warmongering, intelligence, luck etc.

[โ€“] ecstatic_chance@mander.xyz 1 points 2 days ago* (last edited 2 days ago) (1 children)

They currently have the parent class "Action" for their common attributes and methods. Does that cover what you are suggesting?

[โ€“] 0101100101@programming.dev 1 points 2 days ago (1 children)

They currently have the parent class โ€œActionโ€ for their common attributes and methods. Does that cover what you are suggesting?

I didn't see, but if they want a trait that has a completely set of different methods? I'm not a big fan of interface-esque classes unless the API is absolutely solid. In this case it would not be.

[โ€“] ecstatic_chance@mander.xyz 1 points 2 days ago (1 children)

A set of different methods would warrant assignment to a different class. So far no character traits warrant that. What is the alternative to interface-esque classes? If you could provide a dummy code example that would be great. What would make the API solid? Thank you for all the suggestions.

[โ€“] 0101100101@programming.dev 2 points 2 days ago* (last edited 2 days ago) (1 children)

Why not have one class that has a level for each trait, which are scored 0-100, 0-10 etc. so... self.luck = 7.3 self.anger = 4.0 and so on. And then there's one method that determines the action. That's going to be so much easier to maintain, extend, and work with.

class CharacterTraits:
  def __init__(self, luck, anger, magic, ...):
    self.luck = luck
    self.anger = anger
    # and so on

    # maybe keep a list of previous actions which could inform the next action state
    self.history = []

  def get_action(self):
    # do whatever to decide action
    action = ...

    # then add it to history
    self.history.append(action)

    return action

and then the calling code determines what's output to the screen. So, internally, the class is just responsible for one thing - hte business logic. Maybe another class Game could be responsible for outputting the strings, taking user input etc. If the UI were to change at a later date, the CharacterTraits class stays the same, but only the Game class would need to be modified. Instead of - as I understand it - all the classes currently would have to be updated (a maintenance nightmare!)

I only had a really quick look down the code so I may be missing the point entirely, but that's the direction I would go down.

EDIT: the get_action method could take in some args, like opponent_traits or some kind of situation, maybe even add additional methods like is_lucky to return a bool as to whether a situation that requires luck has been successful or not. Another method could be has_won_fight(opponent_traits) and the method compares strength, luck, magic whatever, to the opponent to decide whether the character has won. And so on. By keeping it simple like this, it's a lot easier to work with!

Thank you for all the suggestions. I will take them into deep consideration.