Discussion :

[caramon _majere]

Bonjour,

ci-dessous j'ai un bout de configuration d'un switch Cisco

interface Port-channel112
description *** vers XXXX ***
!
interface Port-channel113
description *** vers YYYYY ***
!
interface Port-channel127
description *** ZZZ ***
!
interface TenGigabitEthernet1/0/1
description *** AAA ***

Je souhaite matcher les lignes concernant le Port-channel et la description associé (non pas le port TenGigabitEthernet1/0/1 par exemple )

J'ai le code suivant qui fonctionne ou chatgpt m'a indiqué l'astuce de créer une variable False pour servir de seconde condition dans un IF pour récupérer la ligne de description de l'interface souhaitée.

Ma question est la suivante. Est-ce la bonne méthode pour mon besoin ? y a t'il une méthode plus générique, plus python ?

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
import re
file_path = Path("H:/CISCO_AUDIT_VLAN/10.255.255.70.config")
 
 
# Define regular expressions
port_channel_pattern = re.compile(r'interface Port-channel\d+$')
description_pattern = re.compile(r'^\s+description .*$')
 
 
with file_path.open('r') as file:
    in_port_channel_block = False
 
    while (line := file.readline()):
        if port_channel_pattern.match(line):
            print(line.strip())
            in_port_channel_block = True
        elif in_port_channel_block and description_pattern.match(line):
            print(line.strip())
            in_port_channel_block = False

OUTPUT
interface Port-channel112
description *** vers XXXX ***
interface Port-channel113
description *** vers YYYYY ***

En vous remerciant.

[Sve@r]

Bonjour

Ma question est la suivante. Est-ce la bonne méthode pour mon besoin ? y a t'il une méthode plus générique, plus python ?

Ce n'est pas une question spécifique à Python mais plus générale d'algo. Ton problème est connu sous le nom de "machine à état" et se résume en "si X trouvé sur ligne n alors afficher ligne n+1".
Il se résout au travers de drapeau qu'on lève si on trouve X sur la ligne n puis qu'on teste pour savoir s'il faut afficher n+1.

Ensuite il peut se complexifier à l'infini (si ligne1 alors si ligne2 alors si ligne3 alors afficher ligne4).

Dans ton code je pense que le "elif" n'est pas utile (ben oui, si la ligne n'a pas été trouvée il te suffit de ne rien faire, le drapeau étant de toute façon baissé quoi qu'il arrive). Tu peux aussi supprimer readline et utiliser un simple for line in file.

[caramon _majere]

Merci pour la simplification avec le for et le mot clé drapeau.

[Sve@r]

et le mot clé drapeau.

Ce n'est pas un mot clef mais une notion qui se concrétise par une variable. Dans ton code c'est la variable in_port_channel_block qui joue ce rôle, avec les valeurs True/False symbolisant l'état "levé/baissé".

Un exemple qui me parait assez proche de ce que tu cherches...

Code python :

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
#!/usr/bin/env python3
# coding: utf-8
 
import re
from pathlib import Path
file_path = Path("fic")
 
 
# Define regular expressions
port_channel_pattern = re.compile(r'^interface Port-channel\d+$')
description_pattern = re.compile(r'^description')
 
with file_path.open('r') as fp:
	in_port_channel_block = False
 
	data=None
	for line in map(str.strip, fp):
		if port_channel_pattern.match(line):
			data=line
			continue
		# if
 
		if data and description_pattern.match(line):
			print("[%s] - [%s]" % (data, line))
 
		data=None
	# for
# with

Dans ce code, la variable "data" sert à la fois de drapeau et de mémorisation des éléments à afficher plus tard (ie la ligne "interface"). Certes j'ai utilisé des techniques Python pour raccourcir certaines étapes (utiliser map() pour simplifier l'étape du strip de la ligne) mais ces raccourcis ne changent pas les fondamentaux: on trouve la valeur on mémorise le fait qu'on l'a trouvée et ensuite on regarde si on avait trouvé la valeur.

Après il y a des machines plus difficiles, exemple checker la ligne n pour afficher la ligne n+4 (gérer alors les lignes n+1, n+2, n+3)...

[fred1599]

Bonjour,

Je suis d'accord avec Sve@r concernant le elif inutile dans le code de départ.

Par contre, je ne comprend pas bien l'utilisation des regex.
Les expressions régulières sont utilisées pour des recherches spécifiques dans un texte. Par ex. elles auraient pu être utile en découpant Port-channel et le port dans Port-channel112.

C'est là où je trouve que oui, chatGPT cherche la complexité, là où nous humain on n'en voit pas !
Voici une proposition humaine

Code :

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

1
2
3
4
5
6
with open("config.txt", 'r') as f:    
    for line in f:
        if line and line.startswith("interface Port"):
            print(line.strip())
            print(next(f).strip())
            next(f)

Tout l'intérêt de cet exercice est de bien analyser le fichier de config.

[Sve@r]

Par contre, je ne comprend pas bien l'utilisation des regex.

Oui en Python d'autres solutions sont bien souvent possibles. Ensuite on peut envisager que les infos du fichier pourraient se complexifier et donc, en ayant placé la regex, le code est déjà prêt pour le changement...

NOTE: Nous ne sommes pas des correcteurs de chatgpt !!!!!

C'était du chatGPT le code d'origine ??? Punaise je m'en suis pas rendu compte !!! Presque ça commence à faire peur (si demain n'importe quel noob qui sait utiliser chatGPT vient se prétendre développeur et commence à placer du chatGPT dans les codes critiques)...

[caramon _majere]

Non, j'ai utilisé chatgpt car j'étais bloqué pour afficher la ligne description après une interface port-channel.

Et le code ne pas en production comme ca, je voulais déjà quelque chose qui fonctionne. Après je le simplifie pour être plus compréhensible.

[Sve@r]

Et le code ne pas en production comme ca,

Je ne parlais pas de toi mais de ce que donnera le futur si (non, pas "si" mais "quand") cela se généralisera.

Sinon effectivement le next() de fred1599 est bien trouvé

[caramon _majere]

Bonjour,

Je suis d'accord avec Sve@r concernant le elif inutile dans le code de départ.

Par contre, je ne comprend pas bien l'utilisation des regex.
Les expressions régulières sont utilisées pour des recherches spécifiques dans un texte. Par ex. elles auraient pu être utile en découpant Port-channel et le port dans Port-channel112.

C'est là où je trouve que oui, chatGPT cherche la complexité, là où nous humain on n'en voit pas !
Voici une proposition humaine

Code :

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

1
2
3
4
5
6
with open("config.txt", 'r') as f:    
    for line in f:
        if line and line.startswith("interface Port"):
            print(line.strip())
            print(next(f).strip())
            next(f)

Tout l'intérêt de cet exercice est de bien analyser le fichier de config.

J'aime cette simplicité

[wiztricks]

C'est là où je trouve que oui, chatGPT cherche la complexité, là où nous humain on n'en voit pas !

ChatGPT se contente de trouver la réponse qui correspond le mieux à ce qu'il a trouvé dans les millions/milliards de lignes de code qu'il a ingurgité. Il ne sait pas faire la différence entre un itérable (qui demandera un drapeau pour se rappeler ce qu'il s'est passé à l'itération précédente) et un itérateur (qui permet d'avancer sur la collection via next).

C'est déjà pas si mal que ça fonctionne mais si un jour on retravaille ça en fonction du bilan carbone, ce sera probablement sans ChatGPT.

Et comme défaire/refaire coute en énergie, je préfère:

Code :

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

1
2
3
4
5
with open("config.txt", 'r') as f:    
    for line in f:
        if line.startswith("interface Port"):
            print(line, end = '')
            print(next(f), end='')

où j'ai supprimé le dernier next(f) car il suppose un fichier bien formé (ce qu'on ne sait pas et qui devient un pré-requis).

- W

[papajoker]

bonjour

Est-ce la bonne méthode pour mon besoin ? y a t'il une méthode plus générique, plus python ?

NOTE: Nous ne sommes pas des correcteurs de chatgpt !!!!! 8 fois sur 10 il va te donner une chose qui fonctionne mais, sans doute 9 fois sur 10 un code pas optimal pour ton besoin spécifique.
C'est notre faute, il faut affiner notre question en fonction de ces réponses (mais pour cela il faut très bien comprendre ces réponses ... )

Nous n'avons aucune idée de ce générique signifie pour toi ! Perso, je vois ce code gpt bon pour la poubelle Car justement beaucoup trop générique pour ton besoin

Un code à la fois plus générique et à la fois beaucoup plus fermé (a un cahier des charges)

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
KEY = "Port-channel"
 
def parse_lines(key, value, key_off=12, key_desc=12):
    return key[key_off:].strip(), value[key_desc:].strip()
 
def parse_conf(fp):
    key = f"interface {KEY}"
    desc = "description "
    for line in fp:
        if line.startswith(key):
            datas = parse_lines(line, next(fp), len(key)+1, len(desc))
            yield datas
 
 
with open("10.255.255.70.config", "r") as fconf:
    for data in parse_conf(fconf):
        print("Port:", data[0], " = ", data[1])

[fred1599]

Sve@r,

Je suis pour le principe YAGNI et ne pas ajouter de fonctionnalité inutile.

Rien n'indique que le PO est besoin de plus dans sa problématique de départ... il a même donné un exemple de la sortie attendue.

Mais la ligne in_port_channel_block = False et le fait de ne pas utiliser cette variable, porte à croire que ce bloc n'est pas terminé... sauf que l'on ne joue pas aux devinettes et que c'est au PO d'expliquer l'ensemble de la problématique.

@papajoker,

En admettant que le PO souhaite une recherche spécifique pour chaque ligne du numéro de port, alors dans ce cas, j'aurai sans doute choisi les expressions régulières.
On pourrait s'imaginer que la ligne est mal écrite, les expressions régulières permettront d'en ressortir le numéro de port tout de même.
Mais comme on ne sait rien de cette suite on va pas trop se questionner sur les besoins futurs du PO.

[fred1599]

Bien vu pour la simplification