Discussion :

[giroll35]

Bonjour,
Je dois comparer des chaînes de caractères pour les classer dans l'ordre alphabétique dans un tableau.
Mon code semble trier correctement mais mon problème est que je dois limiter l'impression à 100 caractères par chaîne et je ne vois pas comment faire...
Je ne sais pas si je dois limiter le nombre de caractères lors de mon scanf, en supprimer avec ma fonction ou en limiter l'impression avec mon printf.
J'envoie donc tout mon code, désolé.
Si quelqu'un a un conseil, merci d'avance.

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
#include <stdio.h>
#include <string.h>
#include <stdlib.h>
#include <stdbool.h>
int compareMots(const void *p1, const void *p2)
{
   int lettre = 0;
   char* Liste1 = (char*) p1;
   char* Liste2 = (char*) p2;
   int long1 = strlen(Liste1);
   int long2 = strlen(Liste2);
 
 
   while (lettre < long1 && lettre < long2 && Liste1[lettre] == Liste2[lettre]){
      lettre++;
      }
   if (Liste1[lettre] < Liste2[lettre] || Liste1[lettre]=='\0'){
      return false;
      }
   else if (Liste2[lettre] < Liste1[lettre] || Liste2[lettre]=='\0'){
      return true;  
      }
   return 0;
  }
int main(){
   int nbLignes;  
scanf("%d\n", &nbLignes);
char Liste[nbLignes][100];
for (int idLigne = 0; idLigne <= nbLignes; idLigne = idLigne + 1)
   {
      scanf("%s\n", Liste[idLigne]);
 
      }
qsort(Liste, nbLignes, 100 * sizeof(char), compareMots);           
 
for (int a = 0; a <= nbLignes; a = a + 1){
         printf("%s\n", Liste[a]);
      }
   }

[foetus]

je dois limiter l'impression à 100 caractères par chaîne et je ne vois pas comment faire...

Utilise la balise code (le bouton #)

C'est trival

printf("%.100s", str);.

Lis la documentation de printf, lien cplusplus.com en anglais

Édit : c'est trivial, mais il faut bien lire la documentation

[WhiteCrow]

ce n'est pas %100s mais %.100s (pourcent point cent s)

[foetus]

Effectivement je n'avais pas testé et effectivement, sans le point/ la précision, c'est le minimum (avec 1 bourrage d'espaces si la chaîne est + courte) - j'ai encore lu trop vite et en zigzag.

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 <stdlib.h>
 
 
int main(int argc, char* argv[])
{
    char str[] = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. In dignissim libero non leo efficitur pretium. Cras elementum malesuada tellus, vitae venenatis metus efficitur at. Sed tincidunt, purus at ullamcorper venenatis, nulla magna egestas justo, nec aliquam sapien quam interdum tortor. Maecenas nec commodo leo. Quisque molestie nec nulla sed egestas. Aenean nec velit lectus. Morbi convallis ligula non libero viverra pulvinar. Fusce sodales vel est eget dignissim.";
    char small_str[] = "Lorem ipsum dolor sit amet, consectetur adipiscing elit. In dignissim libero non leo efficitur pretium.";
 
    printf ("%100s\n%.100s\n120 here->%120s\n%-120s<-120 here", str, str, small_str, small_str);
 
 
    return EXIT_SUCCESS;
}

[Sve@r]

Bonjour

mais mon problème est que je dois limiter l'impression à 100 caractères par chaîne et je ne vois pas comment faire...
Je ne sais pas si je dois limiter le nombre de caractères lors de mon scanf, en supprimer avec ma fonction ou en limiter l'impression avec mon printf.

La réponse est simple: puisque chaque mot de ton tableau est taillé à 100, c'est lors de la saisie que tu dois gérer. Parce que si tu en reçois 150 que tu tentes de claquer dans le tableau c'est trop tard. Maintenant rien ne t'interdit de faire saisir en illimité dans une zone tampon et ne récupérer (strncpy) que ce qu'il faut.

J'envoie donc tout mon code, désolé.

Mouais. T'as raison d'être désolé. Sans déconner, ça te fait plaisir d'écrire aussi porc ? Tu t'es jamais dit qu'un code bien indenté c'était bien plus agréable à lire et à débugguer ?

Donc les erreurs

• tu tailles ton tableau à "nb" mots et tu boucles de 0 (inclus) à nb (inclus). Compte sur tes doigts. Par exemple de 0 (inclus) jusqu'à 5 (inclus) ça fait combien de doigts ?
• ta fonction de comparaison, outre d'être inutilement compliquée (pourquoi compter les lettres identiques? pourquoi gérer '\0' différemment des autres caractères?), doit renvoyer un nombre <0 si mot1<mot2 et >0 si mot1>mot2 or elle renvoie false/true. Est-ce que ça correspond?

Code c :

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

1
2
3
4
5
6
7
8
9
10
int compareMots(const void *p1, const void *p2) {
	const char *mot1=(char*)p1;
	const char *mot2=(char*)p2;
 
 	do {
		if (*mot1 < *mot2) return -1;
		if (*mot1 > *mot2) return 1;
 	} while (*mot1++ != '\0' && *mot2++ != '\0');
	return 0;
}

Voilà. Ecrire indenté n'empêche pas d'écrire efficace.

[kaitlyn]

Salut,

Mouais. T'as raison d'être désolé. Sans déconner, ça te fait plaisir d'écrire aussi porc ? Tu t'es jamais dit qu'un code bien indenté c'était bien plus agréable à lire et à débugguer ?

Si ce n'est pas lui qui a édité son message pour ajouter les balises de code, alors il n'est pas responsable de cette indentation.

[Sve@r]

Si ce n'est pas lui qui a édité son message pour ajouter les balises de code, alors il n'est pas responsable de cette indentation.

Un code sans balise "code" sera tout à gauche de l'écran mais ses indentations s'il en a n'ont pas disparu, simplement elles n'apparaissent plus dans la page. Suffit de rajouter les balises pour les voir réapparaitre telles qu'elles sont. Autrement dit, rajouter les balises ne change rien aux indentations déjà existantes qui sont donc forcément celles du PO.

[giroll35]

Merci pour les conseils...sur la présentation comme sur le fond de mon problème!