Inhalt eines char Pointers konvertieren

was macht eigentlich toupper(0)? das kommt vor wenn das letzte zeichen ' ' oder '-' ist...

if(Name[j]!=0)
    Name[j] = toupper(Name[j]);

toupper(0) gibt 0 zurück also kannst dir meinen letzten post schenken...

noobLolo schrieb:

eigentlich kannst du das auch in-place also ohne kopierung machen
denke dafür mußt du die folgenden zeilen tauschen
char Name[51];
in
char *Name;

strcpy(Name, PtrNamPtrArr[i]);
in
Name = PtrNamPtrArr[i];

und das kommt dann raus
PtrNamPtrArr[i] = Name;

und das am besten unter Name = PtrNamPtrArr[i];
Name[0] = toupper(Name[0]);
lg lolo

void ConvertNamen(char *NamPtrArr[], int NamAnz) {
  char *Name;
  printf("%d ", NamPtrArr[0]);

  for (int i = 0; i < NamAnz; i++) {
    Name = NamPtrArr[i]; 
    Name[0] = toupper(Name[0]);

    for (int j = 0; j < strlen(Name); j++) {
      if(Name[j] == ' ' || Name[j] == '-') {         // Nicht-Buchstaben überspringen
        j++;
        Name[j] = toupper(Name[j]);                  // Erster Buchstabe danach groß
      } else {
        Name[j] = tolower(Name[j]);                  // Alles klein machen
      }
    }
  }
}

Das war jetzt der Versuch. Es kompiliert, bricht dann aber mit folgender Fehlermeldung das Programm ab:

An unhandled exception of type 'System.AccessViolationException' occurred in pa4.exe

Additional information: Attempted to read or write protected memory. This is often an indication that other memory is corrupt.

zeig doch mal den code mit dem du das string array anlegst

btw. du solltest jetzt bei j=1 anfangen da du Name[0] ja schon bearbeitet hast

Was genau meinst du?

char *TstNamPtrArr[10] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };

steht ja schon oben.

Ganz genau:

Main():

int main(void) {
  //char **PtrNamPtrArr = NULL;
  //int NamAnz = 0;
  char *TstNamPtrArr[10] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };
  int NamAnz = 5;

  //GetNamen(PtrNamPtrArr, NamAnz);

  OutNamen("Ausgabe 1: Namen in Original-Reihenfolge und Original-Schreibweise", TstNamPtrArr, NamAnz);  
  ConvertNamen(TstNamPtrArr, NamAnz);
  OutNamen("Ausgabe 2: Namen in Original-Reihenfolge und korrigierter Schreibweise", TstNamPtrArr, NamAnz);
  SortNamen(TstNamPtrArr, NamAnz);
}

ConvertNamen:

void ConvertNamen(char *NamPtrArr[], int NamAnz) {
  char *Name;
  printf("%d ", NamPtrArr[0]);

  for (int i = 0; i < NamAnz; i++) {
    Name = NamPtrArr[i]; 
    Name[0] = toupper(Name[0]);

    for (int j = 0; j < strlen(Name); j++) {
      if(Name[j] == ' ' || Name[j] == '-') {         // Nicht-Buchstaben überspringen
        j++;
        Name[j] = toupper(Name[j]);                  // Erster Buchstabe danach groß
      } else {
        Name[j] = tolower(Name[j]);                  // Alles klein machen
      }
    }
  }
}

nwp2

"lisa-marie" und "LISA MARIE" passen nicht in dein Array.

"lisa-marie" und "LISA MARIE" passen nicht in dein Array.

versuch mal

char TstNamPtrArr[][256] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };
statt
char *TstNamPtrArr[10] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };

so sollts auf jeden fall auf die schnelle klappen, das problem ist das ein string der mit

char *xxx = "asdasdasd";

angelegt wird schreibgeschützt ist
und

char xxx[] = "asdasdasd";

nicht.

Doch das passt rein, das ist erstens in der Aufgabenstellung so vorgegeben und zweitens haben beide Namen 10 "Buchstaben". Da der Array bei 0 beginnt und bis 10 geht passt es rein.

Sobald ich beim aktuellen Code versuche einen einzelnen Char per Name[0] o.Ä. anzusprechen bricht er mit der Access Violation-Message ab.

noobLolo schrieb:

versuch mal
char TstNamPtrArr[][256] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };
statt
char *TstNamPtrArr[10] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };
so sollts auf jeden fall auf die schnelle klappen, das problem ist das ein string der mit
char *xxx = "asdasdasd";
angelegt wird schreibgeschützt ist
und
char xxx[] = "asdasdasd";
nicht.

Dürfen wir leider nicht, die Struktur diesbezüglich ist fest vorgegeben, wir dürfen selbst in den Unterprogrammen nur eingeschränkt arbeiten, wie wir wollen.

Da der Array bei 0 beginnt und bis 10 geht passt es rein.
richtig ist
Da der Array bei 0 beginnt und bis 9 geht passt es rein.
int arr[10];

erstes ist arr[0];
letztes ist arr[9];

Dürfen wir leider nicht, die Struktur diesbezüglich ist fest vorgegeben, wir dürfen selbst in den Unterprogrammen nur eingeschränkt arbeiten, wie wir wollen. :/

also von sowas hab ich ja noch nie gehört... aber man lernt ja nie aus, du kannst auch eine kopie von deinem string array anlegen, alerdings würd ich das nicht in "ConvertNamen" mit ein bauen denn das gehört da nicht rein

zum kopieren

char *from[10] = {"asd","asd","asd"};
char *to[10] = {};
int l = 0;
while( l<10 && from[l] ){
    to = strdup(from[l]);
    l++;
}

hoffe das klappt so...

darüber hinaus ist es ja so das dein programm in zukunft sowieso die namen woanders hernehmen soll also entweder von der tastatur, einer datei, datenbank o.ä. und dann sind die strings idr. nicht schreibgeschützt.

nwp2

"LISA MARIE" ist ein '\0'-terminierter String, deshalb brauchst du 11 chars, 10 für LISA MARIE und 1 fürs '\0'-Zeichen. Wahrscheinlich wird deshalb das '\0'-Zeichen geschluckt und dein Array sieht tatsächlich so aus:

hans\0     FRANZ\0     oTTo\0     lisa-marieLISA MARIE

Das würde auch erklären warum das nicht richtig läuft.
strlen versagt hier weil es bei lisa-marie bis zum '\0'-Zeichen zählt, was aber garnicht da ist.
Du kannst aber statt strlen(Name) einfach 10 hinschreiben, dann sollte es funktionieren.

@nwp2
char *from[10] erstellt bei mir ein array für 10 char* zeiger ist doch bei dir auch so oder?

nwp2

Ups, ich nehme alles zurück und behaupte das Gegenteil.

Ja, später sollen bis zu 10 Namen per fgets eingelesen werden. Vorerst sollen wir aber mit dem Test-Array die Unterprogramme zum Laufen kriegen. Hab jetzt zwar noch etwas weiter rumprobiert, aber finde einfach keine Lösung, wie das Programm nicht abstürzt, wenn ich versuche einzelne Buchstaben von "Name" anzusprechen (Name[0] z.B. was in dem Fall "h" von "hans" sein sollte).

Sobald die Unterprogramme funktionieren, soll ein Feld mit char-Zeigern dynamisch per calloc erzeugt werden und ein Zeiger PtrNamPtrArr auf die Startaddresse zeigen. Das Feld soll dann genau NamAnz (1 bis max 10) Elemente vom Datentyp char* enthalten. Und in die Feldelemente kommen die Addressen der dynamischen Strings, die die Namen enthalten. Bin mir da zwar auch noch nicht 100% sicher wie das implementiert werden soll, aber erstmal will ich einfach nur dass es mit dem Test-Ding klappt.

Also das hauptsächliche Problem gerade ist, dass ich aus der Unterfunktion nicht wirklich in den Ram speichern kann, weil er mir sofort 'ne AccessViolation andreht. Muss ich vielleicht doch vorher schon etwas mit calloc oder so machen?

Und nochmal zum Verständnis: Wenn ich einen Zeiger (char *Name) erstelle und den mit NamPtrArr[0] gleichsetze (Name = NamPtrArr[0]), habe ich in "Name" den ersten Eintrag von NamPtrArr, also "hans" (gerade per fprint getestet). Wieso kann ich dann diesen String charakterweise nicht per Name[0] Schritt für Schritt durcharbeiten? Sobald ich einen Index anhänge kriege ich eine Access-Violation. Gibt es da noch eine andre Methode?

Kuh-Couch schrieb:

Wieso kann ich dann diesen String charakterweise nicht per Name[0] Schritt für Schritt durcharbeiten?

also lesen kannst du schon nur nicht schreiben weil das in einem schreibgeschützten bereich drin steht...

noobLolo schrieb:

versuch mal

char TstNamPtrArr[][256] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };
statt
char *TstNamPtrArr[10] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };

sich selbst zitieren ist normal nicht so mein fall...

nwp2

Yoa, deswegen meine Verwirrung mit den Arrays.

Wenn du

char TstNamPtrArr[][256] = { "hans", "FRANZ", "oTTo", "lisa-marie", "LISA MARIE" };

nimmst wie noobLolo schon meinte dann darfst du dort auch schreiben. Du kannst auch eine 10 statt 256 nehmen wenn es sein muss, allerdings funktioniert dann strlen nicht mehr wie oben beschrieben.

Ach verdammt lolo war wieder schneller.

Ja, aber an dem Array darf ich nichts ändern, das ist fest so vorgegeben. Glaubt mir, ich mach's mit Absicht ungern schwerer als nötig.

ja dann mach ne kopie davon du kannst machen was du willst und wirst nicht in das array schreiben dürfen...

char *from[10] = {"asd","asd","asd"};
char *to[10] = {};
int l = 0;
while( l<10 && from[l] != NULL ){
    to[l] = strdup(from[l]);
    l++;
}

oder so, kpl. ob das geht

char *from[10] = {"asd","asd","asd"};
int l = 0;
while( l<10 && from[l] ){
    from[l] = strdup(from[l]);
    l++;
}

hatte ich schon wieder "einen" drin, das gibts doch nicht, hoffentlich sind da nicht noch mehr