review代码(2017-08-14~2017-08-18)

审查日期:2017-08-14

审查人员:鸣人

代码文件:FindPasswordActivity(找回密码界面)

一、代码规范

注:1、注释过少

2、屏蔽掉的代码未删除

三、注释填充

注:1、优化了键盘输入

 public void keyboardListener() {
        SoftKeyboardUtil.observeSoftKeyboard(this, new SoftKeyboardUtil.OnSoftKeyboardChangeListener() {
            @Override
            public void onSoftKeyBoardChange(int softKeybardHeight, boolean visible) {
                if (visible) { //键盘可见
                    if (!isKeybardScroll) {
                        int bottomNavigationBarHeight = 0; //底部虚拟导航按键高度
                        if (DisplayUtil.checkDeviceHasNavigationBar(PersonalFindPasswordActivity.this)) { //检查设备是否包含底部虚拟导航按键
                            bottomNavigationBarHeight = DisplayUtil.getNavigationBarHeight(PersonalFindPasswordActivity.this);
                        }
                        int screenHeight = DisplayUtil.getScreenHeight(PersonalFindPasswordActivity.this);
                        float scroll = mButton.getY() + mButton.getHeight() - (screenHeight + bottomNavigationBarHeight - softKeybardHeight) + 3;
                        if (scroll > 0) {
                            container.scrollTo(0, (int) scroll); //滚动高度使键盘位于登录按钮下面
                        }
                        isKeybardScroll = true;
                    }
                } else { //键盘不可见
                    container.scrollTo(0, 0);
                    isKeybardScroll = false;
                }
            }
        });
    }

审查日期:2017-08-15

审查人员:鸣人

代码文件:HeadpicClipActivity(图片裁剪界面)

一、代码规范

注:1、统一使用butterknife不使用findViewById方法

 @Bind(R.id.status_view)
    View statusView;
    @Bind(R.id.iv_back)
    ImageView bt_back;
    @Bind(R.id.title)
    TextView titleName;
    @Bind(R.id.src_pic)
    ImageView srcPic;
    @Bind(R.id.clipView)
    ClipView clipview;

2、屏蔽掉的代码未删除

三、注释填充

注:将过多的Log删除

审查日期:2017-08-16

审查人员:鸣人

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

一、代码规范

注: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;  //状态栏

三、注释填充

注:将网络请求方法提出变独立方法,解耦

//重新设置密码,密码的判断
    @OnClick(R.id.button_reset_password)
    public void resetPassword() {
        String newPasswordText = newPassword.getText().toString();
        String checkPasswordText = checkPassword.getText().toString();
        if (newPasswordText.contains(" ")) {
            new DialogUtil(this).showError("新密码中不能含有空格");
            return;
        } else if (newPasswordText.length() > 30 || newPasswordText.length() < 6) {
            new DialogUtil(this).showError("请输入6-30位新密码");
            return;
        } else if (!checkPasswordText.equals(newPasswordText)) {
            new DialogUtil(this).showError("两次新密码输入不一致");
            return;
        }
        TCAgent.onEvent(this, "点击修改密码修改按钮");
        String useid = SharedPreferencesUtil.readString(PersonalResetPasswordActivity.this, SpConstant.FILE_USER_INFO, SpConstant.KEY_USER_ID);
        String token = SharedPreferencesUtil.readString(PersonalResetPasswordActivity.this, SpConstant.FILE_USER_INFO, SpConstant.KEY_TOKEN);
        //防止重复提交
        mButton.setClickable(false);
        requestResetPassword();
    }

审查日期:2017-08-17

审查人员:鸣人

代码文件: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;  //夜间模式

三、注释填充

注:1、因为此页面较少TalkingData的统计,添加了TalkingData的统计

审查日期:2017-08-18

审查人员:鸣人

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

一、代码规范

注:1、注释过少

2、屏蔽掉的代码未删除

三、注释填充

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

private void setHeadImage() {
        String[] titles = {"拍照", "从相册中选取", "取消"};
        List<Map<String, String>> datas = new ArrayList<>();
        for (int i = 0; i < titles.length; i++) {
            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,646评论 4 366
  • 序言:滨河连续发生了三起死亡事件,死亡现场离奇诡异,居然都是意外死亡,警方通过查阅死者的电脑和手机,发现死者居然都...
    沈念sama阅读 67,979评论 1 301
  • 文/潘晓璐 我一进店门,熙熙楼的掌柜王于贵愁眉苦脸地迎上来,“玉大人,你说我怎么就摊上这事。” “怎么了?”我有些...
    开封第一讲书人阅读 110,391评论 0 250
  • 文/不坏的土叔 我叫张陵,是天一观的道长。 经常有香客问我,道长,这世上最难降的妖魔是什么? 我笑而不...
    开封第一讲书人阅读 44,356评论 0 215
  • 正文 为了忘掉前任,我火速办了婚礼,结果婚礼上,老公的妹妹穿的比我还像新娘。我一直安慰自己,他们只是感情好,可当我...
    茶点故事阅读 52,740评论 3 293
  • 文/花漫 我一把揭开白布。 她就那样静静地躺着,像睡着了一般。 火红的嫁衣衬着肌肤如雪。 梳的纹丝不乱的头发上,一...
    开封第一讲书人阅读 40,836评论 1 224
  • 那天,我揣着相机与录音,去河边找鬼。 笑死,一个胖子当着我的面吹牛,可吹牛的内容都是我干的。 我是一名探鬼主播,决...
    沈念sama阅读 32,022评论 2 315
  • 文/苍兰香墨 我猛地睁开眼,长吁一口气:“原来是场噩梦啊……” “哼!你这毒妇竟也来了?” 一声冷哼从身侧响起,我...
    开封第一讲书人阅读 30,764评论 0 204
  • 序言:老挝万荣一对情侣失踪,失踪者是张志新(化名)和其女友刘颖,没想到半个月后,有当地人在树林里发现了一具尸体,经...
    沈念sama阅读 34,487评论 1 246
  • 正文 独居荒郊野岭守林人离奇死亡,尸身上长有42处带血的脓包…… 初始之章·张勋 以下内容为张勋视角 年9月15日...
    茶点故事阅读 30,728评论 2 252
  • 正文 我和宋清朗相恋三年,在试婚纱的时候发现自己被绿了。 大学时的朋友给我发了我未婚夫和他白月光在一起吃饭的照片。...
    茶点故事阅读 32,200评论 1 263
  • 序言:一个原本活蹦乱跳的男人离奇死亡,死状恐怖,灵堂内的尸体忽然破棺而出,到底是诈尸还是另有隐情,我是刑警宁泽,带...
    沈念sama阅读 28,548评论 3 260
  • 正文 年R本政府宣布,位于F岛的核电站,受9级特大地震影响,放射性物质发生泄漏。R本人自食恶果不足惜,却给世界环境...
    茶点故事阅读 33,217评论 3 241
  • 文/蒙蒙 一、第九天 我趴在偏房一处隐蔽的房顶上张望。 院中可真热闹,春花似锦、人声如沸。这庄子的主人今日做“春日...
    开封第一讲书人阅读 26,134评论 0 8
  • 文/苍兰香墨 我抬头看了看天上的太阳。三九已至,却和暖如春,着一层夹袄步出监牢的瞬间,已是汗流浃背。 一阵脚步声响...
    开封第一讲书人阅读 26,921评论 0 201
  • 我被黑心中介骗来泰国打工, 没想到刚下飞机就差点儿被人妖公主榨干…… 1. 我叫王不留,地道东北人。 一个月前我还...
    沈念sama阅读 35,919评论 2 283
  • 正文 我出身青楼,却偏偏与公主长得像,于是被迫代替她去往敌国和亲。 传闻我的和亲对象是个残疾皇子,可洞房花烛夜当晚...
    茶点故事阅读 35,766评论 2 274

推荐阅读更多精彩内容