Python string.replace () не заменяет символы - PullRequest
13 голосов
/ 19 августа 2010

Некоторая справочная информация: у нас есть древняя веб-система баз документов, где я работаю, почти полностью состоящая из документов MS Office с «обычными» расширениями (.doc, .xls, .ppt).Все они названы на основе своего рода произвольного идентификационного номера (например, 1245.doc).Мы переходим на SharePoint, и мне нужно переименовать все эти файлы и отсортировать их по папкам.У меня есть CSV-файл со всеми видами информации (например, какой идентификационный номер соответствует названию какого документа), поэтому я использую его для переименования этих файлов.Я написал короткий сценарий Python, который переименовывает заголовок идентификатора.

Однако некоторые заголовки документов содержат косую черту и другие, возможно, плохие символы в заголовке файла, поэтому я хочузамените их подчеркиванием:

bad_characters = ["/", "\\", ":", "(", ")", "<", ">", "|", "?", "*"]
for letter in bad_characters:
    filename = line[2].replace(letter, "_")
    foldername = line[5].replace(letter, "_")
  • Пример line[2]: "Бла-бла-скучный - встреча 2/19 / 2008.doc"
  • Пример line[5]: "Деловые встречи 2/2008 "

Когда я добавляю print letter внутри цикла for, он напечатает букву, которую он должен заменить, но фактически не заменит этот символ наподчеркивание, как я хочу.

Есть ли что-то, что я делаю здесь неправильно?

Ответы [ 5 ]

25 голосов
/ 19 августа 2010

Это потому, что filename и foldername отбрасываются с каждой итерацией цикла. Метод .replace() возвращает строку, но вы нигде не сохраняете результат.

Вы должны использовать:

filename = line[2]
foldername = line[5]

for letter in bad_characters:
    filename = filename.replace(letter, "_")
    foldername = foldername.replace(letter, "_")

Но я бы сделал это с помощью регулярных выражений. Это чище и (скорее всего) быстрее:

p = re.compile('[/:()<>|?*]|(\\\)')
filename = p.sub('_', line[2])
folder = p.sub('_', line[5])
6 голосов
/ 19 августа 2010

Вы переназначаете переменные filename и foldername на каждой итерации цикла.По сути, заменяется только *.

4 голосов
/ 19 августа 2010

Вы должны посмотреть на метод строки Python translate() http://docs.python.org/library/string.html#string.translate с http://docs.python.org/library/string.html#string.maketrans

Редактирование этого, чтобы добавить пример согласно предложению комментария ниже:
import string
toreplace=''.join(["/", "\\", ":", "(", ")", "<", ">", "|", "?", "*"]) 
underscore=''.join( ['_'] * len(toreplace))
transtable = string.maketrans(toreplace,underscore)
filename = filename.translate(transtable)
foldername = foldername.translate(transtable)

Можно упростить, сделав для замены что-то вроде '/ \ :,' и т. Д. Я просто использовал то, что было дано выше

3 голосов
/ 19 августа 2010

Вы начинаете заново с базовой строки вместо сохранения замененного результата, таким образом, вы получаете эквивалент

filename = line[2].replace('*', '_')
foldername = line[5].replace('*', '_')

Попробуйте следующее

bad_characters = ["/", "\\", ":", "(", ")", "<", ">", "|", "?", "*"]
filename = line[2]
foldername = line[5]
for letter in bad_characters:
    filename = filename.replace(letter, "_")
    foldername = foldername.replace(letter, "_")
1 голос
/ 18 августа 2011

Следует использовать string.replace (str, fromStr, toStr)

bad_characters = ["/", "\\", ":", "(", ")", "<", ">", "|", "?", "*"]
for letter in bad_characters:
    filename = string.replace(line[2], letter, "_")
    foldername = string.replace(line[5], letter, "_")
...