江湖求急之二：又是一个内存不能读的问题，求教高手大侠指点！！

zeng_zong

本人在C++做一个模拟电梯的程序时，在做了CLock模块后调试时，又遭遇内存不能读的头痛问题。本人把代码说明如下：
在CLock.h文件中代码如下：
#ifndef CLOCK_H_INCLUDED_BF66D153
#define CLOCK_H_INCLUDED_BF66D153
#include <time.h>;
#include <sys/timeb.h>;
class Clock
{
  public:
          Clock();
          ~Clock();
          void showToday();
          void showZoneDifference();
          void showTimeZoneName();
          void showPlusMilliseconds();
          void show12HourTime();
          void showUniversalTime();
          void showUNIXTimeAndDate();
          void showSystemDate();
          void showSystemTime();
          void showSecond();

  private:
        char tmpbuf[128],ampm[2];
        time_t ltime;                          //local time
        struct _timeb tstruct;
        struct tm *today,*gmt;
};
#endif /* CLOCK_H_INCLUDED_BF66D153 */
在Clock.cpp中代码如下：
#include "Clock.h"
#include "sys/timeb.h"
#include "sys/types.h"
#include "time.h"
#include "string.h"
#include "stdlib.h"
#include "iostream.h"

void Clock::showSecond()
{

}

void Clock::showSystemTime()
{
        _strtime( tmpbuf );
        cout << "OS Time: " << tmpbuf << '\n';
}

void Clock::showSystemDate()
{
        _strdate(tmpbuf);
        cout << "OS Date: " << tmpbuf << '\n';
}

void Clock::showUNIXTimeAndDate()
{
        time( &ltime );
        cout << "Time in seconds since UTC 1/1/70:\t" << ltime << '\n'
                << "UNIX time and date:\t\t\t" << ctime( &ltime ) << '\n';
}

void Clock::showUniversalTime()
{
        gmt = gmtime( &ltime );
        cout << "Coordinated universal time: \t\t" << asctime(gmt) <<endl;
}

void Clock::show12HourTime()
{
        today = localtime( &ltime );
       
//        if ( today == 0 )
//                exit(1);

        if ( today->;tm_hour >; 12 )
        {
                strcpy(ampm,"M";
                today->;tm_hour -= 12;
        }
       
        if ( today->;tm_hour == 0 )
                today->;tm_hour = 12;

        cout << "12-hour time:\t\t\t\t" << asctime( today ) + 11
                << ampm << endl;
}

void Clock::showPlusMilliseconds()
{
        _ftime( &tstruct );                //additional time information
        cout << "lus milliseconds:\t\t\t" << tstruct.millitm << endl;
}

void Clock::showTimeZoneName()
{
        cout <<"Daylight savings:\t\t\t" << (tstruct.dstflag? "YES":"NO"
                << endl;
}

void Clock::showZoneDifference()
{
        cout << "Zone differnece in seconds from UTC:\t" << tstruct.timezone
                << "Time zone name:\t\t\t\t" << _tzname[0] << endl;
}

void Clock::showToday()
{
        today = localtime( &ltime );
        strftime( tmpbuf, 128,
         "Today is %A, day %d of %B in the year %Y.\n", today );
    cout << tmpbuf << endl;
}

Clock::Clock()
{
        strcpy(ampm,"AM";
        strcpy(tmpbuf,"";
        today = NULL;
        gmt = NULL;
       

        _tzset();        //set time zone from TZ environment variable.
}

Clock::~Clock()
{
}

在驱动程序中代码如下：
int main()
{
        Clock clock;

        clock.show12HourTime();
       
        return 0;
}

问题：编译和连接通过后，就是运行时显示内存不读。错误代码停在箭头处
void Clock::show12HourTime()
{
        today = localtime( &ltime );
       
//        if ( today == 0 )
//                exit(1);

-->;   if ( today->;tm_hour >; 12 )
        {
                strcpy(ampm,"M";
                today->;tm_hour -= 12;
        }
       
        if ( today->;tm_hour == 0 )
                today->;tm_hour = 12;

        cout << "12-hour time:\t\t\t\t" << asctime( today ) + 11
                << ampm << endl;
}
肯请哪位高手不吝指教？ 多谢！！！

yy_yangy

是嵌入式设备？

默难

构造函数中：

1. 
   
2. void Clock::Clock()
   
3. {
   
4. strcpy(ampm,"AM");
   
5. strcpy(tmpbuf,"");
   
6. today = NULL;
   
7. gmt = NULL;
   
8. 
   
9. 
   
10. _tzset(); //set time zone from TZ environment variable.
    
11. }
    

复制代码

没有给指针成员分配内存。改成这样：

1. 
   
2. void Clock::Clock()
   
3. {
   
4. strcpy(ampm,"AM");
   
5. strcpy(tmpbuf,"");
   
6. today = new sizeof(struct tm);
   
7. gmt = new sizeof(struct tm);
   
8. 
   
9. 
   
10. _tzset(); //set time zone from TZ environment variable.
    
11. }
    

复制代码

PS:别忘了在析构函数里面释放

whyglinux

上面的修改还不够充分。 today = localtime( &ltime );  这样以来，你的努力就白费了。

yy_yangy

today = new sizeof(struct tm);
gmt = new sizeof(struct tm);

不需要分配内存。和此无关。

默难

[quote]原帖由 "whyglinux"]上面的修改还不够充分。 today = localtime( &ltime );  这样以来，你的努力就白费了。[/quote 发表：
我错了……偶怎么犯这种低等错误哩。。。。。。。。。。。。。。。。没脸见人了。。。。。。。。。

whyglinux

yy_yangy 说得有理，错误不在于为 today 分不分配内存上，因为 today 指向的是静态存储区。

但是这就让人很困惑。在我的机器上运行并没有出现楼主所说的错误。正如yy_yangy所问的那样：是嵌入式设备？，我也怀疑楼主调试程序的时候不是在通用的 PC 机上。

不过，程序中确实存在着一处错误： char tmpbuf[128],ampm[2]; 这一句中 ampm 字符数组要用来保存 "AM" 或 "M" 显然其容量是不足的；数组的长度至少应该是 3（因为还要加上字符串结束标志）。楼主可把这里改过来试试。

zeng_zong

whyglinux说得不错，ampm[2]的长度应为3，因为要有一个结束标志。
不过改过来以后，还是存在同样的问题。 同时，本人说明一点，不是嵌入设备，只是pc而矣。谢谢各个大侠，请再帮帮忙，再给一点提示先？？？ ：－）

whyglinux

在使用 ltime 之前，先给它赋值。如 time( &ltime ); 难道错误会与此有关？想不通。这个再不行的话，吾已技穷矣！

zeng_zong

whyglinux: Thank you!!!!!
     You说得对，先给它赋值。就是使用time( &ltime );这条语句！！！！
problem is OVER! 但为什么呢？？？？老大，我也是青蛙跳水－－－不通，不通！！