Math: Cannot add two variables

[Lezalith]

Code: Select all

$ InTeaching = "Skill1"
$ Skill1Value = 1
$ Skill1ValueOld = 0
$ MoodAngry = 4
$ MoodHappy = -2
$ MoodAffection = 0
$ TeachingMorning = 0

if InTeaching == "Skill1":
        $ Skill1ValueOld = Skill1Value
        $ MoodAffection = (MoodAngry + MoodHappy)
        $ TeachingMorning = [(MoodAffection + 6)*1.66667]
        $ Skill1Value = (Skill1ValueOld + TeachingMorning)
    else:
        pass

I do not get what isn't understandable here. First, Skill1Value is reflected into Skill1ValueOld. Then, MoodAffection is counted (=2), is calculated in TeachingMorning(=13,33336), and Skill1Value is then calculated (=14,33336).

I get error on the "$ Skill1Value = (Skill1ValueOld + TeachingMorning)" line. TypeError: unsupported operand type(s) for +: 'int' and 'RevertableList'.
What I got from that is, either Skill1ValueOld or TeachingMorning is not an integer value, but when I remove their addition, run it and check developer menu, both of them integers are.
And that line is written the exactly same way as addition of MoodAngry and MoodHappy, and I don't get errors there! What the hell am I doing wrong?!

[Zetsubou]

The problem is this line: $ TeachingMorning = [(MoodAffection + 6)*1.66667]
Those square brackets mean TeachingMorning is a list.
So remove the square brackets, round the result (it'll be a float) to int (might need to import the math module), then you'll be able to add them.

Unrelated, but your else clause needs to be indented to the same level as the if statement.

[Onishion]

Yeah, just do

Code: Select all

        $ TeachingMorning = int((MoodAffection + 6)*1.66667)

Also, in that context, you wouldn't need to establish "Skill1ValueOld," this would work just as well:

Code: Select all

        
        $ Skill1Value = Skill1Value + TeachingMorning

It won't get confused by that. You could even just make it a single line if you prefer,

Code: Select all

        
        $ Skill1Value = Skill1Value + (int((MoodAngry + MoodHappy+ 6)*1.66667)) 

Might be an unnecessary parentheses in there, I tend to err on the side of caution. In any case, that would work, unless you intended to use those other variables someplace else.

Oh, and also, if you're initializing variables in that first bit, it's a good idea to instead use

Code: Select all

default Skill1Value = 1

Using the default thing is a good idea, because it means that if the variable does not exist, even if they are loading a save from the middle of the game, it'll be initialized to that value.