1

わかりました、最初のプログラムで問題が発生しました。2つの関数を見下ろすと、「cur」値が最小値と最大値の間にない場合、ターゲット変数を「0」に変更して、後でプログラムで使用できるようにする必要があります.. . しかし、それはそうではないようです. 「返す」の使い方が間違っていますか? 基本的に、関数は から に変わる必要があるtargetため1、0がri_fit0 に変わり、最終的なifものがトリガーされます...

ひどいコードで申し訳ありません。まだ始めたばかりです。

#Clearance calculator
#clearances are in clearances.txt

targets = open("clearances.txt", "r")
lines = targets.readlines()   #get target clearances from file

in_min_target = float(lines[2])     #minimum intake clearance
in_max_target = float(lines[4])     #maximum intake clearance

ex_min_target = float(lines[8])    #miminum exhaust clearances
ex_max_target = float(lines[10])   #maximum exhaust clearances
targets.close

target_intake = (in_min_target + in_max_target) / 2     #find the ideal intake
target_exhaust = (ex_min_target + ex_max_target) / 2    #find the ideal exhaust

print "Intake Min: ", in_min_target
print "Intake Max: ", in_max_target
print "Exhaust Min: ", ex_min_target
print "Exhaust Max: ", ex_max_target
print """Target intake: %r
Target Exhaust: %r""" % (target_intake, target_exhaust)

print""
print "Enter current RIGHT side Intake clearance"
cur_r_in = float(raw_input(">"))
print ""
print "Enter current RIGHT side Exhaust clearance"
cur_r_ex = float(raw_input(">"))
print ""
print "Enter current LEFT side Intake clearance"
cur_l_in = float(raw_input(">"))
print ""
print "Enter current LEFT side Exhaust clearance"
cur_l_ex = float(raw_input(">"))


target = 1

def in_range(min, max, cur, valve, target):  #figures if intake valves are correct
    if min <= cur <= max:
        print "%r is in range." % valve
        target=1

    else:
        print "%r is OUT OF RANGE." %valve
        target=0

    return target

def ex_range(min, max, cur, valve, target):   #figures if exhaust valves are correct
    if min <= cur <= max:
        print "%r is in range." % valve
        target=1

    else:
        print "%r is OUT OF RANGE." %valve
        target=0
    return target


ri_fit = 1
re_fit = 1        #Assumes all valves are right, until further notice...
li_fit = 1
le_fit = 1

valve = "Right Intake"
print in_range(in_min_target, in_max_target, cur_r_in, valve, target)
print target
if target == 0:
    ri_fit = 0


print ""

valve = "Right Exhaust"
print ex_range(ex_min_target, ex_max_target, cur_r_ex, valve, target)

print ""

valve = "Left Intake"
print in_range(in_min_target, in_max_target, cur_l_in, valve, target)

print ""

valve = "Left Exhaust"
print ex_range(ex_min_target, ex_max_target, cur_l_ex, valve, target)

print ri_fit

if ri_fit==0:
    print "Right intake is out of range."
    print "Enter current right intake shim size."
    ri_cur_shim = int(raw_input(">"))

if re_fit==0:
    print "Right exhaust is out of range."
    print "Enter current right exhaust shim size."
    re_cur_shim = int(raw_input(">"))

if li_fit==0:
    print "Right intake is out of range."
    print "Enter current right intake shim size."
    li_cur_shim = int(raw_input(">"))

if le_fit==0:
    print "Right exhaust is out of range."
    print "Enter current right exhaust shim size."
    le_cur_shim = int(raw_input(">"))
4

4 に答える 4

2

関数の 1 つの中で target に代入すると、そこにある target という名前のパラメーターの値が置き換えられ、グローバルなものには触れません。したがって、関数をグローバル ターゲットに割り当てたい場合、関数で新しいターゲットを定義することはできません。

ただし、関数を変更したくない場合は、return が値を返すという事実を使用できます (出力したので、これは理解しているようです)。それを印刷する代わりにターゲットに割り当てた場合、必要な場所で必要な値を取得でき、計算された値が最終的にどこに行くのかに関係なく、関数は機能します。

于 2012-12-28T02:12:04.583 に答える
1

return関数から値を返します。結果を変数に保存できます。

コードでは、パラメーターとして渡してからその値を設定しようとしてtargetいますが、整数は不変であるため、関数内から変更することはできませんtarget。

あなたの場合、私はあなたのコードを次のようなものに変更します：

def ex_range(min, max, cur, valve):  # You don't need `target` here
    if min <= cur <= max:
        print "%r is in range." % valve
        target=1

    else:
        print "%r is OUT OF RANGE." %valve
        target=0
    return target

target = ex_range(...)  # Store the results in a variable

さらに良いことに、ブール値を使用します。

def ex_range(min, max, cur, valve):  # You don't need `target` here
    if min <= cur <= max:
        print "%r is in range." % valve
        return True

    else:
        print "%r is OUT OF RANGE." %valve
        return False
于 2012-12-28T02:09:08.033 に答える
1

問題はtarget、参照としてではなく、値として渡していることにあると思います。関数内では、targetパラメーターを削除し、代わりに次のように記述します。

def ex_range(min, max, cur, valve):
    global target
    # ...
    # rest of your code

このようにして、インタープリターtargetは、ローカル インスタンスではなく、グローバル変数を設定しようとしていることを識別します。

それ以外の場合は、次のようなことができます。

def in_range(min, max, cur, valve):  #figures if intake valves are correct
    if min <= cur <= max:
        print "%r is in range." % valve
        return 1

    else:
        print "%r is OUT OF RANGE." %valve
        return 0

target = in_range(...) # fill in args
print target

この関連する質問を参照してください。

于 2012-12-28T02:15:07.377 に答える
1

()まず、 afterが必要だと思いますclose：

target.close()

2番目: in_rangeとex_rangeはまったく同じです!

第三に、とにかくそれは問題ないので、正しく実行されます:

in_range(1,10,7,'Right Intake',1)
in_range(1,10,17,'Right Intake',1)

出力：

>>> 'Right Intake' is in range.
>>> 'Right Intake' is OUT OF RANGE.

したがって、関数に渡す引数が正しくない可能性があります。ファイルから正しく読み取っているかどうかを確認してください。

最後に、,はPython 関数であるためmin、max引数として使用しないでください。built-in

于 2012-12-28T02:23:18.233 に答える