pythonについて

Question

def all_the_same(elements):
    for x in range(1,len(elements)):
        if elements[x-1] == elements[x]:
            return True
        elif len(elements) <= 1:
            return True
        else:
            return False

1.print(all_the_same([1,1,1,]))
2.print(all_the_same([]))
3.print(all_the_same([1,1,2,]))
4.print(all_the_same(['a','a']))
5.print(all_the_same(['a','b']))

これは引数の中に配列を与え、違う要素が入っているとFalse
全て同じか、または何も入っていないか、１つだけ要素が入っているとTrue
のように返す関数なのですが、2がNoneとなり３がTrueとなってしまいました。
間違っているところを教えてください。
出来れば、どのようにすれば効率がようなるかコメントなどを書いてくださると嬉しいです。

Answer 1

    for x in range(1,len(elements)):
        if elements[x-1] == elements[x]:
            return True

これですと、連続して同じ値が先頭にあればTrueになってしまいますね。

また、空のListを渡した場合は

for x in range(1,len(elements)):

が1週もしないのでNoneが戻ってしまいます。

いろいろ書き方はあるとおもいますが、

def all_same(elements):
   return len(set(elements)) in (0, 1)

とかはどうでしょうか？set([])では重複をまとめますので、まとめた結果が0(空の場合)か1(1種類)であれなTrueとしています。

Answer 2

間違っているところと言うと、3行目以降が全然駄目ですね。

要素それぞれについて、それ以降末尾までに異なる値があれば、途中でreturn False。
全要素について実行し終わったら、return True。
でしょうか。