2つのソートされた配列をソートされた配列にマージする方法は?[閉まっている]


160

これはインタビューで私に尋ねられました、そしてこれは私が提供した解決策です:

public static int[] merge(int[] a, int[] b) {

    int[] answer = new int[a.length + b.length];
    int i = 0, j = 0, k = 0;
    while (i < a.length && j < b.length)
    {
        if (a[i] < b[j])
        {
            answer[k] = a[i];
            i++;
        }
        else
        {
            answer[k] = b[j];
            j++;
        }
        k++;
    }

    while (i < a.length)
    {
        answer[k] = a[i];
        i++;
        k++;
    }

    while (j < b.length)
    {
        answer[k] = b[j];
        j++;
        k++;
    }

    return answer;
}

これを行うより効率的な方法はありますか?

編集:長さのメソッドを修正しました。


30
私にはかなり良い答えのように見えます。この問題はせいぜいO(n)の複雑さであり、あなたの答えはそれを達成します。それ以外はマイクロ最適化になります。
ドリューホール

3
よく出来ました!これは本質的にマージソートの一部です:(テープまたはディスクからの)2つのソートされたストリームを別のソートされたストリームにマージします。
Vladimir Dyuzhev、

9
あなたは仕事を持っていますか?
Shai 2013

5
また、三項演算子を使用することもできます while (i < a.length && j < b.length) answer[k++] = a[i] < b[j] ? a[i++] : b[j++]; 。Java言語仕様:条件演算子?:
アントンDozortsev 2014年

1
コメントするのを忘れました!!!
LiziPizi

回答:


33

マイナーな改善ですが、メインループの後System.arraycopy、もう一方の配列の最後に到達したときに、いずれかの入力配列の末尾をコピーするために使用できます。O(n)ただし、ソリューションのパフォーマンス特性は変わりません。


109
public static int[] merge(int[] a, int[] b) {

    int[] answer = new int[a.length + b.length];
    int i = 0, j = 0, k = 0;

    while (i < a.length && j < b.length)  
       answer[k++] = a[i] < b[j] ? a[i++] :  b[j++];

    while (i < a.length)  
        answer[k++] = a[i++];

    while (j < b.length)    
        answer[k++] = b[j++];

    return answer;
}

もう少しコンパクトですがまったく同じです!


これが範囲外の例外のインデックスを引き起こしたと言った人に、どの入力を使用していますか?それは私にとってはすべての場合で動作します。
Mike Saull 2013年

1
forループを使用して、ループ変数とループ制御を宣言する行をマージします。二重の空白行は控えめに使用してください-対称的な「テールコピー」の間では呼び出されないように見えます。
greybeard 2018

58

これほどクールで効率的でコンパクトな実装について誰も言及していなかったことに驚いています。

public static int[] merge(int[] a, int[] b) {
    int[] answer = new int[a.length + b.length];
    int i = a.length - 1, j = b.length - 1, k = answer.length;

    while (k > 0)
        answer[--k] =
                (j < 0 || (i >= 0 && a[i] >= b[j])) ? a[i--] : b[j--];
    return answer;
}

興味のポイント

  1. 他のO(n)アルゴリズムと同じか少ない数の演算を実行しますが、文字通り、1つのwhileループ内の1つのステートメントで実行されます。
  2. 2つの配列がほぼ同じサイズの場合、O(n)の定数は同じです。ただし、配列が本当に不均衡である場合はSystem.arraycopy、内部で単一のx86アセンブリ命令を使用してこれを実行できるため、のバージョンが優先されます。
  3. a[i] >= b[j]代わりに注意してくださいa[i] > b[j]。これにより、aとbの要素が等しい場合に定義される「安定性」が保証されます。

これは本当に素晴らしいアプローチです。Swift langでMergeソートアルゴリズムの良いベンチマークを取得するのに苦労しました。これを変換することで、必要なものを
手に入れる

whileループの(j <0)のポイントは何ですか?ところで、+ 1、これは本当にクールです!共有いただきありがとうございます。
ヘンガメ2015

2
@Hengamehの場合j < 0b既に使い尽くされているため、残りのa要素をanswer 配列に追加し続けます
Natan Streppel

6
あまりにも「賢い」と私の心の中で読むのは難しい。特にこのコードではパフォーマンスの改善のほとんどを達成していないため、コードを読みやすくした方がいいと思います。
ケビンM

1
通知のプラスポイント、およびa [i]> b [j]ではなくa [i]> = b [j]。これにより「安定性」が保証されます
Yan Khonski、2017年

16

可能な改善は、マイクロ最適化であり、全体的なアルゴリズムは正しいです。


aが大きく、bが小さい場合、このアルゴリズムは間違っています。
ジャック

7
間違いではありませんが、効率的ではありません。
ジャック

@jackは、n個の項目の配列を生成するときに、O(n)よりも速く実行できますか?
2014年

@will System.arrayCopy()は、CPUに最適化されたmemcpy呼び出しを使用するので、非常に高速です。したがって、チャンクをコピーしてパフォーマンスを向上させる余地があります。境界を二分探索するスコープもあります。
スリム

特に、ソートされた性質を使用してほとんどのエントリをスキップし、それらを決して比較できない場合。あなたは実際にO(n)を倒すことができます。
18

10

このソリューションは、System.arrayCopyを使用して残りの配列要素をコピーすることを除いて、他の投稿と非常に似ています。

private static int[] sortedArrayMerge(int a[], int b[]) {
    int result[] = new int[a.length +b.length];
    int i =0; int j = 0;int k = 0;
    while(i<a.length && j <b.length) {
        if(a[i]<b[j]) {
            result[k++] = a[i];
            i++;
        } else {
            result[k++] = b[j];
            j++;
        }
    }
    System.arraycopy(a, i, result, k, (a.length -i));
    System.arraycopy(b, j, result, k, (b.length -j));
    return result;
}

7

こちらが更新された機能です。重複を削除しますが、うまくいけば誰かがこれを使用できると思います:

public static long[] merge2SortedAndRemoveDublicates(long[] a, long[] b) {
    long[] answer = new long[a.length + b.length];
    int i = 0, j = 0, k = 0;
    long tmp;
    while (i < a.length && j < b.length) {
        tmp = a[i] < b[j] ? a[i++] : b[j++];
        for ( ; i < a.length && a[i] == tmp; i++);
        for ( ; j < b.length && b[j] == tmp; j++);
        answer[k++] = tmp;
    }
    while (i < a.length) {
        tmp = a[i++];
        for ( ; i < a.length && a[i] == tmp; i++);
        answer[k++] = tmp;
    }
    while (j < b.length) {
        tmp = b[j++];
        for ( ; j < b.length && b[j] == tmp; j++);
        answer[k++] = tmp;
    }
    return Arrays.copyOf(answer, k);
}

+1、共有いただきありがとうございます。1つの質問:なぜ配列のタイプと変数「temp」のタイプを長く選択したのですか?
ヘンガメ2015

(メソッド名に疑問があります。)
greybeard '19

5

以下のように4つのステートメントで実行できます

 int a[] = {10, 20, 30};
 int b[]= {9, 14, 11};
 int res[]=new int[a.legth+b.length]; 
 System.arraycopy(a,0, res, 0, a.length); 
 System.arraycopy(b,0,res,a.length, b.length);
 Array.sort(res)


5
なぜこの回答が反対票を投じたのか理解できません。それは効率的ではないことは事実です。しかし、必要なのは、できるだけ早く仕事を終わらせることだけです。非常に小さな配列(100要素未満など)を扱う場合、重要なパフォーマンスの向上をもたらさない長いコードを記述するよりも、上記のコードを使用することをお勧めします。だから、この簡単なソリューションを提供してくれたSudhirと編集のためのSANN3に感謝します。
アーメドフ2014

2
書かれていない前提は、sort関数がそれ自体をソートの方法として使用できないことです。これは、再帰ではなく無限回帰になります。また、merge_arrayはsortを実装する関数であるという前提もあります。したがって、この回答はほとんどの状況では使用できません。
Aki Suihkonen 2015

尋ねられた質問は、必要なコードが小さな配列だけのものであることについては触れていませんでした。したがって、この回答は、その制限を明確に述べていない限り、誤解を招く可能性があります。以下の私の答えも見てください。任意の配列サイズで機能する効率的なコードを書くのに同じ行数が必要です:)
Shital Shah

質問では、配列はすでにソートされた順序になっていると規定されていました。アレイが非常に大きくなる可能性がある場合、このソリューションは停止して性能が低下します。したがって、必要な最終結果が得られることを確認してください。ただし、インタビューを行っている場合、アプリは実行されず、仕事も得られません。
Kevin M

Array.sort()関数はTimSortを使用します。TimSortは、ソートされた実行を検出し、それらにマージソートを適用します。奇妙なことに、このコードは、実際には「非効率的」であることでフォールトすることさえできません。ソートされた実行のため、実際にはO(n)時間で終了することになります。一連のベンチマークを実行できます。オペコードはOPコードを頻繁に上回る可能性があります。
Tatarize

4

私はそれをjavascriptで書かなければなりませんでした、ここにあります:

function merge(a, b) {
    var result = [];
    var ai = 0;
    var bi = 0;
    while (true) {
        if ( ai < a.length && bi < b.length) {
            if (a[ai] < b[bi]) {
                result.push(a[ai]);
                ai++;
            } else if (a[ai] > b[bi]) {
                result.push(b[bi]);
                bi++;
            } else {
                result.push(a[ai]);
                result.push(b[bi]);
                ai++;
                bi++;
            }
        } else if (ai < a.length) {
            result.push.apply(result, a.slice(ai, a.length));
            break;
        } else if (bi < b.length) {
            result.push.apply(result, b.slice(bi, b.length));
            break;
        } else {
            break;
        }
    }
    return result;
}

4

Apacheコレクションは、バージョン4以降のcollat​​eメソッドをサポートしています。次のcollateメソッドを使用してこれを行うことができます。

org.apache.commons.collections4.CollectionUtils

ここにjavadocからの引用:

collate(Iterable<? extends O> a, Iterable<? extends O> b, Comparator<? super O> c)

2つのソートされたコレクションabを1つのソートされたリストにマージして、コンパレータcによる要素の順序が維持されるようにします。

ホイールを再発明しないでください!ドキュメント参照:http : //commons.apache.org/proper/commons-collections/apidocs/org/apache/commons/collections4/CollectionUtils.html


4

GallopSearchマージ:O(n)ではなくO(log(n)* log(i)

私は先に進み、コメントに灰色ひげの提案を実装しました。ほとんどの場合、このコードの非常に効率的なミッションクリティカルバージョンが必要だったからです。

  • コードはO(log(i))であるgallopSearchを使用します。ここで、iは関連するインデックスが存在する現在のインデックスからの距離です。
  • コードは、ギャロップ検索が適切な範囲を識別した後に、binarySearchを使用します。ギャロップはこれをより小さい範囲に制限したため、結果のバイナリサーチもO(log(i))になります
  • ギャロップとマージは逆方向に実行されます。これはミッションクリティカルとは思われませんが、アレイを適切にマージすることができます。配列の1つに結果値を格納するのに十分なスペースがある場合は、それをマージ配列および結果配列として使用できます。このような場合は、配列内の有効な範囲を指定する必要があります。
  • その場合、メモリの割り当ては必要ありません(重要な操作の大幅な節約)。処理されていない値を上書きしないこと、および上書きできないことを確認するだけです(これは逆方向にのみ実行できます)。実際、入力と結果の両方に同じ配列を使用します。悪影響はありません。
  • 私は一貫してInteger.compare()を使用したので、これを他の目的に切り替えることができます。
  • 以前に証明した情報を少し間抜けて利用しなかった可能性があります。1つの値が既にチェックされている2つの値の範囲へのバイナリ検索など。また、メインループを記述するより良い方法があるかもしれません。cの値を順番に2つの演算に組み合わせれば、フリッピングc値は必要ありません。あなたが知っているので、あなたはいつもどちらか一方を毎回行うでしょう。少し磨きをかける余地があります。

これは、O(n)ではなくO(log(n)* log(i))の時間の複雑さで、これを行う最も効率的な方法である必要があります。そしてO(n)の最悪の場合の時間の複雑さ。配列が不揃いで値の長い文字列が一緒にある場合、これは他の方法で行うよりも小さくなります。それ以外の場合は、それらよりも優れています。

マージ配列の最後に2つの読み取り値があり、結果配列内に書き込み値があります。最終値がどちらであるかを見つけた後、その配列に対してギャロップ検索を実行します。1、2、4、8、16、32など。他の配列の読み取り値が大きい範囲が見つかった場合。それはその範囲をバイナリ検索します(範囲を半分にカットし、正しい半分を検索し、単一の値になるまで繰り返します)。次に、配列はそれらの値を書き込み位置にコピーします。コピーは、必然的に、どちらかの読み取り配列から同じ値を上書きできないように移動されます(つまり、書き込み配列と読み取り配列が同じであることを意味します)。次に、他の配列の新しい読み取り値よりも小さいことがわかっている他の配列に対して同じ操作を実行します。

static public int gallopSearch(int current, int[] array, int v) {
    int d = 1;
    int seek = current - d;
    int prevIteration = seek;
    while (seek > 0) {
        if (Integer.compare(array[seek], v) <= 0) {
            break;
        }
        prevIteration = seek;
        d <<= 1;
        seek = current - d;
        if (seek < 0) {
            seek = 0;
        }
    }
    if (prevIteration != seek) {
        seek = binarySearch(array, seek, prevIteration, v);
        seek = seek >= 0 ? seek : ~seek;
    }
    return seek;
}

static public int binarySearch(int[] list, int fromIndex, int toIndex, int v) {
    int low = fromIndex;
    int high = toIndex - 1;
    while (low <= high) {
        int mid = (low + high) >>> 1;
        int midVal = list[mid];
        int cmp = Integer.compare(midVal, v);
        if (cmp < 0) {
            low = mid + 1;
        } else if (cmp > 0) {
            high = mid - 1;
        } else {
            return mid;// key found
        }
    }
    return -(low + 1);// key not found.
}

static public int[] sortedArrayMerge(int[] a, int[] b) {
    return sortedArrayMerge(null, a, a.length, b, b.length);
}

static public int[] sortedArrayMerge(int[] results, int[] a, int aRead, int b[], int bRead) {
    int write = aRead + bRead, length, gallopPos;
    if ((results == null) || (results.length < write)) {
        results = new int[write];
    }
    if (aRead > 0 && bRead > 0) {
        int c = Integer.compare(a[aRead - 1], b[bRead - 1]);
        while (aRead > 0 && bRead > 0) {
            switch (c) {
                default:
                    gallopPos = gallopSearch(aRead, a, b[bRead-1]);
                    length = (aRead - gallopPos);
                    write -= length;
                    aRead = gallopPos;
                    System.arraycopy(a, gallopPos--, results, write, length);
                    c = -1;
                    break;
                case -1:
                    gallopPos = gallopSearch(bRead, b, a[aRead-1]);
                    length = (bRead - gallopPos);
                    write -= length;
                    bRead = gallopPos;
                    System.arraycopy(b, gallopPos--, results, write, length);
                    c = 1;
                    break;
            }
        }
    }
    if (bRead > 0) {
        if (b != results) {
            System.arraycopy(b, 0, results, 0, bRead);
        }
    } else if (aRead > 0) {
        if (a != results) {
            System.arraycopy(a, 0, results, 0, aRead);
        }
    }
    return results;
}

これが最も効率的な方法です。


一部の回答には重複削除機能がありました。実際に各アイテムを比較する必要があるため、O(n)アルゴリズムが必要になります。だから、これはスタンドアロンであり、事後に適用されます。複数のエントリがある場合は、すべてを確認する必要がある場合は、複数のエントリを完全にギャロップすることはできませんが、重複がある場合は、ギャロップできます。

static public int removeDuplicates(int[] list, int size) {
    int write = 1;
    for (int read = 1; read < size; read++) {
        if (list[read] == list[read - 1]) {
            continue;
        }
        list[write++] = list[read];
    }
    return write;
}

更新:以前の回答、恐ろしいコードではありませんが、明らかに上記より劣っています。

別の不必要な超最適化。終了ビットだけでなく、開始ビットのarraycopyを呼び出します。データへのbinarySearchによるO(log(n))での非重複の紹介処理。O(log(n)+ n)はO(n)であり、場合によっては、特に、マージする配列間でオーバーラップがまったくない場合など、効果がかなり顕著になります。

private static int binarySearch(int[] array, int low, int high, int v) {
    high = high - 1;
    while (low <= high) {
        int mid = (low + high) >>> 1;
        int midVal = array[mid];
        if (midVal > v)
            low = mid + 1;
        else if (midVal < v)
            high = mid - 1;
        else
            return mid; // key found
    }
    return low;//traditionally, -(low + 1);  // key not found.
}

private static int[] sortedArrayMerge(int a[], int b[]) {
    int result[] = new int[a.length + b.length];
    int k, i = 0, j = 0;
    if (a[0] > b[0]) {
        k = i = binarySearch(b, 0, b.length, a[0]);
        System.arraycopy(b, 0, result, 0, i);
    } else {
        k = j = binarySearch(a, 0, a.length, b[0]);
        System.arraycopy(a, 0, result, 0, j);
    }
    while (i < a.length && j < b.length) {
        result[k++] = (a[i] < b[j]) ? a[i++] : b[j++];
    }
    if (j < b.length) {
        System.arraycopy(b, j, result, k, (b.length - j));
    } else {
        System.arraycopy(a, i, result, k, (a.length - i));
    }
    return result;
}

1
対称性について何かを始めることに賛成ですが、なぜそこで停止するのですか?ギャロッピング検索を使用して、等しいキーのにインデックス返すようにし ます。3つを超える要素の場合のみ、配列のコピーを使用します。そのコピーの後、何も変更されていないことに注意してください。a)1つの入力の開始インデックスと出力配列b)どの「次の」要素が小さいかについての知識。
greybeard 2018

それは完全に実装されたArrays.sortが行うことです。特にマージソートは最悪です。必要に応じて2つの要素を入れ替えると思いますが、3つ以上の要素の場合は配列複写に分類されます。次の要素を線形的にチェックするのか、それをバイナリ検索するのかはわかりません。あなたがその距離をギャロップできれば、より大きな距離をギャロップできるかどうかを投機的にチェックすることには、かなり大きな利点があります。先のチェック8のように、コピーすることができれば、見る必要のないものの7つの操作を節約できます。
Tatarize

@greybeard ...と完了。また、同じメモリを再利用できるように後退しました。
18

弾道をやる気にさせた良いこと。昼間のタイムシンクの後で詳しく見ていきます。
greybeard 2018

That is totally what the implemented Arrays.sort doesそれはあなたの答えの最初の改訂から-または-私の2月19日のコメントから?)-SunsoftのJDK 8のどちらも見つかりません:どの実装Arrays.sortを参照していますか?
greybeard 2018

2

JavaScriptで書かれた短縮形は次のとおりです。

function sort( a1, a2 ) {

    var i = 0
        , j = 0
        , l1 = a1.length
        , l2 = a2.length
        , a = [];

    while( i < l1 && j < l2 ) {

        a1[i] < a2[j] ? (a.push(a1[i]), i++) : (a.push( a2[j]), j++);
    }

    i < l1 && ( a = a.concat( a1.splice(i) ));
    j < l2 && ( a = a.concat( a2.splice(j) ));

    return a;
}

1
    public class Merge {

    // stably merge a[lo .. mid] with a[mid+1 .. hi] using aux[lo .. hi]
    public static void merge(Comparable[] a, Comparable[] aux, int lo, int mid, int hi) {

        // precondition: a[lo .. mid] and a[mid+1 .. hi] are sorted subarrays
        assert isSorted(a, lo, mid);
        assert isSorted(a, mid+1, hi);

        // copy to aux[]
        for (int k = lo; k <= hi; k++) {
            aux[k] = a[k]; 
        }

        // merge back to a[]
        int i = lo, j = mid+1;
        for (int k = lo; k <= hi; k++) {
            if      (i > mid)              a[k] = aux[j++];
            else if (j > hi)               a[k] = aux[i++];
            else if (less(aux[j], aux[i])) a[k] = aux[j++];
            else                           a[k] = aux[i++];
        }

        // postcondition: a[lo .. hi] is sorted
        assert isSorted(a, lo, hi);
    }

    // mergesort a[lo..hi] using auxiliary array aux[lo..hi]
    private static void sort(Comparable[] a, Comparable[] aux, int lo, int hi) {
        if (hi <= lo) return;
        int mid = lo + (hi - lo) / 2;
        sort(a, aux, lo, mid);
        sort(a, aux, mid + 1, hi);
        merge(a, aux, lo, mid, hi);
    }

    public static void sort(Comparable[] a) {
        Comparable[] aux = new Comparable[a.length];
        sort(a, aux, 0, a.length-1);
        assert isSorted(a);
    }


   /***********************************************************************
    *  Helper sorting functions
    ***********************************************************************/

    // is v < w ?
    private static boolean less(Comparable v, Comparable w) {
        return (v.compareTo(w) < 0);
    }

    // exchange a[i] and a[j]
    private static void exch(Object[] a, int i, int j) {
        Object swap = a[i];
        a[i] = a[j];
        a[j] = swap;
    }


   /***********************************************************************
    *  Check if array is sorted - useful for debugging
    ***********************************************************************/
    private static boolean isSorted(Comparable[] a) {
        return isSorted(a, 0, a.length - 1);
    }

    private static boolean isSorted(Comparable[] a, int lo, int hi) {
        for (int i = lo + 1; i <= hi; i++)
            if (less(a[i], a[i-1])) return false;
        return true;
    }


   /***********************************************************************
    *  Index mergesort
    ***********************************************************************/
    // stably merge a[lo .. mid] with a[mid+1 .. hi] using aux[lo .. hi]
    private static void merge(Comparable[] a, int[] index, int[] aux, int lo, int mid, int hi) {

        // copy to aux[]
        for (int k = lo; k <= hi; k++) {
            aux[k] = index[k]; 
        }

        // merge back to a[]
        int i = lo, j = mid+1;
        for (int k = lo; k <= hi; k++) {
            if      (i > mid)                    index[k] = aux[j++];
            else if (j > hi)                     index[k] = aux[i++];
            else if (less(a[aux[j]], a[aux[i]])) index[k] = aux[j++];
            else                                 index[k] = aux[i++];
        }
    }

    // return a permutation that gives the elements in a[] in ascending order
    // do not change the original array a[]
    public static int[] indexSort(Comparable[] a) {
        int N = a.length;
        int[] index = new int[N];
        for (int i = 0; i < N; i++)
            index[i] = i;

        int[] aux = new int[N];
        sort(a, index, aux, 0, N-1);
        return index;
    }

    // mergesort a[lo..hi] using auxiliary array aux[lo..hi]
    private static void sort(Comparable[] a, int[] index, int[] aux, int lo, int hi) {
        if (hi <= lo) return;
        int mid = lo + (hi - lo) / 2;
        sort(a, index, aux, lo, mid);
        sort(a, index, aux, mid + 1, hi);
        merge(a, index, aux, lo, mid, hi);
    }

    // print array to standard output
    private static void show(Comparable[] a) {
        for (int i = 0; i < a.length; i++) {
            StdOut.println(a[i]);
        }
    }

    // Read strings from standard input, sort them, and print.
    public static void main(String[] args) {
        String[] a = StdIn.readStrings();
        Merge.sort(a);
        show(a);
    }
}

このコピーのa[mid+1 .. hi]目的auxは何ですか?
greybeard 2018

1

より大きなソート済み配列にスキップリストを導入すると、比較の数を減らし、3番目の配列へのコピープロセスを高速化できると思います。これは、配列が大きすぎる場合に適しています。


1
public int[] merge(int[] a, int[] b) {
    int[] result = new int[a.length + b.length];
    int aIndex, bIndex = 0;

    for (int i = 0; i < result.length; i++) {
        if (aIndex < a.length && bIndex < b.length) {
            if (a[aIndex] < b[bIndex]) {
                result[i] = a[aIndex];
                aIndex++;
            } else {
                result[i] = b[bIndex];
                bIndex++;
            }
        } else if (aIndex < a.length) {
            result[i] = a[aIndex];
            aIndex++;
        } else {
            result[i] = b[bIndex];
            bIndex++;
        }
    }

    return result;
}

2
いくつかの説明がいいでしょう。:)
gsamaras 14年

1
public static int[] merge(int[] a, int[] b) {
    int[] mergedArray = new int[(a.length + b.length)];
    int i = 0, j = 0;
    int mergedArrayIndex = 0;
    for (; i < a.length || j < b.length;) {
        if (i < a.length && j < b.length) {
            if (a[i] < b[j]) {
                mergedArray[mergedArrayIndex] = a[i];
                i++;
            } else {
                mergedArray[mergedArrayIndex] = b[j];
                j++;
            }
        } else if (i < a.length) {
            mergedArray[mergedArrayIndex] = a[i];
            i++;
        } else if (j < b.length) {
            mergedArray[mergedArrayIndex] = b[j];
            j++;
        }
        mergedArrayIndex++;
    }
    return mergedArray;
}

これの節約の恵みは何ですか?に縮小することができますfor (int i, j, k = i = j = 0 ; k < c.length ; ) c[k++] = b.length <= j || i < a.length && a[i] < b[j] ? a[i++] : b[j++];Andrewの2014年の回答とどう違うのですか?
greybeard 2018

1

アルゴリズムはさまざまな方法で強化できます。たとえば、a[m-1]<b[0]またはかどうかを確認することは妥当b[n-1]<a[0]です。これらのいずれの場合も、これ以上の比較を行う必要はありません。アルゴリズムは、結果の配列のソース配列を正しい順序でコピーするだけです。

より複雑な拡張機能には、インターリーブパーツの検索と、それらのパーツに対してのみマージアルゴリズムを実行することが含まれます。マージされた配列のサイズが何十回も異なる場合、時間を大幅に節約できます。


この拡張では、最初の要素が2番目の配列のどこにあるかをバイナリサーチで確認してから、そのデータをarraycopyすることをお勧めします。次に、これらのチェックのいずれかがtrueの場合、すべてをarraycopyし、次に3値をarraycopyすると、同じ結果が得られます。ただし、わずかなオーバーラップが発生した場合は、オーバーラップ中に適切なアルゴリズムを実行するだけで、それ以外の時間は必要ありません。事前にクイックO(logn)コマンドを使用してO(n)で立ち往生しているので、コストはかかりません。
酒石化

1

この問題は、2つの並べ替えられたサブ配列が1つの並べ替えられたサブ配列に結合されるmergesortアルゴリズムに関連しています。CLRS本はアルゴリズムの例を与え、最後には、各配列の最後にセンチネル値(「他のどの値よりも大きい」と比較して、何かを)加えることによって到達されたかどうかをチェックする必要性をクリーンアップします。

私はこれをPythonで記述しましたが、Javaにもうまく変換されるはずです。

def func(a, b):
    class sentinel(object):
        def __lt__(*_):
            return False

    ax, bx, c = a[:] + [sentinel()], b[:] + [sentinel()], []
    i, j = 0, 0

    for k in range(len(a) + len(b)):
        if ax[i] < bx[j]:
            c.append(ax[i])
            i += 1
        else:
            c.append(bx[j])
            j += 1

    return c

要素を(巧みに)センチネルを使用して一括コピーする
greybeard

1

2つのスレッドを使用して、前から1つ、後ろから1つ、結果の配列を埋めることができます。

これは、各スレッドが値の半分を挿入する場合など、数値の場合は同期せずに機能します。


0
//How to merge two sorted arrays into a sorted array without duplicates?
//simple C Coding
#include <stdio.h>
#include <stdlib.h>
#include <string.h>

main()
{
    int InputArray1[] ={1,4,5,7,8,9,12,13,14,17,40};
    int InputArray2[] ={4,5,11,14,15,17,18,19,112,122,122,122,122};
    int n=10;
    int OutputArray[30];
    int i=0,j=0,k=0;
    //k=OutputArray
    while(i<11 && j<13)
    {
        if(InputArray1[i]<InputArray2[j])
        {
            if (k == 0 || InputArray1[i]!= OutputArray[k-1])
            {
                OutputArray[k++] = InputArray1[i];
            }
            i=i+1;
        }
        else if(InputArray1[i]>InputArray2[j])
        {
            if (k == 0 || InputArray2[j]!= OutputArray[k-1])
            {
                OutputArray[k++] = InputArray2[j];
            }
            j=j+1;
        }
        else
        {
            if (k == 0 || InputArray1[i]!= OutputArray[k-1])
            {
                OutputArray[k++] = InputArray1[i];
            }
            i=i+1;
            j=j+1;
        }
    };
    while(i<11)
    {
        if(InputArray1[i]!= OutputArray[k-1])
            OutputArray[k++] = InputArray1[i++];
        else
            i++;
    }
    while(j<13)
    {
        if(InputArray2[j]!= OutputArray[k-1])
            OutputArray[k++] = InputArray2[j++];
        else
            j++;
    }
    for(i=0; i<k; i++)
    {
        printf("sorted data:%d\n",OutputArray[i]);
    };
}

0
public static int[] merge(int[] listA, int[] listB) {
        int[] mergedList = new int[ listA.length + listB.length];
        int i = 0; // Counter for listA
        int j = 0; // Counter for listB
        int k = 0; // Counter for mergedList
        while (true) {
            if (i >= listA.length && j >= listB.length) {
                break;
            }
            if (i < listA.length && j < listB.length) { // If both counters are valid.
                if (listA[i] <= listB[j]) {
                    mergedList[k] = listA[i];
                    k++;
                    i++;
                } else {
                    mergedList[k] = listB[j];
                    k++;
                    j++;
                }
            } else if (i < listA.length && j >= listB.length) { // If only A's counter is valid.
                mergedList[k] = listA[i];
                k++;
                i++;
            } else if (i <= listA.length && j < listB.length) { // If only B's counter is valid
                mergedList[k] = listB[j];
                k++;
                j++;
            }
        }
        return mergedList;
    }

0
var arrCombo = function(arr1, arr2){
  return arr1.concat(arr2).sort(function(x, y) {
    return x - y;
  });
};

2
この答えはJavaプログラミング言語には当てはまりませんが、javascriptには良い答えです。
gknicker 2015

これは就職の面接の一部でした。これらの場合、上記のような「通常の」コードを書くことは本当に期待されていません。彼らは、「効率的な」コードと、関与するアルゴリズムを理解できるデモを探しています。
d11wtq

0

私のお気に入りのプログラミング言語はJavaScriptです

function mergeSortedArrays(a, b){
    var result = [];

    var sI = 0;
    var lI = 0;
    var smallArr;
    var largeArr;
    var temp;

    if(typeof b[0] === 'undefined' || a[0]<b[0]){
        smallArr = a;
        largeArr = b;
    } else{
        smallArr = b;
        largeArr = a;
    }

    while(typeof smallArr[sI] !== 'undefined'){
        result.push(smallArr[sI]);
        sI++;

        if(smallArr[sI]>largeArr[lI] || typeof smallArr[sI] === 'undefined'){
            temp = smallArr;
            smallArr = largeArr;
            largeArr = temp;
            temp = sI;
            sI = lI;
            lI = temp;
        }
    }
    return result;
}

0

多分System.arraycopyを使用します

public static byte[] merge(byte[] first, byte[] second){
    int len = first.length + second.length;
    byte[] full = new byte[len];
    System.arraycopy(first, 0, full, 0, first.length);
    System.arraycopy(second, 0, full, first.length, second.length);
    return full;
}

3
あなたはそれらをマージしているだけです。結果の配列自体は、要件であったソートされていません。
Sanjeev Dhiman 2016年

0
public static void main(String[] args) {
    int[] arr1 = {2,4,6,8,10,999};
    int[] arr2 = {1,3,5,9,100,1001};

    int[] arr3 = new int[arr1.length + arr2.length];

    int temp = 0;

    for (int i = 0; i < (arr3.length); i++) {
        if(temp == arr2.length){
            arr3[i] = arr1[i-temp];
        }
        else if (((i-temp)<(arr1.length)) && (arr1[i-temp] < arr2[temp])){
                arr3[i] = arr1[i-temp];
        }
        else{
            arr3[i] = arr2[temp];
            temp++;
        }
    }

    for (int i : arr3) {
        System.out.print(i + ", ");
    }
}

出力は:

1、2、3、4、5、6、8、9、10、100、999、1001


インデックスに名前を付けるために混乱arr2はないind2、しかしtemp
greybeard 2018

0

三項演算子を使用して、コードをもう少しコンパクトにすることができます

public static int[] mergeArrays(int[] a1, int[] a2) {
    int[] res = new int[a1.length + a2.length];
    int i = 0, j = 0;

    while (i < a1.length && j < a2.length) {
        res[i + j] = a1[i] < a2[j] ? a1[i++] : a2[j++];
    }

    while (i < a1.length) {
        res[i + j] = a1[i++];
    }

    while (j < a2.length) {
        res[i + j] = a2[j++];
    }

    return res;
}

0
public static int[] mergeSorted(int[] left, int[] right) {
    System.out.println("merging " + Arrays.toString(left) + " and " + Arrays.toString(right));
    int[] merged = new int[left.length + right.length];
    int nextIndexLeft = 0;
    int nextIndexRight = 0;
    for (int i = 0; i < merged.length; i++) {
        if (nextIndexLeft >= left.length) {
            System.arraycopy(right, nextIndexRight, merged, i, right.length - nextIndexRight);
            break;
        }
        if (nextIndexRight >= right.length) {
            System.arraycopy(left, nextIndexLeft, merged, i, left.length - nextIndexLeft);
            break;
        }
        if (left[nextIndexLeft] <= right[nextIndexRight]) {
            merged[i] = left[nextIndexLeft];
            nextIndexLeft++;
            continue;
        }
        if (left[nextIndexLeft] > right[nextIndexRight]) {
            merged[i] = right[nextIndexRight];
            nextIndexRight++;
            continue;
        }
    }
    System.out.println("merged : " + Arrays.toString(merged));
    return merged;
}

元のソリューションとは少し異なる


0

O(m + n)時間の複雑さで2つのソートされた配列をマージするには、1つのループのみで以下のアプローチを使用します。mとnは、最初の配列と2番目の配列の長さです。

public class MargeSortedArray {
    public static void main(String[] args) {
        int[] array = new int[]{1,3,4,7};
        int[] array2 = new int[]{2,5,6,8,12,45};
        int[] newarry = margeToSortedArray(array, array2);
        //newarray is marged array
    }

    // marge two sorted array with o(a+n) time complexity
    public static int[] margeToSortedArray(int[] array, int[] array2) {
        int newarrlen = array.length+array2.length;
        int[] newarr = new int[newarrlen];

        int pos1=0,pos2=0;
        int len1=array.length, len2=array2.length;

        for(int i =0;i<newarrlen;i++) {     
            if(pos1>=len1) {
                newarr[i]=array2[pos2];
                pos2++;
                continue;
            }
            if(pos2>=len2) {
                newarr[i]=array[pos1];
                pos1++;
                continue;
            }

            if(array[pos1]>array2[pos2]) {
                newarr[i]=array2[pos2];
                pos2++;
            } else {
                newarr[i]=array[pos1];
                pos1++;
            }   
        }

        return newarr;
    }

}

0
var arr1 = [2,10,20,30,100];
var arr2 = [2,4,5,6,7,8,9];
var j = 0;
var i =0;
var newArray = [];

for(var x=0;x< (arr1.length + arr2.length);x++){
    if(arr1[i] >= arr2[j]){                //check if element arr2 is equal and less than arr1 element
        newArray.push(arr2[j]);
      j++;
    }else if(arr1[i] < arr2[j]){            //check if element arr1 index value  is less than arr2 element
        newArray.push(arr1[i]);
        i++;
    }
    else if(i == arr1.length || j < arr2.length){    // add remaining arr2 element
        newArray.push(arr2[j]);
        j++
    }else{                                                   // add remaining arr1 element
        newArray.push(arr1[i]); 
        i++
    }

}

console.log(newArray);

-1

質問は特定の言語を想定していないためです。これがPythonでの解決策です。配列がすでにソートされていると仮定します。

アプローチ1-numpy配列の使用:numpyのインポート

arr1 = numpy.asarray([ 1,  2,  3,  4,  5,  6,  7,  8,  9, 11, 14, 15, 55])
arr2 = numpy.asarray([11, 32, 43, 45, 66, 76, 88])

array = numpy.concatenate((arr1,arr2), axis=0)
array.sort()

アプローチ2-リストを使用して、リストがソートされていると仮定します。

list_new = list1.extend(list2)
list_new.sort()

Since the question doesn't assume any specific language2011/5/11/19:43から、javaとタグ付けされています。
greybeard 2018

以来、事実のリストを利用していないあなたのソリューションは、すでにソートされている、そしてそのランタイムはないO(n)が.sort()あるO(n log n)最高の状態で
dark_ruby

-1

これは、重複を削除する私のJava実装です。

public static int[] mergesort(int[] a, int[] b) {
    int[] c = new int[a.length + b.length];
    int i = 0, j = 0, k = 0, duplicateCount = 0;

    while (i < a.length || j < b.length) {
        if (i < a.length && j < b.length) {
            if (a[i] == b[j]) {
                c[k] = a[i];
                i++;j++;duplicateCount++;
            } else {
                c[k] = a[i] < b[j] ? a[i++] : b[j++];
            }
        } else if (i < a.length) {
            c[k] = a[i++];
        } else if (j < a.length) {
            c[k] = b[j++];
        }
        k++;
    }

    return Arrays.copyOf(c, c.length - duplicateCount);
}
弊社のサイトを使用することにより、あなたは弊社のクッキーポリシーおよびプライバシーポリシーを読み、理解したものとみなされます。
Licensed under cc by-sa 3.0 with attribution required.