对比十二年前的代码,发现了点有意思的东西

今天用vb写一个了一个小函数,功能很简单,就是检测某个项是否在下拉列表里,本来不想加到自己的函数库的,想想还是加一下,结果发现以前已经添加过了,修改时间是2007年,也就是12年前了,那时才学vb一两年。于是我很好奇地用beyondcompare比较下看看。发现了点有意思的地方。于是我决定对12年前的我进行点评。

对比十二年前的代码,发现了点有意思的东西

函数名:isInLst对比现在的isItemInCbo,很显然现在的更加明确和易懂。List没必要略写成Lst,省略一个字母可读性大大增加,另外,list容易让人理解成listbox,这是两个不同的控件。

参数:变量名cbo和str1命名方式都不如现在的好,现在的cbo1,让人一看就明白是combobox的一个实例,后一个参数str1对比现在的strContent,很显然现在的更好点,变量名更有意义。

实现逻辑:12年前用你的do while循环,现在用的for 。。。to遍历

 

对过去的自己吐槽:

1.变量名sameItem竟然不声明,啥玩意!真是太小白了吧 !

2.还有sameItem明显是当做索引用的,可这变量名却让人感觉像是一个字符串。

3.不用for循环竟然用do。。。while循环,自己弄个计数器在那儿辛苦累加,到底是咋想的?这是无语!弄for循环多好,会自动将步长增1,少写多少代码!

4.函数名后面还有个“'True表示列表中有str1”,多余的注释,很显然true就是存在啊。

5.这么多年了,竟然还在搞vb!转眼一个生肖轮回过去了,时间真快啊。

6.破微软能不能给vb开源啊,让热心的网友更新下combobox搞个直接判断的方法.isExist(xxxx)啊,(附一条对微软的吐槽!)