Discussion :

[pinhut]

Bonsoir à tous !

Tout d'abord je tiens à préciser que c'est la première fois que je m'essaye au développement sous Python (via scrapy) mais je suis néanmoins familiarisé avec les concepts orientés objets, mais bien plus à l'aise en C.

La description du projet dans sa globalité n'est pas très importante au regard de la simplicité de ma question:

- je souhaite déclarer deux tableaux, un contenant des chaînes de caractères et un contenant de simples nombres.
-une boucle va venir, par exemple, parcourir toutes les positions de mon tableau contenant mes nombres pour venir me les remplacer dans ce code ci:

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
import scrapy



tab1=[1,2,3]
tab2=['hello','world']

class QuotesSpider(scrapy.Spider):
    name = "histo"

    def start_requests(self):
         for i in tab1:
        urls = [
            'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=60&aggregate=1&toTs=1452680400&extraParams=your_app_name'
           
        ]
        for url in urls:
            yield scrapy.Request(url=url, callback=self.parse)

    def parse(self, response):
        page = response.url.split("/")[-2]
        filename = 'histo-%s.html' % page
        with open(filename, 'wb') as f:
            f.write(response.body)
        self.log('Saved file %s' % filename)

La partie importante qui me pose problème ici est soulignée en gras, je souhaite que "60" soit remplacé par les différentes valeurs présentes dans mon tab1, en langage C j'aurai écris quelque chose comme ça:

Code :

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

1
2
3
4
5
6
for i=0 to 3
 {

        urls = [
            'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit='%d'&aggregate=1&toTs=1452680400&extraParams=your_app_name',tab1[i]

Malheureusement en cherchant sur des forums je ne trouve rien, bien qu'étant conscient que je ne trouve pas ma réponse parce que la formulation de ma question est que bien trop imprécise.

J'espère avoir été assez clair, en vous remerciant par avance et en m'excusant des éventuels affronts que j'aurai pu écrire =)

[VinsS]

Salut,

C'est plus ou moins similaire en Python

Code :

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

1
2
3
4
5
6
7
8
 
>>> indices = [1, 2, 3]
>>> for i in indices:
...     print('indice=%s' % i)
... 
indice=1
indice=2
indice=3

le '%s' peut différer selon le formatage désiré, voir ici:
https://docs.python.org/3/library/st...ing-formatting


Edit: je pense que ta fonction génératrice devrait plutôt être ainsi:

Code :

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

1
2
3
4
5
 
    def start_requests(self, tab1):
         for i in tab1:
             urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%d&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
             yield scrapy.Request(url=url, callback=self.parse)

Note l'argument 'tab1' indispensable parce que inconnu dans l'espace de nom de la fonction.

[pinhut]

Bonjour VinsS et merci pour la rapidité de ta réponse !!!

Enfait j'avais déjà plus ou moins testé de mettre en argument mon tab1 dans la fonction start_requets, mais visiblement en cherchant un peu, j'ai cru comprendre que pour l'instruction start_request il ne pouvait que posséder exclusivement l'argument self et de fait me voilà bien embêté !!

Voila mon code adapté avec ton explication :

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
 
import scrapy
 
 
class QuotesSpider(scrapy.Spider):
    name = "histo"
    tab1=[1000,10]
 
    def start_requests(self, tab1):
        for i in tab1:
            urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%d&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
            yield scrapy.Request(url=url, callback=self.parse)
 
 
        def parse(self, response):
            page = response.url.split("/")[-2]
            filename = 'histo-%s.html' % page
            with open(filename, 'wb') as f:
                f.write(response.body)
            self.log('Saved file %s' % filename)

L'erreur affichée est de type start_requests() takes exactly 2 arguments (1 given), j'ai également essayé pour résoudre ce problème de déclarer mon tab1 ailleurs dans la fonction mais rien n'y fait

Si tu as une idée pour contourner le problème je suis preneur =)

[VinsS]

Si tab1 est attribut de classe alors tu peux le supprimer des arguments et utiliser:

Code :

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

1
2
 
        for i in self.tab1:

Est-ce normal que la fonction parse() soit plus indentée que la précédente ?

[pinhut]

On touche du doigt l'objectif final!!

Concernant ta question sur l'indentation, ma fonction parse l'est en effet plus que la précédente, mais ce n'est absolument pas une volonté de ma part, c'est vrai qu'en revanche l'indentation semble vraiment importante et problématique sous python, chose que je n'avais jamais expérimenté avant ! est-ce un problème selon toi dans notre cas précis ?

Maintenant j'ai fait apparaître un autre problème: avec cette boucle, je souhaite mettre dans "urls" les deux urls que ma boucle for va venir créer (une url dans laquelle à correctement été placé le chiffre 1000 et la deuxieme dans laquelle à correctement été placé le chiffre 10)

Ainsi, je souhaite qu'il m'écrive dans DEUX fichiers différents les informations recueillies sur les deux url différentes (celles crées par ma boucle for) comme s'en suit

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
 
 
import scrapy
 
 
class QuotesSpider(scrapy.Spider):
    name = "histo"
    tab1 = [1000,10]
 
    def start_requests(self):
        for i in self.tab1:
            urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%s&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
 
        for url in urls:
            yield scrapy.Request(url=url, callback=self.parse)
 
    def parse(self, response):
        page = response.url.split("/")[-2]
        filename = 'histo-%s.html' % page
        with open(filename, 'wb') as f:
            f.write(response.body)
        self.log('Saved file %s' % filename)

Cependant rien de spécial ne se passe avec ce code, les page html ne se crée pas. Le seul moyen d'avoir un résultat (création d'une page html avec le contenu d'une des deux url) est de remplacer

Code :

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

yield scrapy.Request(url=url, callback=self.parse)

par

Code :

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

yield scrapy.Request(url=urls, callback=self.parse)

et là encore je ne vais avoir que la création de la page html dont les informations viennent de la deuxième url, j'ai comme l'impression qu'au moment de l'écriture, la page html correspondant à ma première url à bien été crée mais écrasée par la création de la deuxième.

J'ai essayé de conditionner l'écriture par une boucle, mais je pense être sur la mauvaise voie (?)

[VinsS]

Comme indiqué dans mon post #2, ta fonction génératrice devrait être comme ceci:

Code :

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

1
2
3
4
5
 
    def start_requests(self):
        for i in self.tab1:
            url = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%s&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
            yield scrapy.Request(url=url, callback=self.parse)

ce que tu fais est une itération sur la chaîne elle-même (la dernière créée)

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
 
>>> for i in [1, 2]:
...     idx = "indice=%s" % i
... 
>>> for i in idx:
...     print(i)
... 
i
n
d
i
c
e
=
2
>>>

[chimat]

Bonjour à toi Pinhut et bonjour à toi VinsS,

J'essaye de développer un crawler pour raspberry pi à partir des tutoriaux de scrappy mais je rencontre à peu près le même problème que Pinhut.

En reprenant son code suivant modifié selon les conseils de VinsS (ma structure est très similaire):

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
class QuotesSpider(scrapy.Spider):
    name = "histo"
    tab1 = [1000,10]
 
    def start_requests(self):
        for i in self.tab1:
            urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%s&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
        yield scrapy.Request(url=urls, callback=self.parse)
 
    def parse(self, response):
        page = response.url.split("/")[-2]
        filename = 'histo-%s.html' % page
        with open(filename, 'a') as f:
            f.write(response.body)
        self.log('Saved file %s' % filename)

J'obtiens les résultats extraits dans un fichier, mais seules les données de la seconde URL (avec la valeur 10 donc) apparaissent dans mon fichier. Apparemment, ce serait parce qu'une seule URL est scrappée, celle prenant la valeur de 10. ( En "2ème" position donc dans le tableau).



En utilisant le code suivant proposé par Pinhut:

Code :

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

1
2
3
4
5
6
    def start_requests(self):
        for i in self.tab1:
            urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%s&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
 
        for url in urls:       
            yield scrapy.Request(url=urls, callback=self.parse)

Python identifie bien qu'il y a deux URLS qui sont générées, mais cette fois-ci encore, seule l'URL prenant la valeur 10 est prise en compte, et dupliquée, puis Scrappy supprime la première URL afin d'éviter les dupplicatas et renvoie l'info. Je pense que le problème que Pinhut et moi-même rencontrons est à ce niveau là. La capture d'écran ci-dessous montre bien le problème:



Cela signifie t'il que la méthode du tableau pour l'insertion des paramêtres n'est pas viable? Doit-on utiliser d'autres fonctions afin d'appeler les données dans le tableau?

Merci d'avance à tous les deux!

CM.

[VinsS]

Salut,

Le yield doit être indenté au même niveau que la ligne qui le précède. Sinon la boucle sur les deux urls ne sert à rien.

[chimat]

Bonsoir VinsS,

Tu es vraiment le sauveur sur ces problèmes ahah merci infiniment! Je m'y ferais pas non plus à cette histoire d'indentation.

J'ai réussi à progresser depuis la dernière fois (ouf) et ai réussi a maîtriser un peu mieux la fonction url.split pour qu'elle fasse ce que je veux faire..
Néanmoins mon problème se situe à nouveau dans les start_request..
En reprenant l'exemple de pinhut:

Code :

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

1
2
3
4
5
6
7
8
class QuotesSpider(scrapy.Spider):
    name = "histo"
    tab1 = [1000,10]
 
    def start_requests(self):
        for i in self.tab1:
            urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%s&aggregate=1&toTs=1452680400&extraParams=your_app_name' % i
            yield scrapy.Request(url=urls, callback=self.parse)

Ici Pinhut n'a qu'une seule "tab1", mon souhait serait de modifier plusieurs urls de manière séquentielle...
Disons:

Code :

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

1
2
3
4
5
6
7
8
9
class QuotesSpider(scrapy.Spider):
    name = "histo"
    tab1 = [1000,10]
    tab2 = [3,5]
 
    def start_requests(self):
        for i, j in self.tab1, self.tab2:
            urls = 'https://min-api.cryptocompare.com/data/histohour?fsym=ETH&tsym=BTC&limit=%s&aggregate=%s&toTs=1452680400&extraParams=your_app_name' % i % j
            yield scrapy.Request(url=urls, callback=self.parse)

J'ai essayé cette solution (qui ne fonctionne pas), un peu comme on pourrait le faire avec du C.
En parcourant le site je suis tombé sur la fonction Zip, mais de ce que j'en ai compris, elle ne permettrait d'utiliser que des chaines de même nombre de caractères, et je ne sais pas si elle pourrait efficacement résoudre mon problème...

Théoriquement, je souhaiterais que la boucle aille chercher dans l'ordre les valeurs sur l'url en :
1. 1000/ 3
2. 1000/ 5
3. 10/ 3
4. 10/ 5

A terme il faudrait que je puisse définir chacun des paramètres de l'URL. J'ai également croisé la bibliothèque URLLIB mais qui n'a l'air d'être compatible qu'avec Python >3.4, et même dans ce cas je ne suis pas sur de pouvoir modifié les paramètres individuellement.

J'espère avoir été compréhensible, et merci encore milles fois pour ton aide!

[VinsS]

En parcourant le site je suis tombé sur la fonction Zip, mais de ce que j'en ai compris, elle ne permettrait d'utiliser que des chaines de même nombre de caractères, et je ne sais pas si elle pourrait efficacement résoudre mon problème...

tab1 et tab2 sont des listes pas des chaînes de caractères.

Si je comprend bien ce que tu veux faire zip est bien la fonction ad hoc. for i, j in zip(tab1, tab2)

Edit: j'avais regardé ton code mais pas la suite.
Si tu veux la combinaison des valeurs des listes, tu peux le faire avec une listcomprehension.

Code :

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

1
2
3
4
5
6
 
>>> t = [1000, 10]
>>> t1 = [3, 5]
>>> tt = [(i, j) for i in t for j in t1]
>>> tt
[(1000, 3), (1000, 5), (10, 3), (10, 5)]

et puis tu l'utilises avec for i, j in tt