Well, j and k are fine if you're not moving far. A lot of slow scrolling up and down should be { and } or C-d and C-u or <count>[jk]. Even better: Use / and ?. It's bad to be repeatedly tapping j and k or waiting for the keys to repeat.
But h and l are the EMACS approach. You should be moving in a line with w, e, b, f, F, t, T, ^, $, I, A, 0, and the like. Vi provides lots of tools for efficient line editing and h and l are not good.
Beginners should try
:noremap h <nop>
:noremap l <nop>
to keep them from the temptation of the nefarious h and l keys.Lately I use
:no h ^
:no l $
which is even better because ^ and $ require moving your hands off the home row even though they're much better than h and l for you.