Ce site est optimisé pour être consulté depuis un navigateur moderne dans lequel JavaScript est activé.

/* Topic des codeurs [8] */

The Uploader

Bienvenue dans le TdC 0x8.
Ceci est la suite de ce fil.

Historique des précédents fils :
• Topic des codeurs couche-tard [0] : du 14 avril 2010 au 12 juin 2010 (100 pages).
• Topic des codeurs couche-tard [1] : du 12 juin 2010 au 5 septembre 2010 (100 pages).
• Topic des codeurs couche-tard [2] : du 5 septembre 2010 au 16 décembre 2010 (100 pages).
• Topic des codeurs couche-tard [3] : du 16 décembre 2010 au 28 février 2011 (100 pages).
• Topic des codeurs couche-tard [4] : du 28 février 2011 au 6 juin 2011 (101 pages).
• Topic des codeurs couche-tard [5] : du 6 juin 2011 au 10 septembre 2011 (100 pages).
• Topic des codeurs [6] : du 10 septembre 2011 au 4 février 2012 (100 pages).
• Topic des codeurs [7] : du 04 février 2012 au 17 novembre 2012 (100 pages).

N’oubliez pas les Défis du TdC 😉

Le défi actuel est toujours :bah voui, faut croire que les défis sont secondaires :-P

grim7reaper a écritJe propose la mise au point d’une bibliothèque ou d’un module ou truc du genre (faut que le code soit réutilisable ailleurs quoi) qui propose des fonctions pour récupérer des quotes sur Internet.

Vous êtes libre de choisir les sites que vous voulez gérez dans votre code (VDM, DTC, PEBKAC, etc.), au niveau de leur nombre (gestion d’un seul site ou de plusieurs) aussi et au niveau des options offertes (par exemple, pour VDM vous pouvez soit toujours tirer une quote aléatoire soit offrir la possibilité de choisir sa catégorie).

Le seul truc que je fixe c’est le format de sortie : vos fonctions doivent fournir au final juste la quote en texte simple.
Comme ça, c’est plus souple pour la réutilisation ou la combinaison avec d’autres trucs (pour donner des trucs comme ça par exemple).

Amusez-vous bien, et produisez-nous du beau code. 🙂

Et n’oubliez pas notre devise :

#!/usr/bin/env ruby
devise = ['In', 'Code', 'We', 'Trust!'] 
loop do
  devise.each do |str|
    puts str
    sleep 1
  end
end

:!pakman

Une question existentielle me vient soudain à l'esprit :
Le Topic des codeurs [0] existe t-il ?
😛

Edit : * facepalm * Exacte grim, la réponse est dans les liens, m'apprendra à lire 😐

afilmore

The Uploader a écritEt n’oubliez pas notre devise :

#!/usr/bin/env ruby
devise = ['In', 'Code', 'We', 'Trust!'] 
loop do
  devise.each do |str|
    puts str
    sleep 1
  end
end

public static int main (string[] args) {

    string [] devise = {"In", "Vala", "We", "Trust!"};
    foreach (string str in devise) {
        
        stdout.printf ("%s\n", str);
        Thread.usleep (1000000);
    } 
    
    return 0;
}

valac devise.vala && ./devise

😛

The Uploader

Si je compte que les lignes de code (pas le shebang de ma version, et pas tes lignes vides), tu as 8 lignes là où j'en ai 7. :cool:

La ligne en trop :

public static int main (string[] args)

Quant au lignes ruby en elles-mêmes, elles sont bien moins bavardes et pourtant tout à fait claires. :p

(sinon faudrait que je commence à faire du vala. Le peu que j'ai testé m'a plu. Surtout que depuis la sortie de xfce4-vala-bindings 4.10, ça m'donne encore plus envie. Mais paraît que ça fait du C pas propre... Puis j'ai pas trop d'idée d'application Xfce à faire en fait...)

afilmore

L'explorateur de fichiers à gauche et le Panel que tu vois ici, c'est du Vala :
http://pix.toile-libre.org/upload/original/1350033461.png

L'explorateur de fichier est un fork de PcManFm et le Panel est codé avec mes pieds.

https://code.launchpad.net/~axel-filmore

C'est encore très expérimental. 😛

grim7reaper

:!pakman a écritUne question existentielle me vient soudain à l'esprit :
Le Topic des codeurs [0] existe t-il ?
😛

Bah suffit de regarder les liens, et tu verras que oui 😉

afilmore a écrit

public static int main (string[] args) {

    string [] devise = {"In", "Vala", "We", "Trust!"};
    foreach (string str in devise) {
        
        stdout.printf ("%s\n", str);
        Thread.usleep (1000000);
    } 
    
    return 0;
}

valac devise.vala && ./devise

😛

T’auras qu’a ouvrir le prochain si tu veux caler du Vala dans le post d’ouverture :p

The Uploader a écritMais paraît que ça fait du C pas propre... Puis j'ai pas trop d'idée d'application Xfce à faire en fait...)

Y’a deux ans c’était le cas (c’était utilisé pour Vala, et le code généré faisait des warnings à la compilation, c’était très chiant…), je ne sais pas ce que ça donne maintenant.

Mindiell

drapal

Shanx

Bonjour,
aujourd'hui j'ai tenté de faire un mastermind en C. Sur le papier, mon algo me semble correct, mais dans les faits ça ne marche pas des masses... Bon, je trouve que ce que j'ai fait est assez compliqué, je suis sûr qu'on pourrait faire plus simple. Mais avant d'améliorer tout ça, j'aimerais rendre mon code fonctionnel et comprendre pourquoi ça ne fonctionne pas.
Pour expliquer vaguement, je travaille avec deux tableaux : un constant où il y a les couleurs à deviner (t1), l'autre qui varie à chaque essai (t2) et qui correspond aux couleurs entrées par l'utilisateur. Il y a une liste chaînée qui reprend le premier tableau. Ainsi, quand on parcourt les deux tableaux à la recherche de couleurs en commun, on compare t2 et la liste chainée. Si on en trouve une couleur en commun, on retire la case correspondante au niveau de la liste, histoire d'éviter de la comptabiliser plusieurs fois. Je ne vois pas où est la faille (si c'est au niveau de l'algo ou du code), mais je sèche complètement malgré le temps passé dessus... :/
Voici le code :

/**** MASTERMIND ****/

#include<stdio.h>
#include <stdlib.h>
#include <time.h>
#include <string.h>

#define NOMBRE_CASE 2

/* Déclaration d'une liste chainée */
/* J'utilise une liste chainée pour pouvoir retirer les couleurs déjà trouvées */
/* Une partie du code concernant les listes chainées provient du sdz */
typedef struct element element;
struct element {
	int val;
	struct element *nxt;
};
typedef element* llist;

/* Pour remplir la liste */
llist ajouterEnFin(llist liste, char valeur) {
	element* nouvelElement = malloc(sizeof(element));
	nouvelElement->val = valeur;
	nouvelElement->nxt = NULL;
	if(liste == NULL) {
		return nouvelElement; 
	}
	else {
		element* temp=liste;
		while(temp->nxt != NULL) {
			temp = temp->nxt;
		}
	temp->nxt = nouvelElement;
	return liste;
	}
}

/* Pour supprimer un élément */
llist supprimerElement(llist liste, char i) {
	/* Si la liste est vide, on retourne NULL */
	if(liste == NULL)
		return NULL;
	/* Si la liste contient un seul élément */
	if(liste->val == i) {
		free(liste);
		return NULL;
	}
	/* Si la liste contient au moins deux éléments */
	element* tmp = liste;
	element* ptmp = liste;
	while(tmp->val == i | tmp->nxt != NULL) {
		ptmp = tmp;
		tmp = tmp->nxt; 
	}
	if(tmp->val == i) {
		ptmp->nxt = tmp->nxt;
		free(tmp);
	}
	return liste;
}

/* Pour afficher la liste */
void afficherListe(llist liste) {
	element *tmp = liste;
	while(tmp != NULL){
		printf("%d ", tmp->val);
		tmp = tmp->nxt;
	}
}

int main(void) {
	int i, j, a; /* i et j sont des compteurs */
	int z = 0; /* booléen pour savoir quand l'user a trouvé */
	int b, r; /* Nombre de rouge, de blanc */
	char t1[NOMBRE_CASE], t2[NOMBRE_CASE]; /* t1 tableau de départ, t2 tableau proposé par l'user */

	/* Création de la liste chainée */
	llist liste = NULL;

	/* Préparation des couleurs à trouver */
	srand(time(NULL));
	for (i=0; i<NOMBRE_CASE ; i++) {
		a=(rand() % NOMBRE_CASE) + 1;
		printf("%d ", a);
		if (a==1){
			t1[i]='R';
		}
		else {
			t1[i]='B';
		}
	}

	/* Affichage du tableau ainsi fait (à supprimer après) */
	puts(" ");
	puts("Le tableau de départ est :");
	for (i=0; i<NOMBRE_CASE ; i++) {
		printf("%c - ", t1[i]);
	}
	puts(" ");

	/* On remplit la liste chainée */
	for (i=0; i<NOMBRE_CASE ; i++) {
		ajouterEnFin(liste, t1[i]);
	}
	afficherListe(liste); /* Affichage de la liste pour vérifier ; ne fonctionne pas pour l'instant */

	/* À l'utilisateur de jouer */
	puts("À toi de jouer.");
	
	while (z == 0){
		b = 0;
		r = 0;
		/* Remplissage du tableau */
		for (j=0; j<NOMBRE_CASE ; j++) {
			while (t2[j] != 'B' && t2[j] != 'R' /*&& strlen(t2)>2*/) { 
				printf("Que proposes-tu pour la case %d ? ", j+1); 
                                fgets(t2 +j, sizeof t2 + j, stdin);
			}
		}
		/* Comparaison des tableau */
		for (i=0; i<NOMBRE_CASE ; i++) {
			for (j=0; j<NOMBRE_CASE ; j++) {
				if (t2[i] == t1[j]) {
					/* On supprime l'élément de la liste chainée correspondant, 
					pour ne pas qu'il soit recomparé plus tard */
                                        printf("Supression de %c\n", t1[j]);
					supprimerElement(liste, t1[j]);
					/* On incrémente les compteurs de couleurs trouvées */
					if (i == j) {
						r++; /* Si la couleur est à la bonne place, un rouge en plus */
					}
					else if (i != j) {
						b++; /* Si la couleur n'est pas à la bonne place, un blanc en plus */
					}
				}
			}			
		}	
		/* On vérifie si les tableaux sont identiques */
		/* S'ils le sont, tous les éléments de la liste ont été supprimés */
		for (i=0; i<NOMBRE_CASE ; i++) {	
			if (t2[i] == t1[i]) {
				z = 1;
			}
			else {
				printf("Le nombre de pion bien placé est %d, celui de pion mal placé est %d.\n", r, b);
			}
		}
	}
	puts("Bravo, tu as trouvé !");
        return 0;
}

Voilà ce que j'obtiens quand je lance le programme :

$ ./mastermind
1 1  
Le tableau de départ est :
R - R -  
À toi de jouer.
Que proposes-tu pour la case 1 ? B
Que proposes-tu pour la case 2 ? Que proposes-tu pour la case 2 ? R
Supression de R
Supression de R
Le nombre de pion bien placé est 1, celui de pion mal placé est 1.
Bravo, tu as trouvé !

Quelqu'un peut m'aider ?

afilmore

grim7reaper a écritle code généré faisait des warnings à la compilation, c’était très chiant…), je ne sais pas ce que ça donne maintenant.

Il y a parfois quelques bugs dans les bindings, toujours quelques warnings pendant la compilation du code C mais ça c'est rien par rapport aux messages de "déprécation" des mecs de GTK+. :mad:

Une autre difficulté c'est de localiser les fuites mémoires, le même problème qu'en C.

Voili.

grim7reaper

Pour ça :

Que proposes-tu pour la case 1 ? B
Que proposes-tu pour la case 2 ? Que proposes-tu pour la case 2 ? R

C’est parce que tu ne vides pas le buffer clavier entre 2 appels.
Tu tapes B puis Entrée, tu lis B mais après le buffer contient Entrée (que tu lis lorsque tu demandes la case 2) puis il te redemande la case 2.
Pour éviter purge le buffer. Fait comme ça, et n’utilise surtout pas fflush (même si c’est recommandé par un bon paquet d’ignorant, dont une masse incroyable de prof’) car il c’est un UB (Undefined Behavior) sur les flux d’entrées.

Au passage, le fgets pour lire une seul caractère : bof…
Fait plutôt :

t2[j] = getchar();

Je regarde la suite après, si quelqu’un ne passe pas avant.

Édit : Ha ! Et ici je sûr à 100% qu’il te manque un pipe :

	while(tmp->val == i | tmp->nxt != NULL) {

@afilmore : pour les messages de déprécations, c’est pas difficile à éviter, suffit de pas appeler de vieilles fonctions.

grim7reaper

Bon, partie 2 :

		for (i=0; i<NOMBRE_CASE ; i++) {	
			if (t2[i] == t1[i]) {
				z = 1;
			}
			else {
				printf("Le nombre de pion bien placé est %d, celui de pion mal placé est %d.\n", r, b);
			}
		}

Imagine, le premier est bon, le second non.
Tu mets z à 1 au premier tour, au second tu ne le changes pas.
En sortie de boucle z vaut 1, donc tu affiches victoire : remet z à zéro dans le else.
Après cette modif’, tu as le problème inverse : si le premier est mauvais mais le second bon. Tu vas mettre z à 1 et sortir.
Je te conseille de faire un z++ si les lettres sont égales, et dans ton while tu sors si z vaut NOMBRE_CASE.

Deuxième problème, qui va te foutre dans une boucle infinie (si tu ne vois pas pourquoi, teste) : tu ne remets pas t2 à zéro.
Un coup de memset et on n’en parle plus.

Voilà le code mis à jour :

        for(i = 0; i < NB_CELL; i++)
        {
            if (t2[i] == t1[i])
                z++;
            else {
                printf("Le nombre de pion bien placé est %d, celui de pion mal placé est %d.\n", r, b);
                z = 0;
            }
        }
        memset(t2, 0, sizeof t2);

Sinon, évite les caractères non-ASCII dans les chaînes littérales de type char, la sortie va dépendre de l’encodage du terminal, c’est moche.
En fait, évite le français tout court (surtout pour le code lui-même).
Et évite les noms de variables d’une seule lettre comme r et b (red et black c’est beaucoup plus parlant…), sauf pour les compteurs et 2 -3 exceptions.

Shanx

Merci beaucoup grim !
C'est presque bon, il reste juste un problème...

$ ./mastermind                  
1 1  
Le tableau de départ est :
R - R -  
À toi de jouer.
Que proposes-tu pour la case 1 ? B
Que proposes-tu pour la case 2 ? B
Le nombre de pion bien placé est 0, celui de pion mal placé est 0.
Le nombre de pion bien placé est 0, celui de pion mal placé est 0.
Que proposes-tu pour la case 1 ? R
Que proposes-tu pour la case 2 ? B
Supression de R
Supression de R
Le nombre de pion bien placé est 1, celui de pion mal placé est 1.
Que proposes-tu pour la case 1 ?

Il y a deux fois "Supression de R" (au lieu d'une), et normalement il ne devrait pas y avoir de pion mal placé...

Mindiell

Au hasard, tu vides le premier R de ta liste chainée, mais il y en a un deuxième qu'il recomptabilise lors du test suivant. Il faudrait aussi vider ton tableau 2 😉 (ou alors tu remplaces par un X

Shanx

Mindiell a écritAu hasard, tu vides le premier R de ta liste chainée, mais il y en a un deuxième qu'il recomptabilise lors du test suivant. Il faudrait aussi vider ton tableau 2 😉 (ou alors tu remplaces par un X

Mais t2 est déjà vidé par le memset que m'a donné Grim...

Mindiell

Ah ben j'i pas lu, c'tait trop long.
Affiche tes 2 tableaux à chaque fois, avant et après pour être sûr que tout est ok alors 😉

tshirtman

@Grim: connais-tu, as-tu un avis? http://shop.oreilly.com/product/0636920025108.do?green=07806831-7B62-53F9-A415-41CC4E2BCE2D&intcmp=af-mybuy-0636920025108.IP (mon avis est que c'est un peu cher, mais bon ^^)

xapantu

afilmore a écrit
grim7reaper a écritle code généré faisait des warnings à la compilation, c’était très chiant…), je ne sais pas ce que ça donne maintenant.
Il y a parfois quelques bugs dans les bindings, toujours quelques warnings pendant la compilation du code C mais ça c'est rien par rapport aux messages de "déprécation" des mecs de GTK+. :mad:

Boarf, ça se discute, une fois que tu as quelques dizaines de milliers de lignes de code, il y a quand même pas mal de warning 😛

Et sinon, faut pas croire tout ce qui est écrit dans leur manuel, le stdout.printf, c'est peut être vaguement plus propre, mais il faut juste utiliser print, ça va un peu plus vite à écrire 🙂

Pylades

Shanx, ton paste pour le pendu a expiré. 😐

grim7reaper

tshirtman a écrit@Grim: connais-tu, as-tu un avis? http://shop.oreilly.com/product/0636920025108.do?green=07806831-7B62-53F9-A415-41CC4E2BCE2D&intcmp=af-mybuy-0636920025108.IP (mon avis est que c'est un peu cher, mais bon ^^)

Non, je ne connais pas.
À vue de nez le contenu semble pas mal, mais la partie « Use Autotools » me choque un peu pour un truc qui se veut moderne… Il aurait plutôt intérêt à présenter CMake ou Scons :/

xapantu a écrit
afilmore a écrit
grim7reaper a écritle code généré faisait des warnings à la compilation, c’était très chiant…), je ne sais pas ce que ça donne maintenant.
Il y a parfois quelques bugs dans les bindings, toujours quelques warnings pendant la compilation du code C mais ça c'est rien par rapport aux messages de "déprécation" des mecs de GTK+. :mad:
Boarf, ça se discute, une fois que tu as quelques dizaines de milliers de lignes de code, il y a quand même pas mal de warning 😛

Pas si tu codes proprement…
Tu peux en avoir quelques uns ouais, si ta ligne de compil’ est super stricte, mais sinon c’est que tu codes de manière un peu à l’arrache.

grim7reaper

Y’a des gens très très fort 😐

Page suivante »