cual es mi error -POO-

[yuo2]

Hola a todos, estube practicando algo de POO en python y bueno descubri que las variables privadas no existen y el encapsulamiento no es un 99% eficaz.
Bueno saliendo de ese tema, diseñe un ejemplito para practicar y bueno me sale un error; espero que sea error mio y no del lenguaje Python.
Cree 2 Clases y una interface estan en archivos distintos; el problema creo yo, es cuando trato de asignar el valor 18 a maxedad de la clase "PuertaDisco"; se supone que una clase debe poseer atributos y metodos; entonces en el metodo "e_asegurar" trato de comparar la edad del cliente (La cual asigno yo cuando ejecuto el programa) con el maximo de edad que se asigno para que ingrese (esto lo asigno internamente).

Código python:

---------interface.py------------
 
 
import claseCliente
import clasePuerta
 
nom=raw_input("ingrese su nombre: ")
eda=raw_input("ingrese su edad: ")
clie = claseCliente.cliente(nom,eda)
clie.e_irdisco()
 
---------ClaseCliente.py--------
 
import ClasePuerta
 
class cliente:
    def __init__(self,pnomb,pedad):
        self.pnomb=pnomb
        self.pedad=pedad
    def e_irdisco(self):
        op=raw_input("Desea ir a Disco? :")
        if op=="si":
            ClasePuerta.e_asegurar()
 
 
---------ClasePuerta.py-----------
 
import ClaseCliente
 
class PuertaDisco:
    def __init__(self,maxedad):
        self.maxedad=maxedad
    def e_asegurar(self):
        self.maxedad=18
        if self.maxedad > int(claseCliente.pedad)
            print "NO PUEDE PASAR"
        else:
            print "ADELANTE PASE"
 

[David]

Tienes varios errores, por ejemplo, en la línea 35 te faltan los dos puntos, además, tomas a claseCliente como si fuera una clase, cuando no lo es, la clase se llama "cliente" y debe instanciarse para que puedas utilizarla.

Lo mismo ocurre en la línea 23, ClasePuerta no es una clase, la clase es PuertaDisco, y para usarla debes crear una instancia de ella.

[AlvaroG]

Iniciado por yuo2

Hola a todos, estube practicando algo de POO en python y bueno descubri que las variables privadas no existen y el encapsulamiento no es un 99% eficaz.

¡Ojo! que no puedas poner un atributo como "private" no quiere decir que no haya un encapsulamiento eficaz.
Fijate en este tema http://www.forosdelweb.com/f130/get-set-618901/
y esta página http://www.diveintopython.org/object...functions.html

Básicamente, si el nombre de un miembro comienza con __ (dos guiones bajos), es un miembro privado.


Saludos.

[yuo2]

Bueno habia errores (como dijeron arriba en los dos puntos del if y al llamar las clases no estaban escritos bien), estan corregidos

Ahora el problema radica para llamar al evento o metodo 'e_asegurar' de la ClasePuerta desde la ClaseCliente pues me da error. Nose si es porque le asigno el valor self.maxedad=18; ya que ese valor lo deseo asignar directamente desde el codigo.

Código Codigo Corregido:

### interface.py ####
 
import ClaseCliente
import ClasePuerta
 
nom=raw_input("ingrese su nombre: ")
eda=raw_input("ingrese su edad: ")
clie = ClaseCliente.cliente(nom,eda)
clie.e_irdisco()
 
### ClaseCliente.py ####
 
import ClasePuerta
class cliente:
    def __init__(self,pnomb,pedad):
        self.pnomb=pnomb
        self.pedad=pedad
    def e_irdisco(self):
        op=raw_input("Desea ir a Disco?: ")
        if op=="si":
            ClasePuerta.e_asegurar()
 
### ClasePuerta.py ###
 
import ClaseCliente
 
class PuertaDisco:
    def __init__(self,maxedad):
        self.maxedad=maxedad
    def e_asegurar(self):
        self.maxedad=18
        if self.maxedad>int(ClaseCliente.pedad):
            print "NO PUEDE PASAR"
        else:
            print "ADELANTE PASE"
 

imagen del error:
img204.imageshack.us/img204/6706/imgm.jpg

Alvino sobre los get, set ; en realidad no me terminan de convenser ya que cuando inicio la clase con __init__(self,__edad,__nombre) y aplico los get y set + property; igual a la hora de programar en codigo los muestra; por lo cual no oculta esos atributos por otros nombres; no es igual que el VisualStudio (renegando) que remplaza mis atributos por nombres que le puedo dar. Bueno ahora es Python asi que aprobechar lo que se pueda :D

Igual voy a revisarlo bien eso de los set,get.. recien leo los mensajes.

[David]

Iniciado por David el Grande

Tomas a claseCliente como si fuera una clase, cuando no lo es, la clase se llama "cliente" y debe instanciarse para que puedas utilizarla.

Lo mismo ocurre en la línea 23, clasePuerta no es una clase, la clase es PuertaDisco, y para usarla debes crear una instancia de ella.

Eso todavía no corregiste, ojo con eso, claseCliente y clasePuerta son módulos, no son clases. Así que poner clasePuerta.e_asegurar, ya que el módulo clasePuerta no tiene ningún método llamado así, quien tiene ese método es la clase PuertaDisco que está dentro del módulo, ¿entiendes a lo que me refiero?

[yuo2]

Iniciado por David el Grande

Eso todavía no corregiste, ojo con eso, claseCliente y clasePuerta son módulos, no son clases. Así que poner clasePuerta.e_asegurar, ya que el módulo clasePuerta no tiene ningún método llamado así, quien tiene ese método es la clase PuertaDisco que está dentro del módulo, ¿entiendes a lo que me refiero?

Entonces como tendria que llamar a ese metodo 'e_asegurar'?
asi?

Código codigo:

ClasePuerta.PuertaDisco.e_asegurar()
 

ya lo intente y naa.. O,o

[David]

No, debes instanciar un objeto a partir de la clase PuertaDisco.

Código python:

obj = ClasePuerta.PuertaDisco(15)
obj.e_asegurar()
 

[yuo2]

Iniciado por David el Grande

No, debes instanciar un objeto a partir de la clase PuertaDisco.

Código python:

obj = ClasePuerta.PuertaDisco(15)
obj.e_asegurar()
 

Uhmm bueno en realidad queria que el valor de 15 se ejecutara en ese mismo evento de e_asegurar de la ClasePuerta.
Bueno siguiendo tu sugerencia ahora como haria yo para poder evaluar ese valor con la edad de mi cliente?
justo en esta linea de programacion del evento e_asegurar, tendria que hacer este cambio; pero al parecer no logro comparar:

Código Codigo ClasePuerta:

if ClaseCliente.pedad < self.maxedad:
            print "NO PUEDE PASAR"
        else:
            print "ADELANTE PASE"
 

[David]

ClaseCliente tampoco es una clase, es un módulo.

Por lo que tampoco posee un método o atributo con nombre "pedad", deberías recibir como parámetro en la función e_asegurar un objeto cliente o la edad con la cual comparar para evitar instanciar una nueva clase cliente (no tendría sentido en este contexto).

[yuo2]

Iniciado por David el Grande

ClaseCliente tampoco es una clase, es un módulo.

Por lo que tampoco posee un método o atributo con nombre "pedad", deberías recibir como parámetro en la función e_asegurar un objeto cliente o la edad con la cual comparar para evitar instanciar una nueva clase cliente (no tendría sentido en este contexto).

Y si jalo mi variable eda de la interface para comparalo??.. tampoco parece compararlo :p .
Como podria enviar el parametro

[David]

En este caso eres tú quien no está encapsulando las clases correctamente, las clases no deberían tomar variables "de la interface" o directamente de otra clase.

Debes tomar por un parámetro lo que necesites para trabajar.

[yuo2]

Bueno gente ya termine el codigo ya lo hice funcionar, aqui se los dejo para q lo tengan de ejemplito:

Código CODIGO TERMINADO:

////interface.py////
 
import claseCliente
import clasePuerta
 
nom=raw_input("ingrese su nombre: ")
eda=raw_input("ingrese su edad: ")
clien=claseCliente.cliente(nom,eda)
clien.e_irdisco()
 
////claseCliente.py////
 
import clasePuerta
class cliente:
    def __init__(self,pnomb,pedad):
        self.pnomb=pnomb
        self.pedad=pedad
    def e_irdisco(self):
        op=raw_input("Desea ir a Disco?: ")     
        if op=="si":
            obj=clasePuerta.PuertaDisco(self.pedad)
            obj.e_asegurar()
 
//// clasePuerta.py ////
 
class PuertaDisco:
    def __init__(self,maxedad):
        self.maxedad=maxedad
    def e_asegurar(self):      
        if int(self.maxedad) < 18 :
            print "NO PUEDE PASAR"
        else:
            print "ADELANTE PASE"
 

Y gracias a los que se tomaron el tiempo de ayudar: David el Grande y alvlin