Dynamicznie alkowane tablice

Question

Cześć.

Mam utowrzoną nową dynamicznie alokowaną tablicę i chciałbym do niej przepisać wartości(do innego miejsca w pamięci) a następnie usuną starą, ale gdy uruchamiam coś takiego to nie działa tak jak powinno tzn alkouje miejsce na nową tablicę ale wartości zamiast przypisania to jakieś śmieci.

void changeSizeOfVector(int *tabValue, int *tabOffsets, int &oldSize, int newSize) {;
	int *newTabOfValues = new int[newSize];
	int *newTabOfOffsets = new int[newSize];

	int temp=0;
	if (oldSize > newSize)
		temp = newSize;
	if (oldSize < newSize)
		temp = oldSize;

		for (int i = 0; i < temp; i++) {
			newTabOfValues[i] = tabValue[i];
			newTabOfOffsets[i] = tabOffsets[i];
		}

		delete[] tabValue;
		delete[] tabOffsets;

		tabValue = new int[newSize];
		tabOffsets = new int[newSize];

		beg1 = tabValue;
		beg2 = tabOffsets;
		
		for (int i = 0; i < temp; i++) {
			tabValue[i] = newTabOfValues[i];
			tabOffsets[i] = newTabOfOffsets[i];
			}

	oldSize = temp;
	delete[] newTabOfValues;
	delete[] newTabOfOffsets;
}

 

Answer 1

Zaalokowales miejsce na tablice ale nie masz gwarancji że są w niej same zera np. majac sytaucje ze newSize jest wiesze od oldSize to przypisujesz stare wartosci do oldSize a killka zostaje nadal zaśmieconych.

Dodaj do alokowania pamięci {0} na koniec to Ci wypelni samymi zerami lub skorzystaj z jakiegos kontenera.

Co robia zmenne?

beg1 = tabValue;
beg2 = tabOffsets;

 

Jaki jest sens przypisywania oldSize do oldSize?

Comments

Na tak, ale to nie ma znaczenia co tam jest bo i tak potem to usuwam delete[] tabValue

Jeżeli wywołam tą funkcję to dostaje same śmieciowe wartość bez żadnych normalnych.

Zmienne nic nie robią po prostu zapomnialem skasowac

Wywołanie + alokacja :

	int SIZE = 10;

	int * tabOfValues = new int[SIZE];
	int * tabOfOffsets = new int[SIZE];
	for (int i = 0; i < SIZE; i++)
	{
		tabOfValues[i] = i;
		tabOfOffsets[i] = i;
	}

	changeSizeOfVector(tabOfValues, tabOfOffsets,SIZE,8);
	
	for (int i = 0; i < SIZE; i++)
		cout << tabOfValues[i] << " : " << tabOfOffsets[i] << endl;

 

---

Jeśli to jest cały kod który wrzuciles to jeśli zmiejszasz tablice to wyniki wychodzą normalne bynajmniej u mnie.

---

Tutaj masz cały kod, skompiluj to :

void changeSizeOfVector(int *tabValue, int *tabOffsets, int &oldSize, int newSize) {;
	int *newTabOfValues = new int[newSize];
	int *newTabOfOffsets = new int[newSize];

	int temp=0;
	if (oldSize > newSize)
		temp = newSize;
	if (oldSize < newSize)
		temp = oldSize;

		for (int i = 0; i < temp; i++) {
			newTabOfValues[i] = tabValue[i];
			newTabOfOffsets[i] = tabOffsets[i];
		}

		delete[] tabValue;
		delete[] tabOffsets;

		tabValue = new int[newSize];
		tabOffsets = new int[newSize];
		
		for (int i = 0; i < temp; i++) {
			tabValue[i] = newTabOfValues[i];
			tabOffsets[i] = newTabOfOffsets[i];

		}

	oldSize = newSize;
	delete[] newTabOfValues;
	delete[] newTabOfOffsets;
}

int main()
{
	int SIZE = 10;

	int * tabOfValues = new int[SIZE];
	int * tabOfOffsets = new int[SIZE];
	for (int i = 0; i < SIZE; i++)
	{
		tabOfValues[i] = i;
		tabOfOffsets[i] = i;
	}

	changeSizeOfVector(tabOfValues, tabOfOffsets,SIZE,8);
	
	for (int i = 0; i < SIZE; i++)
		cout << tabOfValues[i] << " : " << tabOfOffsets[i] << endl;

	cout << tabOfValues[9] << " : " << tabOfOffsets[9] << endl;
	
	return 0;
}

 

---

#include<iostream>

using namespace std;

void changeSizeOfVector(int *tabValue, int *tabOffsets, int &oldSize, int newSize) {
    int *newTabOfValues = new int [newSize]{0};
    int *newTabOfOffsets = new int [newSize]{0};
 
        for (int i = 0; i < oldSize; i++) {
            newTabOfValues[i] = tabValue[i];
            newTabOfOffsets[i] = tabOffsets[i];
        }
 
        delete [] tabValue;
        delete [] tabOffsets;
 
        tabValue = new int [newSize]{0};
        tabOffsets = new int [newSize]{0};
 
        
         
        for (int i = 0; i < newSize; i++) {
            tabValue[i] = newTabOfValues[i];
            tabOffsets[i] = newTabOfOffsets[i];
        }
 
	oldSize = newSize;
	delete [] newTabOfValues;
	delete [] newTabOfOffsets;
}
int main(){
	
	int SIZE = 10;
 
	int * tabOfValues = new int [SIZE];
	int * tabOfOffsets = new int [SIZE];
	for (int i = 0; i < SIZE; i++)
	{
	    tabOfValues[i] = i;
	    tabOfOffsets[i] = i;
	}
	 
	changeSizeOfVector(tabOfValues, tabOfOffsets,SIZE,11);
	 
	for (int i = 0; i < SIZE; i++)
	    cout << tabOfValues[i] << " : " << tabOfOffsets[i] << endl;
		
	 delete [] tabOfValues;
	 delete [] tabOfOffsets;
}

Masz przykład troszke przerobiony. 

---

Wywala mi dziwne błędy

---

---

tzn. jakie? 

---

To zależy od kompilatora, w Code Blocks wszystko działa ok tylko musiałem usunąć {0}, a w MVS 2015 nie działa wogóle, wywala dziwne błędy typu app.exe has triggered a breakpoint, po czym się zawiesza.

---

Ta changeSizeOfVector jest bez sensu. Popraw ją na:

void changeSizeOfVector(int* &tabValue, int* &tabOffsets, size_t oldSize, size_t newSize) 
{
    int *newTabOfValues = new int [newSize]{0};
    int *newTabOfOffsets = new int [newSize]{0};
  
    size_t toCopy = std::min(oldSize, newSize);

    for (size_t i = 0; i < toCopy; ++i) {
        newTabOfValues[i] = tabValue[i];
        newTabOfOffsets[i] = tabOffsets[i];
    }

    delete [] tabValue;
    delete [] tabOffsets;

    tabValue = newTabOfValues;
    tabOffsets = newTabOfOffsets;
}

 

---

W Twoim rozważaniu LightInWindow jak i moim pomineliśmy fakt, że przy pomocy takiej deklaracji funkcji 

void changeSizeOfVector(int *tabValue, int *tabOffsets, int &oldSize, int newSize)

nie można zmienić wskaźnika. Wiec rozwiązaniem tego jest referencja użyta wyżej przez kolege lub **/

---

Dzięki za pomoc, ułatwiło mi to zrozumienie ! :)