Python関数実装エラー

Question

前提・実現したいこと

Python3.7.4で京都大学のテキスト「プログラミング演習Python2019」を独習しています。
テキストのURL：
https://repository.kulib.kyoto-u.ac.jp/dspace/bitstream/2433/245698/1/Version2020_02_13_01.pdf

p.70演習25で関数を使った処理のカプセル化を試みていますが、うまく実装できません。関数は次の2つです。
get_positive_numeral()　正の数値を得る
square_root()　引数の平方根を求める

それぞれの関数の内容、及び、関数にしないプログラムでは作動が確認できておりますが、2つの関数を使って処理をカプセル化しようとするとうまく作動しません。

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

正の数値（例えば2）を入力しても"正の数値を入力してください"と表示され、平方根の計算が実行されません。

入力値が2であれば、次のような結果がShellに表示されるはずです。
1.5 1.4166666666666665 1.3333333333333333
1.4166666666666665 1.4142156862745097 1.411764705882353
1.4142156862745097 1.4142135623746899 1.41421143847487
1.4142135623746899 1.414213562373095 1.4142135623715002
結果は 1.414213562373095

該当のソースコード

Python3.7.4

ソースコード

def get_positive_numeral():
    '正の数値を得る'
    while True:
        x=input("正の数値を入力してください ")
        if(x=="end"):
            exit()
        try:
            x=float(x)
        except ValueError:
            print(x, "は数値に変換できません")
            continue
        except:
            print("予期していないエラーです")
            exit()
        if(x<=0):
            print(x,"は正の数値ではありません")
            continue
    return x
    
def square_root():
    '引数xの平方根を求める'
    rnew=x
    diff = rnew-x/rnew
    diff=abs(diff)
    while(diff>1.0E-6):
        r1=rnew
        r2=x/r1
        rnew=(r1+r2)/2
        print(r1,rnew,r2)
        diff=r1-r2
        diff=abs(diff)
    return rnew

while True:
    x=get_positive_numeral()
    r = square_root(x)
    print("結果は ",r)

試したこと

関数square_root()のみの実装では正確に作動しましたが、関数get_positive_numeral()を組み合わせるとうまくいきません。
以下、関数square_root()のみ実装したコードです。

while True:
x=input("正の数値を入力してください ")
if(x=="end"):
exit()
try:
x=float(x)
except ValueError:
print(x, "は数値に変換できません")
continue
except:
print("予期していないエラーです")
exit()
if(x<=0):
print(x,"は正の数値ではありません")
continue

def square_root(x):
    '引数ｘの平方根を求める'
    rnew=x
    diff = rnew-x/rnew
    diff=abs(diff)
    while(diff>1.0E-6):
        r1=rnew
        r2=x/r1
        rnew=(r1+r2)/2
        print(r1,rnew,r2)
        diff=r1-r2
        diff=abs(diff)
    return rnew
r = square_root(x)
print("結果は ",r)

ここに問題に対して試したことを記載してください。

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

Windows 10 Home
Version 1903
ここにより詳細な情報を記載してください。

Comments

[meg_]

・リンクは「リンクの挿入」で記入してください。
・デバックはしてみましたか？

Answer 1

2ヵ所修正したら動きました。※結果が正しいかは確認していません。

def get_positive_numeral():
    '正の数値を得る'
    while True:
        x=input("正の数値を入力してください ")
        if(x=="end"):
            exit()
        try:
            x=float(x)
        except ValueError:
            print(x, "は数値に変換できません")
            continue
        except:
            print("予期していないエラーです")
            exit()
        if(x<=0):
            print(x,"は正の数値ではありません")
            continue
        return x ############## インデントを修正 ##############

def square_root(x): ############## 引数を追加 ##############
    '引数xの平方根を求める'
    rnew=x
    diff = rnew-x/rnew
    diff=abs(diff)
    while(diff>1.0E-6):
        r1=rnew
        r2=x/r1
        rnew=(r1+r2)/2
        print(r1,rnew,r2)
        diff=r1-r2
        diff=abs(diff)
    return rnew

while True:
    x=get_positive_numeral()
    r = square_root(x)
    print("結果は ",r)

Comments

[arron]

初めての質問で作法もよくわからないなか、丁寧にご回答いただきありがとうございました。
おかげさまでプログラムがワークしました！
returnのインデントと関数定義の引数の入力だけで結果が変わってしまうのですね。。

[meg_]

Pythonはインデントが意味をもつのでエラーが出た際には確認しましょう。