Il y a deux problèmes avec cela.
La première est que vous pouvez utiliser des requêtes paramétrées. Regardez AOP, cela vous aidera grandement . Non seulement cela est plus rapide pour plusieurs insertions, mais vous n'avez pas à vous soucier autant de l'injection SQL.
La seconde est que vous pouvez utiliser le ON DUPLICATE KEY UPDATE
pour s'occuper de ce problème pour vous. Sinon, lorsque votre requête échoue, vous ne savez pas pourquoi elle a échoué. Ce n'était peut-être pas du tout un problème de clé en double !
Autre que cela, le code du point de vue de or c'est très bien.
