中小学试卷生成系统评测-分析顾同学代码
-
需求实现
-
整体代码通过安卓app实现需求，通过下载app来完成登录和生成试卷等功能，所以功能需求实现的很好，登录界面设计的很干净，操作方便，这点值得我去借鉴学习。
-
-
问题缺陷
-
-
第二个问题是，Activity太多了，虽然这是个小项目，不需要用MVP之类的架构把View和Model完全分离，但是问题在于他在每一个Activity中都调用了initData()和setListener()，既然有通用的模式，为什么不让Activity都继承一个BaseActivity基类呢？此外，Utils类中同时包含了保存文件和生成题库的功能，虽然说这个项目需要模块化的功能并不多，但是把完全不相关的功能全写一块也太过分了吧，而且结队项目还要求加入短信验证功能，向短信运行商发送HTTP请求的功能类总该另写一个类吧，希望进行第二版本开发的时候能注意
-
另外吧，把生成的文件存到手机的文件夹里其实挺蠢的，虽然需求里是这样要求的，但实际上Android的文件夹基本都是有访问权限的，把文件存到App的内部存储文件夹里普通用户是不能查看的，更不用说导出了，其实最好的改进方法应该是把文件存到手机的外部存储的SD卡上，虽然需要在生成时添加运行时权限的代码，不过这样用户应该就能访问生成的文件了
-
还有，好像没有加上记住密码功能呢，虽然是最简单的登录注册页面，但还是要考虑考虑用户体验的吖
-
-
建议方案
-
关于UI优化方面，我提出的第一个建议是加上一个圆形图标，还有给EditText和Button添加自定义样式，像圆角啊，颜色啊
-
首先，引入显示圆形图标的开源库CircleImageView，在build.gradle中添加依赖即可
-
第二，算了，直接附上代码
<RelativeLayout xmlns:android="http://schemas.android.com/apk/res/android"
xmlns:app="http://schemas.android.com/apk/res-auto"
xmlns:tools="http://schemas.android.com/tools"
android:id="@+id/activity_login"
android:layout_width="match_parent"
android:layout_height="match_parent"
android:background="@color/logoScreenBackground">
<de.hdodenhof.circleimageview.CircleImageView
android:id="@+id/iv_icon"
android:layout_width="90dp"
android:layout_height="90dp"
android:layout_centerHorizontal="true"
android:layout_marginTop="60dp"
android:scaleType="centerCrop"
android:src="@mipmap/ic_logo" />
<TextView
android:id="@+id/tv_title"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_centerHorizontal="true"
android:layout_marginTop="160dp"
android:text="AlphaGo"
android:textColor="#000"
android:textSize="23sp"
android:typeface="monospace" />
<RelativeLayout
android:id="@+id/rl_userName"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_below="@id/tv_title"
android:layout_marginTop="40dp"
android:background="@drawable/rectangle_edittext">
<ImageView
android:id="@+id/iv_userIconName"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_centerVertical="true"
android:paddingBottom="15dp"
android:paddingLeft="30dp"
android:paddingTop="15dp"
android:src="@mipmap/ic_edittextuname" />
<View
android:id="@+id/viewName"
android:layout_width="1dip"
android:layout_height="20dp"
android:layout_centerVertical="true"
android:layout_marginLeft="10dp"
android:layout_toRightOf="@id/iv_userIconName"
android:background="@color/colorCursor" />
<EditText
android:id="@+id/et_userName"
android:layout_width="match_parent"
android:layout_height="30dp"
android:layout_centerVertical="true"
android:layout_marginLeft="10dp"
android:layout_toRightOf="@id/viewName"
android:background="@null"
android:ems="19"
android:hint=" 用户名"
android:textColorHint="@color/colorCursor"
android:textCursorDrawable="@drawable/color_cursor"
android:textSize="15sp" />
<ImageView
android:id="@+id/iv_unameClear"
android:layout_width="wrap_content"
android:layout_height="wrap_content"
android:layout_alignParentRight="true"
android:layout_centerVertical="true"
android:layout_marginRight="20dp"
android:src="@mipmap/ic_backspace_grey600_24dp"
android:visibility="invisible" />
-