Bestimmte Textstellen

Jansen

Christian211 schrieb:

Erlaube mir dazu eine andere Meinung zu haben.

Dito.

Hier trifft im kleinen genau dasselbe zu wie für globale Variablen:
Wenn eine Variable nur in einer Funktion (oder Schleife) benötigt wird, dann sollte sie auch nur in der Funktion (oder Schleife) gültig sein, d.h. dort deklariert werden.

Ich bezweifle ausserdem, dass sowas zu Beginn einer Funktion irgendwie der Les- und Wartbarkeit hilft:

int i, j, k, x, y, z; // Zähler für for-Schleifen
// oder
int i; // Zähler erste for-Schleife 
int j; // Zähler zweite for-Schleife
int k; // Zähler dritte for-Schleife
int x; // Zähler Spalten-Schleife
int y; // Zähler Zeilen-Schleife
int z; // Zähler Items-Schleife

Man könnte eine Variable wiederverwenden... Du übertreibst etwas

Christian211 schrieb:

bool found=false;
for(int Zahl=0; Zahl<=Liste->Count; Zahl++)
 {
  if(Liste->Strings[Zahl]==Edit1->Text)
   {
    Label3->Caption=Edit1->Text;
    found=true;
//ggf. könnte hier ein break rein, wenn die anderen nach einem Fund nicht durchsucht werden sollen
   }
 }
if(!found)
  {
   Application->MessageBox("Your name is wrong", "error", 0);
   }

das hat mich schon mal weiter gebracht danke
leider werden immernoch alle zeilen BIS ZUR richtigen Zeile mit einer MsgBox als "Falsch" benannt. Er soll aber nur sagen das nichts gefunden wurde (Your name is wrong) wenn es in der ganzen datei nichts gibt
das ist nervig vor allem wenn man mehrere hundert zeilen hat und das gefundene sagen wir in 246 ist
Ich glaube das wäre dann das letzte problem :p

Das kann nicht sein. In der Schleife wir keine Messagebox mehr ausgegeben. Die wird DANACH ausgegeben. Schau nochmal hin!

Christian211 schrieb:

Das kann nicht sein. In der Schleife wir keine Messagebox mehr ausgegeben. Die wird DANACH ausgegeben. Schau nochmal hin!

Yo hatte es falsch geändert es funzt alles sehr gut danke an alle anmtworten besonders an Christian211

Jansen

Christian211 schrieb:

Du übertreibst etwas

Die C++-Entwickler haben sich sicher etwas dabei gedacht, als sie die Deklaration im Schleifenkopf erlaubt haben.
Wiederverwendung ist natürlich möglich, birgt aber Fehlerpotential (fehlende Neuinitialisierung).

Ohhhhh

for(int Zahl=0; Zahl<=Liste->Count; Zahl++)
  {
  if((Liste->Strings[Zahl]==Edit1->Text)&&(Liste->Strings[Zahl+1]==Edit2->Text));
   {
   Form1->Button1->Enabled=false;
   Form1->Button2->Enabled=true;
   Form1->Button4->Enabled=true;
   gefunden=true;
   Form2->Close();
   break;
   }
  }
 if(!gefunden)
  {
  Application->MessageBox("Your name or/and password is wrong", "error", 0);
  }

Was is jetzt daran falsch? Das problem ist: EGAL ob was gefundne wird oder nicht, er führt die erste if methode aus und lässt die 2. unbeachtet. Also auch wenn nichts gefunden wird wird die erste if methode benutzt

thx

Braunstein

Hast du gefunden vorher mit false initialisiert?
Bei deiner Abfrage gehst du zu Schluß über die Arraygrenzen hinaus.

for(int Zahl=0; Zahl<=Liste->Count; Zahl++) // geht von 0 bis Count, Liste geht aber nur von 0 bis Count-1
  {
// hier testest du bei Zahl=Count sogar mit Count+1
  if((Liste->Strings[Zahl]==Edit1->Text)&&(Liste->Strings[Zahl+1]==Edit2->Text));

richtiger wäre

for(int Zahl=0; Zahl<Liste->Count-1; Zahl++)
  {
  if((Liste->Strings[Zahl]==Edit1->Text)&&(Liste->Strings[Zahl+1]==Edit2->Text));

Braunstein schrieb:

Hast du gefunden vorher mit false initialisiert?
Bei deiner Abfrage gehst du zu Schluß über die Arraygrenzen hinaus.

for(int Zahl=0; Zahl<=Liste->Count; Zahl++) // geht von 0 bis Count, Liste geht aber nur von 0 bis Count-1
  {
// hier testest du bei Zahl=Count sogar mit Count+1
  if((Liste->Strings[Zahl]==Edit1->Text)&&(Liste->Strings[Zahl+1]==Edit2->Text));

richtiger wäre

for(int Zahl=0; Zahl<Liste->Count-1; Zahl++)
  {
  if((Liste->Strings[Zahl]==Edit1->Text)&&(Liste->Strings[Zahl+1]==Edit2->Text));

Problem besteht weiterhin! Hat nicht geholfen trotz beider verbesserungen

caspar_louis

Steht irgendwo vorher geschrieben ?

gefunden=false;

Falls nein, dann ist das ein Fehler (siehe sundays Post).

Da war ein "=" zuviel, jetzt nicht mehr.

Braunstein

@CounterstrikePlayer
Geh doch bitte mal mit dem Debugger durch und schau was dein Code macht.

caspar_louis schrieb:

Steht irgendwo vorher geschrieben ?
gefunden=false;
Falls nein, dann ist das ein Fehler (siehe sundays Post).

Da war ein "=" zuviel, jetzt nicht mehr.

ja hab ich bei mir stehen

Braunstein schrieb:

@CounterstrikePlayer
Geh doch bitte mal mit dem Debugger durch und schau was dein Code macht.

wie ist das gemeint? Sorry Größter Noob aller Zeiten am werke

akari

Hallo

Braunstein meint, du sollst den Debugger benutzen.

bis bald
akari

akari schrieb:

Hallo

Braunstein meint, du sollst den Debugger benutzen.

bis bald
akari

hab ich aber das hilft mir auch nicht
sobald der bei der ersten if-methode ankommt, setzt er gefunden auf true egal ob es wirklich so sein soll oda nich

Dasd

Och nööö. Und ich dachte, du hättest inzwischen dazugelernt. Auch nach If-Abfragen gehört kein Semikolon... Sonst kommt dasselbe raus wie bei deiner Schleife am Anfang...

Braunstein

Mach mal das Semikolon hinter de if-Abfrage weg.

:p KACK DRECK XD
ich hasse dieses semikolon

Ich hoffe es is euch r3cht wenn ich mich mal eben bedanke und ruhig bin ^^