犯了一个低级错误,大家看看是否能雾里看花不迷糊

只看该作者 · 2009-11-15 21:17

呵呵,大家按18楼老X的提示去找,很快能找到.
古道热肠发表于 2009-11-15 10:39

我还是没看出来呀
高手告诉我下，发消息我也行

只看该作者 · 2009-11-15 21:25

假如ucCount很大时
EEPROM的存储不够，会出现问题
是这个问题吗？

只看该作者 · 2009-11-15 22:47

32位，16位？

2万 · 2009-11-16 08:56

那可能就是这两行中的uiTemp++造成的问题了，返回出去的都是++后的地址。
EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)& 0xff);
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++);

1万 · 2009-11-16 09:17

FDB_Current.DIR_FileSize = EEPROMread(uiTemp++)<<24;
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++)<<16;
这两句溢出了,要强制类型转换.

2万 · 2009-11-16 09:17

本帖最后由 xwj 于 2009-11-16 09:47 编辑

yewuyi没找对哦，再去找！

LZ的本意就是要返回++后的地址，每次调用地址是自动增量的——这个不算BUG，BUG出在其他的低级错误上哦~

只看该作者 · 2009-11-16 09:44

嘿嘿，EEPROMread返回的是uchar.........

2万 · 2009-11-16 09:51

FDB_Current.DIR_FileSize += EEPROMread(uiTemp++);
*uiEEPROMAddress = uiTemp;

这个不要加加了吧,加加成下一个地址了

只看该作者 · 2009-11-16 09:53

EEPROM写了之后是需要演示的！
EEPROM真正写的动作是在STOP命令之后，当EEPROM器件接收到STOP之后，内部的升压电路才会打开，从内部升压电路打开到烧写完成时需要时间的，上面的代码在写完一个字节后根本没有延时的动作又马上执行下一次的写，所以会导致写入部成功。

较好的做法是不用单个字节写，而是多个字节的连续写。写完之后并检测是否写入成功(POLLING)

只看该作者 · 2009-11-16 09:56

所以楼主会有时候好像写入，然后读出都是对的，有时不读！

不同的EEPROM读写速度会有差异，就算是相同的型号，不同批次，也会有差异！

2万 · 2009-11-16 10:56

哈哈,BUG已被识破,总结一下.

25楼"t.jm"网友指出了读函数中存在的硬性BUG,属此题正解.

22楼"急驰的蚂蚁"提到的EEPROM空间溢出是有可能,但这个在上层主调函数书写时会自己控制长度的,就象将指针指向内存数组,写数组尤其要小心,弄不好指针跨界,程序重启或出现怪异的现象.

29楼"inter_Zhou"提到的EEPROM函数写可能出错,应考虑批次差异性并校验,这个说法是对的,不过这个处理应该是在EEPROM的写驱动底层函数中完成.该函数目前以Void返回,较为理想的是应该返时写是否成功的结果信息给主调函数进行错误处理.

2万 · 2009-11-16 11:01

贴上测试合格的可用代码,大家一看就明白了,哈哈,老X一眼就看出来了,实战经验丰富.
void WriteFileInfoToEEPROM(uint *uiEEPROMAddress)
{
uint uiTemp;

uiTemp = *uiEEPROMAddress;

EEPROMwrite(uiTemp++,(FDB_Current.DIR_FstClusLO)>>8);
EEPROMwrite(uiTemp++,(FDB_Current.DIR_FstClusLO)&0xff); //
EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)>>24);
EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)>>16);
EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)>>8);
EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)& 0xff);
*uiEEPROMAddress = uiTemp;
}

#if 0
//Old have a Bug in this Function
void ReadFileInfoFromEEPROM(uint *uiEEPROMAddress)
{
uint uiTemp;

uiTemp = *uiEEPROMAddress;

FDB_Current.DIR_FstClusLO = EEPROMread(uiTemp++)<<8;
FDB_Current.DIR_FstClusLO += EEPROMread(uiTemp++);
FDB_Current.DIR_FileSize = EEPROMread(uiTemp++)<<24;
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++)<<16;
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++)<<8;
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++);

*uiEEPROMAddress = uiTemp;
}
//
#endif

//Test OK
#if 1
void ReadFileInfoFromEEPROM(uint *uiEEPROMAddress)
{
uint uiTemp;

uiTemp = *uiEEPROMAddress;

FDB_Current.DIR_FstClusLO = EEPROMread(uiTemp++)<<8;
FDB_Current.DIR_FstClusLO += EEPROMread(uiTemp++);
FDB_Current.DIR_FileSize = ((ulong)EEPROMread(uiTemp++))<<24;
FDB_Current.DIR_FileSize += ((ulong)EEPROMread(uiTemp++))<<16;
FDB_Current.DIR_FileSize += ((ulong)EEPROMread(uiTemp++))<<8;
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++);

*uiEEPROMAddress = uiTemp;
}
#endif

2万 · 2009-11-16 11:03

文件很小时不出错,文件一长就出错.根源就在以下写法并未真正完成预期的功能
FDB_Current.DIR_FileSize = EEPROMread(uiTemp++)<<24;
FDB_Current.DIR_FileSize += EEPROMread(uiTemp++)<<16;

2万 · 2009-11-16 11:05

实事上,2楼网友"ejack"的直觉和猜测也是对的.

2万 · 2009-11-16 11:09

为什么
FDB_Current.DIR_FstClusLO = EEPROMread(uiTemp++)<<8;
FDB_Current.DIR_FstClusLO += EEPROMread(uiTemp++);
不出错了,菜鸟可以试一试,Keil-C51在做这种操作时真不出错,实际上这是一个数据类型的默认转换物性的把握的问题,当然最好的写法是无法确实转换是否有效时,加上强制类型转换,保证万无一失,永不迷糊和犯低级错误.

只看该作者 · 2009-11-16 11:20

void WriteFileInfoToEEPROM(uint *uiEEPROMAddress)
void ReadFileInfoFromEEPROM(uint *uiEEPROMAddress)
为什么参数要用指针传递呢？直接传uint型数据就行吧。
而且看了半天，貌似没找到EEPROMread和EEPROMwrite的原型。

只看该作者 · 2009-11-16 11:38

这个程序是写EEPROM的，程序传递的也是指针，按道理那个变量应该是指针才有意义
void WriteFileInfoToEEPROM(uint *uiEEPROMAddress)
{
      uint *uiTemp;

      uiTemp = uiEEPROMAddress;

      EEPROMwrite(uiTemp++,(FDB_Current.DIR_FstClusLO)>>8);
      EEPROMwrite(uiTemp++,(FDB_Current.DIR_FstClusLO)&0xff);                               //
      EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)>>24);
      EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)>>16);
      EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)>>8);
      EEPROMwrite(uiTemp++,(FDB_Current.DIR_FileSize)& 0xff);
      uiEEPROMAddress = uiTemp;
}