Problem z fstream

[Miroo_007]

Mam problem z zapisem danych do pliku już istniejącego i mającego już jakieś dane. Zapisując dane do pliku usuwają mi się dane, które tam już są. Tak jakby działał specyfikator ios::trunc choć używam ios:ate. Oto kod:

Kod: Zaznacz cały

TSzablon_zapis zapis;
fstream plik;
try {
      plik.open(opcje.sciezka_schematow_email.c_str(), ios::out | ios::ate | ios::binary);
      plik.seekp(0, ios::end);
      int wielkosc_pliku = plik.tellp(); //w celach pogladowych
      plik.seekp(i * sizeof(TSzablon_zapis));
      plik.write((char*)&zapis,sizeof(TSzablon_zapis));
__finally {
      plik.close();
}

Otwierając plik do zapisu zmienna wielkosc_pliku zawsze równa się zero. Jeżeli otworze plik do odczytu to wartość jest poprawna (różna od zera).

[polymorphism]

A ios::app złe?

[Miroo_007]

Z ios::app też usuwa mi wcześniejsze dane zostawiając puste miejsce.

[polymorphism]

Hmm, nie powinno. Spróbuj może z ofstream - nie sądzę, żeby to coś dało, ale kto wie...

[Miroo_007]

Może to być błąd C++ Builder 2007?

[Miroo_007]

Spróbuj może z ofstream - nie sądzę, żeby to coś dało, ale kto wie...

Nie pomogło. Czy da się usunąć ręcznie specyfikator ios::trunc? Wiem, że usuwa się specyfikatory błędu a można specyfikator otwarcia?

[polymorphism]

Może to być błąd C++ Builder 2007?

Może...

Spróbuj jeszcze z taką wiązanką:

Kod: Zaznacz cały

ios::in | ios::out | ios::ate | ios::binary

flaga in powinna zablokować czyszczenie pliku.

[Miroo_007]

Nic to nie zmienia. Przerobię to na TFileStream bo już mnie trafia z tym. Straciłem przez to 2 dni.

[simekone]

Spróbuj użyć zamiast

Kod: Zaznacz cały

c_str()

funkcji

Kod: Zaznacz cały

t_str()

dodaj też parametry zapisu

Kod: Zaznacz cały

ios::in | ios::app

[Miroo_007]

Spróbuj użyć zamiast

Kod: Zaznacz cały

c_str()

funkcji

Kod: Zaznacz cały

t_str()

To nic nie wnosi do sprawy. Używam kompilatora C++ Builder 2007, a Ty mówisz o C++ Builder 2009.
Ten zapis powoduje błąd:

Kod: Zaznacz cały

fstream plik;
try {
      plik.open(sciezka.c_str(), ios::in | ios::out | ios::binary | ios::app);
      if(plik.good()) {
         plik.write((char*)&zapis, sizeof(TSzablon_zapis));
         plik.close();
      }
      else return;
} catch (Exception &e) {
   blad.Raport(e.Message);
}

[polymorphism]

Nic to nie zmienia.

No to być może jest jakiś błąd w strumieniu - w obu przypadkach, które podałem, strumień nie czyścił mi pliku (testowałem to na GCC).

Ten zapis powoduje błąd:

Jaki błąd?

[Cyfrowy Baron]

plik.open(sciezka.c_str(), ios::in | ios::out | ios::binary | ios::app);

Nie można jednocześnie używać operatora ios::in (odczyt danych) i ios::out (zapis danych).

Operatorem umożliwiającym dopisanie danych do pliku jest ios::app.
Używasz biblioteki fstream, ale dla odczytu danych stosujemy klasę ifstream, a do zapisu ofstream, dlatego Tój kod powinien wyglądać tak:

Kod: Zaznacz cały

  TSzablon_zapis zapis;
    ofstream plik;
    try {
          plik.open(opcje.sciezka_schematow_email.c_str(), ios::out | ios::app | ios::binary);
          plik.seekp(0, ios::end);
          int wielkosc_pliku = plik.tellp(); //w celach pogladowych
          plik.seekp(i * sizeof(TSzablon_zapis));
          plik.write((char*)&zapis,sizeof(TSzablon_zapis));
       }
    __finally {
          plik.close();
    }

Problemem wynika prawdopodobnie z klasy TSzablon_zapisu.

Przeprowadziłem testy z Twoim kodem u siebie w środowisku C++ BUilder 2007. Nie znam Twojej struktury TSzablon_zapis, więc użyłem własnej przykładowej:

► 

---

Kod: Zaznacz cały

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

#include <vcl.h>
#pragma hdrstop

#include "Unit1.h"
#include <fstream.h>
#include <stdio.h>
//---------------------------------------------------------------------------
#pragma package(smart_init)
#pragma resource "*.dfm"
TForm1 *Form1;
//---------------------------------------------------------------------------
__fastcall TForm1::TForm1(TComponent* Owner)
    : TForm(Owner)
{
}
//---------------------------------------------------------------------------
struct Baza
{
char Name[5][250];
char Mark[5][50];
float Cost[5];
};

void __fastcall TForm1::Button1Click(TObject *Sender)
{
Baza Hard;

sprintf(Hard.Name[0], "%s", "Bear Paw 1200 TA");
sprintf(Hard.Mark[0], "%s", "Skaner");
Hard.Cost[0] = 350;

sprintf(Hard.Name[1], "%s", "ImageQuest Q770");
sprintf(Hard.Mark[1], "%s", "Monitor");
Hard.Cost[1] = 800;

sprintf(Hard.Name[2], "%s", "Tracer TRK 21MTT");
sprintf(Hard.Mark[2], "%s", "Klawiatura");
Hard.Cost[2] = 187;

sprintf(Hard.Name[3], "%s", "FlashDisk");
sprintf(Hard.Mark[3], "%s", "Napęd Dyskowy USB Flash");
Hard.Cost[3] = 120;

sprintf(Hard.Name[4], "%s", "Epson Stylus C20SX");
sprintf(Hard.Mark[4], "%s", "Drukarka");
Hard.Cost[4] = 320;

static int i = 0;

    ofstream plik;
    try {
          plik.open("c:\\plik.txt", ios::out | ios::app | ios::binary);
          plik.seekp(0, ios::end);
          int wielkosc_pliku = plik.tellp();
          Caption = (String)wielkosc_pliku;
          //plik.seekp(i * sizeof(Baza));
          plik.write((char*)&Hard, sizeof(Baza));
    }
    __finally {
          plik.close();
    }
}
//---------------------------------------------------------------------------


---

i u mnie to działa, dane są dopisywane na końcu pliku, a ten fragment kodu:

plik.seekp(i * sizeof(Baza));

jest zbędny, gdyż argument ios::app odpowiada za dopisywanie danych zawsze na końcu pliku.

podobnie jak ten fragment:

plik.seekp(0, ios::end);

chyba że chcesz pobierać rozmiar pliku.

---

[polymorphism]

Nie można jednocześnie używać operatora ios::in (odczyt danych) i ios::out (zapis danych).

Oczywiście, że można. Odsyłam chociażby do tego.

______________
PS. ios::in, ios::out itd. nie są operatorami - to flagi, zwykłe wartości liczbowe (enum), a fstream nie jest biblioteką, tylko klasą pochodną od iostream.

[Miroo_007]

Problemem wynika prawdopodobnie z klasy TSzablon_zapisu.

W programie operuje na klasie TSzablon:

Kod: Zaznacz cały

class TSzablon {
   public:
      AnsiString         tytul_szablonu,
                  tytul_wiadomosci,
                  sciezka_szablonu;
      bool            szablon_listy,
                  szablon_email,
                  szablon_SMS;
      TStringList      *Tresc_wiadomosci;
      void            Zapisz_szablon(TSzablon &lokalny, int i=0);
      void            Dodaj_szablon(TSzablon &lokalny);
      bool            Wczytaj_szablon(AnsiString sciezka,int i=0);
      TSzablon();
      ~TSzablon();
};

Do zapisywania wykorzystuje strukturę TSzablon_zapis:

Kod: Zaznacz cały

struct TSzablon_zapis {
      char            tytul_szablonu[100],
                  tytul_wiadomosci[300];
      bool            szablon_listy,
                  szablon_email,
                  szablon_SMS;
      char            Tresc_wiadomosci[2000];
};

Problem jest w funkcji TSzablon::Dodaj_szablon():

Kod: Zaznacz cały

void TSzablon::Dodaj_szablon(TSzablon &lokalny) {
   TSzablon_zapis zapis;
   strcpy(zapis.tytul_szablonu,    lokalny.tytul_szablonu.c_str());
   strcpy(zapis.tytul_wiadomosci,    lokalny.tytul_wiadomosci.c_str());
   strcpy(zapis.Tresc_wiadomosci,    lokalny.Tresc_wiadomosci->Text.c_str());
   zapis.szablon_listy          = lokalny.szablon_listy;
   zapis.szablon_email          = lokalny.szablon_email;
   zapis.szablon_SMS         = lokalny.szablon_SMS;
   AnsiString sciezka;
   if(lokalny.szablon_listy) sciezka    = opcje.sciezka_szablonow_listy;
   if(lokalny.szablon_email) sciezka    = opcje.sciezka_szablonow_email;
   if(lokalny.szablon_SMS) sciezka      = opcje.sciezka_szablonow_SMS;
   ofstream plik;
   try {
      plik.open(sciezka.c_str(), ios::out | ios::binary | ios::app);
      if(plik.good()) {
         plik.write((char*)&zapis, sizeof(TSzablon_zapis));
         plik.close();
      }
      else return;
   } catch (Exception &e) {
      blad.Raport(e.Message);
   }
   try {
      plik.open(sciezka.c_str(), ios::in | ios::binary);
      if(plik.good()) {
         plik.seekg(0, ios::end);
         int w = plik.tellg();
         if(zapis.szablon_listy) opcje.liczba_szablonow_listy = w    / sizeof(TSzablon_zapis);
         if(zapis.szablon_email) opcje.liczba_szablonow_email = w    / sizeof(TSzablon_zapis);
         if(zapis.szablon_SMS)    opcje.liczba_szablonow_SMS = w / sizeof(TSzablon_zapis);
         plik.close();
      }
   } catch (Exception &e) {
      blad.Raport(e.Message);
   } 
}

W Symfonii C++ jest napisane: tom 3, str.644 (dotyczy funkcji write):

Podobnie jak przy funkcji read, także i tutaj dodam, że jeśli program ma wykonywać wyjątkowo wiele operacji pisania binarnego - dobrze jest rozważyć posłużenie się klasą streambuf i jej pochodnymi.

[polymorphism]

Kod: Zaznacz cały

ofstream plik;

...

try
{
   plik.open(sciezka.c_str(), ios::in | ios::binary);
   if(plik.good())
   {
      plik.seekg(0, ios::end);
      
      int w = plik.tellg();
      
      ...
   }
   ...
}

Od kiedy to ostream ma takie metody jak seekg i tellg? No i po co te bloki try...catch?