Discussion :

[tyrtamos]

pourquoi tu as utilisé les c1, c2 et c3 ET pourquoi les avoir multiplié comme tu l'as fait ?

J'ai traité le cas général où les IP pouvaient être quelconques (ex: 192.2.56.124, 192.15.89.45, ...)

Toi, tu ne considères que le tri selon le dernier nombre de l'adresse IP (ici: 15, 10, 1 et 11), et si ça te suffit, c'est ok!

[dividee]

On pourrait aussi écrire:

Code :

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

1
2
def cle(ip):
    return [int(c) for c in ip.split('.')]

Vu que l'ordre lexicographique utilisé pour la comparaison de listes convient ici.

PS: Merci à wiztricks pour sa réponse 10 posts plus hauts C'est bon à savoir.

[pfeuh]

Salut,

Sa fonctionne parfaitement, donc pourquoi tu as utilisé les c1, c2 et c3 ET pourquoi les avoir multiplié comme tu l'as fait ?

Pour une raison qui t'a déjà été donnée là:

trier '127.0.0.1' et '127.255.0.0' ne donnera pas forcément le même résultat que trier '127000000001' et '127255000000'.

Je pense que tu négliges le côté algorithme pour trop te focaliser sur le résultat que tu attends, ce qui n'est jamais bon. Tu devrais peut-être écrire avant toute chose quelques tests unitaires.

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
 
def ip2cle(ip):
    c1, c2, c3, c4 = ip.split('.')
    return int(c4) + int(c3)*256 + int(c2)*256**2 + int(c1)*256**3
 
def cle2ip(cle):
    values = [str(cle >> 24 & 255),  str(cle >> 16 & 255), str(cle >> 8 & 255), str(cle & 255)]
    return '.'.join(values)
 
def getSortedIps(ips):
    cles = [ip2cle(ip) for ip in ips]
    cles.sort()
    ips = [cle2ip(cle) for cle in cles]
    return ips
 
ips = ["192.9.202.15", "192.9.202.11", "192.9.202.8", "192.9.202.5", "191.9.202.15", "255.9.202.11", "192.7.0.8", "191.4.0.4"] 
assert getSortedIps(ips) == ['191.4.0.4', '191.9.202.15', '192.7.0.8', '192.9.202.5', '192.9.202.8', '192.9.202.11', '192.9.202.15', '255.9.202.11']

A+

Pfeuh

[BUENOO]

Merci pour vos réponses.

C'est qu'enfête, plus haut dans mon script je demandes une plage à scanner, par exemple 192.9.202.x, 192.168.30.x ainsi de suite ... C'est pourquoi je n'es besoin de faire le tri final que sur le dernier nombre de l'ip ,)