Zgłoszono wyjątek: naruszenie dostępu do zapisu. v było nullptr.

Question

Witam, potrzebuje pomocy- nie wiem gdzie znajduje sie moj blad przy wskaznikach:

#include "pch.h"
#include <iostream>
#include "sort.h"
#include <stdio.h>
#include <stdlib.h>
#include <time.h>

int createTab(int** pTab,int nSize)
{
	*pTab = (int*)malloc(nSize * sizeof(int));
	if (pTab) memset(pTab, 0, nSize * sizeof(int));
	else return 0;
	return 1;
}	
void freeTab(int* pTab)//zwolnij tablice /mallock&memset & czy sie powiodlaelse null
{
	free(pTab);
}
void initTab(int* pTab, int nSize, int nVal)
{
	int *v = pTab; 
	for (int i = 0; i < nSize; i++)
	{
		*v = nVal; //TUTAJ BLAD WSKAZNIKA
		v++;
	}

}
void randomInit(int* pTab, int nSize) //srand&rand //modulo rozmiar -preferencje
{
	
	int *v = pTab;
	time_t tt;
	int zarodek = time(&tt);
	srand(zarodek);   // za zarodek wstawiamy pobrany czas w sekundach
	for (int i = 0; i < nSize; ++i)
	{
		*v = rand() % 999; //zleeeeee
		v++;
	}

void PrintTab(int* pTab, int nSize)
{
	int *v = pTab;
	for (int i = 0; i < nSize; i++)
	{
	printf("%d\t", *v);
	*v++;
	}
	
}

#include "pch.h"
#include <iostream>
#include "sort.h"
#include <stdio.h>

using namespace std;


int* pTab = NULL;
int* pomTab = NULL;

int main()
{
	int nSize = 0;
	printf("Podaj ilosc elementow do posortowania:\n");
	cin >> nSize;

	createTab(&pTab,nSize);
	initTab(pTab, nSize, 0);
	randomInit(pTab, nSize);
	PrintTab(pTab, nSize);
	freeTab(pTab);



	return 0;
}

#ifndef SORT_H
#define SORT_H

void freeTab(int* pTab);//zwolnij tablice /mallock&memset & czy sie powiodlaelse null
void initTab(int*, int nSize, int nVal);
void randomInit(int* pTab, int nSize); //srand&rand //modulo rozmiar -preferencje
void copyTab(int* pTab, int* pTabFrom, int nSize); //kopiuje do tabl z 2 do 1
void PrintTab(int* pTab, int nSize);
int createTab(int** pTab, int nSize);

#endif

 

Comments

https://www.fluentcpp.com/2017/06/13/the-real-difference-between-struct-class/

O jakich fundamentach mówisz dokładnie? 

Zwracanie wartości null jest złym podejściem, gdy zamiast tego można zwrócić, powiedzmy, pustą tablicę.

Powoływałem się na słowa Robert C. Martin, a myślę, że jest jakimś autorytetem w świecie IT.

---

Ok poczytam, przemyślę i odpowiem, ale to pewnie nie wcześniej niż za miesiąc.

EDIT:: A to co pisze o C strukturze i to sam próbowałem wyklepać rozbudowaną strukturę  i no i nie było tak łatwo i przyjemnie. Ale to może wynikać z mojej nie znajomości C w tym zakresie.

Ps. A źródło fajne, bardzo dużo tam ciekawych rzeczy. Tylko ten czas :(;

---

Zwracanie wartości null jest złym podejściem, gdy zamiast tego można zwrócić, powiedzmy, pustą tablicę.

(Um... to raczej jest javowa rada? Bo W C++ie ona raczej nie ma sensu. Ale to już raczej nie jest temat na ten wątek, więc jak ktoś chce ciągnąć to PM)

---

adrian17  jak na wszystko, to zależy...

---

1. Co do zasady, w C++ nie używa się malloc. Zresztą szereg metodyk tego zabrania (branża lotnicza, medyczna, automotive).

2. Co do "manualnej alokacji pamięci" (tak dla C jak i C++), zaleca się by alokacja i zwolnienie pamięci, odbywała się na tym samym poziomie abstrakcji. Czyli funkcja która alokuje, powinna (o ile to możliwe), zwalniać pamięć przed wyjściem z niej. (CERT Secure Coding Standards).

3. Co do sygnalizowania null'em braku alokacji, dla C jest to rzecz normalna. Dla C++ są lepsze sposoby (np. std::optional).

4. Od reguł są wyjątki. Np. Qt ma odmienną filozofię alokacji narzucaną przez framework.

5. Jest możliwość (choć powinna być wybierana z ostrożnością), by alokować dane poprzez argument przekazywany a poprzez return sygnalizować efekt działania. W wielu przypadkach taki kod może być mało czytelny. Czasem takie funkcje obejmuje dany standard (np. NVIDIA dla CUDA czy Khronos dla OpenCL robi to nagminnie). Dla C skutkuje to argumentem podwójnego wskaźnika.

Answer 1

Dzieki za rady - ogolnie to sie sam uczylem troche c++ troche c ,a ze visual kompiluje jedno z drugim to smiga :D. U mnie osobiscie nie zadzialalo przez memset w createtab. Po odkomentowaniu program smiga