'int' object n'est pas appelable

Vous utilisez numerator à la fois comme un nom de méthode (def numerator(...)) et membre nom de la variable (self.numerator = n). Utilisation set_numerator et set_denominator pour les noms de méthode et cela fonctionnera.

En passant, la version 2.6 de Python est doté d'un fraction de la classe.

OriginalL'auteur interjay

• Vous utilisez numerator à la fois comme un nom de méthode et un nom pour une instance de l'attribut. Puisque les méthodes sont stockés sur la classe, lorsque vous recherche de l'attribut, vous obtenez le nombre, pas la méthode. (Python rechercher des attributs de l'instance avant de regarder la classe.)

  C'est-à-dire que sur la ligne où vous dites f.numerator(2), il cherche f.numerator et constate qu'il est 0, puis tente de l'appeler que 0, qui, évidemment, ne devrait pas fonctionner.

• Si vous avez un but pratique pour ce code, vous pouvez utiliser le stdlib fractions module: http://docs.python.org/library/fractions.html

  • Ce qui est nouveau dans la version 2.6 de Python. Si j'avais besoin de représenter des fractions, mais a l'aide d'une version antérieure de Python, je serais probablement utiliser sympy's Rational type.

• Plus pratique valeur par défaut pour denominator est probablement 1. (De cette façon Fraction(5) de cinq, et non pas défini opération tendant vers l'infini.)

• Plutôt qu'un prints méthode, il serait plus typique de définir __str__ et il suffit d'imprimer votre objet.

• Vos méthodes sont seulement l'obtention et la configuration d'un attribut. En Python, nous ne faisons généralement pas utiliser les accesseurs et mutateurs—nous venons de permettre aux utilisateurs de définir nos attributs.

  • Vous êtes en provenance d'un Java arrière-plan, où l'une des règles de base est de toujours utiliser les méthodes getter et setter plutôt que de laisser les utilisateurs d'accéder à des attributs. La justification de cette règle est que si, à une date ultérieure, vous avez besoin de faire plus que simplement get et set (vous avez besoin pour traiter les données), il aurait besoin d'une API de changement. Depuis en Python nous avons des propriétés, nous n'aurions pas besoin d'une API changement dans cette instance afin que nous puissions éviter le standard et les fichiers inutiles de setters et getters.

• Il ne serait pas mal d'hériter numbers.Rational (Python 2.6 et plus), ce qui permet à votre classe automatiquement faire plusieurs choses chiffres devraient. Vous aurez à mettre en œuvre tout ce qu'il a besoin de vous, mais alors il sera automatiquement faire beaucoup plus de travail. Découvrez découvrez http://docs.python.org/library/numbers.html pour en savoir plus.

Spoiler alert:

class Fraction(object):
    """Don't forget the docstring....."""

    def __init__(self, numerator=0, denominator=1):
        self.numerator = numerator
        self.denominator = denominator

    def __str__(self):
        return "%d /%d" % (self.numerator, self.denominator)

    # I probably want to implement a lot of arithmetic and stuff!

if __name__ == "__main__":
    f = Fraction(2, 5)
    # If I wanted to change the numerator or denominator at this point, 
    # I'd just do `f.numerator = 4` or whatever.
    print f

OriginalL'auteur Mike Graham