帮助解决结构中的分段错误

Question

我在调试代码时遇到问题。我有一个结构和一个 函数计算以 HH:MM:SS 格式输入的时差。 我的代码是：

const int hourConv = 3600; // used to get total hours from total seconds 
const int minConv = 60; 
struct MyTime { 
    int hours, minutes, seconds; 
}; 

MyTime *determineElapsedTime(const MyTime *time1, const MyTime *time2) 
{ 
        long timeOneSec = time1->hours*hourConv + time1->minutes*minConv + time1->seconds; 
        long timeTwoSec = time2->hours*hourConv + time2->minutes*minConv + time2->seconds; 
        long ans = timeTwoSec - timeOneSec; 
        cout << ans; 
        MyTime *timeDiff; 
        timeDiff->hours = ans / hourConv; 
        timeDiff->minutes = ans % hourConv / minConv; 
        timeDiff->seconds = ans % hourConv % minConv; 
        return timeDiff; 
} 

我相信问题出在倒数第二行： timeDiff->秒= ans%hourConv%minConv; 自从我评论该行以来， 我没有收到分段错误错误。但我不明白为什么 该行无效。任何帮助将不胜感激。谢谢！

Answer 1

您的代码包含：

MyTime *timeDiff;
timDiff->hours = ...

您已创建一个 MyTime 指针，但未分配任何内容。此时 timeDiff 为空。

Answer 2

您尝试使用以下代码访问未分配的内存：

MyTime *timeDiff;
timeDiff->hours = ans / hourConv;

尽管您可以通过使用 new 手动分配代码来解决此问题，如下所示：

MyTime *timeDiff = new MyTime;
timeDiff->hours = ans / hourConv;

我强烈建议更改您的函数以按值返回 MyStruct 作为堆栈分配的变量。我还建议将参数作为传递常量引用：

MyTime determineElapsedTime(MyTime const &time1, MyTime const &time2)
{
     long timeOneSec = time1.hours*hourConv + time1.minutes*minConv + time1.seconds;
     long timeTwoSec = time2.hours*hourConv + time2.minutes*minConv + time2.seconds;
     long ans = timeTwoSec - timeOneSec;
     cout << ans;
     MyTime timeDiff;
     timeDiff.hours = ans / hourConv;
     timeDiff.minutes = ans % hourConv / minConv;
     timeDiff.seconds = ans % hourConv % minConv;
     return timeDiff;
}

Answer 3

还有一点要注意的是：在这种情况下使用 OOP。它将使您的代码更具可读性。您最终将有更多时间来考虑未初始化的内存。

struct MyTime { 
    int hours, minutes, seconds; 
    int timeInSeconds() const {
        return hours*3600 + minutes*60 + seconds;
    }
    // the difference, in seconds
    int operator-( const MyTime other ) const {
        return timeInSeconds() - other.timeInSeconds();
    }
    void subtract( int seconds ) {
        seconds -= seconds;
        while( seconds < 0 ) { seconds += 60; --minutes; }
        while( minutes < 0 ) { minutes += 60; --hours; }
        assert( hours >= 0 );
    }
}; 

除此之外，区分时差和“绝对”时间值是个好主意。您可以添加两个时间差异，但不能添加两个“日历”值。