Error sin localizar en programa

[Xerelo]

Pues mucho no has entendido cuando tienes dos clases exactamente iguales, ya te dije cuando te puse el código que sólo es necesaria una. Estás yendo contra los principios de la orientación a objetos.

Aparte de eso estás haciendo una cosa muy mal, que es crear una nueva instancia de ventana cada vez que haces un cambio, así lo único que vas a conseguir es petar la memoria.

El hecho de que las pongas a visible=false no significa que desaparezcan, sólo que no las ves.

Es un ejercicio sencillísimo, y te está llevando un montón de tiempo por simple cabezonería.