首先,您的程序具有未定义的行为,因为当i 等于8 时,它会尝试访问循环中数组之外的内存。
for(int i = 0; i < 9; i++) {
if(arr[i] == arr[i + 1]) {
^^^^^^
索引等于9 (arr[8 + 1]) 的元素不存在。
而且逻辑是错误的。当 arr[i] 等于 arr[i+1] 时,您将 i 递增两次
for(int i = 0; i < 9; i++) {
^^^^
if(arr[i] == arr[i + 1]) {
newarr[counter] = arr[i];
i += 1;
^^^^^^
counter++;
} else {
但下一个索引为arr[i+2] 的元素也可以等于arr[i]。因此,相同的值将在目标数组中至少写入两次。
例如,尝试将您的程序应用于这样的数组
int arr[] = { 1, 1, 1, 1, 1, 1 };
所以你需要完全重写你的程序。:)
至于递归函数可以如下所示
#include <stdio.h>
int * unique( const int *a, size_t n, int *b )
{
if ( n == 0 ) return b;
if ( n == 1 || a[0] != a[1] ) *b++ = *a;
return unique( a + 1, n - 1, b );
}
int main( void )
{
int a[] = { 1, 1, 5, 4, 7, 7, 9, 9, 8 };
const size_t N = sizeof( a ) / sizeof( *a );
for ( size_t i = 0; i < N; i++ ) printf( "%d ", a[i] );
printf( "\n" );
int b[N];
int *last = unique( a, N, b );
for ( int *first = b; first != last; ++first ) printf( "%d ", *first );
printf( "\n" );
}
它的输出是
1 1 5 4 7 7 9 9 8
1 5 4 7 9 8
如果你的编译器不支持 C99 标准,那么程序可能看起来像
#include <stdio.h>
int * unique( const int *a, size_t n, int *b )
{
if ( n == 0 ) return b;
if ( n == 1 || a[0] != a[1] ) *b++ = *a;
return unique( a + 1, n - 1, b );
}
#define N 9
int main( void )
{
int a[N] = { 1, 1, 5, 4, 7, 7, 9, 9, 8 };
int b[N];
size_t i;
int *first, *last;
for ( i = 0; i < N; i++ ) printf( "%d ", a[i] );
printf( "\n" );
last = unique( a, N, b );
for ( first = b; first != last; ++first ) printf( "%d ", *first );
printf( "\n" );
}