Klassenfrage :)

Quiche Lorraine

er kopiert den inhalt beginnend der start adressen
Es sind ja beides rohe c-arrays

Ok, du möchtest keine Hilfe.

Und Tschüss..

Schlangenmensch

@EL-europ
Es kann zufällig funktionieren. Es kann blödsinn ausgeben. Es kann abstürzen. Der Code ist einfach inhaltlich falsch, auch wenn es kompiliert und ausgeführt wird.

    int** matrix = (int**)malloc(xres * sizeof(int*));
    for (int i = 0; i < xres; i++)
      matrix[i] = (int*)malloc(yres * sizeof(int));
    int Count = 0;

    for (int x = 0; x < xres; x++)
    {
      for (int y = 0; y < yres; y++)
      {
        matrix[x][y] = Count++;
      }
    }

    int** matrix2 = (int**)malloc(xres * sizeof(int*));
    for (int i = 0; i < xres; i++)
      matrix2[i] = (int*)malloc(yres * sizeof(int));

    memcpy(matrix2, matrix, xres * yres * sizeof(int));
    //int* P = *matrix;

    for (int x = 0; x < xres; x++)
    {
      for (int y = 0; y < yres; y++)
      {
        printf("%i ", matrix2[x][y]);
      }
      printf("\n");

    }
    free(matrix);
    free(matrix2);

Führe das 10 mal aus. Kann gut gehen, muss aber nicht.

int main() {
  int xres = 5;
  int yres = 5;
 

  for (int z = 0; z < 10; ++z)
  {
    int** matrix = (int**)malloc(xres * sizeof(int*));
    for (int i = 0; i < xres; i++)
      matrix[i] = (int*)malloc(yres * sizeof(int));
    int Count = 0;

    for (int x = 0; x < xres; x++)
    {
      for (int y = 0; y < yres; y++)
      {
        matrix[x][y] = Count++;
      }
    }



    int** matrix2 = (int**)malloc(xres * sizeof(int*));
    for (int i = 0; i < xres; i++)
      matrix2[i] = (int*)malloc(yres * sizeof(int));

    memcpy(matrix2, matrix, xres * yres * sizeof(int));
    //int* P = *matrix;

    for (int x = 0; x < xres; x++)
    {
      for (int y = 0; y < yres; y++)
      {
        printf("%i ", matrix2[x][y]);
      }
      printf("\n");

    }

    free(matrix);
    free(matrix2);
  }
  return 0;
}

Schleife drum rum, und noch keinen Durchlauf hinbekommen bei dem das passiert wäre, was du hoffst.

Th69

@EL-europ: Abschließend von mir noch ein paar Hinweise.
Nur die Freigabe von matrix reicht auch nicht aus, denn du hast ja mehrfache malloc-Aufrufe (eben explizit für jeden matrix[i]), so daß du auch diese freigeben mußt (zu jedem malloc-Aufruf gehört ein zugehöriges free).
Du kannst dir auch mal diese Werte (matrix, matrix[i]) ausgeben lassen.
Vielleicht bringt dich das auf die wichtige Erkenntnis.

Und am besten ist es auch, du erzeugst dir viele einzelne Testprogramme (gerade um C++ auch zu verstehen) bzw. nutzt ein Test-Framework (wie z.B. Google Test). Diese kannst du auch dann auf deinem Entwicklungssystem ausführen (anstatt auf dem RASPI).
Dann wirst du auch erkennen, daß du dein Programm anders strukturieren solltest - auch damit du Fehler selber besser und schneller identifizieren kannst.

Edit:
Hier noch der Link zu einer 2D-Array Klasse für C++: Array2D: Evolution von manueller Speicherverwaltung zur STL

EL europ

@Schlangenmensch

int main(){
    int xres=10;
    int yres=10;
    for(int count=0; count<1000; count++){
		int **thr_matrix = (int**)malloc(xres * sizeof(int*));
		for(int i=0; i<xres; i++){
			thr_matrix[i] = (int*)malloc(yres * sizeof(int));
		}
		int **matrix = (int**)malloc(xres * sizeof(int*));
		for(int i=0; i<xres; i++){
			matrix[i] = (int*)malloc(yres * sizeof(int));
		}
		int z = 0;
		for(int x=0; x<xres; x++){
			for(int y=0; y<yres; y++){
				thr_matrix[x][y] = z++;
			}
		}
		memcpy(matrix, thr_matrix, (xres*yres)*sizeof(int));
		for(int x=0; x<xres; x++){
			for(int y=0; y<yres; y++){
				if(matrix[x][y] != thr_matrix[x][y]){
					printf("matrix[x][y]=%d, thr_matrix[x][y]=%d\n", matrix[x][y], thr_matrix[x][y]);
					printf("count %d\n", count);
				}
			}
		}
		free(matrix);
		free(thr_matrix);
	}
}

Beim compilieren meckert er:
test.cpp: In function ‘int main()’:
test.cpp:38:23: warning: ‘void* memcpy(void*, const void*, size_t)’ writing 400 bytes into a region of size 80 overflows the destination [-Wstringop-overflow=]
38 | memcpy(matrix, thr_matrix, (xres*yres)*sizeof(int));
| ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
test.cpp:28:45: note: destination object of size 80 allocated by ‘malloc’
28 | int matrix = (int)malloc(xres * sizeof(int*));

Da hat der Compiler doch vergessen das ich noch yres-mal thr_matrix[i] alloziiert habe? oder?

Bei mir macht das prog hier keine Ausgabe, also alles ok?

wob

@EL-europ sagte in Klassenfrage :

Da hat der Compiler doch vergessen das ich noch yres-mal thr_matrix[i] alloziiert habe? oder?

Sorry, aber was ist wohl wahrscheinlicher:
a) du hast den Code nicht verstanden
b) der Compiler hat etwas vergessen (und noch dazu in einem Beispiel, das extra für dich für dein Verständnisproblem erstellt wurde)

Bei mir macht das prog hier keine Ausgabe, also alles ok?

Im Programm von @Schlangenmensch ist ein printf drin. Wenn dann keine Ausgabe kommt, ist alles ok? Da die Datei bei dir test.cpp heißt: du führst nicht zufällig das shell-builtin test aus?

EL europ

@wob
Lies den Code!

Th69

Gebe mal vor und nach dem memcpy die Werte (Adressen) von matrix[i] und thr_matrix[i] aus!

Und hier noch eine Erklärung zu Multidimensionale Arrays in C (lies besonders "3 Multidimensionale Arrays" - beachte den Unterschied zwischen den beiden Arrays!).

Selbst mit (dem nicht mehr zu einer Warnung führenden)

memcpy(matrix, thr_matrix, xres * sizeof(int*)); // gleiche Angabe wie beim malloc!!!

werden hier nur die Zeiger kopiert, d.h. mit matrix[x][y] != thr_matrix[x][y] greifst du zweimal auf denselben Speicherbereich zu (d.h. es kann also niemals zu einem Ungleich kommen)!!!

PS:

@EL-europ sagte in Klassenfrage : ):
Da hat der Compiler doch vergessen das ich noch yres-mal thr_matrix[i] alloziiert habe? oder?

Das weiß der Compiler aber nicht, denn die Allokation findet ja erst zur Laufzeit statt!

DocShoe

Ich habe mal was dazu geschrieben: 2D Array in C++

Schlangenmensch

@EL-europ Dir erklären hier verschiedene Experten, dass dein Kopieren so nicht geht und trotzdem beharrst du dadrauf, das das kein Problem sei.

Du musst deine Matrix zeilenweise kopieren, sonst wird das nix.

irgendwie sowas

    for (int x = 0; x < xres; x++) {
      memcpy(matrix[x], thr_matrix[x], yres * sizeof(int));
    }

Aber, es geht ja um C++, da macht man das alles nicht:

int main() {
  int xres = 10;
  int yres = 10;
  for (int count = 0; count < 1000; count++) {
    std::vector<std::vector<int>> thr_matrix(xres);
    for(auto& v : thr_matrix)
    {
      v.resize(yres);
    }

    int z = 0;
    for (int x = 0; x < xres; x++) {
      for (int y = 0; y < yres; y++) {
        thr_matrix[x][y] = z++;
      }
    }
    auto matrix = thr_matrix;
    for (int x = 0; x < xres; x++) {
      for (int y = 0; y < yres; y++) {
        if (matrix[x][y] != thr_matrix[x][y]) {
          printf("matrix[x][y]=%d, thr_matrix[x][y]=%d\n", matrix[x][y], thr_matrix[x][y]);
          printf("count %d\n", count);
        }
      }
    }
  }
}

Ist viel einfacher, man kann viel weniger Fehler machen und es gibt auch keine Performance Probleme. Noch schöner wird es, wenn man den Artikel von @DocShoe berücksichtigt und 2 D richtig macht.

EL europ

@Th69
Der speicher wird ja aber erst wieder vor einer neuberechnung, wenn die Werte nicht mehr gebraucht werden, freigegeben; zuvor bleibt er doch erhalten? so das die Zeiger auf die richtigen Werte zeigen

Th69

Nein! Mit dem memcpy überschreibst du die mittels

matrix[i] = (int*)malloc(yres * sizeof(int));

erstellten Werte (und bei einer, hier noch fehlenden, Freigabe würdest du dann einen Laufzeitfehler, sog. "double free", erhalten).

@Schlangenmensch hat dir ja jetzt schon die Lösung verraten, aber eigentlich hätte schon bei deinem C -Code

thr_matrix = matrix;

vollkommen ausgereicht (denn auf das ganze Speicherallozieren und Kopieren kann man verzichten, da die Threads ja direkt den Matrix-Speicher beschreiben können - dein Code wartet ja explizit mit dem pthread_join auf die Abarbeitung aller Threads)...

EL europ

@Th69
ich denke mit
matrix[i] = (int*)malloc(yres * sizeof(int));
alloziere ich nur den speicher, und memcpy schreibt mit die neuen Zeiger in das array. Die doppelte Speicherverwaltung ist mir bewusst, das ist der Preis für Thread-Funktion

EL europ

@Th69 sagte in Klassenfrage :

Nein! Mit dem memcpy überschreibst du die mittels
matrix[i] = (int*)malloc(yres * sizeof(int));
erstellten Werte (und bei einer, hier noch fehlenden, Freigabe würdest du dann einen Laufzeitfehler, sog. "double free", erhalten).

..
thr_matrix = matrix;
vollkommen ausgereicht (denn auf das ganze Speicherallozieren und Kopieren kann man verzichten, da die Threads ja direkt den Matrix-Speicher beschreiben können - dein Code wartet ja explizit mit dem `pthread_join` auf die Abarbeitung aller Threads)...

Das hab ich noch gar nicht ausprobiert! geht das ?

Th69

LOL

EL europ

@EL-europ
Jetzt sind auch keine Abbrüche mehr mit den Sanitizern, aber Meldungen nach Programmende.

EL europ

@Th69
Oje, jetzt hab ich mir selbst geantwortet
Ich pushe git jetzt mal, die Apple->calc sollte soweit funzen?
die rohen c-array one-, ten- und iter- Member werd ich mal in std::vector versuchen, vielleicht gehen dann noch sanitizer Meldungen weg. Da scheint mir in Apple->sort() auch nicht was ganz sauber zu sein.

omggg

Dieser Beitrag wurde gelöscht!

Schlangenmensch

@EL-europ sagte in Klassenfrage :

Das hab ich noch gar nicht ausprobiert! geht das ?

Du musst nur beachten, dass du ausschließlich den Pointer kopierst. D.h, du hast dann zwei Pointer die auf den selben Speicherbereich zeigen

EL europ

Ich hab die unnötoge Speicherallocierung für **matrix und **colormatrix erkannt und die Sanitizermeldungen dieser arrays weg bekommen Jetzt hab ich noch welche durch **iterMembers in Apple->sort(). Könnt ihr mir bitte helfen die noch weg zu bekommen?