Discussion :

[hitachi95]

Bonjour a tous ,
j'ai essayé de réaliser un programme qui insère un nombre x dans un tableau que l'utilisateur donne (l'utilisateur donne des nombre triés)
par exemple si l'utilisateur donne : 1 4 5 6 7
et x=2
le tableau deviendra 1 2 4 5 6 7 .

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
#include <stdio.h>
#include <conio.h>
main()
{
 
 int t[50],x,j,i,i0;  
 int n,*p; 
 p=&t[0];
 printf("Dimension du tableau (max.50) : ");
 scanf("%d", &n );
 for (i=0; i<n; i++){
 scanf("%d", p+i);}
 printf("donnez x ");
 scanf("%d", &x );
for (i=0; i<n; i++)
 printf("%d ", *(p+i));
 for (i=0; i<n; i++){
 if (*(p+i)>x){
 i0=i;
 for (j=n; j>i0; j--);{
 *(p+j)=*(p+j-1);}
 n=n+1;
 break;}}
 *(p+i0)=x;
 for (i=0; i<n; i++)
 printf("\n %d ", *(p+i));
 getch();
}

le programme se compile normalement mais lors de l'execution ça marche pas totalement.

Merci j'éspère que vous saurez m'aider.

[plxpy]

Bonjour

réponse cohérente avec la présentation du source c'est à dire en vrac ! (mais pourquoi tous ces sauts de lignes et caractères espace inutiles ?).

Déjà (mais ce n'est pas tout), le ';' à l'offset 0537 (octal) de ton source (avec fin de ligne sauce Unix, i.e 1 seul caractère (linefeed)) met à mal ton décalage d'éléments.

C'est clair, non ?

Sans plaisanter maintenant : comment veux-tu pouvoir travailler et relire ton source quand il est écrit de la sorte ???

[Neckara]

Bonjour,

je suis d'accord avec plxpy, je ne lis même pas un code source aussi mal indenté.

De plus, c'est bien de donner correctement le prototype de main :

Code :

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

int main(void)

ou

Code :

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

int main(int argc, const char ** argv)

[pythéas]

Bonjour,

En dehors de ce qui a déjà été dit sur la présentation :

- tu devrais afficher ton tableau avant l'insertion de x pour voir à quel moment tes valeurs deviennent incohérentes.

- tu devrais aussi bien regarder cette boucle (j'ai mis vachement de temps à voir ce qui n'allait pas pourtant c'est tout bête) :

Code :

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

1
2
 for (j=n; j>i0; j--);{
 *(p+j)=*(p+j-1);}

- tu devrais enfin te demander ce qu'il se passe si ton x est supérieur à tous les éléments du tableau.

[plxpy]

tu devrais aussi bien regarder cette boucle (j'ai mis vachement de temps à voir ce qui n'allait pas pourtant c'est tout bête)

oui, c'est le :

le ';' à l'offset 0537 (octal) de ton source

qui aurait, sans doute, été évité avec un minimum d'indentation et de "présentation" du code.

Pendant que tu "nettoyes" ton code, hitachi95, oublie ta variable p qui n'est qu'un pointeur sur le premier élément de tab et utilise plutôt tab[i] ou tab+i à la place de *(p+i) ou p+i. C'est tout aussi valable et ça pollue moins le code.

A noter aussi que le décalage des éléments peut se faire via memmove (et pas memcpy à cause du recouvrement entre les zones)