manipulator dwuargumentowy

Question

Zrobiłem program który odczytuje dane z pliku i zapisuje je do kontenera i zlicza ilość.

Problem pojawia się przy "color"

zrobiłem strukturę

struct color										
{
	unsigned char red;
	unsigned char green;
	unsigned char blue;
};

wczytuje plik o rozszerzeniu ppm w którym każdy piksel składa się z 3 kolorów rgb.

w szablonie robię klasę która posiada metodę do wczytywania danych z pliku do kontenera map

 

std::fstream plik(sciezkapliku, std::ios::in);

	while (plik >> tekst)
	{
		kontener[tekst]++;
	}

gdzie zmienna tekst w tym przypadku ma typ color, a mapa w tym przypadku ma wartości map<color,int>kontener.

 

Żeby program działał muszę zrobić operator >> dla color

	friend std::fstream& operator>>(std::fstream &plik,const color &s)
	{
			plik >> s.red >> s.green >> s.blue;
		return plik;
	}

zrobiłem takie przeciążenie operatora>> ale przy próbie włączenia programu wywala błąd o braku odpowiedniego operatora>> dla tego typu.

Gdzie jest błąd ?

Answer 1

friend std::fstream& operator>>(std::fstream &plik,const color &s)

`s` jest oznaczony `const`, ale dwie linie niżej próbujesz go modyfikować.

Comments

tak to jest błąd, ale jest on przez to, że szukałem błędu i wpisałem const i zapomniałem go usunąć. Ale to nie o to chodzi o dalej wywala błąd, że nie odnajduje operatora

 Stan pominięcia Błąd    C2679    dwuargumentowy ">>": nie znaleziono żadnego operatora, który przyjmuje prawostronny operand typu "T" (lub nie istnieje akceptowalna konwersja)  

 

Gdzie T to color

---

Pokaż nowy kod po tym, jak usunąłeś const. W całości.

---

Doszedłem już do tego dlaczego nie łapało tego operatora. Po prostu zamiast w strukturze dałem operator w klasie.

Ale teraz inne pytanie bo wymagany jest operator<

jak powinien wyglądać ten operator

 

 Stan pominięcia Błąd    C2678    dwuargumentowy "<": nie znaleziono żadnego operatora, który przyjmuje lewostronny operand typu "const _Ty"

---

Nic mi to nie mówi bez kodu.

---

masz wszystko wyżej

struktura

struct color																	
{
	unsigned char red;
	unsigned char green;
	unsigned char blue;



	friend std::fstream& operator>>(std::fstream &plik, color &s)
	{
	

		plik >> s.red >> s.green >> s.blue;
		return plik;
	}

};

template

template <typename T>
class freqcounter
{
private:

	std::map < T, int > kontener;
	T tekst;

public:
	
	

	inline void add(std::string sciezkapliku)
	{
		
		std::fstream plik(sciezkapliku, std::ios::in);


		while (plik >> tekst)
		{
			kontener[tekst]++;
		}



		plik.close();
	}

	};

 

---

Ah.

Ale teraz inne pytanie bo wymagany jest operator<
jak powinien wyglądać ten operator

Żeby użyć std::map, przechowywany obiekt musi być porównywalny.

Czyli chodzi o

struct color  {
    bool operator<(const color& other) const {
        // ...
    }
}

W środku porównujesz r,g,b jednego koloru z drugim.

---

@karwyt,

Nasuwają mi się takie pytania:

1. Jeśli struktura color jest typowym VO (ang. Value Object), czyli surowym kontenerem na dane które są dostępne publicznie, w jakim celu robisz friend? Ogólnie friend należy unikać bo niszczy hermetyzację która nie jest tu potrzebna (zakładam że na potrzebę ćwiczenia publiczny dostęp do elementów color jest ok).

2. Nazwa freqcounter, sugeruje że będziesz chciał zliczać ilość wystąpień kolorów. Skąd więc pomysł na rozmycie odpowiedzialności tej klasy na obsługę plików? Nawet jeśli przyjmiesz że to będzie ok, nazwę metody add(..) raczej zmień na readFile(...), inputFromFile(...) czy jakąś podobną.

3. Kontener std::map, pozwoli Ci wyłącznie na szybkie odszukanie danych po kluczu ale już nie zezwoli na posortowanie ich względem ilości wystąpień. To taka uwaga abyś "nie szukał nadaremno". Inną rzeczą jest pytanie czy rzeczywiście niezbędne jest posortowanie wg. klucza? Co to Ci da (oprócz szybkiego wyszukania)? Szybkie wyszukanie osiągniesz także przez std::unordered_map które nie sortuje przy każdym wstawieniu.