Подавление обработки строки как iterable

Ответ 1

Нет никаких способов сделать это автоматически, к сожалению. Решение, которое вы предлагаете (подкласс str, который не является итерируемым), страдает от той же проблемы, что и isinstance()... а именно, вы должны помнить, что используете его везде, где используете строку, потому что нет возможности сделать Python используйте его вместо родного класса. И, конечно же, вы не можете обезопасить встроенные объекты.

Я мог бы предположить, что если вы обнаружите, что пишете функцию, которая принимает либо итерируемый контейнер, либо строку, возможно, что-то не так с вашим дизайном. Иногда вы не можете избежать этого.

На мой взгляд, наименее навязчивое дело - поставить чек в функцию и называть это, когда вы попадаете в цикл. Это, по крайней мере, приводит к изменению поведения, где вы, скорее всего, его увидите: в инструкции for, не скрытой где-то в классе.

def iterate_no_strings(item):
    if issubclass(item, str):   # issubclass(item, basestring) for Py 2.x
        return iter([item])
    else:
        return iter(item)

for thing in iterate_no_strings(things):
    # do something...

Ответ 2

Чтобы расширить и сделать ответ из этого:

Нет, вы не должны этого делать.

• Он изменяет функциональность, которую люди ожидают от строк.
• Это означает дополнительные накладные расходы во всей вашей программе.
• В основном это не нужно.
• Типы проверок очень несовместимы.

Вы можете это сделать, и методы, которые вы указали, вероятно, являются лучшими способами ( для записи, я думаю, что более подходящий вариант. Если вам нужно это сделать, см. @kindall метод), но это просто не стоит делать, и это не очень pythonic. Избегайте ошибок в первую очередь. В вашем примере вы можете спросить себя, является ли эта проблема более ясной в ваших аргументах, и могут ли быть более эффективными решения или аргументы splat.

Например: измените порядок.

def set_fields(record, value, *fields):
  for f in fields:
    record[f] = value

set_fields(weapon1, 'Dagger', *('Name', 'ShortName')) #If you had a tuple you wanted to use.
set_fields(weapon2, 'Katana', 'Name')
set_fields(weapon3, 'Wand', 'Name')

Например: Именованные аргументы.

def set_fields(record, fields, value):
  for f in fields:
    record[f] = value

set_fields(record=weapon1, fields=('Name', 'ShortName'), value='Dagger')
set_fields(record=weapon2, fields=('Name'), value='Katana')
set_fields(record=weapon3, fields='Name', value='Wand') #I find this easier to spot.

Если вы действительно хотите, чтобы порядок был таким же, но не думайте, что идея с именованными аргументами достаточно ясна, то как сделать каждую запись диктоподобным элементом вместо dict (если это еще не так) и иметь

class Record:
    ...
    def set_fields(self, *fields, value):
        for f in fileds:
            self[f] = value

weapon1.set_fields("Name", "ShortName", value="Dagger")

Единственная проблема здесь - это введенный класс и тот факт, что параметр значения должен выполняться с ключевым словом, хотя он сохраняет четкость.

В качестве альтернативы, если вы используете Python 3, у вас всегда есть возможность использовать расширенную распаковку:

def set_fields(*args):
      record, *fields, value = args
      for f in fields:
        record[f] = value

set_fields(weapon1, 'Name', 'ShortName', 'Dagger')
set_fields(weapon2, 'Name', 'Katana')
set_fields(weapon3, 'Name', 'Wand')

Или, для моего последнего примера:

class Record:
    ...
    def set_fields(self, *args):
        *fields, value = args
        for f in fileds:
            self[f] = value

weapon1.set_fields("Name", "ShortName", "Dagger")

Однако при чтении вызовов функций они оставляют некоторые странности из-за того, что обычно предполагается, что аргументы не будут обрабатываться таким образом.

Ответ 3

Проверка типов в этом случае не является беспризорной или плохой. Просто выполните:

if isinstance(var, (str, bytes)):
    var = [var]

В начале вызова. Или, если вы хотите обучить вызывающего:

if isinstance(var, (str, bytes)):
    raise TypeError("Var should be an iterable, not str or bytes")

Ответ 4

Что вы думаете о создании строки без итераций?

class non_iter_str(str):
    def __iter__(self):
        yield self

>>> my_str = non_iter_str('stackoverflow')
>>> my_str
'stackoverflow'
>>> my_str[5:]
'overflow'
>>> for s in my_str:
...   print s
... 
stackoverflow

Ответ 5

Вместо того чтобы пытаться сделать ваши строки неистребимыми, переключите способ поиска проблемы: один из ваших параметров либо итерабелен, либо...

• строка
• ИНТ
• пользовательский класс
• и др.

Когда вы пишете свою функцию, первое, что вы делаете, это проверка ваших параметров, правильно?

def set_fields(record, fields, value):
    if isinstance(fields, str):
        fields = (fields, )  # tuple-ize it!
    for f in fields:
        record[f] = value

Это послужит вам хорошо, поскольку вы будете иметь дело с другими функциями и параметрами, которые могут быть как сингулярными, так и множественными.