C++ Klasy i zamiana wartości.

Question

Witam wszystkich. Z góry uprzedzam że jestem początkującym programistą więc proszę o wyrozumiałość. Otóż mam pewną klasę która ma za zadanie utworzyć tablicę, wypełnić wartością 5,podmienić pierwszą wartość na 0 i wyświetlić wynik. Proszę powiedzcie gdzie leży błąd i jak można go naprawić? Z góry dziękuję za pomoc

 

class GRAFIKA{

public :

 int ROZMIAR_OKNA(int c){


HANDLE handle = GetStdHandle(STD_OUTPUT_HANDLE); // potrzebujesz uchwytu
    SMALL_RECT sr;

    sr.Left = 0;
    sr.Top = 0;
    sr.Right = 49;
    sr.Bottom = 49;
    SetConsoleWindowInfo(handle,true,&sr); // ustawiasz szerokoœæ okna na 50 x 50 (w szerokoœciach i wysokoœciach znaku pojedynczego a nie w pikselach)

    CONSOLE_SCREEN_BUFFER_INFO sbi; // struktura opisuj¹ca okno

    GetConsoleScreenBufferInfo(handle, &sbi);
    int X = sbi.srWindow.Right - sbi.srWindow.Left + 1;
    int Y = sbi.srWindow.Bottom - sbi.srWindow.Top + 1;

    switch(c){
    case 0 : return X;
    case 1 : return Y; } }

    int X = ROZMIAR_OKNA(0);
    int Y = ROZMIAR_OKNA(1);


    int *PLANSZA = new int[5];
    void ZERUJ_PLANSZE(){
    int *PLANSZA = new int[5];

    for(int i = 0; i < 5; i++){
        PLANSZA[i] = 5;       } }

    void ZAMIANA(){
    int *PLANSZA;
    PLANSZA[0] = 0;

    }


    void POKAZ_WYNIK(){
        int *PLANSZA;
        for(int i = 0; i < 5; i++){
        cout << PLANSZA[i];       }
    }

 

Answer 1

Wszystkie nazwy w kodzie pisane CAPSLOCKIEM... Na miły Bóg! Miej litość nad tym nieszczęsnym programistą, bo nie wie co czyni ;-;

Linijka 31:

int *PLANSZA = new int[5];

Linijka 33:

int *PLANSZA = new int[5];

Linijka 46:

int *PLANSZA;

We wszystkich tych wierszach tworzysz nowe wskaźniki o tej samej nazwie. Sposób pisania przez Ciebie klamer w kodzie woła o pomstę do Nieba (tak swoją drogą). Kompilator nie wyświetla błędu, bo mogą istnieć zmienne o takiej samej nazwie w innym zakresie.

Funkcja ZERUJ_PLANSZE tworzy własną tablicę i to ją zeruje. Podobnie funkcja POKAZ_WYNIK tworzy swój wskaźnik i wyświetla zawartość pięciu kolejnych komórek (tym samym wychodząc poza dozwolony zakres — naruszenie pamięci).

Linijka 31 zawiera deklarację i definicję tablicy, która ma zasięg globalny w całej klasie. Do tej tablicy masz się odwoływać za każdym razem. Usuń więc deklaracje tablic z wierszy 33 i 46 i poprawnie utwórz tablicę dla klasy (najlepiej prywatną).

Comments

Wiem że ten kod wygląda jak wygląda ale dopiero się uczę programowania i jestem nowicjuszem co do tematu klas. Jeśli mógłbym prosić to podeślij mi poprawiony kod

Answer 2

Dwia złe nawyki pierwsze się rzucają w oczy. Jeśli nazwa składa się z samych dużych liter to sugeruje, że jet to stała. Nazw klas i funkcji nie powinno sie pisać dużymi literami bo mogą być mylone właśnie ze stałymi. Druga sprawa wcięcia i domknięcia klamer. W kilku miejscach miałem ten problem, że nie wiem czy już wysziśmy z bloku czy jeszcze w nim jesteśmy(np. linia 27). I nie wiem gdzie są jakieś pola klasy.

Twój problem jest w tym, że za każdym razem jak chcesz coś zrobić z planszą to tworzysz nową w lokalnym scopie(zasięgu), a ta planasza w scopie wyżej zostaje niezmieniona. Pozatym ani razu nie dealokujesz pamięci.

Najłatwiej jest to rozwiązać w taki sposób. Tworzysz pole planszy int* plansza. Najlepiej prywatne. W kostruktorze alokujesz pamięć na tą planszę, a w destruktorze ją dealokujesz. W metodach nie tworzysz nowej zmiennej plansza tylko odwołujesz się do pola klasy.