// accessor
string get_host() const {
return host;
}
string get_guest() const {
return guest;
}
Te metody tworzą kopie stringów host oraz guest, co jest dosyć czasochłonne. Dostałem AC po zmianie tych getterów w taki sposób, żeby zwracały stałe referencje.
Kilka uwag:
- gettery są zwykle używane, gdy pola klasy są prywatne; w Twoim przypadku są one publiczne, więc używanie accessorów się chyba mija z celem.
- popracuj trochę nad wcięciami. Aktualnie wygląda to tak, jakbyś rzucał kostką przed każdym wcięciem, i na podstawie uzyskanego wyniku decydował ile spacji/tabów użyć.