Про 7 пункт - там же еще и неиспользуемый параметр метода.
Свич по строке - уууу. Особенно если это обработка сетевого респонза.
Урл-порт не в константах, а в коде.
Бесконечный цикл без возможности выхода. И чет меня его содержимое пугает - останавливаем и тут же запускам листенер - зачем?