Discussion :

[benois123]

Bonsoir
je suis débutant en C et j'ai un problème avec un programme

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
 
int first(int arr[], int low, int high)
{
  if(high >= low)
  {
 
    int mid = low + (high - low)/2;
 
 
    if ( ( mid == 0 || arr[mid-1] == 0) && arr[mid] == 1)
      return mid;
 
    // if the element is 0, recur for right side
    else if (arr[mid] == 0)
    return first(arr, (mid + 1), high);
 
    else // If element is not first 1, recur for left side
      return first(arr, low, (mid -1));
  }
  return -1;
}
int main()
{ int R=3,C=3;
    int mat[R][C] = { {0, 0, 0, 1},
        {0, 1, 1, 1},
        {1, 1, 1, 1},
        {0, 0, 0, 0}
    };
    int i,k=0,index;
    int tab[5];
    for (i = 0; i < R; i++)
    {
     index = first (mat[i],0, C-1);
     tab[k]=index;
       k++;
      //printf("Index of row is %d \n", index);
 
    }

quand je compile :

attention : passing argument 1 of 'first' from incompatible pointer type [enabled by default]|
note: expected 'int *' but argument is of type 'float *'|
attention : variable 'tab' set but not used [-Wunused-but-set-variable]|
||=== Build finished: 2 errors, 0 warnings (0 minutes, 0 seconds) ===|

[imperio]

Tu déclares un tableau de 4x4 en lui donnant une taille de 3x3. Pour le reste ça compile nickel sur mon pc.

[bpy1401]

Bonjour,

Tu passe en argument un tableau à deux dimensions, mais ta fonction s'attend à un tableau à une dimension.

[imperio]

@bpy1401: j'ai beau chercher, je vois pas du tout où il le fait...

@benois123: la ligne 17 est inutile au passage, le else ne sert à rien, tu peux mettre directement ton return.

[Bktero]

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
int R=3,C=3;
 
    int mat[R][C] = { {0, 0, 0, 1},
        {0, 1, 1, 1},
        {1, 1, 1, 1},
        {0, 0, 0, 0}

Ca, ça fait 4*4 et non 3*3 !

Je n'ai d'ailleurs pas d'erreur avec des flottants, mais comme imperio, des tonnes d'erreurs et de warning à cause de cet extrait de code.

EDIT : il semblerait que gcc n'accepte pas cette écriture, même en donnant la valeur 4 à R et C. En effet, il semble considérer que cette taille variable pourrait contredire la taille de l'initialisation statique et il le fait savoir :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
int main(void)
{ 
 
    int R=4, C=4;
    int mat[R][C] = 
    {   {0, 0, 0, 1},
        {0, 1, 1, 1},
        {1, 1, 1, 1},
        {0, 0, 0, 0}
    };
 
  return 0;
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
14
15
16
17
18
19
20
21
22
23
24
25
26
27
28
29
30
31
32
33
34
35
36
37
38
39
40
41
42
43
44
Compilation finished with errors:
source.c: In function 'main':
source.c:6:5: error: variable-sized object may not be initialized
source.c:7:5: warning: excess elements in array initializer [enabled by default]
source.c:7:5: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:7:5: warning: excess elements in array initializer [enabled by default]
source.c:7:5: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:7:5: warning: excess elements in array initializer [enabled by default]
source.c:7:5: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:7:5: warning: excess elements in array initializer [enabled by default]
source.c:7:5: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:7:5: warning: excess elements in array initializer [enabled by default]
source.c:7:5: warning: (near initialization for 'mat') [enabled by default]
source.c:8:9: warning: excess elements in array initializer [enabled by default]
source.c:8:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:8:9: warning: excess elements in array initializer [enabled by default]
source.c:8:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:8:9: warning: excess elements in array initializer [enabled by default]
source.c:8:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:8:9: warning: excess elements in array initializer [enabled by default]
source.c:8:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:8:9: warning: excess elements in array initializer [enabled by default]
source.c:8:9: warning: (near initialization for 'mat') [enabled by default]
source.c:9:9: warning: excess elements in array initializer [enabled by default]
source.c:9:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:9:9: warning: excess elements in array initializer [enabled by default]
source.c:9:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:9:9: warning: excess elements in array initializer [enabled by default]
source.c:9:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:9:9: warning: excess elements in array initializer [enabled by default]
source.c:9:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:9:9: warning: excess elements in array initializer [enabled by default]
source.c:9:9: warning: (near initialization for 'mat') [enabled by default]
source.c:10:9: warning: excess elements in array initializer [enabled by default]
source.c:10:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:10:9: warning: excess elements in array initializer [enabled by default]
source.c:10:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:10:9: warning: excess elements in array initializer [enabled by default]
source.c:10:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:10:9: warning: excess elements in array initializer [enabled by default]
source.c:10:9: warning: (near initialization for 'mat[0]') [enabled by default]
source.c:10:9: warning: excess elements in array initializer [enabled by default]
source.c:10:9: warning: (near initialization for 'mat') [enabled by default]
source.c:6:9: warning: unused variable 'mat' [-Wunused-variable]

Alors que :

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
6
7
8
9
10
11
12
13
int main(void)
{ 
 
    int R=4, C=4;
    int mat[4][4] = 
    {   {0, 0, 0, 1},
        {0, 1, 1, 1},
        {1, 1, 1, 1},
        {0, 0, 0, 0}
    };
 
  return 0;
}

Code :

Sélectionner tout - Visualiser dans une fenêtre à part

1
2
3
4
5
Compilation finished with warnings:
source.c: In function 'main':
source.c:6:9: warning: unused variable 'mat' [-Wunused-variable]
source.c:5:14: warning: unused variable 'C' [-Wunused-variable]
source.c:5:9: warning: unused variable 'R' [-Wunused-variable]

Avec gcc et les options -std=c99 -Wall -Wextra.

@benois123: la ligne 17 est inutile au passage, le else ne sert à rien, tu peux mettre directement ton return.

Il ne mange pas de pain et aide à la lisibilité du code, bien qu'effectivement inutile

[imperio]

Il ne mange pas de pain et aide à la lisibilité du code, bien qu'effectivement inutile

Je pense que ça dépend de la personne. Pour moi c'est moins agréable, pour d'autre c'est plus clair. Les deux cas sont corrects.