no, no es mejor... ya que las expresiones regulares son mas lentas y hay que compilarse... y por lo general se consideran solo en casos especiales, regulares y hasta dinámicos!!
en cambio, las funciones naturales para cadenas de texto son mas veloces y practicas...
además viendo la forma en que se aplica en dicho código, usar expresiones regulares sería tonto, y gastaría mas recursos... pues, ¿hay que ser mas claro?
y es verdaderamente simple:
MSIE 5 es un valor literal sencillo, no es necesario buscarlo con expresiones regulares... ¿o porque piensas lo contrario?