質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

新規登録して質問してみよう
ただいま回答率
85.48%
マージ

複数のデータベースやファイル、プログラムなどを決まった手順や規則に従って一つに結合すること。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

ソート

複数のデータを、順序性に従って並べ替えること。 データ処理を行う際に頻繁に用いられ、多くのアルゴリズムが存在します。速度、容量、複雑さなどに違いがあり、高速性に特化したものにクイックソートがあります。

再帰

情報工学における再帰とは、プログラムのあるメソッドの処理上で自身のメソッドが再び呼び出されている処理の事をいいます。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

Q&A

解決済

1回答

1290閲覧

マージソートで再帰を繰り返し過ぎてしまう

langhtorn

総合スコア104

マージ

複数のデータベースやファイル、プログラムなどを決まった手順や規則に従って一つに結合すること。

アルゴリズム

アルゴリズムとは、定められた目的を達成するために、プログラムの理論的な動作を定義するものです。

ソート

複数のデータを、順序性に従って並べ替えること。 データ処理を行う際に頻繁に用いられ、多くのアルゴリズムが存在します。速度、容量、複雑さなどに違いがあり、高速性に特化したものにクイックソートがあります。

再帰

情報工学における再帰とは、プログラムのあるメソッドの処理上で自身のメソッドが再び呼び出されている処理の事をいいます。

Python

Pythonは、コードの読みやすさが特徴的なプログラミング言語の1つです。 強い型付け、動的型付けに対応しており、後方互換性がないバージョン2系とバージョン3系が使用されています。 商用製品の開発にも無料で使用でき、OSだけでなく仮想環境にも対応。Unicodeによる文字列操作をサポートしているため、日本語処理も標準で可能です。

0グッド

0クリップ

投稿2020/11/13 07:50

###実現したいこと
マージソートを完成させたいです。
再帰で分割、併合をしています。
###問題点
下記のようなエラーが出ます。
list index out of rangeこれは、配列の外を参照しているという意味だと認識しています。
どうしてそうなるのか考えるためリストを可視化するサイトで見てみました。すると、再帰を繰り返し過ぎて最終的にリストLanRanの中身が空っぽになっていました。だからmerge(Lan,Ran)をしようがなくなってしまったのだと思います。
再帰を繰り返し過ぎない方法はないでしょうか。いろいろなマージソートのサンプルコードを見ていますがよくわかりません。
アドバイスをお願いします!

7 2 8 5 3 2 1 Traceback (most recent call last): File "kadai07c.py", line 48, in <module> mdiv(a) File "kadai07c.py", line 18, in mdiv Lan=mdiv(L) File "kadai07c.py", line 20, in mdiv return merge(Lan,Ran) File "kadai07c.py", line 30, in merge if L[0]<R[0]: #二つの配列を見比べたときにL[0]が大きかったら IndexError: list index out of range

###コード

python

1#分割 2def mdiv(a): 3 n=len(a) 4 l=0 5 r=n 6 if n==1: #そのまま出力 7 return a 8 else: 9 mid=(l+r)//2 #分割した右端+左端 10 print(mid) 11 L=a[0:mid] #0~midまでの値を入れる 12 R=a[mid:n] #mid~最後までの値を入れる 13 #print("L=",L) 14 #print("R=",R) 15 #再帰的に分割 16 Lan=mdiv(L) 17 Ran=mdiv(R) 18 return merge(Lan,Ran) 19 20#併合 21def merge(L,R): 22 sort=[] 23 nL=len(L) #リストLのデータ数 24 nR=len(R) #リストRのデータ数 25 26 #比較 27 while (nL>0) and (nR>0): #リストLとリストRのデータ数が0になるまで繰り返す 28 if L[0]<R[0]: #二つの配列を見比べたときにL[0]が大きかったら 29 sin=L[0] 30 sort.append(sin) 31 del L[0] #L[0]を削除 32 else: ##二つの配列を見比べたときにR[0]が大きかったら 33 sin=R[0] 34 sort.append(sin) 35 del R[0] #R[0]を削除 36 37 return sort 38 39 40 41a=[] #リスト 42#リストに入力していく 43a=[int(i) for i in input().split()] 44n=len(a) #リストの数 45 46mdiv(a) 47 48#print(a)

気になる質問をクリップする

クリップした質問は、後からいつでもMYページで確認できます。

またクリップした質問に回答があった際、通知やメールを受け取ることができます。

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

guest

回答1

0

ベストアンサー

関数mergeについて、片方のリストが空の場合の考慮が欠けています。
空のリストには当然先頭要素などありませんから、範囲外アクセスを吐くでしょう。

Python

while (nL>0) and (nR>0): #リストLとリストRのデータ数が0になるまで繰り返す

リストLやRの要素をdelしても、nLやnRの値が勝手に書き換わることはありません。
ループ中で書き換えるようにするか、条件部で直接長さを判定して下さい。


なお、その部分を直してもまだ足りません。
試しに merge([], [1, 2, 3]) などを試してみると良いでしょう。

投稿2020/11/13 08:13

編集2020/11/13 08:19
LouiS0616

総合スコア35660

バッドをするには、ログインかつ

こちらの条件を満たす必要があります。

LouiS0616

2020/11/13 08:16

あれ、ちょっと外した気がします。修正しますので少々お待ちください。
LouiS0616

2020/11/13 08:19

修正しました。
langhtorn

2020/11/15 02:18

その部分の方法を変えたらできました。アドバイスありがとうございます!
guest

あなたの回答

tips

太字

斜体

打ち消し線

見出し

引用テキストの挿入

コードの挿入

リンクの挿入

リストの挿入

番号リストの挿入

表の挿入

水平線の挿入

プレビュー

15分調べてもわからないことは
teratailで質問しよう!

ただいまの回答率
85.48%

質問をまとめることで
思考を整理して素早く解決

テンプレート機能で
簡単に質問をまとめる

質問する

関連した質問