Mala Práctica de Programación?

[ocbardales]

Hola! solo necesito saber si a la larga me va a causar problemas o es una mala práctica de programación, tengo una clase donde están todos mis metodos:

Código:

Public Class miClase

     Public Sub miMetodo()
     End Sub

End Class

e hice un módulo donde instancío esta clase:

Código:

Module miModulo

    Public miObjetoClase As New miClase

End Module

luego en los forms simplemente la llamo:

Código:

Public Class miForm
    Private Sub miForm_Load(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles MyBase.Load
        miObjetoClase.miMetodo()
    End Sub
End Class

Hay algún problema en hacer este tipo de prática o implementación? El programa me corre todo bien! Sólo necesito saber si es una mala práctica de programación.

Gracias de antemano.

[alexg88]

Pues sí es una mala práctica porque estás usando clases pero como si no, porque sólo creas un objeto y lo usas como si fueran funciones normales de programación no OO.

En la programación OO tienes varios objetos que se relacionan entre sí mediante llamadas (que són los métodos).

Sí a pesar de esto quieres usarlo como funciones normales, crea los métodos como estáticos con la palabra Shared y así no tendrás que crear un objeto, porque serán métodos de la clase, no del objeto.

Código vb:

Public Class miClase
 
     Public Shared Sub miMetodo()
     End Sub
 
End Class
 

Código vb:

Public Class miForm
    Private Sub miForm_Load(ByVal sender As System.Object, ByVal e As System.EventArgs) Handles MyBase.Load
        miClase.miMetodo()
    End Sub
End Class
 
Pero lo mejor es que adoptes un estilo orientado a objetos.
 
Saludos.
 

[ocbardales]

El problema es que tengo un método de conexión a la base de datos en una clase, cómo utilizo este metodo de conexión en mis demás clases?

Aprovechándote, debería de instanciar la clase cada vez que la utilizo en cada evento de control de un Form o no hay problema en usar New una sola vez en las declaraciones y utlizar ese mismo objeto en todos los procedimientos, ejemplo:

Código vb:

Public Class miForm
 
    Private miClase As New miClase
 
End Class
 

y en los procedimientos(load, activate, click de un boton, etc.) solo ejecutar los métodos, o, tengo que darle New en cada procedimiento que la vaya a usar?