Discussion :

[Jiji30]

Bonjour tout le monde,

Je vous prie de trouver le code que j'ai écris ainsi que son exécution, je voudrai affiner le résultat obtenu de la manière suivante:

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
data_EPG = {}
 
ligne = data.split("\n")
 
for i in filter(None, ligne):
    col = i.split(";")
    event = col[9]
 
    if   '//activate' in col[9]  or col[9]=='/app//standby' :
 
 
            data_EPG[event] = {
 
 
        '@mac':{
 
          col[6],
 
        },
 
 
        'Ev':{
 
          col[9],
 
        }
 
 
    }
 
            print(data_EPG[event]['Ev'] )

Une parie du résultat obtenu résultat c'est:
.
.
.
{'/homepage//activate'}
{'/epg//activate'}
{'/homepage//activate'}
{'/homepage//activate'}
{'/canal//activate'}
{'/homepage//activate'}
{'/epg//activate'}
{'/search//activate'}
{'/live//activate'}
{'/app//standby'}
{'/live//activate'}
{'/main_process//activate'}
{'/app//standby'}
{'/homepage//activate'}
{'/epg//activate'}
{'/app//standby'}


Ce que je souhaite avoir en affinant mon code c'est d’éliminer tous les {'/xxx//activate'} tq: 'xxx'! = 'epg' et tous les {'/app//standby'} qui ne succèdent directement pas les {'/epg//activate'}. Donc le résultat que je souhaite avoir c'est le suivant:

{'/epg//activate'}
{'/homepage//activate'}
{'/epg//activate'}
{'/search//activate'}
{'/epg//activate'}
{'/app//standby'}

Si vous avez une idée sur comment je peux avoir ce résultat en affinant mon code, svp aidez-moi. Merci d'avance !

[Sve@r]

et tous les {'/app//standby'} qui ne succèdent directement pas les {'/epg//activate'}

Si vous avez une idée sur comment je peux avoir ce résultat en affinant mon code, svp aidez-moi. Merci d'avance !

Bonjour

Etant donné que tu traites les lignes une à une, tu ne peux à priori pas savoir si {'/app//standby'} succède ou ne succède pas un {'/epg//activate'}.
Donc il faut que quand tu trouves un {'/epg//activate'} tu mets un flag à True. Si ensuite tu as {'/app//standby'} et que le flag est à True tu ne fais rien sinon tu le supprimes. Et dans tous les cas tu remets le flag à False.

Et putain quand on voit ta façon de coder (tabulé n'importe comment, des lignes vides anarchiques) ben on n'a vraiment pas envie de s'impliquer plus.

[Jiji30]

Bonjour

Etant donné que tu traites les lignes une à une, tu ne peux à priori pas savoir si {'/app//standby'} succède ou ne succède pas un {'/epg//activate'}.
Donc il faut que quand tu trouves un {'/epg//activate'} tu mets un flag à True. Si ensuite tu as {'/app//standby'} et que le flag est à True tu ne fais rien sinon tu le supprimes. Et dans tous les cas tu remets le flag à False.

Et putain quand on voit ta façon de coder (tabulé n'importe comment, des lignes vides anarchiques) ben on n'a vraiment pas envie de s'impliquer plus.

Merci beaucoup Mr. Sve@r pour ta réponse! J'ajoute juste que si le flag est à True et que sois j'ai {'/app//standby'} ou un {'/xxx//activate'} je ne fais rien sinon je les supprime.
Un petit souci c'est que je n'ai jamais travaillé avec les flags avant , je serai reconnaissante si vous pouvez me montrer à quoi ressemblera mon code.

[zancrows]

Salut ce que veut dire Sve@r par flag, c'est juste une variable
ex :

Code :

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

flag = False

[fred1599]

je serai reconnaissante si vous pouvez me montrer à quoi ressemblera mon code.

Oufff, à mon avis ça va saigner

Code :

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

for i in filter(None, ligne):

Pourquoi cette fonction filter si en premier argument tu ne mets pas de fonction ?

Code :

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

ligne = data.split("\n")

N'y a-t-il qu'une seule ligne dans le fichier ? Si oui le nom est bien choisi, si non...

--------------------------------------

Il y a un mélange d'anglais et de français dans le nom de tes variables, pas bien !

Pour lire chaque ligne de ta variable data, on préfère

Code :

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

1
2
for line in data:
    # suite du code

Comme le dis Sve@r, le flag est la bonne manière en lisant ligne par ligne de la façon que je présente juste au-dessus, et le code ressemblera à (non testé évidemment) quelque chose comme ci-dessous

Code :

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

1
2
3
4
5
6
7
8
for line in data:
    if flag:
        if line.startswith('/app//standby'): # Si la ligne commence par '/app//standby'
            # on agit
    flag = False
    if line.startswith('/epg//activate'):
        flag = True
    # etc

--------------------------------------

Autre manière de faire et de jouer avec les index d'une liste de lignes du fichier

Code :

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

1
2
3
4
5
lines = data.splitlines()
for ind, line in enumerate(lines):
    if line.startswith('/epg//activate'):
        if lines[ind+1].startswith('/app//standby'):
            # on agit

Bref il y a du boulot et de bonnes bases à avoir avant d'attaquer cela sereinement...

[Jiji30]

Oufff, à mon avis ça va saigner

Pourquoi cette fonction filter si en premier argument tu ne mets pas de fonction ?

N'y a-t-il qu'une seule ligne dans le fichier ? Si oui le nom est bien choisi, si non...

--------------------------------------
Il y a un mélange d'anglais et de français dans le nom de tes variables, pas bien !

Merci Mr @fred1599 pour ta réponse ainsi que pour tes précieux conseils.
En effet mon fichier texte comprend plusieurs lignes. J'ai utilisé la fonction filter car sinon j'avais un message d'erreur de 'Index out of range' puisque dans mon fichier texte entre une ligne et une autre il y avait juste un espace du coup j'ai été dans l'obligation de le filtrer.
J'ajoute juste que mes lignes ne commencent pas par '/app//standby'.
Voici le code que j'ai pu écrire en utilisants les flags et en se basant sur le votre :

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
data_EPG = {}
lines = data.split("\n")
for line in filter(None, lines):
    col = line.split(";")
    #print(col[9])
    event = col[9]
    if flag:
        if   '//activate' in col[9]   or col[9]=='/app//standby' :
            data_EPG[event] = {
        '@mac':{
          col[6],
        },
        'Ev':{  
          col[9],
        }
    }
    flag = False
    if col[9]== '/epg//activate':
        flag = True
    print(data_EPG[event] )

Or je reçois l'erreur suivante:

KeyError Traceback (most recent call last)
<ipython-input-28-15974b8cf359> in <module>()
18 if col[9]== '/epg//activate':
19 flag = True
---> 20 print(data_EPG[event] )
21
22

KeyError: '/app//wakeup'


Sachant que '/app//wakeup' est la col[9] de la 1ère ligne de mon fichier texte.

J'ai beau essayer de comprendre d’où ça vient cette erreur mais sans résultat . Si vous avez une idée, je vous prie de m'aider !

[Sve@r]

J'ai beau essayer de comprendre d’où ça vient cette erreur mais sans résultat . Si vous avez une idée, je vous prie de m'aider !

M'étonne que wiztricks ne soit pas venu te dire qu'avant de coder il faut commencer par acquérir les bases (lire des tutos, faire les exercices, etc). Il doit être en vacances...

Un "KeyError", comme sa traduction française le laisse suggérer, est une erreur de clef. Ca signifie que la clef que tu utilises n'existe pas dans ton tuple/liste/dictionnaire. Donc ici, ça veut dire qu'au moment où tu demandes data_EPG[event] ton dictionnaire "data_EPG" n'a pas de clef dont la valeur est dans la variable "event".

Et pour arriver à comprendre le pourquoi d'une erreur, rien ne vaut le bon vieux print() des familles.

PS: dans un forum oui les intervenants peuvent aider... mais ils ne peuvent pas apprendre les bases à ta place.

[Jiji30]

M'étonne que wiztricks ne soit pas venu te dire qu'avant de coder il faut commencer par acquérir les bases (lire des tutos, faire les exercices, etc). Il doit être en vacances...

Un "KeyError", comme sa traduction française le laisse suggérer, est une erreur de clef. Ca signifie que la clef que tu utilises n'existe pas dans ton tuple/liste/dictionnaire. Donc ici, ça veut dire qu'au moment où tu demandes data_EPG[event] ton dictionnaire "data_EPG" n'a pas de clef dont la valeur est dans la variable "event".

Et pour arriver à comprendre le pourquoi d'une erreur, rien ne vaut le bon vieux print() des familles.

PS: dans un forum oui les intervenants peuvent aider... mais ils ne peuvent pas apprendre les bases à ta place.

Merci pour votre réponse Mr.Sve@r,. En effet avant de poster cette question j'ai cherché avant et j'ai bien compris l'erreur comme tu l'a expliqué, toutefois je ne suis pas arrivée à résoudre le problème. Certes je ne suis pas professionnelles en python mais je fais de mon mieux

[Sve@r]

Merci pour votre réponse Mr.Sve@r,.

Donc ce n'est pas "Mr Sve@r" mais "Sve@r". C'est le principe du pseudo.

toutefois je ne suis pas arrivée à résoudre le problème

dataEPG[event] n'est créé que si flag vaut "True". Si tu ne passes jamais le bloc if flag de la ligne 7 ben dataEPG[event] n'est jamais créé. Et quand tu tentes de l'afficher ligne 20...

Donc il faut soit le créer dans tous les cas... ou bien soit ne l'afficher que s'il a été créé.. ou bien soit ne l'afficher que s'il existe => exemple: if event in dataEPG.keys() print dataEPG[event].

[wiztricks]

M'étonne que wiztricks ne soit pas venu te dire qu'avant de coder il faut commencer par acquérir les bases (lire des tutos, faire les exercices, etc). Il doit être en vacances...:mrgreen.

Yup... Et je regarde comment d'autres traitent les soucis de ceux qui ont oublié de débuter. Il faut dire que si on trie les lignes, il y a des chances pour que le dictionnaire ne soit pas créé dès le départ. De plus je n'aurais pas pensé au flag: si on veut récupérer la ligne suivante lorsque la ligne courante vaut une certaine valeur autant le faire immédiatement.
Si je veux filtrer les 0 et le caractère suivant dans la séquence:

Code :

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

>>> z = '110111102111111103111'

J'écrirai plutôt:

Code :

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

1
2
3
4
5
6
7
>>> x = iter(z)
>>> for ch in x:
...     if ch == '0': print (ch, next(x))
...
0 1
0 2
0 3

mais bon...

- W

[Jiji30]

Donc ce n'est pas "Mr Sve@r" mais "Sve@r". C'est le principe du pseudo.

dataEPG[event] n'est créé que si flag vaut "True". Si tu ne passes jamais le bloc if flag de la ligne 7 ben dataEPG[event] n'est jamais créé. Et quand tu tentes de l'afficher ligne 20...

Donc il faut soit le créer dans tous les cas... ou bien soit ne l'afficher que s'il a été créé.. ou bien soit ne l'afficher que s'il existe => exemple: if event in dataEPG.keys() print dataEPG[event].

Merci infiniment Sve@r pour ton aide.

[Sve@r]

De plus je n'aurais pas pensé au flag

Mmmoui. C'est dû à la façon dont j'ai claqué Python sur mes méthodes d'algorithmie acquises à partir de langages plus ancien. Je sais qu'il me manque certains réflexes...