16,471
社区成员
发帖
与我相关
我的任务
分享又是一个函数问题
定义写函数
extern "C" __int16 __stdcall writecard(int icDev,UCHAR *cardNo)
{
if(icDev!=514) return error_comp; //错误厂商
CString strfirst,strfun,strprice,stramount,strmoney;
UCHAR buf[128];
UCHAR a;
a=cardNo[0]*16+cardNo[1];
buf[0]=a;
a=cardNo[2]*16+cardNo[3];
buf[1]=a;
a=cardNo[4]*16+cardNo[5];
buf[2]=a;
a=cardNo[6]*16+cardNo[7];
buf[3]=a;
swr_24c01a(0,0,4,buf);
return 0;
}
编辑未报错,调试报access violation
UpdateData(true);
int ret=writecard(514,(UCHAR *)m_cardsn.GetBuffer(0));
if(ret!=0)
{
AfxMessageBox("写卡失败");
}
m_cardsn为cstring类型
extern "C" __int16 __stdcall writecard(int icDev,UCHAR *cardNo)
{
if(icDev!=514) return error_comp; //错误厂商
CString strfirst,strfun,strprice,stramount,strmoney;
UCHAR buf[128];
UCHAR a;
a=cardNo[0]*16+cardNo[1];
buf[0]=a;
a=cardNo[2]*16+cardNo[3];
buf[1]=a;
a=cardNo[4]*16+cardNo[5];
buf[2]=a;
a=cardNo[6]*16+cardNo[7];
buf[3]=a;
swr_24c01a(0,0,4,buf);
return 0;
}
编辑未报错,调试报access violation
UpdateData(true);
int ret=writecard(514,(UCHAR *)m_cardsn.GetBuffer(0));
if(ret!=0)
{
AfxMessageBox("写卡失败");
}
m_cardsn为cstring类型
...全文
请发表友善的回复…
发表回复
chuting1 2010-05-05
- 打赏
- 举报
谢谢各位,是我的cardsn转化过程中出的错,长度不等于8了
Eleven 2010-05-05
- 打赏
- 举报
lz的这个函数writecard(int icDev,UCHAR *cardNo)，最好加个参数，指明UCHAR[]的长度
__int16 __stdcall writecard(int icDev,UCHAR *cardNo, int len);
__int16 __stdcall writecard(int icDev,UCHAR *cardNo, int len);
chuting1 2010-05-05
- 打赏
- 举报
m_cardsn.GetBuffer(0) 没有开辟内存空间，writecard直接操作这些内存肯定报错。
???如何开辟存在空间
???如何开辟存在空间
chuting1 2010-05-05
- 打赏
- 举报
这是细节问题,以后会加判断的.m_cardsn长度是等于8的
jyh_baoding 2010-05-05
- 打赏
- 举报
是的，上面说的有道理
快乐鹦鹉 2010-05-05
- 打赏
- 举报
[Quote=引用 7 楼 chuting 的回复:]
m_cardsn是从窗体上UpdateData(true)得到的,不会超过8
[/Quote]
那可能少于8个啊，你必须不少于8个
m_cardsn是从窗体上UpdateData(true)得到的,不会超过8
[/Quote]
那可能少于8个啊，你必须不少于8个
chuting1 2010-05-05
- 打赏
- 举报
m_cardsn是从窗体上UpdateData(true)得到的,不会超过8
cnzdgs 2010-05-04
- 打赏
- 举报
1、要检查m_cardsn的长度及内容是否符合要求；
2、GetBuffer用完后需要ReleaseBuffer，对于读取操作最好不要用GetBuffer，可以把参数定义为const char*类型，直接传CString变量。
3、从你的代码来看，貌似是输入卡号，经过处理后传给另一函数。需要注意，输入的内容是字符串而不是数值，如果要使用数值则需要转换一下，0~9这10个数字对应的字符值是0x30~0x39，字符值减去0x30就是对应的数值。
2、GetBuffer用完后需要ReleaseBuffer，对于读取操作最好不要用GetBuffer，可以把参数定义为const char*类型，直接传CString变量。
3、从你的代码来看，貌似是输入卡号，经过处理后传给另一函数。需要注意，输入的内容是字符串而不是数值，如果要使用数值则需要转换一下，0~9这10个数字对应的字符值是0x30~0x39，字符值减去0x30就是对应的数值。
向立天 2010-05-04
- 打赏
- 举报
m_cardsn有内容么？
不要用GetBuff
直接传m_cardsn试一下
或者声明一个UCHAR数组然后把字符串拷过去
不要用GetBuff
直接传m_cardsn试一下
或者声明一个UCHAR数组然后把字符串拷过去
尹成 2010-05-04
- 打赏
- 举报
楼上正解，要给m_cardsn.GetBuffer(0)开辟一段空间
aa3000 2010-05-04
- 打赏
- 举报
m_cardsn.GetBuffer(0) 没有开辟内存空间，writecard直接操作这些内存肯定报错。
也不确保m_cardsn开辟的空间足够供writecard操作
也不确保m_cardsn开辟的空间足够供writecard操作
13880079673 2010-05-04
- 打赏
- 举报
(UCHAR *)m_cardsn.GetBuffer(0)
改成,(UCHAR *)(LPCTSTR)m_cardsn 看看
改成,(UCHAR *)(LPCTSTR)m_cardsn 看看
快乐鹦鹉 2010-05-04
- 打赏
- 举报
你这个函数很容易出问题啊。安全检查都没有。指针直接就拿来用了。你应该先判断cardNo是否为空，然后其长度是否为8，否则很容易爆掉。
或者int ret=writecard(514,(UCHAR *)m_cardsn.GetBuffer(0));之前判断m_cardsn串的长度不小于8
或者int ret=writecard(514,(UCHAR *)m_cardsn.GetBuffer(0));之前判断m_cardsn串的长度不小于8
