コンマコード - つまらないものを自動化する

@pjzには非常に良い点がいくつかありましたので、レビューでそれらをスキップします。

1. 関数名と変数名はどちらもpythonの snake_case に書かれています。
2. エラーメッセージを表示する代わりに、適切な例外を発生させます。
3. 長さ1と2の入力にピリオドを追加しますが、それ以上の出力には追加しません。
4. Pythonリストスライスを使用すると、特別な場合を実際に削除できます。

これは2つの提案された改善、1つはより簡単、もう1つはpythonicです。また、例外が発生したことを検証するための短いテストスニペットも含めました。

def comma_code(input_list):
    if len(input_list) == 0:
        # Raise an exception rather than just printing the error
        raise ValueError('List cannot be empty')
    # Convert everything to string (could also raise an exception if not string)
    string_list = [str(i) for i in input_list]
    # Handle the trivial case
    if len(string_list) == 1:
        return string_list[0]

    # This could be even more pythonic, but it would lose its readability
    more_than_two_items = len(string_list) > 2
    first_part = ', '.join(string_list[:-2])
    optional_separator = ', ' * more_than_two_items
    last_part = ', and '.join(string_list[-2:])

    formatted_string = first_part + optional_separator + last_part
    return formatted_string     

def comma_code_pythonic(input_list):
    if len(input_list) == 0:
        raise ValueError('List cannot be empty')

    string_list = [str(i) for i in input_list]

    last_part = ', and '.join(string_list[-2:])
    first_part = string_list[:-2]

    formatted_string = ', '.join(first_part + [last_part])
    return formatted_string     


# Try to place non-global variables below the function
spam = ['apples', 'bananas', 'tofu', 'cats']

for i in range(5):
    try:
        print("comma_code:", comma_code(spam[:i]))
    except ValueError as e:
        print(repr(e))

for i in range(5):
    try:
        print("comma_code_pythonic:", comma_code_pythonic(spam[:i]))
    except ValueError as e:
        print(repr(e))

この答えは、あなたが作成したコードに関するものではありませんが、それをもう一度やろうとした場合の問題への取り組み方に関するものです。

Pythonスタイルガイドでは、 camelCase の命名規則よりも snake_case が推奨されているので、ここで使用します。

"リストの値が次のようになっているとします。spam = ['apples'、 'bananas'、 'tofu'、 'cats']

     

引数としてリスト値を取り、すべての項目をカンマとスペースで区切った文字列を返し、最後の項目の前に挿入する関数を作成します。

ここで実行する必要がある主な機能は何ですか？

簡単に言うと、コードの主な機能は、リスト内の値を取得してこれを文字列に変換することです。
これは、すぐにstring.join（）がこれに適した関数になることを示唆しているはずです。

結局のところ、文字列を好きな部分文字列と結合することができます。

', '.join(list_of_strings)

次のような変換が得られます。

['apples', 'bananas', 'tofu', 'cats'] -> 'apples, bananas, tofu, cats'

これでほぼすべての作業が完了しました。 （ 'join（）'は0と1の長さのリスト配列を正しく処理するので、必要はありません）
'と' を挿入するだけです。

A quick analysis of the problem shows that we only need the 'and ' when there are at least two items, so we write a modification to do just that.
We could just add 'and' in the penultimate location in the list, but we don't want to end up with ['x', 'y'] -> 'x, and, y' so the simple solution to this is to replace the final input in this case with 'and ' plus the input.

次のいずれかの行でこれを実行できます。

#python 2+:
list_of_strings[-1] = 'and %s' % list_of_strings[-1]
#python 2.6+:
list_of_strings[-1] = 'and {}'.format(list_of_strings[-1])
#python 3.6+:
list_of_strings[-1] = f'and {list_of_strings[-1]}'

これによって入力が変更されないようにするため（他の場所で再利用される可能性があります）、まずそのコピーを作成してください。簡単な方法は、元のものから新しいリストを作成することです。

list_of_strings = list(input_list_of_strings)

これらすべてをまとめると、結果としてかなり単純な関数が得られます。

def comma_code(input_list_of_strings):
    list_of_strings = list(input_list_of_strings)
    if len(list_of_strings) > 1:
        list_of_strings[-1] = f'and {list_of_strings[-1]}'
    return ', '.join(list_of_strings)

すでに与えられた他の良い答えを補強するために、これはすでに与えられたアイデアのいくつかを利用する短いバージョンと、さらにいくつかのアイデアです：

>>> spam = ['apples', 'bananas', 'tofu', 'cats']
>>> comma_code = lambda l: ", ".join(l[:-1] + ['and ' + l[-1]])
>>> comma_code(spam)
'apples, bananas, tofu, and cats'

lambda 節は、完全な def ... 構文を使用するのではなく、より小さな関数を定義するための簡単な方法です。

また、リストスライス、および文字列とリストの両方の連結も使用します。入力リストが文字列のみであることを知っている限り、これで問題ありませんが、特にリストの最後の要素がわからない場合は、 .format（）を使用する方法の1つが考えられます。同様に統合されています。

更新：2項目未満のリストではこれが失敗することに気付いたばかりです。それで、それはあなたがあなたの入力でなさなければならないもう一つの仮定です。上記の2つのケースを扱うわずかに大きい関数で上記をラップするのは比較的簡単です。

他の答えについていくつかの優れた指針があります、しかし私はそれらが最もpythonic解決策を欠いていると感じます：

def comma_code(words):
    *head, final = words

    if head:
        final = f'and {final}'

    return ', '.join(*head, final)

この解決法はどの読者にとっても明らかであるべきです、そして私には問題定義のエンコーディングとして読んでください：空でない単語のリストを与えられたら、それらをコンマで結合してください。複数ある場合は、最後の単語の前に単語「and」を追加します。

アンパック操作は空の入力に対してValueErrorを送出するので、このコードは空のリストを特別な場合に扱う必要はありません。これが望ましい動作であればこれをキャッチして空の文字列を返すことができます。あるいは、varadiac引数を使用してインターフェイスを明確にすることもできます。

def comma_code_args(first, *rest):
    *head, final = first, *rest

    if head:
        final = f'and {final}'

    return ', '.join(*head, final)

これは次のように呼ばれます。

>>> spam = ['apples', 'bananas', 'tofu', 'cats']

>>> comma_code(spam)
'apples, bananas, tofu, and cats'

>>> comma_code_args('apples', 'bananas', 'tofu', 'cats')
'apples, bananas, tofu, and cats'

>>> comma_code_args(*spam)
'apples, bananas, tofu, and cats'

>>> spam = ['apples', 'bananas', 'tofu', 'cats']
>>> print(', '.join(spam[:-2] + [spam[-2] + ' and ' + spam[-1]]))
apples, bananas, tofu and cats

tofu の後のコンマは存在しません。

ステップ：

1. ', '.join(...) creates a comma-separated string of the values contained in spam
2. spam[:-2] slices spam and creates a sublist from the first to the second-to-last item. Have a look at python's slice notation.
3. [spam[-2] + ' and ' + spam[-1]] creates a list containing: spam's second-to-last item ('tofu'), the word 'and' and spam's last item ('cats'). Creating a list is necessary because python doesn't know how to concatenate a list + another type of object. We can only concatenates lists.

Edit: Oxford Comma is a thing, apparently. Well, hooray english.

>>> spam = ['apples', 'bananas', 'tofu', 'cats']
>>> print(', '.join(spam[:-1] + [' and ' + spam[-1]]))
apples, bananas, tofu, and cats