美文网首页
review代码(2017-08-21~2017-08-25)

review代码(2017-08-21~2017-08-25)

作者: tjjlhst | 来源:发表于2017-08-27 18:51 被阅读0次

审查日期:2017-08-21

审查人员:鸣人

代码文件:MoreUserServerListActivity(服务列表页)

一、代码规范

注:1、注释过少

2、屏蔽掉的代码未删除

二、注释填充

1、添加注释

    @Bind(R.id.status_view)
    View statusView;  //顶部布局
    @Bind(R.id.service_list_recycleview)
    RecyclerView serviceListRecycleview; //服务列表
    @Bind(R.id.iv_back)
    ImageView mIvBack;  //返回按钮
    @Bind(R.id.tv_title)
    TextView mTvTitle;  //标题栏
    @Bind(R.id.tv_right)
    TextView mTvRight;  //控件预留的右边文字,并无展示、无作用,已隐藏
    @Bind(R.id.iv_point)
    ImageView mIvPoint;   //控件预留的右边图片,并无展示、无作用,已隐藏
    @Bind(R.id.progress_relative)
    RelativeLayout progressRelative;  //刚进来的加载进度遮盖
    MoreUserServiceListAdapter serviceListAdapter;  //服务列表的适配器
    MoreUserServerListPresenter moreUserServerListPresenter;  //网络请求
    List<ServiceListModel.DataBean> serviceList = new ArrayList<ServiceListModel.DataBean>();  //列表数据

审查日期:2017-08-22

审查人员:鸣人

代码文件:MoreUserServiceListAdapter(个人服务列表适配器)

一、代码规范

1、增加注释
2、命名不规范的变量修改

修改前:
Bitmap bitmap1 = FilterColorUtil.getGrayBitmap(bitmap);
serviceListViewHolder.customView.setImageBitmap(bitmap1);

修改后:
 //没有服务,将图标变灰
Bitmap bitmapGray = FilterColorUtil.getGrayBitmap(bitmap);
serviceListViewHolder.customView.setImageBitmap(bitmapGray);

2、屏蔽掉的代码未删除

二、逻辑调整
将重复的跳转代码提出到一个方法

修改前:
serviceListViewHolder.llContent.setOnClickListener(new View.OnClickListener() {
         @Override
         public void onClick(View v) {
                   Intent intent = new Intent();
                   intent.setClass(mContext, ServicesPayActivity.class);
                   intent.putExtra("servicename", serviceList.get(position).getName());
                   intent.putExtra("dbname", serviceList.get(position).getDbname());
                   mContext.startActivity(intent);
         }
    });
 ...

修改后:
   serviceListViewHolder.llContent.setOnClickListener(new View.OnClickListener() {
         @Override
         public void onClick(View v) {
                 //跳转到服务界面
                 clickStartActivity(position);
         }
    });

    //跳转到购买页面
    public void clickStartActivity(int position) {
        Intent intent = new Intent();
        intent.setClass(mContext, ServicesPayActivity.class);
        intent.putExtra("servicename", serviceList.get(position).getName());
        intent.putExtra("dbname", serviceList.get(position).getDbname());
        mContext.startActivity(intent);
    }

三、注释填充
holder的注释

      ImageView customView;  //服务图标
      TextView serviceNameTextview;  //服务名称
      TextView serviceBuyTextview;  //购买文字
      TextView serviceDeadlineTextview;  //截止日期
      RelativeLayout relativeLayout;  //服务的大布局
      LinearLayout llContent;  //线性布局,包括起文字

审查日期:2017-08-23

审查人员:鸣人

代码文件:PersonalResetPasswordActivity(重新设置密码界面)

一、代码规范

注:1、注释过少

   @Bind(R.id.id_password_old)
   EditText oldPassword;  //旧密码输入框
   @Bind(R.id.id_password_new)
   EditText newPassword;  //新密码输入框
   @Bind(R.id.id_password_check)
   EditText checkPassword;  //检查新密码输入框
   @Bind(R.id.button_reset_password)
   Button mButton;  //设置密码按钮
   @Bind(R.id.back)
   ImageView back;   //返回按钮
   @Bind(R.id.base_toolbar_title)
   TextView baseToolbarTitle;  //顶部标题
   @Bind(R.id.status_view)
   View statusView;   //状态栏
   @Bind(R.id.id_password_old_textview)
   TextView idPasswordOldTextview;  //旧密码
   @Bind(R.id.tip_text)
   TextView tipText;  //错误提示
   boolean isOld;  //两次输入的密码是否一致,不一致显示  显示  文字,这个是标识
   ResetPasswordPresenter resetPasswordPresenter = new ResetPasswordPresenter(); //请求网络
   private boolean isNight;  //夜间模式
   private String mUserName;  //获取上一个页面传过来的用户名

二、逻辑调整
注:将状态栏、用户名、按钮状态、网络请求初始化提出变独立方法

修改前:
   protected void onCreate(Bundle savedInstanceState) {
        super.setPageName("重设密码页面");
        super.onCreate(savedInstanceState);
        setContentView(R.layout.activity_personal_reset_password);
        isNight = SharedPreferencesUtil.readBoolean(this, SpConstant.FILE_SYSTEM_SETTING, SpConstant.KEY_SEND_NIGHT);
        ButterKnife.bind(this);
        setHidden(statusView);
        mUserName = getIntent().getStringExtra("userName");
        resetPasswordPresenter.attachView(this, RetrofitPostClient.POSt_REQUEST);
        mButton.setClickable(false);
        idPasswordOldTextview.setOnClickListener(this);
    }
修改后:
  //初始化状态栏、用户名、按钮状态、网络请求
    public void initView(){
        setHidden(statusView);
        mUserName = getIntent().getStringExtra("userName");
        resetPasswordPresenter.attachView(this, RetrofitPostClient.POSt_REQUEST);
        mButton.setClickable(false);
        idPasswordOldTextview.setOnClickListener(this);
    }

审查日期:2017-08-24

审查人员:鸣人

代码文件:RenameNicknameActivity(重新设置昵称界面)

一、代码规范

注:1、注释过少

    @Bind(R.id.rename_nickname_text)
    EditText etRename; //重命名输入框
    @Bind(R.id.tv_rename_nickname_tip)
    TextView textViewHint;  //提示文字
    @Bind(R.id.button_rename_nickname)
    Button buttonRename;  //点击设置重命名 
    private final String NICKNAME_PATTERN = "[\\w_-\\u2E80-\\u9FFF]+";  //正则表达式
    @Bind(R.id.back)  //返回按钮
    ImageView back;
    @Bind(R.id.base_toolbar_title)
    TextView baseToolbarTitle;  //头部按钮
    @Bind(R.id.status_view)
    View statusView;  //状态栏
    private boolean isNight;  //夜间模式

审查日期:2017-08-25

审查人员:鸣人

代码文件:RegisterSettingActivity(注册设置界面)

一、代码规范

1、注释过少

 @Bind(R.id.back)
    ImageView back;
    @Bind(R.id.edit_nickname)
    EditText editNickname;  //编辑名字框
    @Bind(R.id.edit_password)
    EditText editPassword;  //设置密码框
    @Bind(R.id.invote_code_edittext)
    EditText invoteCodeEdittext;  //邀请码框
    @Bind(R.id.edit_nickname_img)
    ImageView editNicknameImg;  //昵称前的文字
    @Bind(R.id.btn_complete)
    Button btnComplete;  //完成 按钮
    @Bind(R.id.textview)
    TextView textview;  //注册协议文字
    @Bind(R.id.tip_textview)
    TextView tipTextview;  //提示文字
    @Bind(R.id.edit_password_img)
    ImageView editPasswordImg;  //密码前的图片
    @Bind(R.id.textview_permiss)
    TextView textviewPermiss; //用户注册协议
    @Bind(R.id.hidden_tip)
    TextView hiddenTip;  //隐藏 文字
    @Bind(R.id.set_head_image)
    CircleImageView headImage;  //头像
    @Bind(R.id.clear_password)
    ImageView clearPassword;  //清除密码按钮
    @Bind(R.id.clear_nickname)
    ImageView clearNickname;  //清除昵称按钮
    @Bind(R.id.clear_invide)
    ImageView clearInvide;  //清除邀请码按钮
    @Bind(R.id.load_progress_container)
    LinearLayout load_progressContainer;  //加载框

二、注释填充

注:1、选择图片的模式,增加了注释

 private void setHeadImage() {
        String[] titles = {"拍照", "从相册中选取", "取消"};
        List<Map<String, String>> datas = new ArrayList<>();
        for (int i = 0; i < titles.length; i++) {
            //设置adapter的数据,选择的三种类型
            Map<String, String> itemData = new HashMap<>();
            itemData.put("name", titles[i]);
            datas.add(itemData);
        }
        SimpleAdapter adapter = new SimpleAdapter(this, datas, R.layout.simple_text_list_item, new String[]{"name"}, new int[]{R.id.simple_text_name});
        //选择对话框
        DialogPlus dialogPlus = DialogPlus.newDialog(this)
                .setAdapter(adapter)
                .setGravity(Gravity.BOTTOM)
                .setOnItemClickListener(new OnItemClickListener() {
                    @Override
                    public void onItemClick(DialogPlus dialog, Object item, View view, int position) {
                        if (dialog.isShowing()) {
                            dialog.dismiss();
                        }
                        Intent intent;
                        switch (position) {
                            case 0:
                                //跳转到拍照界面
                                intent = new Intent(MediaStore.ACTION_IMAGE_CAPTURE);
                                intent.putExtra(MediaStore.EXTRA_OUTPUT, Uri.fromFile(tempFile));
                                startActivityForResult(intent, REQUEST_CAPTURE);
                                break;
                            case 1:
                                //跳转到原生的选择照片界面
                                intent = new Intent(Intent.ACTION_PICK, MediaStore.Images.Media.EXTERNAL_CONTENT_URI);
                                startActivityForResult(Intent.createChooser(intent, "请选择图片"), REQUEST_PICK);
                                break;
                        }
                    }
                })
                .create();
        dialogPlus.show();
    }

相关文章

网友评论

      本文标题:review代码(2017-08-21~2017-08-25)

      本文链接:https://www.haomeiwen.com/subject/bolidxtx.html