Не работает кнопка

@ComputerMouse · Регистрация: 06.11.2012

Author24 — интернет-сервис помощи студентам

Код программы:

C++

//---------------------------------------------------------------------------
#include <stdio.h>
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma link "sBitBtn"
#pragma link "sEdit"
#pragma link "sLabel"
#pragma link "sMemo"
#pragma link "sButton"
#pragma resource "*.dfm"
TForm1 *Form1;
FILE *work;
char *path;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::SaveClick(TObject *Sender)
{
Txt->Text=0;
strcpy(path, Day->Text.c_str());
strcat(path, "-");
strcat(path, Month->Text.c_str());
strcat(path, "-");
strcat(path, Year->Text.c_str());
strcat(path, ".dpf");
work=fopen(path, "w");
fprintf(work, "%s", Txt->Text);
}
//---------------------------------------------------------------------------

На этапе компиляции выдаёт Warning:

Код

[C++ Warning] Unit1.cpp(5): W8058 Cannot create pre-compiled header: write failed

При попытке нажать на кнопку "Save" выдаёт ошибку как на картинке.

@kzru_hunter · 04.03.2013, 18:15

Не выделил память под указатель path

@ComputerMouse · 04.03.2013, 19:02 **[ТС]**

Спасибо!

@Avazart · 04.03.2013, 22:15

Сообщение от ComputerMouse

strcpy(path, Day->Text.c_str());

Опасно ...

Добавлено через 1 минуту
Или у вас Builder6 ?

@ComputerMouse · 04.03.2013, 22:29 **[ТС]**

Да, 6

@Avazart · 04.03.2013, 22:33

А ну тогда нормально...

@ComputerMouse · 04.03.2013, 23:20 **[ТС]**

Хмм... Опять тоже самое, только после "Write on adress" число другое.
Полный код:

C++

//---------------------------------------------------------------------------
#include <stdio.h>
//#include <iostream>
#include <vcl.h>
#pragma hdrstop
 
#include "Unit1.h"
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma link "sBitBtn"
#pragma link "sEdit"
#pragma link "sLabel"
#pragma link "sMemo"
#pragma link "sButton"
#pragma resource "*.dfm"
TForm1 *Form1;
FILE *work;
char path [200];
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
        : TForm(Owner)
{
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::SaveClick(TObject *Sender)
{
strcpy(path, "DATA\\");
strcat(path, Day->Text.c_str());
strcat(path, "-");
strcat(path, Month->Text.c_str());
strcat(path, "-");
strcat(path, Year->Text.c_str());
strcat(path, ".dpf");
work=fopen(path, "w");
fprintf(work, "%s", Txt->Text);
fclose (work);
}
//---------------------------------------------------------------------------
 
void __fastcall TForm1::LoadClick(TObject *Sender)
{
SYSTEMTIME st;
GetSystemTime(&st);
strcpy(path, "DATA\\");
if (st.wDay>=10)
{
strcat(path, IntToStr(st.wDay).c_str());
}
else
{
strcat(path, "0");
strcat(path, IntToStr(st.wDay).c_str());
}
strcat(path, "-");
if (st.wMonth >=10)
{
strcat(path, IntToStr(st.wMonth).c_str());
}
else
{
strcat(path, "0"); strcat(path, IntToStr(st.wMonth).c_str());
}
strcat(path, "-");
strcat(path, IntToStr(st.wYear).c_str());
strcat(path, ".dpf");
Txt->Text=path;
    if((work = fopen(path, "rb")) == NULL){
        Txt->Text="Óðà! Íè÷åãî íå íàäî äåëàòü!\n";
        Month->Text=path;
    }
    else{
    fopen(path, "rb");
    Txt->Text=path;
    fseek(work, 0L, SEEK_END);
    long size = ftell(work);
    fseek(work, 0L, SEEK_SET);
    char* buf = (char*)malloc(sizeof(char) * size);
    fread(buf, 1, size, work);
    Txt->Text=buf;
    free(buf);
    fclose(work);
    }
}
//---------------------------------------------------------------------------

@kzru_hunter · 05.03.2013, 13:12

На глаз проверил, заметил следующее:
74-я строчка:

C++

1	fopen(path, "rb");

замени на

C++

1	work = fopen(path, "rb");

@ComputerMouse · 11.03.2013, 21:12 **[ТС]**

Ооооййй... Спасибо!

Добавлено через 6 минут
Однако дело не в этом. Ошибка-то где-то здесь:

C++

void __fastcall TForm1::SaveClick(TObject *Sender)
{
strcpy(path, "DATA\\");
strcat(path, Day->Text.c_str());
strcat(path, "-");
strcat(path, Month->Text.c_str());
strcat(path, "-");
strcat(path, Year->Text.c_str());
strcat(path, ".dpf");
work=fopen(path, "w");
fprintf(work, "%s", Txt->Text);
fclose (work);
}

А тут всё с самого начала нормально.

Добавлено через 3 часа 17 минут
Причём ошибка в строке 4, но где?..

@nick42 · 12.03.2013, 00:40

Сообщение от ComputerMouse

Причём ошибка в строке 4, но где?..

Ошибка не в строке, а в отсутствии каталога DATA. Если хотите этими средствами (stdio.h), то сначала нужно создать каталог на диске, прежде чем в него записывать.

@ComputerMouse · 12.03.2013, 08:52 **[ТС]**

Ну да, я понял. Сейчас попробую найти как "чистым" способом создать каталог, а то мой

C++

1	system("md DATA");

открывает консоль, и вообще некрасиво.

@nick42 · 12.03.2013, 10:21

C++

/*  #include <dir.h>  */
  _mkdir("DATA\\"); Sleep(20); 
  strcpy(path, "DATA\\");
strcat(path, Day->Text.c_str());
strcat(path, "-");
strcat(path, Month->Text.c_str());

@ComputerMouse · 12.03.2013, 10:24 **[ТС]**

Эммм...

C++

1	MkDir("DATA");

@nick42 · 12.03.2013, 10:28

Вообще-то все это можно короче записать__

C++

1
2
3

work=fopen(("DATA\\"+Day->Text+"-"+Month->Text+"-"+Year->Text+".dpf").c_str(),"w");
fprintf(work, "%s", Txt->Text);
fclose (work);

@ComputerMouse · 12.03.2013, 13:54 **[ТС]**

Сообщение от nick42

Sleep(20);

Зачем?

@nick42 · 12.03.2013, 15:27

Сообщение от ComputerMouse

Зачем?

каталог моментально не создается; попробуйте без этого, может у вас пройдет...

@nick42 Практикантроп 4823 / 2717 / 524 Регистрация: 23.09.2011 Сообщений: 5,777
	12.03.2013, 15:27	16
	Сообщение от ComputerMouse Зачем? каталог моментально не создается; попробуйте без этого, может у вас пройдет... 0

@kzru_hunter 1123 / 794 / 101 Регистрация: 01.02.2011 Сообщений: 1,878 Записей в блоге: 1
	04.03.2013, 18:15	2
	Не выделил память под указатель path 1

@ComputerMouse 6 / 6 / 4 Регистрация: 06.11.2012 Сообщений: 254
	04.03.2013, 19:02 [ТС]	3
	Спасибо! 0

@Avazart 8385 / 6147 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	04.03.2013, 22:15	4
	Сообщение от ComputerMouse strcpy(path, Day->Text.c_str()); Опасно ... Добавлено через 1 минуту Или у вас Builder6 ? 0

@ComputerMouse 6 / 6 / 4 Регистрация: 06.11.2012 Сообщений: 254
	04.03.2013, 22:29 [ТС]	5
	Да, 6 0

@Avazart 8385 / 6147 / 615 Регистрация: 10.12.2010 Сообщений: 28,683 Записей в блоге: 30
	04.03.2013, 22:33	6
	А ну тогда нормально... 0

@nick42 Практикантроп 4823 / 2717 / 524 Регистрация: 23.09.2011 Сообщений: 5,777
	12.03.2013, 00:40	10
	Сообщение от ComputerMouse Причём ошибка в строке 4, но где?.. Ошибка не в строке, а в отсутствии каталога DATA. Если хотите этими средствами (stdio.h), то сначала нужно создать каталог на диске, прежде чем в него записывать. 0

@ComputerMouse 6 / 6 / 4 Регистрация: 06.11.2012 Сообщений: 254
	12.03.2013, 13:54 [ТС]	15
	Сообщение от nick42 Sleep(20); Зачем? 0