Klassenfrage :)

[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?