c++ Rozszerzanie tablicy (czy ten kod jest poprawny?)

Question

Witam, stworzyłem taki kod i chciałem się spytać o jego poprawność.

Stworzyłem zmienne globalne co niektórym może się nie spodobać. Zwykle nie daje globalnych ale trudno.

#include <iostream>

using namespace std;

int il=9;
int *tablica = new int[9];

void przestaw(int x)
{
    int *sch = new int[il];

    for(int i=0;i<=il;i++)
    {
        sch[i] = tablica[i];
    }

    tablica = new int[x];
    il = x;

    for(int i=0;i<=il;i++)
    {
        tablica[i] = sch[i];
    }
}

int main()
{
    tablica[0]=0;
    tablica[1]=1;
    tablica[2]=2;
    tablica[3]=3;
    tablica[4]=4;
  
    for(int i=0;i<=4;i++)
        cout<<tablica[i]<<endl;

    cout<<endl;
    przestaw(16);


    for(int i=0;i<=15;i++)
    {
        cout<<tablica[i]<<endl;
    }
    return 0;
}

Czy takie wydłużanie i skracanie tablic jest poprawne? Jest może lepszy sposób? Może wprowadzić jakąś poprawkę?

Answer 1

Błąd w kodzie:

    for(int i=0;i<=il;i++)

wychodzisz poza zakres tablicy dokładnie przez zapis i<=il (indeksowanie zaczyna się od 0.

A i jeszcze jedno, przydzielasz nową pamięć do zmiennej tablica a nie zwalniasz startej pamięci (będziesz miał wyciek pamięci). To samo dotyczy zmiennej tymczasowej sch przydzielasz jej pamięć a później jej nie zwalniasz.

Zacytuję tutaj Pana Mirosława: "Pamiętaj, za każdym razem jak nie zwalniasz pamięci gdzieś na świecie ginie mały kotek!" :)

Comments

Dzięki za odpowiedź, dodałem li-1 i zwalnianie pamięci. Wystraszyłem się że to nie działa na tablicach obiektów klas, ale wszystko chodzi ;)

Answer 2

Możesz stworzyć tablicę dynamiczną o określonym maksymalnym rozmiarze, a następnie dodawać elementy za pomocą iteratora (zmiennej określającej aktualny rozmiar).