Модераторы: LSD, AntonSaburov
  

Поиск:

Ответ в темуСоздание новой темы Создание опроса
> Код неправильно читает файл 
:(
    Опции темы
3,14
Дата 20.4.2005, 16:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник Клуба
Сообщений: 1614
Регистрация: 18.6.2004
Где: Н. Новгород

Репутация: 6
Всего: 24
Никак не пойму почему код неправильно читает файл:

Код

package techlib;

import java.io.*;
import java.net.*;
import javax.mail.*;
import javax.mail.internet.*;
import java.util.*;
import javax.activation.*;
import javax.servlet.*;
import javax.servlet.http.*;

public class install extends HttpServlet
{
    public void service( HttpServletRequest request, HttpServletResponse response ) throws ServletException, IOException 
    {
        response.setContentType("text/html");
        PrintWriter out = response.getWriter();
        out.println("<html><body>");

        String file_name = "/users/techlib/techlib_books.sql";
        FileReader sql_dump = new FileReader(file_name);
        char [] cbuff = new char [1024];
        int received = 0;
        String sqlQueries = "";
        while ((received = sql_dump.read(cbuff)) > 0)
        {
            sqlQueries += new String(cbuff);
        }

        out.println(sqlQueries.replaceAll("[\\n\\r]+", "<br>"));
        out.println("</body></html>");
    }
}

Добавлено @ 17:02
Торможу smile, всё понял


--------------------
Может быть, это только мой бред,
Может быть, жизнь не так хороша,
Может быть, я не выйду на свет,
Но я летал, когда пела душа...
PM MAIL   Вверх
batigoal
Дата 20.4.2005, 17:04 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Нелетучий Мыш
****


Профиль
Группа: Участник Клуба
Сообщений: 6423
Регистрация: 28.12.2004
Где: Санктъ-Петербургъ

Репутация: 24
Всего: 151
А в чем именно неправильность?

Вот эта строка
Код

while ((received = sql_dump.read(cbuff)) > 0)

Здесь получается так, что значение boolean (которое, кстати, всегда true) сравнивается с числом. Имхо, не очень хорошо.


--------------------
"Чтобы правильно задать вопрос, нужно знать большую часть ответа" (Р. Шекли)
ЖоржЖЖ
PM WWW   Вверх
AntonSaburov
Дата 20.4.2005, 18:12 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Штурман
****


Профиль
Группа: Модератор
Сообщений: 5658
Регистрация: 2.7.2002
Где: Санкт-Петербург

Репутация: 51
Всего: 118
Цитата(Lamer @ 20.4.2005, 17:04)
Здесь получается так, что значение boolean (которое, кстати, всегда true) сравнивается с числом.

Что-то ты путаешь. Где тут boolean ?
Код

while ((received = sql_dump.read(cbuff)) > 0)


(received = sql_dump.read(cbuff) - вернет число считаных байт. И его потом сравнивают с нулем. Тут все чистенько.

Скорее проблема в том, что строка формируется из полного массива, а это странно. Конец-то не всегда полностью покрывает весь массив.


--------------------
Мои проекты для Винграда
PM MAIL WWW ICQ   Вверх
batigoal
Дата 20.4.2005, 22:29 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Нелетучий Мыш
****


Профиль
Группа: Участник Клуба
Сообщений: 6423
Регистрация: 28.12.2004
Где: Санктъ-Петербургъ

Репутация: 24
Всего: 151
Пардон, стормозил под под конец рабочего дня.


--------------------
"Чтобы правильно задать вопрос, нужно знать большую часть ответа" (Р. Шекли)
ЖоржЖЖ
PM WWW   Вверх
igon
Дата 20.4.2005, 23:35 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 327
Регистрация: 30.10.2004

Репутация: 1
Всего: 21
Цитата
Скорее проблема в том, что строка формируется из полного массива, а это странно. Конец-то не всегда полностью покрывает весь массив.

Да, нужно что-то типа
Цитата
sqlQueries += new String(cbuff, 0, received);

Кроме того, достаточно ли анализировать только на
Цитата
received > 0 ?

Возможна, хотя и маловероятна ситуация, когда
sql_dump.read(cbuff) вернет 0, но конец файла еще достигнут не будет.
ИМХО, читать нужно до тех пор, пока не получим -1.
Странным кажется и размер буфера - 1024. Файл не может быть длиннее? sql_dump.read(cbuff) может ведь за раз считать и гораздо бОльшие фрагменты.
Похоже, безопаснее использовать

Цитата
public int read(char[] cbuf,
                int offset, // 0
                int length) // 1024
        throws IOException
Read characters into a portion of an array.





--------------------
Хотите поговорить об этом?
PM   Вверх
3,14
Дата 21.4.2005, 08:18 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Эксперт
***


Профиль
Группа: Участник Клуба
Сообщений: 1614
Регистрация: 18.6.2004
Где: Н. Новгород

Репутация: 6
Всего: 24
Цитата(igon @ 20.4.2005, 23:35)
Да, нужно что-то типа
Цитата
sqlQueries += new String(cbuff, 0, received);

Именно в этом и была проблема


--------------------
Может быть, это только мой бред,
Может быть, жизнь не так хороша,
Может быть, я не выйду на свет,
Но я летал, когда пела душа...
PM MAIL   Вверх
AntonSaburov
Дата 21.4.2005, 15:58 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Штурман
****


Профиль
Группа: Модератор
Сообщений: 5658
Регистрация: 2.7.2002
Где: Санкт-Петербург

Репутация: 51
Всего: 118
Цитата(igon @ 20.4.2005, 23:35)
Странным кажется и размер буфера - 1024.

Мы когда-то замеряли - получилось, что читать больше чем 4 Кб смысла особого уже нет. Но 1 Кб наверно маловато будет. Если конечно файлы не маленькие. А если маленькие, то тут вообщем-то по барабану.


--------------------
Мои проекты для Винграда
PM MAIL WWW ICQ   Вверх
igon
Дата 22.4.2005, 00:16 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 327
Регистрация: 30.10.2004

Репутация: 1
Всего: 21
Цитата
читать больше чем 4 Кб

А разве можно управлять размером блока, считываемого из Stream в каждой итерации? Сколько Stream успеет подготовить, столько и выдаст - конкретное значение суть величина непредсказуемая. Единственное, что можем - "держать карман шире" smile При маленьком буфере для варианта
Код

read(cbuff) 
рискуем получать "Переполнение массива".
При маленьком length для варианта
Код

read(cbuf, offset, length) 
придется сделать больше итераций. Оптимальные значения - 4K smile



--------------------
Хотите поговорить об этом?
PM   Вверх
AntonSaburov
Дата 22.4.2005, 14:17 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Штурман
****


Профиль
Группа: Модератор
Сообщений: 5658
Регистрация: 2.7.2002
Где: Санкт-Петербург

Репутация: 51
Всего: 118
Цитата(igon @ 22.4.2005, 00:16)
рискуем получать "Переполнение массива".

Ну переполнение - это как-то странно.
А 4К - больше просто диск обычно не позволяет. Может сейчас можно и больше. В принципе IO операции ограничены работой с диском и тут большой массив просто будет дольше считываться и если обработка не очень сложная, то пока подготовиться следующие 4К можно успеть что-то сделать с предыдущими. А больше - ну а что лишний раз память есть.


--------------------
Мои проекты для Винграда
PM MAIL WWW ICQ   Вверх
igon
Дата 24.4.2005, 02:09 (ссылка) | (нет голосов) Загрузка ... Загрузка ... Быстрая цитата Цитата


Опытный
**


Профиль
Группа: Участник
Сообщений: 327
Регистрация: 30.10.2004

Репутация: 1
Всего: 21
Риск "переполнения" действительно мнимый, sorry smile (RTFM - и еще одной иллюзией меньше smile)


--------------------
Хотите поговорить об этом?
PM   Вверх
  
 Ответ в темуСоздание новой темы Создание опроса
Правила форума "Java"
LSD   AntonSaburov
powerOn   tux
javastic
  • Прежде, чем задать вопрос, прочтите это!
  • Книги по Java собираются здесь.
  • Документация и ресурсы по Java находятся здесь.
  • Используйте теги [code=java][/code] для подсветки кода. Используйтe чекбокс "транслит", если у Вас нет русских шрифтов.
  • Помечайте свой вопрос как решённый, если на него получен ответ. Ссылка "Пометить как решённый" находится над первым постом.
  • Действия модераторов можно обсудить здесь.
  • FAQ раздела лежит здесь.

Если Вам помогли, и атмосфера форума Вам понравилась, то заходите к нам чаще! С уважением, LSD, AntonSaburov, powerOn, tux, javastic.
 
0 Пользователей читают эту тему (0 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема | Java: Общие вопросы | Следующая тема »

Подписаться на тему | Подписка на этот форум | Скачать/Распечатать тему


 




[ Время генерации скрипта: 0.0988 ]   [ Использовано запросов: 22 ]   [ GZIP включён ]


Реклама на сайте     Информационное спонсорство

 
 По вопросам размещения рекламы пишите на vladimir(sobaka)vingrad.ru
Отказ от ответственности     Powered by Invision Power Board(R) 1.3 © 2003  IPS, Inc.