違うディレクトリから呼び出すとき

Question

前提

曲線をランダムな制御点で描くプログラムを作るために、ディレクトリを分けてきれいにしようとしています。

一つのファイルで実行すると動作しますので、コードに問題はないはずです。

ランダムな曲線の生成を繰り返し、それを画像として保存するプログラムを作りたいと思い
そのためにファイルを分けた方が見やすいし、構成しやすいかなと考え構築しています。

実現したいこと

bezier.pyで定義した関数の引数？をtrain.pyで呼び出せるようにしたい。

該当のソースコード

pythonコード

import numpy as np
import math

class bezier_class:
    # Bernstein多項式を計算する関数
    def bernstein(n, t):
        B = []
        for k in range(n + 1):
            # 二項係数を計算してからBernstein多項式を計算
            nCk = math.factorial(n) / (
                    math.factorial(k) * math.factorial(n - k))
            B.append(nCk * t ** k * (1 - t) ** (n - k))
            # print(nCk, k, n - k)
        return B

    # ベジェ曲線を描く関数
    def bezier_curve(Q):
        self = len(Q) - 1
        dt = 0.03333333
        t = np.arange(0, 1, dt)
        B = Q.bernstein(self, t)
        px = 0
        py = 0
        for i in range(len(Q)):
            px += np.dot(B[i], Q[i][0])
            py += np.dot(B[i], Q[i][1])
        return px, py, t

from bezier import bezier_class
import numpy as np
from matplotlib import pyplot as plt
import random

def control_point():
    q1 = [0, 0]
    q2 = [random.uniform(0, 1), random.uniform(0, 1)]
   q3 = [random.uniform(0, 1), random.uniform(0, 1)]
   q4 = [random.uniform(0, 1), random.uniform(0, 1)]
   q5 = [random.uniform(0, 1), random.uniform(0, 1)]
   q6 = [1., 1.]
   Q = [q1, q2, q3, q4, q5, q6]     ＃←ここのQが定義されてない判定？になる（暗い文字になる）

   # ベジェ曲線を描く関数を実行


px, py, t = bezier_curve(Q)　　＃Qに赤波線

＃実行すると「Qが見つかりません」と出ます。。。

試したこと

以前別の質問をした際にclassのやり方を教えてもらったのでやってみましたがうまくいきませんでした。
別のファイルから呼び出すとなると、コードを書き直す必要があるのでしょうか

補足情報（FW/ツールのバージョンなど）

Comments

[TakaiY]

インデントが微妙にずれていますが、これはソースコードのままでしょうか。

[Tyutohannpa_]

q1, q2, ・・・のところに関しては移すときの、ミスと思われます。申し訳ありません
それ以外に関しては、そのままです

[8524ba23]

いまの実際のコードでは以下のいずれかではないでしょうか？
・「file1.py」→「bezeir.py」
・「from bezier import ～」→「from file1 import ～」

なお「px, py, t = bezier_curve(Q)」の行のインデントも提示のとおりであれば
たしかにQは定義されていないので、ファイルを分ける、分けないに関係なく発生します。

[Tyutohannpa_]

分かりにくくてすみません、修正します。
現物ではなく、例えて書いてしまいました。
file1.pyがbezier.pyでfile2.pyはtrain.pyです。

Answer 1

提示のようなケースであればbezeir.pyでは単純にやりたい処理の関数を定義し
bezeir.py

def bezier_curve(Q):
    # Qを用いて何らかの処理をおこなう
    return 4,5,6 # その結果を返す

# 他の関数など
def hoge():
    return 'fuga'

以下のような感じで呼び出せばよいかと思います。

import bezier as bz # 書きやすいようにbzという短い別名をつける

Q = [1,2,3]
px, py, t = bz.bezier_curve(Q)
print(px, py, t)# 4 5 6
print(bz.hoge())# fuga

Answer 2

ぱっと見たところのおかしなところをいくつか指摘します。
これだけでは解決しませんが。

(1) from bezier import bezier_class となっていますが、bezier_classが定義されているのは、file1.pyファイルではないのでしょうか。 これではインポートできないはずです。
違うのであれば、質問を修正してください。

(2) 変数Qは関数cortrol_pointのローカル変数ですので、関数外からはアクセスできません。
これはファイルの分割とは関係ありません。
使うのであれば、control_point関数がQを返すようにして、使うまえに呼び出すようにすればいいでしょう。

(3) bezier_curveメソッドは、 bezier_classのメソッドなので、この形で呼び出すことはできません。
「以前別の質問をした際にclassのやり方を教えてもらったのでやってみました」ということでクラスに入れているのだと思いますが、クラスの基本的な使いかたができていないようです。
もともとは1つのファイルでクラスにしていなかったのでしょうから、まずは、file1.py の中のクラスは無くして関数定義のみにし、file2.pyからその関数を呼ぶようにするのがいいでしょう。

import file1

px, py, t = file1.bezier_curve(Q)　

のような使いかたになります。