Detect undefined attribute in child class (when defined in parent)
Nadie ha tomado este issue todavía.
- Lenguaje dominante
- Python
- Estrellas
- 20.6k
- Forks
- 3.3k
- Métricas de merge de PR
- Métricas de PR pendientes
Descripción
mypy does not detect an undefined attribute in a child class if it is defined in a parent class.
Given the following code, where the __init__ of a child class forgets to call super().__init__:
class Base():
def __init__(self, x: int) -> None:
self.x = x
def whatsx(self) -> int:
return self.x
class Child(Base):
def __init__(self, x: int) -> None:
pass
# Note: no call to super(), nor defining self.x.
c = Child(3)
print(c.whatsx()) # mypy does not catch undefined c.x
The real issue is obviously that the child class does not define the x attribute, failing Liskov.
Perhaps this is out-of-scope for mypy, but since programmers make stupid mistakes like this (/me raises hand), it would be awesome if mypy could warn about this error, making said programmers look less stupid ;)
Guía de contribución
Primeros pasos
- Lee el issue completo y luego la guía de contribución del proyecto.
- Comenta en el issue que vas a ocuparte — evita que dos personas hagan lo mismo.
- Haz un fork del repositorio y trabaja en una rama.
- Abre un pull request que haga referencia al número del issue.
Línea de trabajo
Comienza ejecutando mypy en el ejemplo proporcionado de Base y Child, centrándote en el init de la clase hija, el atributo x heredado y la llamada faltante a super().init. Rastrea cómo mypy gestiona los atributos heredados y las sobrescrituras de constructores. Se considera terminado cuando mypy advierte de forma fiable sobre la inicialización faltante sin rechazar incorrectamente subclases válidas.
Escrito por el modelo de indexación a partir del texto del issue.
Evaluación
- Stack tecnológico
- python
- Área
- compilers
- Tipo de issue
- Nueva funcionalidad
- Dificultad
- 5/5
- Tiempo estimado
- Más de una semana
- Estado de actividad
- Estancado
- Claridad
- Bastante claro
- Aptitud para principiantes
- 35/100