2つのアレイを1つの大きなアレイにコピーしますか？

Question

I、次の質問があります。

は「マージ」という名前の関数を使用してプログラムを書くことをコピーデータより大きなサイズの配列に1列の整数、とにコピー二番目の配列のデータの整数をIは、アレイ2

の配列1及び{3,4}の{1,2}を入力した場合、私の関数

に問題があるだけで最初の配列

の内容の後に大きなアレイ

その後、出力が1 2 -57574 -658675
それは1つの2 3 4

void merge (int a[], int n, int b[],int m) { 

int c[100]; 
int x=n+m ; //size of merge aray c[] 

for(int i = 0; i < n; i++) 
c[i]=a[i]; 

for(int j = n ; j < x ; j++) 
c[j] = b[j]; 

cout<<endl<<endl; 


for(int k = 0; k < x; k++) 

cout<<c[k]<<" "; 


} 

Answer 1

問題でなければなりません：あなたは動的に正しいサイズの配列を作成する必要が

1. - もっとあるかもしれませんその100項目。
2. ではなく、b[0]からコピーを開始する必要があります。

Answer 2

c[j] = b[j]; 

です。最初のjは正しいですが、2番目のjは実際にはj - nです。第二ループ内

Answer 3

for(int j = n ; j < x ; j++) 
c[j] = b[j]; <---- b[j] not defined, you need to start from b[0] 

これを試してください：

for(int j = n ; j < x ; j++) 
c[j] = b[j-n]; 

Answer 4

void merge (int a[], int n, int b[],int m) 
{ 
    int* c = new int[n+m]; 

    for(int i = 0; i < n; i++) 
    c[i]=a[i]; 

    for(int j = 0 ; j < m ; j++) 
    c[n+j] = b[j]; // <-- there was your fault 

    cout<<endl<<endl; 
    for(int k = 0; k < n+m; k++) 
    cout<<c[k]<<" "; 

    delete [] c; 
} 

Answer 5

インナーループとして書き直すことができる：

for(int j=0;k=n;j<m,k<x;j++,k++) 

{

​​

}

私はyouuがアサート上の点...

Answer 6

#include<iostream> 
#include<assert.h> 


void merge(int first[], int nLenFirst, int second[], int nLenSecond, int merged[]) 
{ 
    int nTotal = nLenFirst + nLenSecond; 

    for(int i= 0; i < nLenFirst; ++i) 
     merged[i] = first[i]; 

    for(int i= nLenFirst, j = 0; i < nTotal; ++i,++j) 
     merged[i] = second[j]; 

} 

void main() 
{ 
    int a[] = {2, 4, 5, 7}; 
    int b[] = {3, 7, 11, 19, 25}; 

    int nLenA = sizeof(a)/sizeof(int); 
    int nLenB = sizeof(b)/sizeof(int); 

    int c[100] = {0}; 

    int nTotal = nLenA + nLenB; 
    assert(sizeof(c)/sizeof(int) >= nTotal); 

    merge(a, nLenA, b, nLenB, c); 

    for(int i = 0; i < nTotal; ++i) 
    { 
     std::cout << c[i] << std::endl; 
    } 
} 

フォーカスを得た願っています！

Answer 7

ベクターを使用するだけではどうですか？このような何か：これは私の単純なコードが

std::vector<int> concat(const std::vector<int>& a, const std::vector<int>& b) { 
    std::vector<int> c; 
    c.reserve(a.size() + b.size()); 
    c.insert(c.end(), a.begin(), a.end()); 
    c.insert(c.end(), b.begin(), b.end()); 
    return c; 
} 

Answer 8

void merge (int a[], int n, int b[],int m) 
{ 

    int c = new int[n + m]; 
    std::copy(a, a + n, c); 
    std::copy(b, b + m, c + n); 

    cout<<endl<<endl; 
    for(int k = 0; k < n+m; k++) 
     cout<<c[k]<<" ";  

    delete[] c; 
} 

Answer 9

です。マージ機能の目的に達するように変更します。

int main() { 
    int a[5]; 
    int b[5]; 
    int c[10]; 

    cout << "Enter elements for array a[5]:" << endl; 
    int i = 0; 
    do { 
     cin >> a[i]; 
     i++; 
    } while (i <= 4); 

    cout << "Enter elements for array b[5]:" << endl; 
    i = 0; 
    do { 
     cin >> b[i]; 
     i++; 
    } while (i <= 4); 

    for (register int x = 0; x <= 5; x++) { 
     if (x == 5) { 
      for (register int h = 0; h < 5; h++) { 
       c[x] = b[h]; 
       x++; 
      } 
      break; 
     } 
     c[x] = a[x]; 
    } 

    for (register int x = 0; x < 10; x++) { 
     cout << c[x] << " "; 
    } 
    return (0); 
}