Помогите подправить код (php)

А какой размер файла?

И для чего explode, если в итоге выводится полная строка?

какой размер файла 1.txt?

зачем вот эта строка?

$exstr = explode('=', $rstr1);

(я понимаю, что $exstr может еще где-нибудь использоваться)

Может в файл 1.txt кто-то пишет, возникают коллизии доступа.

1.тхт до 1 мб. Просто интересно ,нет ли ошибок в коде?

Используйте просто echo $rstr1;

WhiteSmartFox добавил 18-02-2011 в 16:33

Если не надо удалите.

WhiteSmartFox добавил 18-02-2011 в 16:33

Файл 1 мб это слишком много, мало того что прочитать 1Мб долго + каждый процесс будет резервировать 1-2Мб'ний массив => что тоже не быстрое занятие, да и память кстати ограниченный ресурс.

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

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

Можно так попробовать:

$file_cont = file("1.txt");

$rand_line = $file_cont[array_rand($file_cont)];

Может тогда переделать алгоритм, типа

<?php 
      $file_name = "1.txt";
      $len = 300; // Это максимальный код строки * 2
      $file = fopen($file_name,"r");
      flock($file, LOCK_SH);
      $size = fstat($file);
      $n = rand(0,$size); 
      fseek($file,$n);    
      $str = fread($file,$len);  
      flock($file, LOCK_UN);     
      fclose($file);
      $tmp = explode("/n", $str1."/n"); 
      echo $tmp[1];
?>

Это код также выведет случайную строчку, правда немного изменится вероятность (чем больше строка, тем более вероятно что она будет выведена) + Есть некоторая вероятность что последняя строка может вывестись не полностью. Зато этот код не будет читать больше максимального размера строки * 2.

P.S. Можно создать дополнительный файл в который будет записывается размер начальный и конечный индекс строк при каждом обновлении 1.txt, при этом можно получать строку намного проще т.е. есть файл 1.txt вида

Привет Мир

Привет Мир!Ура!

создать для него доп файл 1.ind вида (средствами php конечно)

00000 00010

00012 00015

...

в коде при чтении будет что-то вида (не проверял)

      $file = fopen('1.ind',"r");
      $size = fstat($file);
      $n = rand(0,$size/13); 
      fseek($file,$n*13);    
      $str = fread($file,11);  
      $tmp = explode(' ', $str1); 
      fclose($file);
      $file1 = fopen('1.txt',"r"); 
      fseek($file1,$tmp[0]);      
      $str = fread($file1,$tmp[1]);         
      fclose($file1); 
      echo $str;

<?php
	$fhandle = fopen('1.txt', 'r');
	fseek($fhandle, 0, SEEK_END);
	$length = ftell($fhandle);
	$position = mt_rand(0, $length)-20;
	fseek($fhandle, $position);
	$line = fgets($fhandle);
	$line = fgets($fhandle);
	fclose($fhandle);
	echo $line;
?>

Смысл такой, получаем размер файла, получаем случайное число от 0 до размера файла, перемещаем указатель в файле на полученную случайную позицию. Получаем строку в указанной позиции (там идет два раза $line = fgets($fhandle); - это не опечатка, т.к. первая полученная строка может быть не полной, читаем следующую).

Работает с файлами любого размера, не занимает кучу памяти. Минус - чем меньше строк в файле, тем больше шанс попасть на последнюю строку и прочитать ее не полностью (следующей строки не будет). Для предотвращения этого там стоит -20, можно увеличить до максимальной длины строки.