VPF::Обновление текстового файла. Покритикуйте код.

abibok

Дата 31.5.2008, 17:02 (ссылка)

(нет голосов)

Загрузка ...

Новичок

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

Репутация: нет
Всего: нет

Код


/*
Привет!

Возникла необходимость написать следующий код: 
На вход получаем имя файла и номер строки.
Необходимо добавить в начало указанной строки определенный текст и 
перезаписать файл с этими измениями. 

Ниже код, который я написал, он работает. 
Но быть может у него есть минусы, и можно написать лучше? (у меня есть сомнения)
*/

$n = 3;    //номер строки

$data = file("1.txt");                //открываем файл, читаем его в массив
$data [$n] = "TEXT! ".$data [$n];    //добавляем в начало указанной строки нужный текст

$final_file = fopen("1.txt", "w"); //открываем файл на чтение, старое содержимое удаляется

while (list($linenum, $value) = each ($data)) //обходим массив
    fputs($final_file, $value); //построчно записываем весь файл, в том числе и нашу измененную строку

fclose($final_file);

Это сообщение отредактировал(а) abibok - 31.5.2008, 17:05

Feldmarschall

Дата 31.5.2008, 17:45 (ссылка)

(голосов:3)

Загрузка ...

Новичок

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

Репутация: 22
Всего: 32

начнем с того, что чтение всего файла в память является хрестоматийно кривой операцией.
как и использование массивов там, где они не нужны.

читать надо построчно.
не забывать о блокировках
и начать использовать базу данных

abibok

Дата 2.6.2008, 13:59 (ссылка)

(нет голосов)

Загрузка ...

Новичок

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

Репутация: нет
Всего: нет

Feldmarschall, благодарю за ответ.

1 Пользователей читают эту тему (1 Гостей и 0 Скрытых Пользователей)
0 Пользователей:
« Предыдущая тема \| PHP: Общие вопросы \| Следующая тема »