Gdzie jest błąd w tym kodzie ? (C++ Builder)

pytanie zadane 8 sierpnia 2015 w C i C++ przez Rappa2 Obywatel (1,540 p.)
edycja 18 czerwca 2018 przez Arkadiusz Waluk

Gdzie jest błąd w tym kodzie?

//---------------------------------------------------------------------------

#include <vcl.h>
#pragma hdrstop

#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;

AnsiString nazwapliku="";

//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------

 

void __fastcall TForm1::Otwrz1Click(TObject *Sender)
{
   if(OpenDialog1->Execute())
   try
   {
   Tekst->Lines->LoadFromFile(OpenDialog1->FileName);
   nazwapliku=OpenDialog1->FileName;
   }
   catch(...)
   {
   ShowMessage("Błąd otwarcia pliku. Upewnij się że plik jest na dysku");
   }

}
//---------------------------------------------------------------------------

 

void __fastcall TForm1::Zapiszjako1Click(TObject *Sender)
{
   if(SaveDialog1->Execute())
   try
   {
   Tekst->Lines->SaveToFile(SaveDialog1->FileName);
   nazwapliku=SaveDialog1->FileName;
   }
   catch(...)
   {
   ShowMessage("Zapis zakończył się niepowodzeniem");
   }

}
//---------------------------------------------------------------------------

void __fastcall TForm1::Zapisz1Click(TObject *Sender)
   {
   if (nazwapliku!="")
   {
   Tekst->Lines->SaveToFile(nazwapliku));
   else
   Form1->Zapiszjakoclick(MainMenu1);
}
//---------------------------------------------------------------------------

3 odpowiedzi

odpowiedź 8 sierpnia 2015 przez Dorion300 Szeryf (90,250 p.)

odpowiedź 8 sierpnia 2015 przez figorin Bywalec (2,040 p.)

Może tutaj:

Tekst->Lines->SaveToFile(nazwapliku));

po co ten trzeci nawias?

komentarz 8 sierpnia 2015 przez Rappa2 Obywatel (1,540 p.)

Ok usunołęm nawias, teraz się podświetla else

komentarz 8 sierpnia 2015 przez figorin Bywalec (2,040 p.)

void __fastcall TForm1::Zapisz1Click(TObject *Sender)
   {
   if (nazwapliku!="")
   {
   Tekst->Lines->SaveToFile(nazwapliku));
   else
   Form1->Zapiszjakoclick(MainMenu1);
}

usuń nawias klamrowy po ifie.

odpowiedź 18 czerwca 2018 przez MsMaciek123 Pasjonat (24,760 p.)

//Na przyszłość, dawaj kod w znaczniku
//Lepiej się czyta

void __fastcall TForm1::Zapisz1Click(TObject *Sender)
{
   if (nazwapliku!=""){
   Tekst->Lines->SaveToFile(nazwapliku);
    }
   else
   {
   Form1->Zapiszjakoclick(MainMenu1);
   }
}