Помогите с ошибкой сегментации в структуре

Question

У меня возникла проблема с отладкой моего кода. У меня есть функция struct и , чтобы вычислить разницу во времени, введенную в формате 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; 
} 

Я считаю, что проблема быть с 2-го по последней строке: timeDiff->seconds = ans%hourConv%minConv; , так как, когда я комментировать эту линию, я не получаю ошибку ошибки сегментации. Но я не понимаю, почему эта строка недействительна. Любая помощь будет оценена по достоинству. Благодаря!

Answer 1

Ваш код содержит:

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

Вы создали указатель Mytime, но ничего не выделяется. timeDiff имеет значение null в этот момент.

Answer 2

Вы пытаетесь получить доступ к нераспределенной памяти со следующим кодом:

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

Хотя вы могли бы решить эту проблему вручную выделений кода с использованием нового, как:

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

Просто еще одно замечание: используйте ООП в этом случае. Это сделает ваш код более читабельным. У вас будет больше времени, чтобы подумать о неинициализированной памяти.

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); 
    } 
}; 

В дополнение к этому, рекомендуется отличать временные разности и абсолютные значения времени. Вы можете добавить два разных раза, но вы не можете добавить два значения «календарь».