Python関数実装エラー

前提・実現したいこと

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
ここにより詳細な情報を記載してください。

meg_

2020/03/08 12:03 編集

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

行動規範の内容に同意します

回答1件

ベストアンサー

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

Python
1def get_positive_numeral():
2    '正の数値を得る'
3    while True:
4        x=input("正の数値を入力してください ")
5        if(x=="end"):
6            exit()
7        try:
8            x=float(x)
9        except ValueError:
10            print(x, "は数値に変換できません")
11            continue
12        except:
13            print("予期していないエラーです")
14            exit()
15        if(x<=0):
16            print(x,"は正の数値ではありません")
17            continue
18        return x ############## インデントを修正 ##############
19
20def square_root(x): ############## 引数を追加 ##############
21    '引数xの平方根を求める'
22    rnew=x
23    diff = rnew-x/rnew
24    diff=abs(diff)
25    while(diff>1.0E-6):
26        r1=rnew
27        r2=x/r1
28        rnew=(r1+r2)/2
29        print(r1,rnew,r2)
30        diff=r1-r2
31        diff=abs(diff)
32    return rnew
33
34while True:
35    x=get_positive_numeral()
36    r = square_root(x)
37    print("結果は ",r)
38