A estrutura do On Error seria esta:
Private Sub Form_Load()
On Error GoTo TrataErro
Exit Sub
TrataErro:
MsgBox Err.Number & Err.Description
End Sub
O Resume Next serve para retorna a execução na linha que vem logo após à linha que gerou o erro, ou seja usando este código seria quase a mesma coisa que dizer ao sistema: Ignore os erros. Ou seja, se der erro ele vai passar para a linha de baixo e continuar executando.
Em lógica de programação é difícil falar o que esta errado e o que esta certo, vai depender da necessidade, mas particularmente eu não usaria desta forma.
Veja mais exemplos sobre o On Error
