Los IDE no siempre aciertan al refactorizar código automaticamente en estos casos, por ejemplo lo que ha hecho en este punto:
if (!!this.socket.isClosed()) break;
es totalmente absurdo, pues la doble negación !!
es equivalente a no tener ninguna negación!
Lo que te quería indicar el IDE probablemente fuese que necesitabas añadir la excepción, y ha buscado una manera de meterla un poco extraña (para mi gusto), pues los bucles while(true)
no son lo más "correcto", dado que cuando el contenido del mismo crece, buscar la condición de salida (el break
) puede ser complicado, condición que con
while(!this.socket.isClosed())
se ve mucho mejor (O con cualquier otro flag, dado el caso).
Por lo demás, meter el try/catch dentro o fuera del bucle no tiene mayores implicaciones respecto a la optimización, aunque si no me equivoco, lo ideal es intentar tener cuanta menos lógica sea posible dentro del try, simplemente por facilitar la lectura del código. Como dije, meter el try en un lado u otro es indiferente, pues si no hay excepciones, los bloques try/catch no tienen implicación alguna a la hora de ejecutar el código, es decir el compilador los "ignora".
Como prueba de esto dejo el siguiente enlace a una respuesta de la versión inglesa donde se discute la viabilidad de tener el try/catch dentro o fuera del bucle.
Entonces, da igual dónde pongamos el try/catch?
No!
Un try/catch con un bucle dentro del bloque try detendrá la ejecución del bucle si se lanza una excepción, es decir, se sale del bucle tras la excepción!.
Al contrario, un bucle con un try/catch dentro del bucle continuará la ejecución del bucle tras la excepción como si no hubiese pasado nada.
En definitiva, dónde lo metas dependerá un poco de lo que necesites hacer cuando ocurra la excepción