#include <iostream>
#include <windows.h>
#include <conio.h>
#include <fstream>
#include <string>

using namespace std; 

int personalNr,zugangscode; 
int pnr =1234; 
int zc  =5678; 

int main(void)
  { 

    cout << "===============================================================\n";
    cout << "----------Willkommen zur Datenbank der Firma Müller----------\n";
    cout << "===============================================================\n";
    cout << "\n\n\n";
    cout << "Geben Sie bitte Ihre personalNr ein.\n";

  cin >> personalNr;

  if (personalNr.length() > 4) // <----- Da sitzt der WURM
  {
  cerr << "Die PersonalNR ist zu Lang bitte versuchen sie es erneut\n";

  getch();
  return EXIT_SUCCESS;
}

    cout << "\n\n\n"; 
    cout << "Bitte geben Sie Ihren Zugangscode ein.......:\n";

  cin >>zugangscode; 

    cout << "\n\n\n"; 

  if ((personalNr == pnr) && (zugangscode == zc)) 
  { 
    cout << "Zugang gewaehrt - Guten Tage Herr Müller\n\n\n"; 
  } 
  else 
  { 
    cout << "Personalnummer oder Zuganscode falsch\n\n\n"; 
  } 

ifstream dat_ein;
string dateiname;

cout << "Wählen sie ein Menüpunkt aus und drücken sie die EENTER-Taste\n"; 
cout << "=============================================================\n";
cout << "= Nr. == 1 ==== Mitarbeiter-Daten     =======================\n";
cout << "= Nr. == 2 ==== Finanz-Daten          =======================\n";
cout << "= Nr. == 3 ==== Aussattungsdaten-Daten=======================\n";
cout << "= Nr. == 4 ==== Ausliefer-Daten       =======================\n";
cout << "= Nr. == 5 ==== Anzahl verkaufter Systeme-Daten==============\n";
cout << "=============================================================\n";
cin >> dateiname;
cout << endl;

         string dat_end(".txt");
         dateiname.append(dat_end);

dat_ein.open (dateiname.c_str(), ios_base::in);

if(!dat_ein)
{

cout << "Falsche Eingabe oder Datenbank beschädigt bitte versuchen sie es erneut!";
cout << endl;

system ("PAUSE");
return EXIT_SUCCESS;
}

char zeichen;
while (!dat_ein.eof())
{
dat_ein.get(zeichen);
cout << zeichen;
}
 getch();
   return 0;

}

in der 28. Zeile kommt folgender Compiler-Fehler :

In function `int main()': 
`length' has not been declared 
request for member of non-aggregate type before '(' token

Haber aber gesehen das es bei Anderen problemlos klappte bzw. deren Problem gelöst wird.

oder wird die Funktion gar nicht mehr benutzt (veraltet )????

Danke im Voraus

jetzt überleg dir nochmal was integer datentypen sind, und was klasseninstanzen sind

lordcesarius schrieb:

Haber aber gesehen das es bei Anderen problemlos klappte

das halte ich aber mal sowas von für ein gerücht. nicht mit diesem code

edit: !!111!!11!!1onehundredelevenandone1!!11!

Gruß Tobi.

#include <cmath>

if(log10(static_cast<double>(personalNr))+1>4)

Ja, eben. Wieso einfach, wenn's auch kompliziert geht?

Also wirklich …

if (personalNr > 9999)

@Tobi
Jemand die show stehlen ist nicht nett .
Der hat Recht denn nur durch nachenken kommt man später fast immer alleine auf die Lösung

PS: Dein Tipp funzt nicht einmal :p

@cmath.pro
Tut mir leid deine Lösung habe ich gegen

int (personalNr.length() > 4)

und darunter noch einen

else {}

ausgetauscht und funzt trotzdem nicht!

kommt folgendes bei raus:

In function `int main()': 
37  expected primary-expression before "else" 
37  expected `;' before "else"

wenn ich

else {}

Weglasse dann passiert das jede Zahl die ich eingebe zu lang ist (auch die zahl 1 oder 1234)

verstehe jetzt gar nichts mehr !!

Danke im Vorraus

Was willst du genau? Erkennen, ob eine Zahl mehr als 4 Stellen hat? Wenn ja, nimm doch Folgendes:

if (personalNr > 9999)
{
   // Zahl ist mindestens 5-stellig
}
else
{
   // Zahl ist höchstens 4-stellig
}

Du kannst else nur nach einem if benützen, deshalb deine Fehlermeldung.
Du solltest erst einmal richtig C++ lernen. Wenn dein Kopf dafür zu klein ist, können wir dir leider auch nicht helfen

cerr << "Die PersonalNR ist zu Lang bitte versuchen sie es erneut\n";

lies DU doch bitte vorher den Beitrag.

dann habe ich doch einen IF vorher gepackt, habe mich nur verschrieben.
und Konrad Rudolph habe ich übersehen.

nicht alles bei c++ ist logisch und nachvollziehbar.Viele Bücher, Interntetforen und PDF's sind entweder veraltet, schlecht erklärt, gar nicht erklärt, auf fachenglisch oder gar falsch.

Daher gibt es auch www.c-plusplus.net oder stellst die ganze Internetpräsenz in frage?Wenn man c++ doch einfach nur Lernen muss,dann gibt es ja deiner Meinung nach ja gar kein Problem und man könnte die Seite ja ganz schließen, oder?

Das dazu

so trotzdem Danke es funzt jetzt.
Wie kann ich aber bei einer falsch eingabe zurück zu

cin >> personalNr

kommen?

ohne dabei eine Switchverzweigung einzuleiten.

Zu deiner Frage: Du kannst eine Schleife erstellen, die bei einer Falscheingabe noch einmal von vorne gestartet wird:

int personalNr;
while (true)
{
   cin >> personalNr;
   if (/* Eingabe nicht gut */)
   {
      cout << "Falsche Eingabe. Bitte nochmals eingeben:" << endl;
      continue; // beginnt die Schleife von vorne
   }
   else
   {
      // hier machst du etwas mit der Eingabe
      break; // verlässt die Schleife
   }
}

Eigentlich könntest du es auch mit goto machen, würde ich dir aber nicht raten.

Danke für den Tipp werde es sofort versuchen.