Hier das Problem:

Wir sollen ein Programm schreiben, bei dem ein Nachname und ein Vorname in jeweils einer char Variable gespeichert werden. <= Das war noch der leichte Teil.

Für die spätere Ausgabe jedoch sollen beide char-Variablen in EINE char kopiert werden. Falls das nicht klar genug ist "Name = Vorname + Nachname".

Ich hab die ganze Sache jetzt auf verschiedene Arten probiert, habe aber lediglich hinbekommen das beim Kompilieren kein Fehler mehr angezeigt wird.
Bei der Ausgabe fehlt dann der vollständige Name, die dritte char-Variable ist leer.

Kann mir irgendjemand mit dem Problem helfen. Wenn nötig kann ich auch meinen bisherigen Code posten. Sollten ansonsten noch Infos nötig sein reiche ich die auf Wunsch gerne nach.
Danke im Voraus,
lg
Shruikan

Ist der Typ char bzw. char* Pflicht? Falls nicht, kannst du es ganz einfach mit der Klasse std::string machen (da funktioniert dann auch Vorname+Nachname). Falls du mit char-Arrays arbeiten musst, kannst du z.B. sprintf oder strcat benutzen, um deine beiden Strings zusammenzufügen.

Aber hier erstmal meinen Code:

#include <cstdlib>
#include <iostream>

using namespace std;

int main(int argc, char *argv[])
{
system("COLOR F0"); //Definition der Farbe

char name[20];
char vorname[20]; //Deklaration der Variablen
char gesamt[40]; //Gesamtvariable
char abfrage; //Variable zur Abfrage der Gültigkeit der Eingabe

while(abfrage !='j')
{
cout <<"Geben Sie einen Nachnamen ein: ";
cin >> name;
cout <<"Geben Sie einen Vornamen ein: ";
cin >> vorname;
cout << endl << endl;
cout <<"Ihr eingegebener Name lautet "<< name << endl;
cout <<"Ihr eingegebener Vorname lautet "<<vorname << endl << endl;
cout <<"Sind diese Eingaben korrekt?" <<endl;
cout <<"W„hlen Sie j wenn die Eingaben korrekt sind"<<endl;
cout <<"W„hlen Sie n wenn die Eingaben falsch sind"<<endl << endl;
cin >> abfrage;
cout << endl;
}
gesamt == vorname && name;
cout <<"Ihr vollst„ndiger Name lautet" << gesamt << endl << endl;
system("PAUSE");
return EXIT_SUCCESS;
}

Hoffe das hilft weiter. Falls ich nicht direkt antworte tuts mir leid, aber ich kann hier nur sporadisch reingucken.

1. std::strncpy ist dein Freund

Ich bin kein C-Progger aber generell glaub ich zusätzlich auch: (???)
http://www.cplusplus.com/reference/clibrary/cstring/strncat/
und natürlich
http://www.cplusplus.com/reference/clibrary/cstring/strlen/
wobei man letzteres eigentlich als Kenntnis voraussetzen kann (wie Fellhuhn es getan hat)

Fellhuhn schrieb:

2. Sag deinem Lehrer das das Bullshit ist und man seit langer Zeit std::string statt char-Arrays verwendet.

char a[20];  //das sind Arrays vom Typ char
char b[20];  //
char c[50];  //

a="Hallo "; //geht nicht!

strncpy(a,"Hallo ",20);  //klappt schon besser!
strncpy(b,"Welt!",20);

c=a+b;  //Nö... hier würdest du nicht die Inhalte/Strings von a+b addieren, 
        //sondern deren Adressen (schlag vielleicht mal array-to-pointer-decay nach)
        //Daher gibt's hier auch einen Fehler (du kannst 2 Zeiger nicht addieren)

_snprintf(c,50,"%s%s",a,b);  //schon besser. Hier werden die Strings, auf die
                             //a und b zeigen, in c zusammengeführt.

gesamt == vorname && name;

Sorry, aber das ist so falsch, wie es nur sein kann!

Du hast wohl krampfhaft versucht, die Compiler-Fehler wegzubekommen. Aber "==" ist der Vergleichsoperator, nicht der Zuweisungsoperator. Und "&&" ist ein logisches Und, kein Addieren oder Zusammenführen, wie du es vorhattest.

Du bildest hier einfach ein boolsches Ergebnis, dass nicht weiter ausgewertet wird. Genauso hättest du auch eine solche Zeile notieren können:

1;

Um jetzt zu den string-lösungen zu kommen die erwähnt wurden.
Ich sagte meinem Lehrer schon das es so meiner Meinung nach schrott ist.
(Aber der schreibt auch Widerstand mit "ie")

Frage ist jetzt wie ich das Programm funktionsfähig UND richtig codiere.
Bin für den restlichen Tag jetzt unterwegs, lasst euch also ruhig Zeit mit den Antworten.

Lg und danke für die bisheringen Posts

Frage ist jetzt wie ich das Programm funktionsfähig UND richtig codiere.
Bin für den restlichen Tag jetzt unterwegs, lasst euch also ruhig Zeit mit den Antworten.

Eine Möglichkeit habe ich dir schon gezeigt. Die Frage ist eher, was genau du machen sollst. Wenn lediglich die Datentypen (char-Array) und das Ziel definiert sind, kannst du mit meiner Methode arbeiten (eine Alternative wäre noch strncat statt _snprintf). Wenn - wie schon vermutet wurde - ein strcat/strncat nachprogrammiert werden soll, musst du das bitte sagen.

Aufgabenstellung unseres Lehrers lautet wortgemäß
"Codiere ein Programm bei dem Vorname und Nachname in jeweils einer eigenen char Variable abgespeichert werden. Für die Ausgabe sollen die Werte aus den beiden chars in eine dritte char kopiert werden"

Wir haben daraufhin in der Schule schonmal nach einem befehl wie "copy" gesucht, aber nichts gefunden. Also muss die Lösung anders aussehen.

Hoffe das gibt jetzt aufschluss.
LG

Aufgabenstellung unseres Lehrers lautet wortgemäß
"Codiere ein Programm bei dem Vorname und Nachname in jeweils einer eigenen char Variable abgespeichert werden. Für die Ausgabe sollen die Werte aus den beiden chars in eine dritte char kopiert werden"

Ich hoffe, dass das nicht wirklich "wortgemäß" widergegeben ist. Bringt euch der Hausmeister programmieren bei?

strncat und snprintf sagt mir beides NICHTS, sorry.

CSpilles Links gesehen?

Shruikan schrieb:

Aufgabenstellung unseres Lehrers lautet wortgemäß
"Codiere ein Programm bei dem Vorname und Nachname in jeweils einer eigenen char Variable abgespeichert werden. Für die Ausgabe sollen die Werte aus den beiden chars in eine dritte char kopiert werden"

Mein Beileid. Wenn jemand einen String als "char-Variable" bezeichnet, lässt das schon mal stark an seiner Kompetenz zweifeln.

habe jetzt mal zwei beispiele aus besagten Links ausprobiert, aber die sind beide auch mit strings.

ich denke ich werd das erstmal so an den guten mann weitergeben.
Lösung wie er sie sich vorstellt werd ich danngerne hier posten.

(vielleicht meinte er ja strings und sagte char)

wird jedenfalls nicht mein letzter Beitrag hier sein.
Bis zum nächsten Problem dann.

Spatz beiseite: mach Dir Dein char-Arry-Copy doch selber:

char vorname[20];
char nachname[20];
char name[42];

// Einlesen ...
cin >>...

// Konkatenieren

int pos_name=0;

for(int pos_vn=0; vorname[pos_vn]!=0 && pos_vn<20; ++pos_vn)
  name[pos_name++]=vorname[pos_vn];

name[pos_name++]=' '; //Leerzeichen dazwischen

for(int pos_nn=0; nachname[pos_nn]!=0 && pos_nn<20; ++pos_nn)
  name[pos_name++]=nachname[pos_nn];

name[pos_name]='0'; // Vorsicht in der Porzellankiste

Sicher nicht die schönste Lösung, aber sollte grob tun
Angestrebter Lerneffekt: Strings als Null-terminierte Char-Arrays (klassische C-Strings) selbst verwurschdeln.

Zusatzaufgabe für Übermotivierte: Zeiger-Arithmetik statt Feld-Indizes verwenden!

heißt das (übersetzt für alle wie mich) das ich einen copy befehl selbst erstellen könnte wenn ich den code aus deinem post entnehme?

hm, dafür das der post nicht lang ist hast du es geschafft mich während der geschätzen minute die ich zum lesen brauchte komplett zu verwirren

lol ja bei solchen kommentaren

löl schrieb:

//Konkatenieren
//Vorsicht in der Porzellankiste

kein wunder

char vorname[]="vorname";
char nachname[]="nachname";
char name[42]={0};

strcat(name,vorname);
strcat(name," ");
strcat(name,nachname);

cout << name << endl;

ist zwar viel? langsamer aber dafür kürzer

Zusatzaufgabe für Übermotivierte: Zeiger-Arithmetik statt Feld-Indizes verwenden!

jipee

char vorname[20];
char nachname[20];
char name[42];

char *pvorname = vorname;
char *pnachname = nachname;
char *pname = name;

while((*pname++ = *pvorname++));

*pname++ = " ";

while((*pname++ = *pnachname++));

*pname = "\0";

Paulomat schrieb:

Zusatzaufgabe für Übermotivierte: Zeiger-Arithmetik statt Feld-Indizes verwenden!

jipee

char vorname[]="vorname";
char nachname[]="nachname";
char name[42]={0};

char *pvorname = vorname;
char *pnachname = nachname;
char *pname = name;

//alles inkl. '\0' kopieren
while((*pname++ = *pvorname++));

//'\0' durch ' ' ersetzen
*--pname = ' ';

//den rest inkl. '\0' kopieren
while((*++pname = *pnachname++));

hoffe jetzt stimmts

cppAnfängerÜbermotiviert schrieb:

Paulomat schrieb:

Zusatzaufgabe für Übermotivierte: Zeiger-Arithmetik statt Feld-Indizes verwenden!

jipee

char vorname[]="vorname";
char nachname[]="nachname";
char name[42]={0};

char *pvorname = vorname;
char *pnachname = nachname;
char *pname = name-1;

//alles inkl. '\0' kopieren
while((*++pname = *pvorname++));

//'\0' durch ' ' ersetzen
*pname = ' ';

//den rest inkl. '\0' kopieren
while((*++pname = *pnachname++));

hoffe jetzt stimmts

jetzt noch bischen unrollen und fertig ist der porsche

Undefiniert aber kürzer. :p