code review（自审）

代码的自我评审能够帮助我们找出代码的错误，优化维护代码，这里我选取了以前安卓做的一个小demo进行评审。

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    xmlns:tools="http://schemas.android.com/tools"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    tools:context="com.example.mrwu.myapplication.MainActivity"
    android:orientation="vertical">


    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="登录界面"
        android:textSize="30dp"
        android:background="#DBDBDB"
        android:layout_gravity="center"></TextView>

    <EditText
        android:id="@+id/et1"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:hint="请输入用户名"/>

    <EditText
        android:id="@+id/et2"
        android:layout_width="match_parent"
        android:layout_height="wrap_content"
        android:password="true"
        android:hint="请输入密码"/>
    <Button
        android:id="@+id/bt1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="登录"
        android:layout_gravity="center"
        />
</LinearLayout>

<?xml version="1.0" encoding="utf-8"?>
<LinearLayout xmlns:android="http://schemas.android.com/apk/res/android"
    android:layout_width="match_parent"
    android:layout_height="match_parent"
    android:orientation="vertical">
    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="头像"
        android:layout_gravity="center"/>
    <ImageView
        android:id="@+id/Iv1"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:src="@mipmap/ic_launcher"
        android:layout_gravity="center"/>
    <RelativeLayout
        android:layout_width="wrap_content"
        android:layout_height="wrap_content">
        <TextView
            android:id="@+id/tv1"
            android:layout_width="wrap_content"
            android:layout_height="wrap_content"
            android:text="姓名："/>
        <EditText
            android:layout_width="350dp"
            android:layout_height="wrap_content"
            android:layout_alignParentTop="true"
            android:layout_alignParentEnd="true" />
    </RelativeLayout>
    <RelativeLayout
        android:layout_width="match_parent"
        android:layout_height="wrap_content">
    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="我的性别是："/>
    <RadioButton
        android:layout_marginLeft="100dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="男"/>
    <RadioButton
        android:layout_marginLeft="200dp"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="女"/>

    </RelativeLayout>
    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="我喜欢的专业："/>

    <CheckBox
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="JAVA"/>
    <CheckBox
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="Android"/>
    <CheckBox
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="英语"/>
    <CheckBox
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="高数"/>
    <RelativeLayout
        android:layout_width="wrap_content"
        android:layout_height="wrap_content">
    <TextView
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:text="ip:"
        android:textSize="25dp"/>
      <EditText
        android:id="@+id/et3"
        android:layout_marginLeft="25dp"
        android:layout_width="350dp"
        android:layout_height="wrap_content" />
</RelativeLayout>
    <TextView
        android:id="@+id/tv2"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content" />

</LinearLayout>

package com.example.mrwu.myapplication;

import android.content.Intent;
import android.support.v7.app.AppCompatActivity;
import android.os.Bundle;
import android.text.TextUtils;
import android.view.View;
import android.widget.Button;
import android.widget.EditText;
import android.widget.Toast;

public class MainActivity extends AppCompatActivity {
       EditText et1;
       EditText et2;

    @Override
    protected void onCreate(Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_main);
         et1=(EditText) findViewById(R.id.et1);

         et2=(EditText) findViewById(R.id.et2);

        Button bt1 =(Button)findViewById(R.id.bt1);
        bt1.setOnClickListener(new View.OnClickListener() {
            @Override
            public void onClick(View v) {
                String str1=et1.getText().toString();
                String  str2=et2.getText().toString();
                if(TextUtils.isEmpty(str1)&&TextUtils.isEmpty(str2)){
                        Toast.makeText(MainActivity.this,"请输入用户名和密码",Toast.LENGTH_SHORT).show();

                }else if(TextUtils.isEmpty(str1)&&str2!=null){
                    Toast.makeText(MainActivity.this,"请输入用户名",Toast.LENGTH_SHORT).show();


                }else if(TextUtils.isEmpty(str2)&&str1!=null){
                    Toast.makeText(MainActivity.this,"请输入密码",Toast.LENGTH_SHORT).show();

                }else{
                    Toast.makeText(MainActivity.this,"登录成功",Toast.LENGTH_SHORT).show();
                    Intent myIntent = new Intent(MainActivity.this,SecondActivty.class);
                    startActivity(myIntent);
                }


                }

        });
    }
}

public class SecondActivty extends AppCompatActivity {
    ImageView Iv1;
    EditText et3;
    TextView tv2;

    @Override
    protected void onCreate(@Nullable Bundle savedInstanceState) {
        super.onCreate(savedInstanceState);
        setContentView(R.layout.mylayout);
        Iv1 = (ImageView) findViewById(R.id.Iv1);
        et3 = (EditText)findViewById(R.id.et3);
        tv2 = (TextView)findViewById(R.id.tv2);
        Iv1.setOnTouchListener(new View.OnTouchListener() {
            @Override
            public boolean onTouch(View v, MotionEvent event) {
                if (event.getAction() == MotionEvent.ACTION_UP){
                    float x= event.getX();
                    float y=event.getY();
                    String info = "X=" + x + "Y=" + y;
                    Toast.makeText(SecondActivty.this, info, Toast.LENGTH_LONG).show();
                }
                return true;
            }

    });
        et3.setOnKeyListener(new View.OnKeyListener() {
            @Override
            public boolean onKey(View v, int keyCode, KeyEvent event) {
                switch (event.getAction()){
                    case KeyEvent.ACTION_UP:
                        String ip = et3.getText().toString();
                        tv2.setText(Subs(ip));
                    case KeyEvent.ACTION_DOWN:
                        break;
                }

                return false;
            }
            String  Subs(String total){
                String news = "";
                for (int i =0;i<=total.length()/3;i++ ){
                    if(i*3+3<total.length()){
                 news = news+total.substring(i*3,Math.min(i*3+3,total.length()))+"-";
                    }else{
                 news = news+total.substring(i*3,Math.min(i*3+3,total.length()));
                    }
                }
               return news;
            }



        });
        }


    }

1.代码没有块缩进：4个空格

2.控件ID的命名是随便命名的，没有意义

3.必要的地方没有注释

4.类名没有以UpperCamelCase风格编写

5.参数名是随意定义的，没有以LowerCamelCase风格编写

6.UI控件成员变量没有统一加上控件缩写作为后缀，这样容易与普通成员变量混淆