En effet, il y a moyen de coupler la vérification d’url dans le code. La raison de ma méthode externe urlExists est bête : j’ai fait ce code pour mon application “Skyblog Photos Downloader” où je teste plusieurs fois si une URL existe.
D’où ce petit bout de code. Mais, je te l’accorde, il serait plus optimal de faire tout en un et inviter ainsi 2 instructions.

Pour les accolades, bien vu ! Je corrige ça tout de suite

Mais si je peux me permettre une petite remarque.

Je pense que ta fonction urlExist est inutile. dans le sens où a cause de celle ci tu fait appel 2 fois à new URL et 2 fois a openstream(). ce qui prend donc beaucoup plus de temps.

Alors que le but principale (il me semble) de cette fonction est de tester si l’url est du bon format et si le site existe. ces deux try{}catch{} peuvent judicieusement etre integré au code principale afin d’appeler qu’une seule fois new URL et openStream()

Et aussi il y a dans ton code un truc du genre { { (instructions) } } il me semble que tu peux en enlever une paire