Давайте называть вещи своими именами: если у вас есть чужой код и вы хотите, чтобы вам его кто-то переписал — это не "рефакторинг", это "техническое задание" ака "напишите всё за меня". Такие вопросы могут набрать очень много минусов от постоянных посетителей и быть закрытыми. Рефакторингом у нас на [codereview](https://codereview.stackexchange.com/) называется "мы вам расскажем, чем ваш код плох -- но переписывать вам придётся самому". Мы не злыдни и может быть даже какие-то фрагменты кода покажем, как лучше — но не обязаны приводить полностью код. Кроме того, если код не работает — на рефакторинг он не принимается. Подробнее можно почитать по ссылкам: * [Политика относительно вопросов с инспекцией кода (code review)](https://ru.meta.stackoverflow.com/q/1761/213987) * [jОписание метки инспекция-кода](https://ru.stackoverflow.com/tags/%d0%b8%d0%bd%d1%81%d0%bf%d0%b5%d0%ba%d1%86%d0%b8%d1%8f-%d0%ba%d0%be%d0%b4%d0%b0/info) Ну и просто полистайте [вопросы по этой метке](https://ru.stackoverflow.com/questions/tagged/%d0%b8%d0%bd%d1%81%d0%bf%d0%b5%d0%ba%d1%86%d0%b8%d1%8f-%d0%ba%d0%be%d0%b4%d0%b0), чтобы посмотреть хорошие и плохие образцы. А вот начать разбираться — это похвальный вариант. Будут конкретные вопросы — приходите, задавайте вопросы, поможем.