С точки зрения эффективности, вы не победите
s.translate(None, string.punctuation)
Для более поздних версий Python используйте следующий код:
s.translate(str.maketrans('', '', string.punctuation))
Он выполняет необработанные строковые операции в C с помощью таблицы поиска - не так много, что побьет это, кроме написания собственного кода на C.
Если скорость не беспокоит, другой вариант:
exclude = set(string.punctuation)
s = ''.join(ch for ch in s if ch not in exclude)
Это быстрее, чем s.replace с каждым символом, но не будет работать так же хорошо, как подходы с не-чистым питоном, такие как регулярные выражения или string.translate, как вы можете видеть из приведенных ниже моментов времени. Для такого типа проблем выполнение на минимально возможном уровне окупается.
Временной код:
import re, string, timeit
s = "string. With. Punctuation"
exclude = set(string.punctuation)
table = string.maketrans("","")
regex = re.compile('[%s]' % re.escape(string.punctuation))
def test_set(s):
return ''.join(ch for ch in s if ch not in exclude)
def test_re(s): # From Vinko's solution, with fix.
return regex.sub('', s)
def test_trans(s):
return s.translate(table, string.punctuation)
def test_repl(s): # From S.Lott's solution
for c in string.punctuation:
s=s.replace(c,"")
return s
print "sets :",timeit.Timer('f(s)', 'from __main__ import s,test_set as f').timeit(1000000)
print "regex :",timeit.Timer('f(s)', 'from __main__ import s,test_re as f').timeit(1000000)
print "translate :",timeit.Timer('f(s)', 'from __main__ import s,test_trans as f').timeit(1000000)
print "replace :",timeit.Timer('f(s)', 'from __main__ import s,test_repl as f').timeit(1000000)
Это дает следующие результаты:
sets : 19.8566138744
regex : 6.86155414581
translate : 2.12455511093
replace : 28.4436721802