Версия для печати темы
Нажмите сюда для просмотра этой темы в оригинальном формате
Форум программистов > Delphi: Для новичков > TFileStream и русский язык


Автор: pushok 5.10.2009, 21:40
Здравствуйте = )

Ситуация: использую TFileStream для записи и чтения из файла переменных разных типов.

Проблема: записываю русский текст в переменную (string). Переменную записываю в файл. Открываю файл и он мне выводит вместо строки чушь какую-то. Соответственно весь оставшийся файл испорчен (видимо русский текст занимает больше чем 4 Кбайта.

При этом заметил такую особенность: если я записываю файл и открываю его не выходя из программы - все в порядке!

В чем дело? Может кто-то сталкивался?

Автор: Qu1nt 5.10.2009, 21:50
Телепаты на юге. Код в студию!

Автор: AntonN 5.10.2009, 21:58
смотря как записывать переменную string, у нее нулевой байт не нужно записывать

Автор: Beltar 5.10.2009, 22:37
Помимо кода версию Delphi не забудь.

Автор: bems 6.10.2009, 01:21
Цитата(pushok @  5.10.2009,  21:40 Найти цитируемый пост)
Переменную записываю в файл

Именно переменную? smile    Так это ж указатель!

Цитата(pushok @  5.10.2009,  21:40 Найти цитируемый пост)
Соответственно весь оставшийся файл испорчен (видимо русский текст занимает больше чем 4 Кбайта.
Откуда взялось 4 кило? Что это вообще за число?

Цитата(pushok @  5.10.2009,  21:40 Найти цитируемый пост)
если я записываю файл и открываю его не выходя из программы - все в порядке!
Ну да, это может быть. Записал указатель, потом прочитал его. Если строка куда указывает указатель все еще на месте то будет казаться что все работает.

Цитата(Qu1nt @  5.10.2009,  21:50 Найти цитируемый пост)
Телепаты на юге. Код в студию! 
Да, было бы не плохо.

Цитата(AntonN @  5.10.2009,  21:58 Найти цитируемый пост)
смотря как записывать переменную string, у нее нулевой байт не нужно записывать 
Нулевой байт имеет особый смысл только у ShortString. Но в этом случае его записывать как раз нужно.

Цитата(Beltar @  5.10.2009,  22:37 Найти цитируемый пост)
Помимо кода версию Delphi не забудь. 

Ну в новых версиях дельфи строки стали уникодными, но вполне возможно писать так, чтобы оно корректно работало под любой версией (ну кроме первой). Нужно просто вместо 1 байта на симбол использовать SizeOf(Char)

Добавлено через 57 секунд
Модератор: И да, вопрос-то для начинающих!

Автор: pushok 6.10.2009, 02:46
Вообщем суть такая:

Код

...
type
t=record
s:string;
...
var
tf: TFileStream;
...
tf.Write(t.s,SizeOf(String));
...
tf.Read(t.s,SizeOf(String));
...


ПРо русский я наврал = )) Попробовал и английский тоже косячит.

4 кило взялось из строки: ShowMessage(IntToStr(SizeOf(String))) или ShowMessage(IntToStr(SizeOf(t.s))).

Автор: bems 6.10.2009, 03:12
Ну так и есть, пишешь указатель. Не все телепаты в отпуске smile
SizeOf возвращает значение в байтах. 4 байта - потому что строка на самом деле указатель на символы. а размер указателя - 4 байта. Чтобы записать собственно строку, делай что-то такое
Код

tf.Write(t.s[1], Length(t.s) * SizeOf(Char));

Только когда ты захочешь прочитать, то окажется что длина строки неизвестна. Поэтому записывай сначала длину, потом строку как я показал. Тогда читаешь длину и определяешь сколько байт в строке (длина строки умножить на размер символа)

Автор: CodeMonkey 6.10.2009, 08:24
http://www.delphikingdom.ru/asp/viewitem.asp?catalogid=1206#06.
http://transl-gunsmoker.blogspot.com/2009/09/pchars.html.

Автор: pushok 6.10.2009, 08:45
Спасибо всем за обсуждение и помощь.
Думаю проблемы больше нет = )

За ссылки отдельное спасибо, доступно написано.


Автор: pushok 6.10.2009, 23:56
Код

...
tf.Read(t.q[i].img,SizeOf(TBitmap));
...
 tf.Read(e,SizeOf(Integer));
showmessage(IntToStr(e));
 SetLength(t.q[i].a1,e);
...


При выводе переменной e на экран получается  больше 20 тыс, однако должно быть меньше.

Подскажите в чем дело.

??

Автор: bems 7.10.2009, 00:08
выглядит нормально. Ошибка где-то в другом месте.

Автор: kami 7.10.2009, 00:10
Цитата(pushok @  6.10.2009,  23:56 Найти цитируемый пост)
tf.Read(t.q[i].img,SizeOf(TBitmap));

У битмапов есть свой метод чтения из потока.
ограничение этого метода - Tbitmap читает от текущей позиции указателя в потоке и до конца потока, вне зависимости от того, его данные содержатся там или нет.
А в вашем коде считывается вместо полных данных битмапа только первые 4 байта. И то, что считывается в переменную e на самом деле должно было считаться в битмап.
Для анализа работы с потоками нужно помнить, что Read и Write - это функции, и они возвращают количество реально считанного/записанного.
В режиме пошаговой отладки бывает полезно вывести в Watches myStream.Position.

Это было раз.

Второе - из кода непонятно, а почему собственно должно было считаться число меньшее 20000? Может, вы записали в поток больше 20000?

Автор: bems 7.10.2009, 00:13
эээ... То есть TBitmap это Graphics.TBitmap? мда... Тогда читаем kami smile

Автор: kami 7.10.2009, 00:17
Цитата(bems @  7.10.2009,  00:13 Найти цитируемый пост)
То есть TBitmap это Graphics.TBitmap?

Я без малейшего понятия. Но думаю, что автор не должен был додуматься перекрыть своим объявление этого класса smile Кроме того - не представлена конкретная задача, и даже полная структура 
Цитата(pushok @  6.10.2009,  02:46 Найти цитируемый пост)
type t=record

Что затрудняет работу телепатора в автономном режиме smile

Автор: bems 7.10.2009, 00:32
Цитата(kami @  7.10.2009,  00:17 Найти цитируемый пост)
Но думаю, что автор не должен был додуматься перекрыть своим объявление этого класса
Я подумал про Windows.TBitmap. его можно читать вышеуказаным кодом. Хотя если подумать смысла в этом не то что бы вагон.

Автор: kami 7.10.2009, 01:16
Цитата(bems @  7.10.2009,  00:32 Найти цитируемый пост)
Я подумал про Windows.TBitmap. 

Упс... забыл про него...
Чего-то у нас много получается "подумал-забыл". smile  Надо подождать разъяснений ТС 

Для ТС: на всякий случай (не знаю, будет полезно или нет) - процедуры чтения и записи того, что спрашивалось (само собой - применять только парами).
Код

const
  MaxStrLength = 8192;

procedure LoadStringFromStream(var Str: string; Stream: TStream);
var
  i: integer;
begin
  Str := '';
  if Stream.Read(i, SizeOf(i)) = SizeOf(i) then
    begin
      if (i > 0) and (i < MaxStrLength) then
        begin
          SetLength(str, i);
          if Stream.Read(Str[1], i) <> i then
            raise EStreamError.Create('Ошибка при чтении строки из потока');
        end
      else
        if i <> 0 then
          raise EStreamError.Create('Ошибка: недопустимый размер строки '+IntToStr(i));
    end
  else
    raise EStreamError.Create('Ошибка при чтении размера строки из потока');
end;

procedure SaveStringToStream(const Str: string; Stream: TStream);
var
  i: integer;
begin
  i := Length(Str);
  if i < MaxStrLength then
    begin
      Stream.Write(i, SizeOf(i));
      if i > 0 then
        Stream.Write(str[1], i);
    end
  else
    raise EStreamError.Create('Ошибка! Размер строки превышает допустимый (' + IntToStr(i) + ')');
end;

// следующие процедуры пишу прямо здесь, посему обработку ошибок оставляю "за кадром".
procedure LoadBitmapFromStream(Bitmap:TBitMap; Stream:TStream);
var
  tmpStream:TMemoryStream;
  i:integer;
begin
  Stream.Read(i, SizeOf(i));
  if i>0 then
    begin
      tmpStream:=TMemoryStream.Create;
      tmpStream.CopyFrom(Stream, i);
      tmpStream.Seek(0, soBeginning);
      Bitmap.LoadFromStream(tmpStream);
      tmpStream.Free;
    end;
end;

procedure SaveBitmapToStream(Bitmap:TBitMap; Stream:TStream);
var
  tmpStream:TMemoryStream;
  i:integer;
begin
  tmpStream:=TMemoryStream.Create;
  Bitmap.SaveToStream(tmpStream);
  i:=tmpStream.Size;
  Stream.Write(i, SizeOf(i));
  Stream.CopyFrom(tmpStream, 0);
  tmpStream.Free;
end;

Автор: THandle 7.10.2009, 08:43
kami, а как же try..finally в последних двух процедурках?


pushok, вообще глянь еще такую штуку как http://forum.vingrad.ru/forum/topic-250409.html. Там реализовано много методов для записи и чтения различных данных.

Автор: kami 7.10.2009, 09:14
Цитата(THandle @  7.10.2009,  08:43 Найти цитируемый пост)
а как же try..finally в последних двух процедурках?

а я так и написал:
Цитата(kami @  7.10.2009,  01:16 Найти цитируемый пост)
// следующие процедуры пишу прямо здесь, посему обработку ошибок оставляю "за кадром".

А если честно - "ниахоооота" (с) мультик про Великого Нехочуху.

Автор: pushok 9.10.2009, 14:40
По аналогии написал две функции для TStringList. Выкладываю на всякий случай (вроде работает, но если какие-то ошибки). А строки решил пока по-обычному записывать (string[1]).

Код


procedure SaveStringListToStream(SL:TStringList; Stream:TStream);
var
  tmpStream:TMemoryStream;
  i:integer;
begin
  tmpStream:=TMemoryStream.Create;
  SL.SaveToStream(tmpStream);
  i:=tmpStream.Size;
  Stream.Write(i, SizeOf(i));
  Stream.CopyFrom(tmpStream, 0);
  tmpStream.Free;
end;

procedure LoadStringListFromStream(SL:TStringList; Stream:TStream);
var
  tmpStream:TMemoryStream;
  i:integer;
begin
  Stream.Read(i, SizeOf(i));
  if i>0 then
    begin
      tmpStream:=TMemoryStream.Create;
      tmpStream.CopyFrom(Stream, i);
      tmpStream.Seek(0, soBeginning);
      SL.LoadFromStream(tmpStream);
      tmpStream.Free;
    end;
end;



Всем большое спасибо.

Автор: THandle 10.10.2009, 09:23
Цитата(kami @  7.10.2009,  10:14 Найти цитируемый пост)
А если честно - "ниахоооота" (с) мультик про Великого Нехочуху. 


Такие вещи, как обработка ошибок должны всегда в первую очередь писаться.  smile  

Вот у pushok теперь потенциальноошибконеобрабатываемый код.

pushok, используй try..finally при создании доп потоков, плюс я бы еще проверял на всякий случай все эти входящие параметры на существование(if not Assigned() then Exit).

Powered by Invision Power Board (http://www.invisionboard.com)
© Invision Power Services (http://www.invisionpower.com)