请教大家平日编程时应该注意哪些细节问题
最近编程时总觉得自己的一些细节方面有很多困惑，比如...
情况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));
           }
       }