yes, but think about it, this code doesn't really make sense and is kind of wasteful. On one hand, you are calculating the *total* amount of armies but then you throw away the results at the end of every loop.

__cavalries += territory.cavalries__ really serves no purpose, you might as well just use __territory.cavalries__ and get rid of all the assignments.

On the other hand, you *might* want to display the **total** amount of armies a player owns, in which case you can rewrite the loop as:

    total_cavalries = 0
    total_infantries = 0
    total_artilleries = 0
    for territory in territories:
        total_cavalries   += territory.cavalries
        total_infantries  += territory.infantries
        total_artilleries += territory.artilleries
        print '%s:Cavlries: %sInfantries: %sArtilleries: %s' % (territory, territory.calvaries, territory.infantries, territory.artilleries)

\# print grand total here