My Partner‘s Code View

1、建议修改的代码：

switch (msg.what) {
            case SystemConfig.MSG_TYPE_CHANNEL:
                String response = msg.obj.toString();
                //创建JSONObject对象
                try {
                        ...
                }
            ...
 }

意见：创建JSONObject对象后应该取出所需要的object对象。

修改后的代码：

switch (msg.what) {
            case SystemConfig.MSG_TYPE_CHANNEL:
                String response = msg.obj.toString();
                //创建JSONObject对象,并且取出所需要的object对象
                JSONObject object = null;
                try {
                        ...
                }
            ...
}

2、建议修改的代码：

 if (showapi_res_code == 0) {
                        JSONObject body_object = object.getJSONObject("showapi_res_body");
                        String channelId = channelListArray.getJSONObject(0).getString("channelId");
                        HttpUtil.httpGetData_Get(SystemConfig.GET_NEWS + "?channelId=" + channelId + "&page=1&needContent=0" + "&needHtml=1", handler, SystemConfig.MSG_TYPE_NEWS);
 }

意见：没有注释，最后一行代码过长不便于观看，还应该创建JSONArray对象

修改后的代码：

 if (showapi_res_code == 0) {
                        JSONObject body_object = object.getJSONObject("showapi_res_body");
                        //创建JSONArray对象
                        JSONArray channelListArray = body_object.getJSONArray("channelList");
                        //取出频道列表第一个channelId
                        String channelId = channelListArray.getJSONObject(0).getString("channelId");
                        HttpUtil.httpGetData_Get(SystemConfig.GET_NEWS + "?channelId
                           =" + channelId + "&page
                                =1&needContent=0" + "&needHtml
                                    =1", handler, SystemConfig.MSG_TYPE_NEWS);
}

3、建议修改的代码：

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    lv_datas = (ListView) findViewById(R.id.lv_datas);
    myAdapter = new MyAdapter(this);
    HttpUtil.httpGetData_Get(SystemConfig.GET_CHANNEL, handler, SystemConfig.MSG_TYPE_CHANNEL);
}

意见：最后应该绑定Adapter

修改后的代码：

@Override
protected void onCreate(Bundle savedInstanceState) {
    super.onCreate(savedInstanceState);
    setContentView(R.layout.activity_main);

    lv_datas = (ListView) findViewById(R.id.lv_datas);
    myAdapter = new MyAdapter(this);
    HttpUtil.httpGetData_Get(SystemConfig.GET_CHANNEL, handler, SystemConfig.MSG_TYPE_CHANNEL);
    //绑定Adapter
    lv_datas.setAdapter(myAdapter);
}

总的来讲，整体代码没有大的错误，只是大部分代码没有注释，而且过于冗长，不利于观看、检查，建议下次将部分重要代码注释，细心编程。