Algumas observações antes de responder o que quer. Então algumas pessoas podem pular para a solução lá embaixo, essa primeira parte é só para quem quer fazer certo e não apenas funcionar.
Não faz o menor sentido usar esse global $pdo
. Evite global tanto quanto pode, e só use se souber bem o que está fazendo. Nesse caso específico é pior porque está fazendo algo que não é global ser e não tem necessidade. Na verdade não use qualquer mecanismo sem antes dominar como ele funciona, vai evitar muita dor de cabeça. Isso é especialmente importante porque esse código tem vários problemas por usar o que não conhece.
Seu código provavelmente sofre de SQL Injection. Pode ser que o dado foi sanitizado antes, mas duvido, essa não é a forma correta de fazer. Sua aplicação ficará vulnerável.
Não sei se minimizo dizendo para não se preocupar, a gente vê uns 80% dos códigos assim, porque isso é terrível, está quase todo mundo fazendo errado. Não é totalmente culpa das pessoas que fazem, mas é um pouco. Quase todo lugar na internet está ensinando errado. Aqui mesmo no nosso site que classificamos conteúdo acontece muito isso. No começo não acontecia, não deixávamos passar nada que ensinasse errado. Agora passa muita, em geral a maioria das respostas ensinam isso errado e a quantidade de problema é tão grande quem nem conseguimos mais lidar com todos, até porque a maioria dos usuários não liga se estão ensinando errado. É um pouco culpa da pessoa que pega a informação errada porque ela não deveria em confiar em informações na internet, essa atitude fará ela cometer erros a vida inteira induzidas por pessoas bem ou mal intencionadas. Eu sei que isso ficou um pouco longo, fora do tópico, mas é o melhor que posso te ensinar para tudo na vida.
A esmagadora maioria das pessoas adotam PDO por motivos errados. Não sei se deveria adotar, ainda que não seja o fim do mundo.
Não sei se deveria adotar classes, é pouco código para eu afirmar, mas parece que essa classe está mal estruturada, então usar um mecanismo do jeito errado não é muito bom, e se está treinando, acostumará fazer errado pro resto da vida. Conexão misturando com os casos do que seria o usuário, parece muito errado. Pelo menos não fez herança pra piorar mais ainda, a solução talvez não seja ficar sem.
Não é sobre o código, mas quase sempre deletar algo do banco de dados é algo errado a se fazer. Inativar costuma ser melhor. Em códigos reais isso tem vasta consequência.
Enfim, eu gosto de ajudar de verdade as pessoas, mas essa pergunta não é sobre isso não é o caso de ir a fundo. A solução é sempre não usar o que não sabe fazer certo.
Algumas pessoas não gostam deste tipo de resposta porque não foquei só no que foi perguntado, mas deixar a pessoa aprender errado é muito pior. O SOpt seria muito mais útil para as pessoas se todas respostas fizessem esses alertas e teríamos códigos melhores, menos perguntas precisariam ser fechadas.
A solução
Por fim, sem falar de problemas menores, a solução mais óbvia para fazer o que deseja é colocar pelo menos o execute()
dentro de um try catch
para pegar o erro já que essa opção está ligada e já usou em outro ponto do código. Porém tem um jeito mais fácil:
return $sql->execute();
Coloquei no GitHub para referência futura.
Só isso. Se você olhar a documentação, e só deveria usar algo depois de ler a documentação daquilo e entender por completo, diz que essa função retorna se a operação foi bem sucedida ou não, que é exatamente o que deseja.
Só que isso não funciona bem se optar por receber os erros como exceções.
Mesmo que quisesse verificar alguma coisa não faz sentido usar um if
para entregar um valor true
ou false
, o if
já é esse valor, é um código redundante.