There is a file (query.txt) which has some keywords/phrases which are to be matched with other files using grep. The last three lines of the following code are working perfectly but when the same command is used inside the while loop it goes into an infinite loop or something(ie doesn't respond).
import osf=open('query.txt','r')
b=f.readline()
while b:cmd='grep %s my2.txt'%b #my2 is the file in which we are looking for bos.system(cmd)b=f.readline()
f.close()a='He is'
cmd='grep %s my2.txt'%a
os.system(cmd)
First of all, you are not iterating over the file properly. You can simply use for b in f:
without the .readline()
stuff.
Then your code will blow in your face as soon as the filename contains any characters which have a special meaning in the shell. Use subprocess.call
instead of os.system()
and pass an argument list.
Here's a fixed version:
import os
import subprocess
with open('query.txt', 'r') as f:for line in f:line = line.rstrip() # remove trailing whitespace such as '\n'subprocess.call(['/bin/grep', line, 'my2.txt'])
However, you can improve your code even more by not calling grep
at all.
Read my2.txt
to a string instead and then use the re
module to perform the search. In case you do not need a regex at all, you can even simply use if line in my2_content