引数が定義されるようにしたいです。

Question

python3.7でとあるアルゴリズムを作成しているのですが、実行すると下記のエラーメッセージが出てしまいます。

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

Failed test #1. Runtime error
Traceback (most recent call last):
  File "jailed_code", line 19, in <module>
    print(random_motifs(Dna, k, t))
NameError: name 'Dna' is not defined

該当のソースコード

import random

def random_motifs(Dna, k, t):
    Dna = ["TTACCTTAAC", "GATGTCTGTC", "ACGGCGTTAG", "CCCTAACGAG", "CGTCAGAGGT"]
    k = 3
    t = len(Dna)
    random_sample = []
    for i in range(t):
        r = random.randint(0, 6)
        random_sample.append(Dna[i][r:r+k])
    
    return random_sample
print(random_motifs(Dna, k, t))

疑問

該当のソースコードではDnaが定義できない理由は何でしょうか。

Answer 1

「引数とは何か」が理解できていないので、そこから復習しましょう。

Comments

[UT100]

分かりました。
回答ありがとうございました。

Answer 2

NameError: name 'Dna' is not defined

これは　
"print(random_motifs(Dna, k, t))　の行で Dna をつかっているけれど、Dna は未定義ですよ"
というエラーです。

Dna は 関数宣言部 random_motifs(Dna, k, t) で使っています。
でも, これは関数の引数宣言であり、変数定義ではありません。

関数宣言部 random_motifs(Dna, k, t)　は、 random_motifs を呼び出すときは 3 つの引数が必要といっているだけです。
呼び出すときに、Dna, k, t を使うという意味ではありません。

呼び出すときは、random_motifs には３つの引数を渡しさえすれば、変数名は何でも良いです。
変数でなくて 3 とか "abc" みたいな具体的な値をそのまま渡してもよいのです。
random_motifs の定義部で処理内容を書いていく際に, 受け取った値に名前 (大抵は人間にとって意味がある名前)をつけて
その名前を使うようにしているのです。

windows の bat ファイルや linux の shell スクリプトでは、
受け取る引数の名前は $0, $1, ...という名前で受けとるように決められていますね。

Answer 3

1.関数の中で宣言した変数は、その関数の処理が終了すると全て破棄されます。
2.global宣言をすれば、関数の外で宣言したと同じ扱いになり変数が破棄されませんが、関数を呼び出してから変数が宣言されます。なのでいずれにしろこのコードでは、関数を呼び出す前にDna k t を使おうとしているので動きません。

またこのコードの引数は全て不要であるように思います。

import random

def random_motifs():
    global Dna
    Dna = ["TTACCTTAAC", "GATGTCTGTC", "ACGGCGTTAG", "CCCTAACGAG", "CGTCAGAGGT"]
    k = 3
    t = len(Dna)
    random_sample = []
    for i in range(t):
        r = random.randint(0, 6)
        random_sample.append(Dna[i][r:r+k])

    return random_sample
print(random_motifs())

Answer 4

print文は関数の外にあるので、関数で定義してる変数は見えません

Comments

[UT100]

ありがとうございました。