Problème : Mon prog ne marche pas.

Question

Bonjour,

VOici un programme qui lit le code source d'une page web, et est sensé repérer une balise annonçant un lien.
Il ne marche pas. Quelqu'un peut il me dire pourquoi.

EL barto ! · Answer

au fait ! voici le code du programme : #include #include #include #include //Variables globales char buffer[50000]; int offset = 0; /*là c'est la définition de la fontion buffer_write pour allouer la bonne taille au buffer*/ size_t buffer_write (const void *ptr, size_t size, size_t nmemb, FILE *stream) { int i; char *myptr = (char *) ptr; for (i=0; i < size * nmemb; i++) { if (i + offset >= sizeof(buffer)) break; buffer[i + offset] = myptr[i]; offset++; } return size * nmemb; } int main(int argc, char *argv[]) { char url[1024]; int u; int y; const int TAILLE_CHAINE_MAX = 55555; char * debut = "

Utilisateur anonyme · Answer

copie le programme ic pour que chacun puisse l'analyser!

Utilisateur anonyme · Answer

Si tu oublies de coller ton code qui ne fonctionne pas correctement ça va être difficile

EL barto ! · Answer

En fait il est fait à partire de 2 programmes :

le prog 1 : permet de lire le code source d'une page web et de l'afficher dans un fenêtre d'execution mais NE LE TELECHAGRE PAS ! Il ne fait que le lire.

le prog 2 : sert à repérer toutes les 'sous_chaine' ( c'est à dire l'adresse des liens' compris entre les caractère 'a href=' et ' " '.

si vous n'y comprenez rien, je peux vous poster les deux programmes qui marchent séparément  (ça je suis sûr qu'il marchent, je les ai testé).

Qu'en pensez vous ? peut être que ça serait mieu pour que vous puissiez juger non ?

EL barto ! · Answer

Oui, ça marche mais dans une page autonome. En fait ce n'est pas un téléchargement. tiens je te colle, son code. en fait si il y a un erreure ça doit être dans la fusion des deux programmes. VOici, le code qui permet de LIRE : #include #include #include #include //Variables globales char buffer[50000]; int offset = 0; /*là c'est la définition de la fontion buffer_write pour allouer la bonne taille au buffer*/ size_t buffer_write (const void *ptr, size_t size, size_t nmemb, FILE *stream) { int i; char *myptr = (char *) ptr; for (i=0; i < size * nmemb; i++) { if (i + offset >= sizeof(buffer)) break; buffer[i + offset] = myptr[i]; offset++; } return size * nmemb; } /*là c'est l'ouverture de la séssion vers l'url, et la lecture grâce à la fonction buffer_write*/ int main(int argc, char **argv) { char url[1024]; int i; scanf("%s", &url); CURL *session = curl_easy_init(); /*Initialisation de la session */ curl_easy_setopt(session, CURLOPT_URL, url); /*Ici on précise l'url */ curl_easy_setopt(session, CURLOPT_WRITEDATA, stdout); //On fait semblant d'avoir un fichier curl_easy_setopt(session, CURLOPT_WRITEFUNCTION, &buffer_write); //On passe notre fonction curl_easy_perform(session); curl_easy_cleanup(session); /*Enfin, on libère nos ressources utilisées */ for (i=0; i #include #include #include int main(int argc, char *argv[]) { int y; const int TAILLE_CHAINE_MAX = 555; char * debut = " J'ai di commettre des erreure en essayant de des les intégrer l'un à l'autre, car il y a des variable qui avait le même non. dans la version que je t'ai envoyé, ça bloque à la ligne int u = 0;

Utilisateur anonyme · Answer

Je sais bien, j'appelle ça "télécharger" quand même à partir du moment où tu obtiens le code source de la page. J'ai déjà utilisé plusieurs fois cette libraire libcurl ;-).

Je viens de remarquer quelque chose qui peut avoir son importance mais je ne sais pas si ça vient de là.
Ton buffer ne fait "que" 50000. Ce qui est peu pour une page web. Par exemple, ce sujet (au moment où j'écris ce message) fait 112 542 octets. Dans mes programmes je fais un char *source = malloc(500000);. Si, dans les pages que tu testes, les liens sont après le 50millième caractère, tu ne les auras pas dans ton buffer.

EL barto ! · Answer

En fait  je n'arrive même pas à tester puisque le compilo s'arrête à la ligne :
int u = 0;

le prg ne se lance tout simplement pas.

EL barto ! · Answer

dans quels programme tu regardes ?

J'ai beau me crever les yeux  ( :)  ) je ne trouve pas les deux bout de code que tu m'a signalé ?
Es tu sûr que tu regardes dans la première version ( la version totale que j'ai envoyé des mon second message) ?

EL barto ! · Answer

OK merci ! 

ça y 'est je les vois !

Donc ces deux lignes en gras je devrais les mettre où exactement ?
et puis si tu n'y vois pas d'inconvenient je crois que je vais arrêter la pour aujourd'hui.

tu es présent sur le forum demain ?

EL barto ! · Answer

Non, ça ne fait rien.

ça bloque toujours à la même ligne

EL barto ! · Answer

En fait( une précision), je ne sias pas si tu as remarqué, mais dans la version complète, j'ai remplacé tout les i par des u (istoire qu'il n'y ai aucune confusion de variable, cela ne concerne évidemnt que les i du prog servant la création du buffer.

Et bien le compilo (il s'agit de dev ++) ne dit que :

in function main
lorsqu'il s'arrête à la ligne:
 int u = 0;

EL barto ! · Answer

non c'est plutôt l'inverse je veux dire, j'ai changé tou les i en u pour la fonction qui reconnait la sous chaine.

une fois encore sir tu essaye les deux programme séparément il marchent. Mais c'est dans la version où je les ai mis dans la même feuille de code que ça ne marche pas.

EL barto ! · Answer

J'ai la version 4.9.9.2 de dev ++

mype · Answer

ça doit etre parce que tu declare deux fois u...
int main(int argc, char *argv[])
{
    char url[1024];
    int u; // <-----------------------ICI
   
    int y;
    const int TAILLE_CHAINE_MAX = 55555;

    char * debut = "<a href=\"";
    char * fin = "\"";
    char buffer[TAILLE_CHAINE_MAX];
    buffer[50000] = -1;
    buffer[TAILLE_CHAINE_MAX-1] = '\0';
    char sous_chaine[TAILLE_CHAINE_MAX];

    int u = 0; // <----------------------------- et LA
    int indice = 0;
    int taille = strlen( debut );
    int ok = 0;
le deuxieme ça serait plutot u=0 je pense

EL barto ! · Answer

JE te remet le code sur la page de discussion, tel quel. Tu n'as qu'a le copier et le coller. (Mais je te l'avais déjà mis dès la seconde reponse). tiens. le voici texto ( je ne sais pas comment me servir de la page que tu as envoyé) #include #include #include #include //Variables globales char buffer[50000]; int offset = 0; /*là c'est la définition de la fontion buffer_write pour allouer la bonne taille au buffer*/ size_t buffer_write (const void *ptr, size_t size, size_t nmemb, FILE *stream) { int i; char *myptr = (char *) ptr; for (i=0; i < size * nmemb; i++) { if (i + offset >= sizeof(buffer)) break; buffer[i + offset] = myptr[i]; offset++; } return size * nmemb; } int main(int argc, char *argv[]) { char url[1024]; int u; int y; const int TAILLE_CHAINE_MAX = 50000; char * debut = "

EL barto ! · Answer

maintenant que je remplace 'int u=0' par 'u=0', le compilateur s'arrête à cette ligne :

  if ( strncmp( buffer[50000], debut, taille ) == 0 ) {

il indique encore 
'in function main'

EL barto ! · Answer

Bon, il s'est passé plusieurs truc.

ne bougez pas je vais chercher mon clébar qui s'est encore faufilé sous le grillage !

J'ne ai pour aproximativement 20 minutes

EL barto ! · Answer

Bon me revoilà. 

non, je crois que vous n'avez pas vraiment compris comment ce programme a été construit, et c'est pour ça que vous avez du mal à "entrer" conceptuellement dans les variables.

le mieux, c'est que je vous passe les programme détaillé l'un aprés l'autre.

Ils sont assez simple à comprendre et ne font qu'une 15 n de lignes.
Quest ce que vous en pensez ?

(parce que j'ai fait vos modification, et elles m'ont amené à d'autres, et au bout du compte plus rien ne fonctionnait.)

EL barto ! · Answer

Je pige pas comment elle marche la page que tu m'envoies  ! :)

il me faut ton adresse non ?
c'estpas plus simple si je te poste les deux pregramme séparemment ?

EL barto ! · Answer

Bon ok, mais je te l'ai pourtant envoyé !!! tu ne l'a pas recu ? Bon voilà le PRG N°1 : connection_lecture d'un code source d'un page web : #include #include #include #include //Variables globales char buffer[4096]; int offset = 0; /*là c'est la définition de la fontion buffer_write pour allouer la bonne taille au buffer*/ size_t buffer_write (const void *ptr, size_t size, size_t nmemb, FILE *stream) { int i; char *myptr = (char *) ptr; for (i=0; i < size * nmemb; i++) { if (i + offset >= sizeof(buffer)) break; buffer[i + offset] = myptr[i]; offset++; } return size * nmemb; } /*là c'est l'ouverture de la séssion vers l'url, et la lecture grâce à la fonction buffer_write*/ int main(int argc, char **argv) { char url[1024]; int i; scanf("%s", &url); /*là on indique l'url de la page qu'on veut lire/* CURL *session = curl_easy_init(); /*Initialisation de la session */ curl_easy_setopt(session, CURLOPT_URL, url); /*Ici on précise l'url */ curl_easy_setopt(session, CURLOPT_WRITEDATA, stdout); //On fait semblant d'avoir un fichier curl_easy_setopt(session, CURLOPT_WRITEFUNCTION, &buffer_write); //On passe notre fonction curl_easy_perform(session); curl_easy_cleanup(session); /*Enfin, on libère nos ressources utilisées */ for (i=0; i" et "< itre>" ------------------------------------------------------------------------------------------------------------------------------------------------------- #include #include #include int main(int argc, char *argv[]) { int y; const int TAILLE_CHAINE_MAX = 555; char * debut = ""; char * fin = "< itre>"; char chaine[TAILLE_CHAINE_MAX]; chaine[0] = -1; chaine[TAILLE_CHAINE_MAX-1] = '\0'; char sous_chaine[TAILLE_CHAINE_MAX]; int u = 0; int indice = 0; int taille = strlen( debut ); int ok = 0; printf( "Ce programme trouve une sous-chaine comprise entre les chaines %s et %s. ", debut, fin ); printf( "Ecrivez un texte de la forme : [...] %s [...] %s [...] (Maximum %i caracteres) ", debut, fin, TAILLE_CHAINE_MAX ); // Tant que le caractère saisi n'est pas un saut de ligne et qu'on n'a pas atteint // le nombre de caractères maximum, on lit et on enregistre dans le tableau. scanf("%c", &chaine[u]); while ( chaine[u] != ' ' && u et < itre>.

mype · Answer

moi je comprend pas pourquoi tu modifie ton programme dans cette partie...
 while ( u < strlen( chaine ) && ok == 0 ) {
        // Si on trouve debut dans la chaine saisie
        if ( strncmp( &chaine[u], debut, taille ) == 0 ) {
            u = u + taille;
            taille = strlen( fin );
            // On cherche maintenant la chaine fin
            while ( u < strlen( chaine ) ) {
                sous_chaine[indice] = chaine[u];
                // Si on trouve la chaine fin...
                if ( strncmp( &chaine[u], fin, taille ) == 0 ) {
                    // ...on enregistre l'indice
                    sous_chaine[indice] = '\0';
                    // ...on sort
                    ok = 1;
                    break;

la tu fais bien des strncmp( &chaine[u], fin, taille )  mais dans ton code final quand tu mixe les deux programme tu fais strncmp( buffer[50000], fin, taille )  pourquoi enlevé le & ?

EL barto ! · Answer

POUrquoi je modifie ?

 ha la la :)

Si je le savais. Je pense que si je remplace chaine par buffer,

-chaine représente la chaine de caractère total dans le programe 2
-buffer représente l'espace mémoire alloué à la lecture des 50 000 caractère.

Donc je me suis dit qu'il fallait que ces deux variable n'en soit qu'une dans le programme.
Mais peut être que je ne devrait pas. TU as essayé sans modifier?

mype · Answer

je peux pas tester ton programme j'ai pas la lib ;)

EL barto ! · Answer

Aïe !

tu ne devrais pas sortir à poil !:)

mype · Answer

ben j'ai pas encore eu a utiliser le C sur des page web c'est pour ça ;)
mais essaye et tu verras bien ça change quelquechose

Utilisateur anonyme · Answer

En fait je ne comprends pas pourquoi tu as deux programmes. J'ai déjà fait plusieurs programmes utilisant libcurl (entre autre pour exploiter dans mes programmes des résultats des recherches sur google).

Il suffit d'une fonction qui te télécharge le code source de la page et qui te l'écrit dans un char*.
Après tu n'as qu'à parcourir ce char* dans ta main (ou dans une autre fonction appelée depuis la main).

Je ne voudrais pas que tu te décourages mais ta "while if while if", pour ma part je la trouve tellement laide que je n'ai pas envie de me pencher à fond dessus, résultat : je ne la comprends pas. Il ne faut pas oublier qu'un bon code est un code bien structuré et facilement "reprenable", et éventuellement commenté (mais ce n'est pas parce qu'on commente qu'il faut pour autant se permettre de faire du code moche).


size_t ecrire(void *ptr, size_t size, size_t nmemb, void *stream)
{
	sprintf(stream, "%s%s", stream, ptr); //concatène stream et str dans stream

	return nmemb;
}

void dlwebsite(char *url, char *source)
{
	CURL *session;
	CURLcode res;

	source[0] = '\0'; //initialisation source vide

	session = curl_easy_init();
	curl_easy_setopt(session, CURLOPT_URL, url);
	curl_easy_setopt(session, CURLOPT_WRITEDATA, source);
	curl_easy_setopt(session, CURLOPT_WRITEFUNCTION, ecrire);
	curl_easy_perform(session);
	curl_easy_cleanup(session);
}

void analyse(char *source) //éventuellement d'autres entrées
{

... je te laisse le soin d'y mettre le contenu si tu prends en considération ma présentation de code

}

int main()
{
	char *url = "https://www.google.com/?gws_rd=ssl";
	char *source = malloc(500000);

	dlwebsite(url, source);
	analyse(source);
}

et pas besoin de variables globales !

Un autre avantage d'avoir un code bien structuré en fonction est que, même quelqu'un qui n'aurait pas cette librairie pourrait se faire une petite main vite fait pour tester ta fonction sans dépendre du reste de ton code (sans avoir à faire le tri..).

Problème : Mon prog ne marche pas.

26 réponses

Votre réponse

Newsletters