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

审查日期: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();
    }
最后编辑于
©著作权归作者所有,转载或内容合作请联系作者
  • 序言:七十年代末,一起剥皮案震惊了整个滨河市,随后出现的几起案子,更是在滨河造成了极大的恐慌,老刑警刘岩,带你破解...
    沈念sama阅读 160,165评论 4 364
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 67,720评论 1 298
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 109,849评论 0 244
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 44,245评论 0 213
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 52,596评论 3 288
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,747评论 1 222
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 31,977评论 2 315
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,708评论 0 204
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,448评论 1 246
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,657评论 2 249
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 32,141评论 1 261
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,493评论 3 258
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 33,153评论 3 238
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,108评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,890评论 0 198
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,799评论 2 277
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,685评论 2 272

推荐阅读更多精彩内容