発生したエラーメッセージを直して配列をマージしたい

前提・実現したいこと

javaのプログラミングを勉強中です。
与えられた二つの配列を１つにマージするプログラムを作成したいのですが
以下のエラーメッセージが発生しました。

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

Exception in thread "main" java.lang.ArrayIndexOutOfBoundsException: 5
at 演習12.Practice12.merge(Practice12.java:25)
at 演習12.Practice12.main(Practice12.java:9)

java
ソースコード

public class Practice12 {
	public static void main(String[] args) {
		int[] data1 = {1, 2, 3, 4, 5};
		int[] data2 = {10, 9, 8, 7, 6};

		int[] result;
		result = merge(data1, data2);
		printArray(result);
	}

	static void printArray(int[] data) {
		for(int i=0; i<data.length; i++) {
			System.out.print(data[i]+ " ");
		}
		System.out.println("");
	}
	static int[] merge(int[] first, int[] last) {
		int[] margedata = new int[10];
		for(int i=0; i<first.length; i++) {
			margedata[i]  = first[i];
		}
		for(int i=first.length; i<first.length + last.length; i++) {
			margedata[i] = last[i];
		}
		return margedata;
	}
}

このエラーメッセージを直すにはどうすれば良いのでしょうか？

m.ts10806

2020/05/16 05:07

どういう結果を得たいのでしょう

退会済みユーザー

2020/05/16 05:11

下記のような実行結果を得たいです 1 2 3 4 5 10 9 8 7 6 10 9 8 7 6 1 2 3 4 5

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

回答2件

ベストアンサー

for(int i=first.length; i<first.length + last.length; i++) {

lastは0~4のindexを持つ配列なので、first.lengthから開始では最初の時点で超えてます。
つまり「lastにはindex番号5はない」という意味。

margedata におさめる配列番号は別途持つべきです。

java
1    static int[] merge(int[] first, int[] last) {
2        int index = 0;
3        int[] margedata = new int[10];
4        for(int i=0; i<first.length; i++) {
5            margedata[index]  = first[i];
6            index++;
7        }
8        for(int i=0; i<last.length; i++) {
9            margedata[index] = last[i];
10            index++;
11        }
12        return margedata;
13    }