Последнюю строку массива мне выводит очень много раз, в чем проблема?

Question

[Паша Максименко]

КОД:

cout <<"--------------GenComb---------------"<<endl;

for (int i=0; i<n; i++)
a[i]=i+1;
for (int j=0; j<k; j++)
{
    cout<<a[j]<<"";
}
   cout<<endl;


   while (n>k)
   {
       GenComb (a, n, k);

for (int j=0; j<k; j++)
       {
           cout <<a[j]<<"";
       }
       cout<<endl;
   }

return 0;
}

ОШИБКА:

3456
3456
3456
3456
3456
3456
3456
3456
3456
3456
3456
3456

Comments

[Wataru]

Введи код в редакторе. Выдели его весь и нажми на кнопку </>, прокрути до C++ и ткни туда.
Вокруг кода появятся теги code.

Выглядить это будет вроде:

#include <iostream>

int main() {
  std::cout << "Hello world";
  return 0;
}

Никакие части кода не исчезнут.

[Паша Максименко]

Wataru,

cout <<"--------------GenComb---------------"<<endl;

for (int i=0; i<n; i++)
a[i]=i+1;
for (int j=0; j<k; j++)
{
    cout<<a[j]<<"";
}
   cout<<endl;


   while (n>k)
   {
       GenComb (a, n, k);

for (int j=0; j<k; j++)
       {
           cout <<a[j]<<"";
       }
       cout<<endl;
   }

return 0;
}

Answer 1

[Wataru]

Если вы повнимательнее посмотрите на свой код, то заметите, что там вывод массива a происходит в цикле while(n>k). Где именно ошибка - непонятно, ибо неясно, что ваш код должен делать. Или вы вывод массива вставили не туда, или GetComb делает что-то не то и цикл исполняется больше раз, чем должен.

Comments

[Паша Максименко]

ну вот здесь, мне кажется все правильно нет?

int GenComb(int* arr, int n, int k)
{
    for(int i = k - 1; i>=0; i--) {
    if (arr[i] < n - k + i + 1){
    ++arr[i];
    for (int j = i + 1; j<k; ++j){
    arr [j] = arr [j - 1]+ 1;
                }
    return  1;
         }
  }

[Wataru]

Паша Максименко, Не могу сказать, потому что не знаю, что этот код должен делать.