내 코드에 문제가

Question

을 file.txt를 위해 sys.argv에에서 비교에 대해 : 모두가 잘 작동하지만, 단지

for hosts in s: 
    for line in fin: 
     if hosts==line: 
      print line 

이 dosn't 작업을 비교하고에 출력하지 아무것도하지 않고입니다 cmd.

import os.path 
import sys 
'''////if not enouth suffix in the code////''' 
if len(sys.argv)<2: 
    print "Please write %s <FileLocation> <hostname1> <hostname2> " % sys.argv[0] 
    sys.exit(1) 

'''////check if the file is exist in the dir////''' 
filelocation = sys.argv[1] 
if os.path.exists(filelocation) == False: 
    print "File dosnt exist please write after %s <exist file> " % sys.argv[0] 
    sys.exit(1) 

'''////compare from the sys.argv to the file////''' 
with open(filelocation,"r") as fin: 
    s = set(sys.argv[2:]) 
    for hosts in s: 
     for line in fin: 
      if hosts==line: 
       print line 

Answer 1

당신은 with 블록 이후에 추가로 들여 쓰기를 할 필요가있다. 그렇지 않은 경우 fin은 None입니다. 그래서, 당신의 코드는 다음과 같이 표시됩니다

with open(filelocation,"r") as fin: 
    s = set(sys.argv[2:]) 
    for hosts in s: 
     for line in fin: 
      if hosts==line: 
       print line 

Answer 2

문제는 line 텍스트 파일에서 줄 끝에 줄 바꿈 (\n)가 포함되어 있다는 점이다.

str.strip()을 사용하여 제거하십시오.

with open(filelocation,"r") as fin: 
    s = set(sys.argv[2:]) 
    for host in s: 
     for line in fin: 
      if host==line.strip(): 
       print line 

Answer 3

첫 번째 문제 : 파일의 내용에서 공백과 개행 문자를 제거해야합니다. 두 번째 문제 : 첫 번째 문제가 해결되면 내부 루프가 첫 번째 외부 루프에 대해서만 실행된다는 것을 알 수 있습니다. 첫 번째 외부 루프 이후에 파일의 끝에 도달 했으므로 반복 할 항목이 없기 때문입니다 .

파일이 정말 큰 경우를 제외하고, 가장 좋은 방법은, 먼저에게 집합을 교차 얻기 위해 일련의 작업을 사용하는 것입니다 : 당신은 당신이 반전 할 것이다 큰 파일을 기대한다면

s1 = set(sys.argv[2:]) 
with open(filelocation) as fin: 
    s2 = set(filter(None, (line.strip() for line in fin))) 
print "\n".join(s1 & s2) 

을 루프 :이 파일을 소진하고, 또한 세트 조회로 더 효율적입니다 피한다

s = set(sys.argv[2:]) 
with open(filelocation) as fin: 
    for line in fin: 
     line = line.strip() 
     if line in s: 
      print line 

는 (현재의 알고리즘을 대신 O의 (N)) O (1)이다.