Discussion :

[Flav78]

Bonjour à vous !

Je suis débutant en C et j'essaye d'implémenter la fonction cosinus. Cependant, j'obtiens toujours le message d'erreur : "conflicting types for built-in function 'cos'"

voici mon code :

cos.c

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
#include "power.h"
#include "facto.h"
 
double cos (double x, int n)
{
  double s;
  int i,k;
  s=0;
  k=0;
  for (i=0;i<=n;i=i+2) {s=s+(power(x,i)*power(-1,k))/facto(i); k=k+1;}
  return(s);
}

cosmain.c

Code :

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

1
2
3
4
5
6
7
8
9
10
11
12
13
14
#include <stdio.h>
#include "facto.h"
#include "power.h"
#include "cos.h"
 
int main ()
{
  int n;
  double x;
  printf("Indiquez le calcul souhaité sous la forme cos(x),n\n");
  scanf("cos(%lf),%d",&x,&n);
  printf("%lf\n",cos(x,n));
  return(0);
}

cos.h

Code :

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

double cos (double x, int n);

Je ne vous ai pas remis les fonctions facto et power car je les ai déjà testées et elles fonctionnent (en créant de même un powermain.c)


Voilà j'espère que vous pourrez m'aider. L'erreur est la même et porte sur cos.c et cos.h.

Merci d'avance !

[gabrielS]

Hello Flav78,

Tu essaies (je pense) de redefinir une fonction qui est deja existante dans ton environnement de travail... Essaie en renommant ta fonction cos (appelle la par exemple my_cos) ou, si tu veux garder le meme nom, supprime le header qui contient deja la fonction cosinus

Bonne continuation !

[Flav78]

Merci pour ta réponse rapide !

J'ai remplacé cos par cosinus et effectivement ça marche !
Par contre je ne comprends pas pourquoi ça ne marchait pas avant sachant que je n'avais pas mis #include <math.h> qui contient justement cos.

Merci

[Bktero]

Inclure <math.h> permet de connaitre le prototype de la fonction (et non sa définition [= implémentation] entière). Je pense que l'erreur que tu as a été émise par ton éditeur de liens. Une fois tous les morceaux compilés, cet éditeur les assemble pour créer le programme. A ce moment là, il doit trouver ta fonction cos() et la fonction cos() de sa bibliothèque. Tu dois pouvoir faire la même chose avec malloc() sans inclure stdlib.h et printf() sans inclure stdio.h.

[Obsidian]

Il est aussi probable que tu aies inclus certains headers qui — eux — ont inclus <math.h> à un niveau ou à un autre pour leurs propres dépendances.

Par ailleurs, selon la façon dont ton compilateur est réglé, il se peut qu'il estime que la bibliothèque mathématique fasse partie des fondamentaux également et te l'inclue de façon implicite.

[Sve@r]

Par contre je ne comprends pas pourquoi ça ne marchait pas avant sachant que je n'avais pas mis #include <math.h> qui contient justement cos.

Merci

Salut
Un header ne contient pas de fonction mais seulement leurs signatures (type et paramètres).
Les fonctions sont situées dans des librairies (libm pour le cosinus). Et comme le dit Obsidian, il est probable que ton compilo l'ait incluse automatiquement...

[jlliagre]

Le problème ne vient pas du lien avec la bibliothèque mathématique (libm) ou de l'inclusion automatique de math.h. C'est une particularité de gcc qui est la cause du problème. Ce compilateur redéfinit cos, entre autres fonctions, avec une macro interne pour des raisons d'efficacité.

Il est possible d'éviter le message "conflicting types for built-in function 'cos'" en utilisant l'option fno-builtin-xxx:

Code :

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

gcc ... -fno-builtin-cos ...

[Bktero]

Ce matin, j'ai essayé d'avoir le même message d'erreur avec malloc() ou puts() et je n'y suis pas arrivé. Je me suis ensuite souvenu que gcc définissait sa propre fonction cos() et je me suis demandé si ce n'était pas ça qui causait le warning. Merci d'avoir confirmer cette hypothèse !

[Sve@r]

Code C :

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

for (i=0;i<=n;i=i+2) {s=s+(power(x,i)*power(-1,k))/facto(i); k=k+1;}

Petite astuce: quand deux variables évoluent en parallèles (comme ici "i" et "k"), tu peux les grouper dans le for. Ca accentue la notion de "groupe"...

Code C :

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

1
2
for (i=0, k=0;i<=n;i+=2, k++)
    s=s+(power(x,i)*power(-1,k))/facto(i)

Autre astuce: ici la factorielle de i sera calculée à chaque itération. Donc à l'itération 5, il caclulera facto(5). Et à l'itération 6, il calculera facto(6) ce qui revient à recalculer facto(5) pour multiplier le résultat par 6.
Tu peux éviter ces redondances (surtout pour une factorielle qui est super gourmande en temps). Bon ici c'est plus sioux parce que i monte de 2 en 2 mais c'est quand-même faisable.

Et 3° astuce: au départ de la boucle, i et k valent 0. Donc power(x, 0) vaut 1 et power(-1, k) vaut aussi 1. Et facto(i) vaut aussi 1. Bref s vaut 1. Donc tu peux directement partir de 1...

Ce qui donnerait le code suivant

Code c :

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

1
2
3
4
5
6
7
8
9
10
11
12
double cos (double x, int n)
{
  double s=1.0;
  unsigned int i,k;
  unsigned long f=1;
  for (i=2, k=1;i<=n;i+=2, k++)
 {
    f=f*(i-1)*i;
    s=s+(power(x,i)*power(-1,k))/f;
 }
 return s;
}

Dernière astuce mais là très très subtile: je présume que la variable "n" sert à limiter le développement pour ne pas partir dans l'infini.
Eh bien il y a une façon de limiter le développement sans utiliser de variable choisie par l'utilisateur: utiliser la limite du processeur lui-même. En effet, il te suffit de mémoriser le calcul de chaque itération et ensuite le comparer au calcul suivant. Et lorque le processeur ne sera plus capable de faire la différence, cela veut dire qu'il a atteint sa limite de calcul et dans ce cas on peut quitter la boucle. Ce qui donnerait

Code c :

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
double cos (double x)
{
  double s=1.0;
  double mem=0;
  unsigned int i=2,k=1;
  unsigned long f=1;
  while (1)
 {
    f=f*(i-1)*i;
    s=s+(power(x,i)*power(-1,k))/f;
    if (s == mem) break;
    mem=s;
    i+=2;
    k++;
 }
 return s;
}