this question is seriously wrong!
because any of the many wallets currently existing can be very good and very bad at the same time.
wallets like Coinbase and Xapo which are centralized and don't give you your private keys are bad because you have no control over your money and only have an account with them. but it is good for beginners who want to avoid all the technical details and are too lazy to learn anything about bitcoin and using it! also it is good to buy and store bitcoin in same place.
the web wallets such as blockchain.info gives the user access and control over the private keys but don't offer much options like RBF, CPFP, setting fees appropriately,... and are risky because of being online but at the same time they are convenient to use
the full verification wallets such as Core and Armory are good if you don't want to rely on anybody else for your tx verifications,... but it takes up space and needs a good internet connection. and obviously you can no longer be lazy and not learn more about how it works.
the Simplified Verification wallets (SPV) such as Electrum and Multibit are lacking the full verification benefits but are not taking up much space and can be used by people with low HDD space or slow internet speed or low cap. the options they offer may vary, for instance eletrum offers more options for advanced usage while Multibit keeps things as simple as possible.
in short you can't really say which wallet is better than others. depending on you and your usage they can be very suitable or far from it.
p.s. you can also read this topic:
https://bitcointalksearch.org/topic/general-bitcoin-wallets-which-what-why-1631151