SIGBUS on thread : -1717046736

Question

Gdy kompiluje program na jednym kompilatorze pojawia sie komunikat:

SIGBUS on thread : -1717046736

Na innym zaś nie ma tego problemu i kompiluje się dobrze co może powodować ten błąd

#include <iostream>
using std::cout;
using std::endl;
class Tuplyw_czasu;

std::ostream& operator<<(std::ostream&, const Tuplyw_czasu&);

class Tuplyw_czasu{
	unsigned long long czas;
	public:
	Tuplyw_czasu(unsigned long long a):czas{a}{
		}
	Tuplyw_czasu operator+(const Tuplyw_czasu& temp){
		return Tuplyw_czasu{czas+temp.czas};
	  }
	  friend std::ostream& operator<<(std::ostream&, const Tuplyw_czasu&);
};

Tuplyw_czasu operator"" _d(unsigned long long t){
	return Tuplyw_czasu{t*3600*24};
}

Tuplyw_czasu operator"" _godz(unsigned long long t){
	return Tuplyw_czasu{t*3600};
}

Tuplyw_czasu operator"" _min(unsigned long long t){
	return Tuplyw_czasu{t*60};
}

Tuplyw_czasu operator"" _sek(unsigned long long t){
	return Tuplyw_czasu{t};
}

int main(){
	Tuplyw_czasu	czas = 3_sek + 1_min;
	cout << "Krotszy czas = "<< czas << endl;

	czas =  24_godz + 2_d + 5_godz;
	cout << "Dluzszy czas = " << czas << endl;  
}
	
std::ostream& operator<<(std::ostream& entrance, const Tuplyw_czasu& temp){
  entrance<<"To w sekundach czas = "<<temp.czas<<endl;
}

 

Comments

Sam SIGBUS oznacza problem z przydzieleniem pamięci.
To jest cały kod?
Czym kompilujesz?
Możliwe, że to kwestia przypadku, spróbuj zresetować komputer, może adresy się poustawiają odpowiednio.

Answer 1

W przeładowaniu operatora << nic nie zwracasz ( brak return entrance;) co może powodować SIGBUS, gdyż w instrukcji

cout << "Krotszy czas = "<< czas << endl;

ostatni człon ( endl ) jest parametrem funkcji operator<<(endl) wywoływanej na rzecz obiektu typu void , a nie typu ostream (gdyż nic nie jest zwracane z przeładowania operatora), co prowadzi do UB.

Comments

Jak może być wysyłane w nieokreślone miejsce skoro jest wywoływane na rzecz ostream'a? Poza tym jest określone gdzie, przez pierwszy argument w funkcji.

---

Trochę to za bardzo uprościłem. W rzeczywistości brak zwracanej wartości w tym przypadku prowadzi do Undefined Behavior, czyli może równie dobrze spowodować błąd SIGBUS.

---

Może wyjaśnie to za bardzo prosto, ale może będą to czytać początkujący. Zapis

cout << costam << cosinnego << jeszczecos << endl;

nie oznacza wywołanie jakiegoś tajemniczego operatora wieloargumentowego, tylko serię wywołań operatora dwuargumentowego, gdzie wynik wcześniejszego działania jest jednym z argumentów następnego. W zależności od tzw wiązania operatora gupowanie jest od lewej lub prawej.

W tym przypadku będzie to coś takiego:

(((cout << costam) << cosinnego) << jeszczecos) << endl;

W przypadku przeciążania (a tak jest z <<) każdy operator to może być inna funkcja i jeśli któraś zwróci zły wynik to następna dostanie zły argument.

Może sprawdź, czy twój kompilator nie potrafi generować ostrzeżeń.

 

Trochę podejrzana wydaje mi się też linijka w operatorze +. temp.czas oznacza dostęp do prywatnego pola innego obiektu. Pamiętam, że początkowo (w początkach C++) kompilatory to puszczały (w sumie ta sama klasa), ale później przestały, bo to jednak jest inny obiekt, nie ten identyfikowany przez this.

---

Trochę podejrzana wydaje mi się też linijka w operatorze +. temp.czas oznacza dostęp do prywatnego pola innego obiektu. Pamiętam, że początkowo (w początkach C++) kompilatory to puszczały (w sumie ta sama klasa), ale później przestały, bo to jednak jest inny obiekt, nie ten identyfikowany przez this.

Który kompilator tego nie puszcza? Kwestią, która by mogła zaważyć jest jedynie opcja użycia {}, zamiast (). Poza tym jest to operator klasy, nie jakiś zewnętrzny dodatek. Reszta jest jak najbardziej poprawna. 

Może wyjaśnie to za bardzo prosto, ale może będą to czytać początkujący. Zapis

C/C++

Kopiuj

?

1	cout << costam << cosinnego << jeszczecos << endl;

nie oznacza wywołanie jakiegoś tajemniczego operatora wieloargumentowego, tylko serię wywołań operatora dwuargumentowego, gdzie wynik wcześniejszego działania jest jednym z argumentów następnego. W zależności od tzw wiązania operatora gupowanie jest od lewej lub prawej.

Z tego co mówi dokumentacja, ostream& operator<< jest operatorem jednoargumentowym. Od c++ 11 przyjmującym 17 przeciążeń. A cout jest obiektem ostream mimo wszystko.