C++ dzielenie klas na mniejsze

pytanie zadane 12 marca 2022 w C i C++ przez Billy Użytkownik (680 p.)

Siemka,

Stworzyłem ostatnio program, który obsługuje i manipuluje obrazami, oraz zamienia je na ich wersje ASCII. Cały kod umieściłem w jednej klasie, odpowiednio podzielonej na funkcje. Funkcje uporządkowałem grupami (funkcje operujące na pliku, filtry, obracanie obrazu itp.).

I tu jest moje pytanie. Jak wydzielić te poszczególne grupy? Żeby to wszystko nie siedziało w jednej klasie, tylko było bardziej zorganizowane? Myślałem o rozwiązaniu, gdzie wydzieliłbym kod do pomocniczych klas, w klasie głównej tworzyłbym odpowiednie pod-obiekty i dostęp do funkcji wyglądałby następująco:

img.filters.changeToGrayscale();

Ale nie wiem, czy jest to optymalne rozwiązanie. Może jakaś porada od bardziej doświadczonych kolegów? :P

Link do githuba: github

komentarz 13 marca 2022 przez VBService Ekspert (253,420 p.)

BTW, int ratio - nie używane.

float calcSize(int pixels, int orginalSize, int ratio)
{
	return (float)pixels / (float)orginalSize * 0.5;
}

pomimo wywołania calcSize(width, img.getW(), 0.5)

int main()
{
	...
	img.reduce(width, calcSize(width, img.getW(), 0.5));

    ...

	return 0;
}

komentarz 13 marca 2022 przez Billy Użytkownik (680 p.)

O faktycznie, dzięki za wypunktowanie :P Wyciąłem kod do funkcji, i zapomniałem zmienić 0.5 na parametr :P

A masz jakieś pomysły, jak można podzielić klasę na mniejsze, tak jak w pytaniu?

komentarz 14 marca 2022 przez tkz Nałogowiec (42,000 p.)

https://github.com/Billypl/Ascii-Generator/blob/b22c3c446f4f1d90fb27f567d4943188fedd327b/Image.cpp#L313 wychodzisz poza zakres tablicy, chyba, że to tak specjalnie...

komentarz 29 marca 2022 przez Billy Użytkownik (680 p.)

Podświetlało mi, że możliwe jest wyjście poza zakres, ale przecież mam `i += channels`, czyli `i` w ostatniej iteracji będzie wynosiło `size() - channels`, bo przy następnej i będzie większe od size(). Czyli wszystko powinno się zmieścić, nie? Czy coś źle kombinuję?