Foro de elhacker.net

Buenas, una duda, resulta que estoy intentando diseñar una clase en Python pero cada vez que imprimo los objetos se me agrega
un None que no consigo ver de dónde sale...
Todo está en un mismo archivo
Por cierto, estoy empezando en este lenguaje por lo tanto tal vez ignore alguna práctica que sea más recomendable...

La clase es la siguiente y abajo el main.

class Producto:
    contador_productos = 0
    def __init__(self,nombre: str, precio: float):
        self._id = Producto._generar_id()
        self._nombre = nombre
        self._precio = self._verificar_precio(precio)
 
    @staticmethod
    def _generar_id():
        Producto.contador_productos += 1
        return Producto.contador_productos
 
    def obtener_id(self):
        return self._id
 
    def obtener_nombre(self):
        return self._nombre
 
    def obtener_precio(self):
        return self._precio
 
    def _verificar_precio(self,precio):
        if precio > 0:
            return precio
        else:
            return -1
 
    def imprimir_producto(self):
        print(f'{self._id}, {self._nombre}, {self._precio}')
 

def main():
    a = Producto("Mouse",1700)
    b = Producto("Teclado",8000)
    c = Producto("Monitor",15000)
 
    print(a.obtener_id(), a.obtener_nombre(), a.obtener_precio())
    print(b.obtener_id(), b.obtener_nombre(), b.obtener_precio())
    print(c.obtener_id(), c.obtener_nombre(), c.obtener_precio())
 
    print(a.imprimir_producto())
    print(b.imprimir_producto())
    print(c.imprimir_producto())
 
if __name__ == '__main__':
    main()
 

salida:

1 Mouse 1700
2 Teclado 8000
3 Monitor 15000
1, Mouse, 1700
None
2, Teclado, 8000
None
3, Monitor, 15000
None

Hola!

Prueba reemplazando el print con return imprimir_producto (quita los paréntesis de la función)

Saludos!

Es cierto, era un problema en esa función jaja gracias genio
se solucionó así

    def imprimir_producto(self):
        return f'{self._id}, {self._nombre}, {self._precio}'
 

veamos:
1)que raro que usas return en casi todas las funciones y justo esa tiene un print. Es un error pésimo usar funciones sin que retornen algo, de ahí el none.
2) está bien que estás empezando con el lenguaje pero hay muchas cosas innecesarias, muchas faltantes y lo peor una mala lógica en lo que esperas lograr, por ejemplo imaginate que tenés que agregar miles de productos, no sirve declararlos como lo estás haciendo, para eso existe como mínimo los nodos y los árboles, aunque es otro tema anda viendo otra forma de hacer lo mismo.
3)tengo un ratito y como estoy aburrido cambiaré algunas cosas para ver si te sirven de guía:

class Producto:
    contador_productos = 0
    def __init__(self,nombre: str, precio: float):
        self._id = Producto._generar_id()
        self._nombre = nombre
        self._precio = self._verificar_precio(precio)
 
    @staticmethod
    def _generar_id():
        Producto.contador_productos += 1
        return Producto.contador_productos
 
    def __str__(self):
        return f'{self._id},{self._nombre},{self._precio}'
 
    def _verificar_precio(self,precio):
        return precio if precio > 0 else  -1 
 
def main():
    a = Producto("Mouse",1700)
    b = Producto("Teclado",8000)
    c = Producto("Monitor",15000)
 
    print(f"""{a._id},{a._nombre},{a._precio}
{b._id},{b._nombre},{b._precio}
{c._id},{c._nombre},{c._precio} \n""")
 
    print(f"{a}\n{b}\n{c}")
 
if __name__ == '__main__':
    main()
 

Donde obtenés lo mismo, agregando solo una función y eliminando varias,
obviamente los print del main son repetitivos ya que ambos obtienen lo mismo,
pero como dije antes es una muy mala idea la forma en que estas creando los objetos.

Sí desde luego no está pensado para que se pueda implementar en un sistema, es solo una práctica. Pero aún así entiendo las observaciones que hiciste, saludos!