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:
Ver originalPublic Class miClase
Public Shared Sub miMetodo()
End Sub
End Class
Código vb:
Ver originalPublic 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.