It's probably preferred to use $PAGER, if set. If not, then fallback to "more" or "less" or "view". "vi" may (?) have an unfamiliar interface to some (requiring ":q" to quit) whereas "more" and "less" require only "q" to quit. "vi" is also read-write, whereas the others are read-only. PAGER would be set to a user's preference, so that would be preferred above any others.
It's probably preferred to use $PAGER, if set. If not, then fallback to "more" or "less" or "view". "vi" may (?) have an unfamiliar interface to some (requiring ":q" to quit) whereas "more" and "less" require only "q" to quit. "vi" is also read-write, whereas the others are read-only. PAGER would be set to a user's preference, so that would be preferred above any others.