Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Code Review #3

Open
ZiGNicK opened this issue Nov 6, 2018 · 0 comments
Open

Code Review #3

ZiGNicK opened this issue Nov 6, 2018 · 0 comments

Comments

@ZiGNicK
Copy link

ZiGNicK commented Nov 6, 2018

В коде обнаружены следующие недостатки:
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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

1 participant