К первой строчке претензий вообще нет:
Код |
---|
$x = file('file.txt'); |
тут без вопросов - все отлично, если файл не будет иметь размер в пару гигов конечно...
Но все, что внутри цикла - ужас.
Начать с того, что по условию разбивать на до по 25 строк, а не по 20. Оп, уже садиться этот кусок переписывать? А если завтра по 5 надо будет или по 1? 100 раз скопипастите? а если строк не 100, а 10000?
Сейчас в КАЖДОЙ итерации открывается файл на запись...

Почему бы не подготовить несколько строк сперва и не скинуть их в файл только 1 раз, а не 20 раз как сейчас - все продуктивнее будет и быстрее по времени.