Si bien ya detectaste cuál era tu error sintáctico (=+
), quiero proveer una solución más pythonic que aborde tu caso.
Lo que quieres hacer es, básicamente, tener una variable que contenga la información de cuántas instancias de Articulo
(Article
, de ahora en adelante ;) ) se han creado. Esto, en efecto, es un trabajo ideal para una variable de clase.
Ahora bien, ¿cómo podemos actualizar esta variable? Debemos colocar ese código de actualización del contador en un lugar que sepamos que siempre será llamado cuando una instancia nueva de Article
sea creada.
Las clases en Python incluyen el método __new__
(acá hay un artículo buenísimo que explica a detalle el raciocinio detrás de la concepción de este método, en palabras del mismísimo Guido) que es llamado cada vez que se crea una instancia de ellas. Por lo tanto, este parece ser el sitio ideal para el código de nuestra misión.
La clase Article
, entonces, se vería de manera similar a lo mostrado a continuación:
class Article:
instances = 0
def __new__(class_, *args, **kwargs):
class_.instances += 1
return super().__new__(class_)
De ahora en adelante, cada vez que se cree un objeto de la clase Article
, la variable instances
será actualizada. Cabe destacar que es necesario el uso de super()
para no alterar el proceso de instanciación que Python provee, a través del llamado del método __new__
en la clase padre (object
).
Aplicando esto, y editando el código para hacerlo más pythonic, lo que tú tenías quedaría ahora de la siguiente manera:
class Article:
_instances = 0
def __new__(class_, *args, **kwargs):
class_._instances += 1
return super().__new__(class_)
def __init__(self, name, price):
self.id = self._instances
self.name = name
self.price = price
def __str__(self):
return (
f'Articulo número: {self.id} '
f'{self.name} \n'
f'Precio: {self.price}'
)
@classmethod
def get_number_of_articles(class_):
return class_._instances
A notar:
Usamos interpolación de strings, específicamente F-Strings, en lugar de concatenar las cadenas de caracteres.
Indicamos con un leading underscore que la variable _instances
no debe ser ni modificada ni accedida fuera de la clase Article
.
Usamos un @classmethod
que sirva como interfaz pública para obtener la variable "privada" (y nótense las comillas) _instances
, a través de la misma clase Article
, en lugar de a través de sus instancias.
Se podrá usar la clase así:
>>> article = Article('Cereal', 45.0)
>>> print(article)
Articulo número: 1 Cereal
Precio: 45.0
>>> article_2 = Article('Yogurt', 22.0)
>>> print(article_2)
Articulo número: 2 Yogurt
Precio: 22.0
>>> Article.get_number_of_articles()
2
Nota final: Si bien esto funciona y mediante mi respuesta intenté transformar tu código para que se apegar más al lenguaje, si estás usando este código en realidad, te sugiero volver a pensar la estructura de tu sistema. Las clase Article
no debería preocuparse de cuántas instancias se han creado. Me parece que debes separar los concerns y hacer una clase aparte (algo tipo ArticleRepository
) que sea la encargada de gestionar y llevar registro de los artículos creados.