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

                                                                         朱金灿

 

晚上一位朋友问我有关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这一项，这显然是不符合逻辑嘛。

 

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