日期:2014-05-18  浏览次数:21119 次

请教大家平日编程时应该注意哪些细节问题
最近编程时总觉得自己的一些细节方面有很多困惑,比如...
情况1:
C# code

        private void UpdateMPointsInStations()
        {
            foreach (BaseItem item in itemPanel3.Items)
            {
                ButtonItem btn = item as ButtonItem;
                int MPID = Convert.ToInt32(btn.Tag);
                Dictionary<string, string> data = new Dictionary<string, string>();
                data.Add("StatID", "null");
                db.Update("MPoints", data, string.Format("MPID = {0}", MPID));
            }
        }


这样的代码在循环里重复声明和生成变量再销毁,是不是很浪费资源或者说性能会很受影响?

情况2:
C# code

       private void Initialize() {

            //初始化传感器名字的列表
            InitSenList(SenList);
            //显示在当前工作站下,已选择的工作站
            showSelectedPointsInLeftPanel(itemPanel4);
            //初始化剩余未被部署的测点于itemPanel中
            InitRestUnSelectedMPointsInRightPanel(itemPanel3);
            //初始化剩余未被分配的传感器于itemPanel中
            InitRestUnSelectedSensorInRightPanel(itemPanel5);
            //初始化表格信息
            InitDataGridViewForMap(dataGridViewX1);
            //每次刷新时先清空itemPanel6
            InitItemPanel6(itemPanel6);
        }


我习惯于在调用方法的时候每一步写上注释说明,而不是每个函数定义的部分写上注释,这也是我个人的编程习惯和爱好。但是同事读我代码时他老是说不方便。

所以我想问问平时编写代码时应该注意哪些细节?我知道我现在编写的程序只是一些小的应用,但我只希望趁我还年轻,能够有一个好的编程习惯和风格。
真心求助,谢谢大家~

------解决方案--------------------
1.
C# code
Dictionary<string, string> data = new Dictionary<string, string>();
                data.Add("StatID", "null");

------解决方案--------------------
因为不知道 你的类是什么类型,ButtonItem btn = item as ButtonItem;这个是否有必要呢
------解决方案--------------------
private void UpdateMPointsInStations()
{
Dictionary<string, string> data = new Dictionary<string, string>();
data.Add("StatID", "null");
foreach (BaseItem item in itemPanel3.Items)
{
ButtonItem btn = item as ButtonItem;
int MPID = Convert.ToInt32(btn.Tag);
db.Update("MPoints", data, string.Format("MPID = {0}", MPID));
}
}