编程要准确理解程序里面的逻辑

朱金灿

晚上一位朋友问我有关ado访问数据库的问题。说实话我已经有相当一段时间没有做数据库了，感觉把知识忘得差不多了。

朋友的程序是这样的：

int CGMSDlg::LoginCheck(CString strID, CString strPwd)

{

if( m_adoConnection.Open("GMS.mdb")==FALSE )

{

return LINK_FAILURE;

}

m_adoRecordset.SetAdoConnection(&m_adoConnection);

m_adoRecordset.PutCursorLocation(adUseClient);

if( m_adoRecordset.Open("users", adCmdTable)==FALSE )

{

return LINK_FAILURE;

}

if( m_adoConnection.BeginTrans()==BEGINTRANS_FAILURE )

{

return LINK_FAILURE;

}

////查找用户名

m_strFind.Format("SELECT pwd FROM users WHERE id=/"%s/"",strID);

if ( m_adoRecordset.Find(m_strFind)==FALSE )

{

return WRONG_ID;

}

////若找到，取出用户密码，和登录密码进行比较

if( m_adoRecordset.GetCollect("pwd", m_varValue)==FALSE

|| strcmp(strPwd, (_bstr_t)m_varValue)!=0 )

{

return WRONG_PWD;

}

m_strUserName = m_adoRecordset.GetCollect("name",m_varValue);

m_strUserName = (LPCSTR)(_bstr_t)m_varValue;

return CHECK_PASS;

}

这里的m_adoConnection是ado中的_ConnectionPtr一个封装类的实例，m_adoRecordset是_RecordsetPtr一个封装类的实例（我个人感觉没必要对ado进行封装）。

我觉得朋友没有准确理解程序的逻辑。比如m_adoRecordset.Find函数（其实是封装_RecordsetPtr的Find函数），它的参数放的就不是sql语句，而是查询条件。更为关键的问题是他没有理解ado的内在逻辑。其实我们看到对某个记录取字段值，都是基于查询结果集的。那么下面一段代码：

m_strFind.Format("SELECT pwd FROM users WHERE id=/"%s/"",strID);

if ( m_adoRecordset.Find(m_strFind)==FALSE )

{

return WRONG_ID;

}

看似是比较ID，但实际上执行sql语句的结果记录集却没有id这一项，这显然是不符合逻辑嘛。

我觉得，在现在很多库层层封装之下，学会理解被封装的程序的内在逻辑对开发人员尤为重要。