为什么我的代码不打印排序后的数组?

时间:2018-10-06 17:44:44

标签: c arrays function pointers multidimensional-array

为什么代码不打印任何内容?这段代码应该做的就是以这种方式对2d数组进行排序: 2d数组表示一个{x,y},那么代码需要对其进行排序,包含x <0的行需要排在第一位,而x> = 0的行需要排在后面。 这里的交换功能是在两行之间交换。现在,当我尝试打印排序后的数组时,输出中什么也没得到

#include <stdio.h>
void swap(int p1[], int p2[]);
int arrange(int p[][2], int n);


void swap(int p1[], int p2[]){
  for(int i=0; i<2; i++){
    int temp=p1[i];
    p1[i]=p2[i];
    p2[i]=temp;
  }
}

int arrange(int p[][2], int n){
  int idx=0;
  for(int i=0; i<n; i++){
    if(p[i][0] >= 0 && (i+1)<n)
        if(p[i+1][0] <0) {
            swap(&p[i][0],&p[i+1][0]);
            idx++;
        }
    else if(p[i][0]<0)
        idx++;
  }
return 1;
}

int main()
{
  int a[4][2]={{1,2},{6,7},{-10,5},{0,1}};
  arrange(a[4][2], 4);
  for(int i=0; i<4; i++){
        printf("{%d, %d}, ", a[i][0], a[i][1] );
  }

}

1 个答案:

答案 0 :(得分:3)

  

为什么代码不打印任何内容?

因为它无法编译,因为您的编译器应该给出错误和警告。

为了解决错误,请更改此内容:

arrange(a[4][2], 4);

对此:

arrange(a, 4);

附录:

这是传递Wall标志的GCC警告给我的:

prog.cc: In function 'int arrange(int (*)[2], int)':
prog.cc:17:7: warning: suggest explicit braces to avoid ambiguous 'else' [-Wdangling-else]
17 |     if(p[i][0] >= 0 && (i+1)<n)
   |

为了解决该警告,我将您的代码更改为:

if(p[i][0] >= 0 && (i+1)<n) {
    if(p[i+1][0] <0) {
        swap(&p[i][0],&p[i+1][0]);
        idx++;
    }
} else if(p[i][0]<0) {
    idx++;
}
相关问题