Aide sur les erreurs de segmentation dans la structure

Question

J'ai des problèmes pour déboguer mon code. J'ai une structure et une fonction pour calculer la différence de temps entrée au format HH: MM: SS. Mon code est:

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

Je crois que le problème soit avec la 2e à la dernière ligne: timeDiff->seconds = ans%hourConv%minConv; depuis quand je commente cette ligne sur, je ne reçois pas une erreur de défaut de segmentation. Mais je ne comprends pas pourquoi cette ligne est invalide. Toute aide serait appréciée. Merci!

Answer 1

Votre code contient:

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

Vous avez créé un pointeur MyTime mais rien alloué. timeDiff est null à ce stade.

Answer 2

Vous essayez d'accéder à la mémoire non alloué avec le code suivant:

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

Bien que vous puissiez résoudre cette situation en allouant manuellement le code à l'aide de nouveau, comme:

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

je serais très recommande de changer votre fonction pour retourner le MyStruct par valeur, comme une variable allouée par pile. Je vous suggère également de prendre les arguments comme référence passe par const:

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

Juste une autre remarque: utiliser OOP dans ce cas. Cela rendra votre code beaucoup plus lisible. Vous aurez plus de temps pour réfléchir à la mémoire non initialisée.

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

À côté de cela, il est une bonne idée de faire la différence entre les différences de temps-et valeurs de temps « absolues ». Vous pouvez ajouter deux fois diff, mais vous ne pouvez pas ajouter deux valeurs de «calendrier».