1、背景说明

        虽然不止一次说明代码规范的问题，但是在协同作战的时候，代码规范这问题总是被遗忘，希望大家能够引以为戒。协同作战的时候，如果不能遵守代码规范，很容易出现各种问题，而且不知道从何处入手解决，消耗时间成本。

        实际上，对于代码规范这种问题，应该有人专门来审核代码的，但是公司小，项目也不大，参与的人员也不多的情况下，绝大多数公司都没有这种机制，尤其是在移动端编程上面 ，这就需要开发人员自我监督，避免出现不必要的麻烦。

        本文就举例说明，在近期笔者所带领的团队协作中，出现的一些问题。

2、命名

2.1 控件名称

       笔者在合并代码的过程中，看到如下的一段代码，大家注意看看id命名：

<TextView
        android:id="@+id/zongnumber"
        android:layout_width="wrap_content"
        android:layout_height="wrap_content"
        android:layout_marginBottom="0dp"
        android:layout_marginLeft="10dp"
        android:layout_marginTop="6dp"/>

2.2 实体名称

       看到一个实体类的命名为：

Part_3_JsonBean

jinriremen_gridview.xml

getExit();

3 代码要简洁

        代码要简介，意思是说，能够一句话写完，就不要分开成两句话，那么我们看看，如下两个例子：

3.1 不够简洁

        这里提供一段代码，如下所示：

Intent intent = new Intent();
intent.putExtra("detail", detail.getText().toString());
intent.setClass(mActivity, NewestActivity.class);
startActivity(intent);

Intent intent = new Intent(getActivity(), NewestActivity.class);
intent.putExtra("detail", detail.getText().toString());
startActivity(intent);

3.2 乱用简洁

       有时候，程序员误以为，简洁就是所有代码都写在一行，所有的同类项都可以合并，这是不对的，我们举例如下：

public static class ViewHolder {
	public ImageView remen_gallery_img;
	public TextView title, zongnumber, canyu, shenyu, detail;
	public ProgressBar progressBar;
}

4、字段注释

       这里就不过多解释了，上面已经说了，严格来说需要使用doc注释。先上一个反例：

public class JsonBean {
	private String already = null; // 以参与人数
	private String thumb = null;
	private String title = null;
	private String detail = null;
	private String total = null;// 总参与人数
	private String remaining = null;// 剩余人数
	private String issue = null;// 期数
	
	public JsonBean() {
		// TODO Auto-generated constructor stub
	}
}

public class UserBean {

	/** 参与次数 */
	private int count;
	/** 用户名 */
	private String username;
	/** 开奖时间 */
	private String publish_time;
	/** 用户ID */
	private String user_id;
	/** 幸运号码 */
	private String luck_num;
	/** ip地址 */
	private String ip;
	/** 头像 */
	private String head_picture;
	/** 抢单时间 */
	private String create_time;

	public UserBean() {
		// TODO Auto-generated constructor stub
	}
}

5、方法模块化

       一个方法里面，尽可能只做一件事情，不要同时执行了很多不同的事情，尤其是不要再一个方法，嵌套多层的网络请求，案例如下：

/**
 * 请求json数据
 * **/
private void setJson(){
	
	new Thread(new Runnable() {
		
		@Override
		public void run() {
			try {
				String json = HttpRequestUtils.doGet(PublicUtils.EG_URL + "/index");
				JSONObject response = new JSONObject(json);
				String msg = response.optString("msg");
				JSONObject object = new JSONObject(msg);
				String part_1 = object.optString("part_1");
				String part_2 = object.optString("part_2");
				String part_3 = object.optString("part_3");
				JSONArray array2 = new JSONArray(part_1);
				JSONArray array3 = new JSONArray(part_2);
				JSONArray array4 = new JSONArray(part_3);
				lists1 = new ArrayList<Part_1_JsonBean>();
				lists2 = new ArrayList<Part_2_JsonBean>();
				lists3 = new ArrayList<Part_3_JsonBean>();
				/** 解析part_1数据 **/
				for (int j = 0; j < array2.length(); j++) {
					// 代码省略
				}

				Message message = new Message();
				message.what = 200;
				handler.sendMessage(message);

				/** 解析part_2数据 **/
				for (int j = 0; j < array3.length(); j++) {
					// 代码省略
				}
				Message message2 = new Message();
				message2.what = 201;
				handler.sendMessage(message2);

				/** 解析part_3数据 **/
				for (int j = 0; j < array4.length(); j++) {
					// 代码省略
				}
				Message message3 = new Message();
				message3.what = 202;
				handler.sendMessage(message3);

			} catch (JSONException e) {
				e.printStackTrace();
			} catch (ClientProtocolException e) {
				e.printStackTrace();
			} catch (IOException e) {
				e.printStackTrace();
			}
		}
	}).start();
}

       修正案例如下：

/** handler消息处理机制 */
private Handler handler = new Handler() {
	public void handleMessage(android.os.Message msg) {
		switch (msg.what) {
		case ConstantValues.GET_NET_SUCCEED:
			jsonString = (String) msg.obj;
			if (jsonString != null && !jsonString.equals("")) {
				new DelJsonThread().start();
			} else {
				Logger.ii(TAG, "There is no data");
			}
			break;
		case ConstantValues.GET_NET_FAILED:
			// 获取网络失败
			break;
		case ConstantValues.DEL_RESULT_SUCCEED:
			listView.setAdapter(new HistoryListAdaptet(list,
					HistoryActivity.this));
			break;
		case ConstantValues.DEL_RESULT_FAILED:
			// 字符串处理失败
			Logger.ii(TAG, "del result failed");
			break;
		default:
			break;
		}
	}

};

if (!StringUtils.isEmpty(url)) {
	HttpGetThread httpGetThread = new HttpGetThread(handler, url);
	new Thread(httpGetThread).start();
}

6、逻辑处理

       在代码规范中，逻辑处理也是比较难遵守的，因为在测试的时候很难出现。这里的逻辑问题，主要是空指针这样的问题，主要有以下几个方面：

6.1 异步加载图片

       这里使用的是开源组件，如果url为空的时候，会在后台提示异常。

ImageLoader.getInstance().displayImage(url,holder.remen_gallery_img, options);

if (!StringUtils.isEmpty(url)) {
	ImageLoader.getInstance().displayImage(url,holder.remen_gallery_img, options);
}

6.2 从上一个界面获取数据

      我们从一个Activity跳转到另一个Activity的时候，会使用如下方法获取数据，这里都需要做判断：

String url = getIntent().getStringExtra("history");
if (!StringUtils.isEmpty(url)) {
	HttpGetThread httpGetThread = new HttpGetThread(handler, url);
	new Thread(httpGetThread).start();
}else{
	handler.sendEmptyMessage(ConstantValues.THE_URL_ERROR);
}

6.3 处理JSON字符串

      当我们向服务器发出请求，服务器返回一段字符串，这个时候，我们需要先判断一下数据是否正确：

if (jsonString != null && !jsonString.equals("")) {
	new DelJsonThread().start();
} else {
	Logger.ii(TAG, "There is no data");
}

7、总结

       程序开发的规范，是一个很考验程序员功力的问题，希望大家一开始就养成良好的习惯。

       以上，是笔者在公司做指导开发的时候，发现的一些问题，从中截取了一些代码。这里只是提供一些说明，并不提供完整可运行的程序，希望读者自己注意。

代码规范中常见问题举例说明