Ich will einen mehrzeiligen Text eingeben.
Das Abbruchktriterium soll eine leere Zeile sein.
Beim kompilieren kommt keine Fehlermeldung.
Beim Ausführen jedoch kommt nach dem ersten enter eine Windows Fehlermeldung!!

Kann mir jemand weiterhelfen ? Wäre sehr nett...

#include <stdio.h>
#include <stdlib.h>
#include <iostream>

using namespace std;

void EINGABE (char[]);

int main ()
{
char *INPUT ;
cout <<endl<<"Gib mehrere Zeilen ein!"<<endl<<endl;

EINGABE(INPUT);

cout <<endl<<endl;
system ("pause");
return 0;
}

void EINGABE (char INPUT[])
{
do
{
cin.getline (INPUT,1000);
char* p = new char[ strlen(INPUT) +1 ];

if (p==NULL)
{cerr <<endl<<endl<<"------Speicherreservierung M I S S L U N G E N------";}

strcpy (p,INPUT);
p++;
} while (strlen(INPUT)!= 0);
}

Naja so direkt sehe ich jetzt auch nicht was du mit dem Code bezwecken willst, meiner meinung nach ist das was jetzt da steht eh nicht lauffähig..

Zweitens: "new" gibt nie NULL zurück, sondern wirft eine Ausnahme, wenn kein Speicherplatz mehr da ist

vor deinem´:
cin.getline (INPUT,1000);

löst das Fehler, dein Input war nich Dimensioniert, kann also keine Daten zum kopieren aufnehmen..

INPUT = new char[1000];

vor deinem´:
cin.getline (INPUT,1000);

löst das Fehler, dein Input war nich Dimensioniert, kann also keine Daten zum kopieren aufnehmen..

Ist aber auch nur eine halbe Lösung, der Zeiger in main kriegt von der Zuweisung nämlich nix mit; wenn dann muss das in der main passieren.

Nur als Aufklärung was ich mit meinem Programm will!!
________________________________________

1. cin.getline (INPUT,1000);

Also als erstes will ich eine Zeile (vom Typ char)eingeben können die max. 1000 Zeichen besitzt.
______________________________________________
2) char* p = new char[ strlen(INPUT) +1 ];

Danach soll diese Zeile soviel Speicherplatz bekommen wie sie auch nur wirklich benötigt. Des +1 ist für das Zeichen \0.
Jedoch soll auf diese Zeile jetzt nicht mehr der INPUT zeigen, sondern der neue Zeiger p.

Danke Jungs jetzt funktioniert es ohne Fehler!!!!

int main ()
{
char INPUT[1000] ; <<<<<<<Hier war der Fehler, mit Speicherplatzreservierung funktionierts

cout <<endl<<"Gib mehrere Zeilen ein!"<<endl<<endl;

EINGABE(INPUT);

Danke für die schnelle und kompetente Hilfe !!!

...
char INPUT[1000] ; <<<<<<<Hier war der Fehler, mit Speicherplatzreservierung funktionierts
...

... aber auch nur solange nicht mehr als 1000 Zeichen eingegeben werden (spätestens, wenn Du Dateien einlesen lässt, ist das gar nicht mehr so viel, wie es jetzt scheint).

Warum machst Du es nicht einfach mit C++ ? ... und C++ hat ja schließlich string !

#include <string>
using std::string;

void EINGABE (string&);

int main () {
   string INPUT ;
   cout <<endl<<"Gib mehrere Zeilen ein!"<<endl<<endl;

   EINGABE(INPUT);

   cout <<endl<<endl;
   system ("pause");
   return 0;
}

void EINGABE (string& INPUT) {
   string einzelInput;
   do  {
      getline(cin, einzelInput); // OHNE "cin." davor !!
      INPUT += einzelInput; 
      // oder falls Du das abschließende \n nicht dabei haben willst:
      // INPUT += einzelInput.substr(0, einzelInput.length()-1);
   } while (einzelInput.length());
}

Dieses ganze char-Geraffel macht doch keinen glücklich und sieht auch noch blöd aus....
So hast Du noch Alternativen wie z.B.:

void EINGABE(string& INPUT) {
   string einzelInput;
   do  {
      cin >> einzelInput;
      if(einzelInput != "-") INPUT += einzelInput;
   } while (einzelInput != "-");
}

oder (mein persönlicher Favorit):

#include <algorithm>
#include <iterator>
#include <vector>

void EINGABE(vector<string>& INPUT) {
 std::copy(
         std::istream_iterator<string>(cin),
         std::istream_iterator<string>(),
         std::back_inserter(INPUT));
}

(hier wird auch klar, warum eine separate Eingabe-Funktion überflüssig sein kann)

Letzteres weist Dich auf die logisch folgende Frage hin: Sollen wirklich alle Eingaben in demselben string bearbeitet werden ... oder ist es bei Dir nicht auch so wie bei 99,9% aller Anderen:
Du möchtest schon die einzelnen Eingaben getrennt arbeiten, hältst es aber erstmal für einfacher, alles erst in einen String zu packen und hinterher wieder auseinander zu puhlen.
Dann nimm doch lieber gleich wieder den vector<string> - da hast Du alles, was Du brauchst...

Gruß,

Simon2.

P.S.: Deine Bezeichner sind ein wenig ungewöhnlich... so komplett Großgeschriebenes wird üblicherweise höchstens für Konstanten verwendet (und selbst da immer weniger) - macht den Code natürlich nicht falsch, aber hakt eben ein wenig beim Lesen.

Warum machst Du es nicht einfach mit C++ ? ... und C++ hat ja schließlich string !

#include <string>
using std::string;

void EINGABE (string&);

int main () {
   string INPUT ;
   cout <<endl<<"Gib mehrere Zeilen ein!"<<endl<<endl;

   EINGABE(INPUT);

   cout <<endl<<endl;
   system ("pause");
   return 0;
}

void EINGABE (string& INPUT) {
   string einzelInput;
   do  {
      getline(cin, einzelInput); // OHNE "cin." davor !!
      INPUT += einzelInput; 
      // oder falls Du das abschließende \n nicht dabei haben willst:
      // INPUT += einzelInput.substr(0, einzelInput.length()-1);
   } while (einzelInput.length());
}

Hallo Simon2,
getline liest bis inklusive newline, aber hängt das newline nicht an den Eingabestring an. D.h. man braucht nie ein '\n' entfernen, sondern muss höchstens eines hinzufügen, wenn man die Struktur der Zeilen erhalten möchte.

Simon2 schrieb:

Dieses ganze char-Geraffel macht doch keinen glücklich und sieht auch noch blöd aus....
So hast Du noch Alternativen wie z.B.:

void EINGABE(string& INPUT) {
   string einzelInput;
   do  {
      cin >> einzelInput;
      if(einzelInput != "-") INPUT += einzelInput;
   } while (einzelInput != "-");
}

Die Einlesen eines Strings liest i.A. ein Wort. D.h. mit diesem Code liest Du alle Worte ohne Zwischenräume (White Space) und hängst sie aneinander. Eine Eingabe von

das ist die erste Zeile
und die zweite

hätte ein

INPUT = "dasistdieersteZeileunddiezweite"

zur Folge.

Simon2 schrieb:

oder (mein persönlicher Favorit):

#include <algorithm>
#include <iterator>
#include <vector>

void EINGABE(vector<string>& INPUT) {
 std::copy(
         std::istream_iterator<string>(cin),
         std::istream_iterator<string>(),
         std::back_inserter(INPUT));
}

dummerweise funktioniert das nicht, da der istream_iterator() so immer bis Fehler oder End-Of-File liest. Nur EOF gibt es bei cin nicht und Fehler passiert auch nicht, da jede Eingabe als string gelesen werden kann.

Hier mein Vorschlag:

#include <iostream>
#include <string>

std::string EINGABE()
{
    using namespace std;
    string in;
    for( string line; getline( cin, line ) && !line.empty(); )
        in += line + '\n';
    return in;
}

int main()
{
    using namespace std;
    cout <<endl<<"Gib mehrere Zeilen ein!"<<endl<<endl;
    string INPUT = EINGABE();

    cout << "-- Ihre Eingabe:\n" << INPUT << endl;
    return 0;
}

Simon2 schrieb:

(hier wird auch klar, warum eine separate Eingabe-Funktion überflüssig sein kann)

Letzteres weist Dich auf die logisch folgende Frage hin: Sollen wirklich alle Eingaben in demselben string bearbeitet werden ... oder ist es bei Dir nicht auch so wie bei 99,9% aller Anderen:
Du möchtest schon die einzelnen Eingaben getrennt arbeiten, hältst es aber erstmal für einfacher, alles erst in einen String zu packen und hinterher wieder auseinander zu puhlen.

Hier gebe ich Dir Recht. Man sollte das lesen, was man auch nachher im Programm haben will.

Gruß
Werner

dummerweise funktioniert das nicht, da der istream_iterator() so immer bis Fehler oder End-Of-File liest. Nur EOF gibt es bei cin nicht und Fehler passiert auch nicht, da jede Eingabe als string gelesen werden kann.

Hier muss ich widersprechen. Natürlich gibt es EOF auch für cin, sonst würde kein UNIX-Konsolenprogramm funktionieren welches sowohl auf der Standardeingabe als auch auf Dateien arbeiten kann, und das sind so ziemlich alle Standard-Tools. Der Standardeingabestrom ist dann EOF wenn der Schreibende (Terminal oder Programm in einer Pipe) die Standardausgabe schließt.

danke für Deine Hinweise.
Das Problem mit dem vector<string> bei cin ist mir das noch nicht untergekommen , da ich eigentlich immer Files und/oder Non-Strings einlese - aber Du hast natürlich Recht.

Ganz kurz zu den einzelnen Punkten:

Werner Salomon schrieb:

...
getline liest bis inklusive newline, aber hängt das newline nicht an den Eingabestring an. ...

öh ... das habe ich anders erfahren. Bei meinem Compiler (gcc 3.4.4) wird auch das newline mit aufgenommen.
Im Standard steht das aber tatsächlich anders (habe jetzt mal nachgeschlagen, Kap 21.3.7.9.6) ...

Simon2 schrieb:

...Die Einlesen eines Strings liest i.A. ein Wort....

Weiß ich - ich wollte auch nicht 1:1 die Anforderung umsetzen, sondern mehr zeigen, "wie man in C++ aus streams einliest"...

Werner Salomon schrieb:

...dummerweise funktioniert das nicht, ...

Stimmt auch - wie gesagt: Es ging mir mehr um "den Umgang mit streams" und "Inputmengen".

Gruß,

Simon2.