I think ledger nano s or Trezor wallet is the best but I wallet does not support all coins. So exactly which coins are you want to keep safe?
I am new to crypto and just invested in OUICOIN which is now in ICO and will be on exchange in near June, so i don't think ledger nano s or Trezor will support it.