You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
В коде обнаружены следующие недостатки:
1 - в методе onCreate класса MainActivity следует разделять
логические блоки действий пустой строкой, т.к. это стиль кода данного приложения.
2 - мусор в виде закомментированных блоков кода (class MainActivity)
3 - есть слишком короткие и непонятные переменные типа (int id = item.getItemId();) к чему id не понятно (class MainActivity, method onNavigationItemSelected)
4 - есть пустые блоки try/catch (class MainActivity, method vkRequestForHeader)
5 - не выровнены перенесенные на новую строку параметры/вызовы методов (class Util; class MainActivity, method fillNavigationDrawerHeader; class UserChatAdapter, method bind; class DialogsListAdapter, method getView; class UserProfileFragment, methods parseUserProfile, onClick)
6 - ненужные пустые строки (class Util, method isConnectedToInternet; class DialogsListFragment, method onScrollStateChanged; class FriendListFragment, method onActivityCreated)
Из плюсов:
1 - в основном код легко читабельный и понятный для быстрого вникания
2 - стиль названия переменных и методов соответствует их назначению и выполнены в стиле Camel
The text was updated successfully, but these errors were encountered:
В коде обнаружены следующие недостатки:
1 - в методе onCreate класса MainActivity следует разделять
логические блоки действий пустой строкой, т.к. это стиль кода данного приложения.
2 - мусор в виде закомментированных блоков кода (class MainActivity)
3 - есть слишком короткие и непонятные переменные типа (int id = item.getItemId();) к чему id не понятно (class MainActivity, method onNavigationItemSelected)
4 - есть пустые блоки try/catch (class MainActivity, method vkRequestForHeader)
5 - не выровнены перенесенные на новую строку параметры/вызовы методов (class Util; class MainActivity, method fillNavigationDrawerHeader; class UserChatAdapter, method bind; class DialogsListAdapter, method getView; class UserProfileFragment, methods parseUserProfile, onClick)
6 - ненужные пустые строки (class Util, method isConnectedToInternet; class DialogsListFragment, method onScrollStateChanged; class FriendListFragment, method onActivityCreated)
Из плюсов:
1 - в основном код легко читабельный и понятный для быстрого вникания
2 - стиль названия переменных и методов соответствует их назначению и выполнены в стиле Camel
The text was updated successfully, but these errors were encountered: