Delete line from text file if line contains one of few specified strings Python [duplicate]

Question

I presently have code that deletes all lines from a text file that contain one specific string. Here it is:

import os  
with open(r"oldfile") as f, open(r"workfile", "w") as working:    
    for line in f:   
       if "string1" not in line:  
           working.write(line)  
os.remove(r"oldfile")  
os.rename(r"workfile", r"oldfile")

My question is: how can I include other strings? In other words, I want to tell the script that if a line contains "string1" or some other string "string2", then delete that line. I know I could just repeat the code I put above for every such string, but I'm certain there's some shorter and more efficient way to write that.
Many thanks in advance!

This might help: stackoverflow.com/questions/6531482/…

yinnonsanders
– yinnonsanders

2017-07-07 18:29:06 +00:00
Commented Jul 7, 2017 at 18:29 — yinnonsanders
– yinnonsanders, Commented Jul 7, 2017 at 18:29

havanagrawal · Accepted Answer · 2017-07-07 18:30:35Z

2

Just abstract it out into a function and use that?

def should_remove_line(line, stop_words):
    return any([word in line for word in stop_words])

stop_words = ["string1", "string2"]
with open(r"oldfile") as f, open(r"workfile", "w") as working:    
for line in f:   
   if not should_remove_line(line, stop_words):  
       working.write(line)

answered Jul 7, 2017 at 18:30

havanagrawal

1,0396 silver badges12 bronze badges

Sign up to request clarification or add additional context in comments.

Comments

PYA · Accepted Answer · 2017-07-07 18:29:28Z

1

might be good to have a function

def contains(list_of_strings_to_check,line):
  for string in list_of_strings_to_check:
    if string in line:
      return False
  return True

list_of_strings = ["string1","string2",...]
...
for line in f:   
       if contains(list_of_strings,line):

answered Jul 7, 2017 at 18:29

PYA

8,8123 gold badges24 silver badges42 bronze badges

Comments

jacoblaw · Accepted Answer · 2017-07-07 18:32:02Z

0

You can loop through a list of your blacklisted strings while keeping track of if one of the blacklisted strings was present like this:

import os  
blacklist = ["string1", "string2"]
with open(r"oldfile") as f, open(r"workfile", "w") as working:    
    for line in f:   
        write = True
        for string in blacklist:
           if string in line:  
               write = False
               break
        if write:
               working.write(line) 
os.remove(r"oldfile")  
os.rename(r"workfile", r"oldfile")

answered Jul 7, 2017 at 18:32

jacoblaw

1,2839 silver badges9 bronze badges

Comments

J0hn · Accepted Answer · 2017-07-07 18:32:16Z

0

if "string1" in line or "string2" in line:

This should work I think

edited Jul 7, 2017 at 18:32

answered Jul 7, 2017 at 18:28

J0hn

5805 silver badges19 bronze badges

1 Comment

J0hn Over a year ago

Updated, yeah I tried running and it only worked this way. Depending on how many strings OP has to check, some of the other methods people have posted involving lists might be better.

Collectives™ on Stack Overflow

Delete line from text file if line contains one of few specified strings Python [duplicate]

4 Answers 4

Comments

Comments

Comments

1 Comment

Linked

Hot Network Questions

Collectives™ on Stack Overflow

4 Answers 4

Comments

Comments

Comments

1 Comment

Linked

Related