lvalue required as left operand of assignment

Question

#include <iostream>

class Tablica {
public:
    Tablica(unsigned r) {
        rozmiar = r;
        tab = new int[rozmiar];
    }

    ~Tablica() {
        delete[] tab;
    }

    Tablica(const Tablica& inna) = delete;
    Tablica& operator=(const Tablica&) = delete;

    unsigned getRozmiar() const {
        return rozmiar;
    }

    //typ this: Tablica* const
    //typ tab: int*
    //zwracamy referencje na tab[idx]
     int& operator[](unsigned idx) {
        std::cout << "nieconstowy\n";
        return tab[idx];
     }

     //typ this: const Tablica* const
     int operator[](unsigned idx) const {
         std::cout << "CONST\n";
        return tab[idx];
     }
     void dodaj(int x){
        getRozmiar() = new int(x);
    for (unsigned i = tab.getRozmiar; i < x(); i++) {
        std::cout << i << ":\t " << tab[i] << '\n';
    }
     }

private:
    unsigned rozmiar;
    int* tab;
};

Witam, otóż mam problem ze swoją funkcją void dodaj (zaznaczona na biało). Podczas kompilacji wyskakuje mi komunikat "lvalue required as left operand of assignment", a ogólnie w tej funkcji void chodziło mi o dodanie nowego elementu x na końcu tablicy. Macie jakieś porady co poszło nie tak?

Comments

35 linia, co to jest?

---

Chodziło mi o dopisanie kolejnych tablic przez alokacje pamięci

---

getRozmiar() = new int(x); żeby to miało jakikolwiek sens musiałbyś zwracać referencję, ale i tak do innej zmiennej. Ogólnie nie ma to sensu. http://www.cplusplus.com/reference/cstdlib/realloc/ możesz skorzystać z tego, albo memcpy, albo memmove.

 

Answer 1

Przepis na dodanie nowych elementów do tablicy utworzonej na stercie.

1. Alokacja nowej tablicy z powiększonym rozmiarem
2. Kopiowanie elementów ze starej do nowej
3. Ustawienie dodanych elementów w tablicy na zero
4. Ustawienie nowego rozmiaru
5. Usunięcie starej tablicy ze sterty
6. Przypisanie do wskaźnika tab nowej tablicy

void dodaj( int n )
{
    int *new_tab = new int[rozmiar+n]; // (1)
    
    std::copy(tab, &tab[rozmiar], new_tab); // (2)
    memset(&new_tab[rozmiar], 0, n*sizeof(int)); // (3)
    
    rozmiar += n; // (4)
    delete[] tab; // (5)
    tab = new_tab; // (6)    
}