[Python3.x] 同じプログラムなのにエラーが出る場合とでない場合とがある

Question

問題

パスワード生成プログラムを作ったのですが、同じプログラムで同じ実行方法なのに、ちゃんと動作するときとしないときとがあります。

該当のソースコード

from itertools import chain

import sys
import re
import random

# ASCIIコード
abc = list(range(97, 123)) # アルファベット小文字
ABC = list(range(65, 91)) # アルファベット大文字
numbers = list(range(48, 58)) # 数字
sign = list(chain(range(33, 48), range(58, 65), range(91, 97), range(123, 127))) # 記号

chars = [abc, ABC, numbers, sign]

def create_password():
    u = list()
    uc = list()
    c = sys.argv[1]
    l = sys.argv[2]
    for i in c:
        u += chars[int(i)-1]
    for i in range(0, int(l)):
        uc.append( chr(u[random.randint(0, len(u))]) )

    print("".join(uc))

create_password()

# プログラムの出来には目をつぶってください????

発生している問題・エラーメッセージ

# 失敗
MacBookPro:Python ***$ python3 python.py 1234 64
Traceback (most recent call last):
  File "python.py", line 28, in <module>
    create_password()
  File "python.py", line 24, in create_password
    uc.append( chr(u[random.randint(0, len(u))]) )
IndexError: list index out of range

# 成功
MacBookPro:Python ***$ python3 python.py 1234 64
WUqanVxsuq1PBV|bIELU_[5vo[Uxqc0>6Q#w/L|%jM|XXb_'fF"/Pq-jBz-1f5SJ

# 成功
MacBookPro:Python ***$ python3 python.py 1234 64
Z*znue`u;&5^n*0abLu>*]*<,`%rDz.B],Ji9B:c*Ru?U}Rg%4!LO1*@(wO=K1y&

# 失敗
MacBookPro:Python ***$ python3 python.py 1234 64
Traceback (most recent call last):
  File "python.py", line 28, in <module>
    create_password()
  File "python.py", line 24, in create_password
    uc.append( chr(u[random.randint(0, len(u))]) )
IndexError: list index out of range

###質問
IndexErrorが発生する理由が全くわかりません。

上のように、実行が成功するときとしない時との違いはなんですか?
どうすればつねに正しく実行されるようになるのでしょうか。

補足

実行コマンド python3 1234 64の1234はパスワードに使用する文字の種類を指定し、64はパスワードの長さを指定しています。

pythonのバージョンは 3.7.4 で 実行環境は MacOS Mojave 10.14.6 の付属ターミナルです。

Answer 1

random.randint(a, b)¶
a <= N <= b であるようなランダムな整数 N を返します。

random --- 擬似乱数を生成する — Python 3.7.4 ドキュメント

ということで、ちょうどlen(u)が出てきたときIndexErrorになります。

randintのかわりにrandrangeを用いるのが適切でしょう。

Comments

[退会済みユーザー]

なるほど！
まったく気がつきませんでした。
ありがとうございます。