Code to bring copy the unique value from array A to B. And return the number of unique value which is the number of element in B
#include <stdio.h>
int func(int A[], int B[], int n)
{
int i, j, inc = 0;
for(i = 0; i < n; i++)
{
for(j = 0; j < i; j++)
{
if (A[i] == A[j])
break;
this break supposed to break out the inner loop incrementally
if (i == j)
{
B[inc] = A[i];
inc++;
}
}
}
}
int main()
{
int A[100] = {1, 3, 4, 5, 3, 2, 5, 4, 1, 0};
int B[100];
int c;
c = func(A, B, 10);
printf(" %d", c);
}
Даже если вы добавите отсутствующую строку return inc;
в свой func
, у вас все еще будет проблема, что if (A[i] == A[j])
будет всегда истинным на первой итерации внутреннего цикла... потому что i
и j
будут одинаковыми (оба равны нулю).
if (i == j)
во внутреннем цикле также бессмысленно. По определению ограничителя внутреннего цикла i
и j
там будут равны никогда. Во всяком случае, он принадлежит внешнему циклу, снаружи (и после) внутреннего цикла.
Ваш тест на i
против j
неуместен. Он принадлежит снаружи, но после внутренней петли. И, конечно же, нужно вернуться inc
.
#include <stdio.h>
int func(int A[], int B[], int n)
{
int i, j, inc = 0;
for (i = 0; i < n; i++)
{
for (j = 0; j < i; j++)
{
if (A[i] == A[j])
break;
}
if (i == j)
{
B[inc] = A[i];
inc++;
}
}
return inc;
}
int main()
{
int A[100] = {1, 3, 4, 5, 3, 2, 5, 4, 1, 0};
int B[100];
int c;
c = func(A, B, 10);
printf("%d\n", c);
for (int i=0; i<c; ++i)
printf("%d ", B[i]);
fputc('\n', stdout);
}
Выход
6
1 3 4 5 2 0
Это правильный ответ. в исходном массиве шесть уникальных значений, от 0 до 5 включительно.
Мои бедные глаза слишком болят. Застрял здесь примерно на час. Я как бы бездумно смотрю на вещи. Должен ли я отдыхать, когда дела идут плохо?
Из любопытства, где значение
return
, указанное в вашей функции?inc
не собирается возвращаться кmain
по волшебству.