Condition qui ne s'effectue pas

Version imprimable

Bonjour. :)

J'ai fais un script simple que je développerais, modifierais par la suite.
Ici il y a que le commencement de base. J'ai un souci concernant les conditions dans ce script.
Que je sois connecté avec le vpn ou non j'ai toujours la même réponse ( ip a changer).
Code:

1 2 3 4 5 6 7 8 9 10 11 12 import os vpn = 1316232503 # ip vpn "78.116.29.55" str converti en int = 1316232503 curl = os.system("curl ipinfo.io/ip") # int if curl == vpn: print("bonne ip") elif curl != vpn: print("ip a changer") else: print("none")
La chaîne vpn est un string avec l'ip "78.116.29.55" , curl un entier.
J'ai alors converti vpn en entier à la place de la chaîne, mais j'ai toujours le même résultat ( ip a changer) en déconnectant le vpn ou en le remettant.

Je remarque qu'en tapant curl sur la console, j'obtiens le code 0 et non l'ip.
Hors je pense que le problème vient de là. Je suis en apprentissage de python autodidacte mais je n'ai pas assez d'expérience pour remédier à ce problème.

Citation:

j'execute et vpn connecter:
>>>
78.116.29.55
ip a changer

j'execute sans vpn:
>>>
78.116.29.55
ip a changer

>>> curl
0

:salut:

Vous ne pouvez pas comparer un int avec le contenu de curl :

Code:

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
32
33
34
import struct
import socket
import os
 
curl = os.system("curl ipinfo.io/78.116.29.55")
 
print(curl)
 
'''
{
  "ip": "78.116.29.55",
  "hostname": "55.29.116.78.rev.sfr.net",  
  "city": "Douai",
  "region": "Hauts-de-France",
  "country": "FR",
  "loc": "50.3707,3.0792",
  "org": "AS8228 SFR SA",
  "postal": "59351 CEDEX",
  "timezone": "Europe/Paris",
  "readme": "https://ipinfo.io/missingauth"
}0
'''
 
 
def ip2int(addr):
    return struct.unpack("!I", socket.inet_aton(addr))[0]
 
 
def int2ip(addr):
    return socket.inet_ntoa(struct.pack("!I", addr))
 
 
print(int2ip(1316232503))  # 78.116.29.55
print(ip2int('78.116.29.55'))  # 1316232503

J'ai testé avec l'adresse de mon modem, les renseignements fournis par curl ne sont pas fiables.

09/08/2020, 18h32
wiztricks
Salut,

Citation:

Envoyé par ati00

Je suis en apprentissage de python autodidacte mais je n'ai pas assez d'expérience pour remédier à ce problème.

Ce n'est pas un problème Python à proprement parler mais de compréhension de ce que fait os.system.

Il se contente de lancer la commande et de retourner un status sous la forme d'un entier entre 0 et 255 qui dira si la commande s'est exécutée ou pas sans problème.
Vous ce que vous voulez récupérer, c'est ce que la commande affiche à l'écran (après avoir testé le status).

Pour çà il est préférable d'utiliser subprocess:
Code:

1 2 3 from subprocess import check_output curl = check_output("curl ipinfo.io/ip", shell=True) print(curl)
Et comme curl sera une chaine de caractères (mais des bytes), vous savez à quoi la comparer pour tester.

- W
09/08/2020, 19h42
Sve@r

Bonjour

En plus de ton erreur sur le comportement de os.system(), moi je m'interroge sur ton alternative binaire à 3 sorties.
Tu commences par tester si curl est égal à vpn, puis si curl est différent de vpn. Et sinon tu affiches "none".

Et donc c'est quoi pour toi ce 3° état mystérieux où curl ne serait ni égal ni différent de vpn ???
09/08/2020, 21h42
ati00

Citation:

danielhagnoul: Vous ne pouvez pas comparer un int avec le contenu de curl

Sur un navigateur l'adresse http://ipinfo.io/ip donne que l'ip, mais après l'échec du script je me doutais un peu que curl ne donner pas que l'adresse ip, surtout que le test de booléen entre vpn et curl donner False. Merci d'avoir essayer ton script vérificateur.

Citation:

wiztricks: Ce n'est pas un problème Python à proprement parler mais de compréhension de ce que fait os.system.

C'est que os.system je ne dois pas l’utiliser pour ce cas, je me rend compte que je capture que le code de retour de la commande.
Pour le code avec subprocess et mon code, cela ne fonctionne pas mais merci de ton code, je le retenterais demain ou Mardi.
Mais je vais écarter os.system du script et le remplacer.

Citation:

Sve@r[: En plus de ton erreur sur le comportement de os.system(), moi je m'interroge sur ton alternative binaire à 3 sorties

Pour l'alternative à 3 sorties :) else est inutile tu as raison je l'ai mis et cela ne mange pas de pain dans ce test. les conditions c'est pour simplifier le script ici mais après les conditions seront remplacer par une boucle, etc ... pour faire une alternative à Ip Indicator, j’utiliserais notify-send etc.. Mais je comprend pour toi à la base cela peut paraître sans valeur ce test.
Le test des conditions doit fonctionner pour que je puisse ensuite finaliser mon script, parce que le principal c'est si le l'ip du vpn est connecter ou si elle ne l'ai pas.
Bonne soirée.
10/08/2020, 09h29
danielhagnoul
Pour interroger https://ipinfo.io/, je vous recommande l'usage du module Python Geocoder : https://geocoder.readthedocs.io/

Pour l'installer : pip install geocoder voir : https://pypi.org/project/geocoder/
Code:

1 2 3 4 5 6 import geocoder g = geocoder.ipinfo("78.116.29.55") print(g.ip) print(g.json)
10/08/2020, 10h04
Sve@r
Citation:

Envoyé par ati00

je l'ai mis et cela ne mange pas de pain dans ce test.

Si, cela mange du pain. Le processeur fera 2 comparaisons là où une suffit

Citation:

Envoyé par ati00

Les conditions c'est pour simplifier le script ici

Simplifier je comprends (une des philosophies Python est justement que la simplicité est préférable à la complexité). Toutefois ta façon de simplifier je comprends moins... print("bonne ip" if curl == vpn else "ip à changer"). ;)

Citation:

Envoyé par ati00

Pour le code avec subprocess et mon code, cela ne fonctionne pas mais merci de ton code, je le retenterais demain ou Mardi.
Mais je vais écarter os.system du script et le remplacer.

Tu peux essayer ceci
Code:

1 2 import subprocess curl=subprocess.Popen(("curl", "ipinfo.io/ip"), stdin=None, stdout=subprocess.PIPE).stdout.read()
Et pour plus de détails sur subprocess, ses paramètres et la façon de récupérer son résultat, voir ici.

Super!! @ danielhagnoul
je te remercie de me faire connaître geocoder. je vais lire la documentation tout à l'heure, ça à l'air top je le garde à la fin, et je ferai un essai juste après. Je te tiens au courant. Merci :)

@Sve@r
Lorsque j'avais écris les conditions, je me suis en plus dit " else: none " est inutile de l'ajouter mais je l'ai mis, pour tester quand même. Sur 20 ou 30 fois d'échec sur '==' ou '!=' je suis tombé 1 seul fois sur 'none' et je sais plus comment. lol
Merci de ta nouvelle commande, j'ai essayé mais cela me donne une ip de type Bytes.
Code:

1 2 3 4 5 6 7 8 9 10 11 12 13 import os import subprocess curl = subprocess.Popen(("curl", "ipinfo.io/ip"), stdin=None, stdout=subprocess.PIPE).stdout.read() vpn = "78.116.29.55".encode() # ip str converti en bytes if curl == vpn: print("bonne ip") if curl != vpn: print("autre ip")
Bon cela me gène pas au contraire cela me fais apprendre.
Je suis parti alors rechercher sur le net pour convertir un str en bytes, et ici je converti l'ip vpn en bytes. Le problème l'ip de curl en byte à le retour chariot à la fin \n.
Si je la compare à l'ip vpn bytes elles apparaissent semblable mais ne le sont pas pour Python.
La condition ne peut fonctionner pour '=='.

Citation:

>>> curl
b'78.116.29.55\n'
>>> vpn
b'78.116.29.55'

Bon je repose ma tête chaude aujourd'hui, je ne sais pas si c'est vraiment Python ou bien la chaleur du moi d'Août lol

10/08/2020, 15h03
wiztricks

Salut,

Citation:

Envoyé par ati00

Jje ne sais pas si c'est vraiment Python ou bien la chaleur du moi d'Août

Pourquoi écrire:

Code:

vpn = "78.116.29.55".encode()

plutôt que:

Code:

vpn = b"78.116.29.55"

ou encore:

Code:

vpn = b"78.116.29.55\n"

Et si vous ne voulez pas triturer "vpn": curl.decode().strip()

De toutes façons quand vous récupérer la sortie d'un programme, ce seront des chaines de caractères qu'il faudra retravailler... Autant connaitre les méthodes de base pour le faire car peu importe dans ce que vous voudrez coder, il faudra les connaître.

- W
10/08/2020, 15h43
ati00

Salut wiztricks.

Citation:

Pourquoi écrire:
vpn = "78.116.29.55".encode()

Forcément que je ne peux savoir ces commandes, je ne les ai pas étudier, je sais bien qu'il faut étudier avant ( et j'ai appris déjà des bases d'un livre, c'est impossible de tout savoir en tant que débutant et python c'est vaste ) ,et je ne pensais partir sur d'autre voie et en l’occurrence sur la conversion en bytes, donc pour mon problème j'ai voulu le résoudre vite et j'en suis pas loin. J'ai passé la mâtiné à régler mon problème. :)
10/08/2020, 15h46
ati00
Formidable !! cela fonctionne directement avec geocoder. :D
Je te remercie beaucoup @danielhagnoul :D c'est ce qui me fallait, en plus il y a des options intéressante pour le futur.
Trop content que ça fonctionne, vpn connecter ou déconnecter la condition fonctionne parfaitement.

Voici le code corriger:
Code:

1 2 3 4 5 6 7 8 9 10 11 12 import geocoder g = geocoder.ipinfo('me') g = g.ip data = "78.116.29.55" if g == data: print("bonne ip") elif g != data: print("autre ip")
En python il y a tant de façon pour arriver au but.
Il me reste maintenant, à modifier mon script en plus sophistiquer mais cela devrait aller.

Mon problème est résolu. J’étudierais subprocess et bytes.
Merci les amis, je met en résolu. :)
10/08/2020, 18h05
Sve@r
Citation:

Envoyé par ati00

En python il y a tant de façon pour arriver au but.

Oui mais tester l'égalité est sinon tester la différence ça fait quand-même partie des pires.
Code:

1 2 import geocoder print("bonne ip" if geocoder.ipinfo('me').ip == "78.116.29.55" else "autre ip")
Citation:

Envoyé par ati00

Sur 20 ou 30 fois d'échec sur '==' ou '!=' je suis tombé 1 seul fois sur 'none' et je sais plus comment. lol

C'était un état quantique intriqué.
Plus sérieusement c'est totalement impossible. Deux choses sont soit égales entre elles, soit non. Il n'y a pas de 3° choix. Je ne dis pas que tu n'as pas eu "none" à l'écran mais il est possible que ton code ait été alors un peu différent et que le "none" ait été mis (par exemple) dans le cas du "else" (une tabulation malencontreuse et une instruction qu'on croyait en dehors d'un bloc se retrouve en fait dans le bloc). Ou autre situation qui fait que "none" s'affiche alors qu'on reste dans l'une des deux seules alternatives possibles.

Citation:

Envoyé par ati00

Mon problème est résolu. J’étudierais subprocess et bytes.

Les bytes sont les chaines encodées en ascii (c'étaient les "str" sous P2). Or le nom "str" a été conservé sous P3 mais fait maintenant référence à l'unicode. Et donc un des soucis du portage P2 => P3 c'est que certaines librairies qui retournaient des str sous P2 ont eu à choisir entre "retourner la même chose" (donc des bytes) ou "retourner ce qui s'appelle pareil" (donc changer l'encoding du truc retourné puisque le même nom fait référence à une chaine encodée différemment) et ont choisi de retourner la même chose sous P3 => donc des bytes
Je vais pas critiquer (je pense que les concepteurs ont eu à faire un choix et ont choisi ce qu'ils estimaient être le moins pire). Il faut juste en être conscient. Perso quand j'utilise une librairie qui me retourne des trucs déjà je regarde ce qu'elle retourne puis j'essaye de convertir tout ce qui entre en unicode (ce qui maintenant se nomme "str"sous P3) le plus tôt possible pour "tenter" d'être le plus serein ensuite dans mon code.
10/08/2020, 20h17
ati00

@Sve@r
Oui je suis d'accord avec ta bonne analyse et l'apparition de none cela du arriver comme tu le décrit.
Je confirme ton code est nickel en remédiant avec ma mauvaise conversion bytes corriger par @wiztricks le script fonctionne parfaitement.

Merci de ton commentaire sur Bytes, je ne savais rien de tout ça.
Merci à vous tous et bonne soirée.:)