[bachus]

Głupie, może prostackie pytanie z programowania o którym nie mam pojęcia. Mam gotowy skrypt, który wykona się, gdy wersja obecnego oprogramowania jest w wersji 6.7.1 lub wyższej. Na 6.8 wykonuje się bez problemu, jednak na 6.10 już nie. W skrypcie jest taki fragment:

version = version_utils.get_cluster_version(numeric=True)
  if not ("master" in version or version >= '6.7.1'):

czy wg. logiki pythona "6.10" >= "6.7.1"?
#programowanie #python

[Fattek]

@bachus: na moje oko on tu nie porównuje "wartości" liczb, a ich kolejność alfabetyczną. Stąd jak przechodzi do porównania trzeciego znaku, to uznaje, że 1 jest mniejsze niż 7, więc sprawdzany warunek nie jest prawdą

[m_ba]

@bachus from pkgresources import parseversion

version = versionutils.getclusterversion(numeric=True)
if not ("master" in version or parseversion(version) >= parse_version('6.7.1')):

[krzaczor93]

@bachus: '6.7.1' to nie liczba. Zapisz te wartości w formie krotki i wtedy porównuj.

[heniek_8]

@bachus: nie, porównywanie tekstów to jak sortowanie w porządu alfabetycznym

In [2]: "6.7" > "6.10"
Out[2]: True

[bachus]

@m_ba: cześć, to skrypt od vendora, ja tego nie chcę poprawiać - czyli Twoim zdaniem warunek nie jest spełniony? Wiadomo, ja tam sobie mogę podstawić or 1=1, chodzi mi o samo to czemu skrypt się nie wykonuje.

[yojo2]

chodzi mi o samo to czemu skrypt się nie wykonuje.

@bachus: skrypt się nie wykonuje, bo ktoś błędnie zapisał sprawdzanie wersji. Tak samo "10.0" nie spełniłoby warunku >= "6.7.1"

[bachus]

@yojo2 dzięki :-)

[yojo2]

@bachus: tak dla szerszego wyjaśnienia - "6.7.1" zapisane w takiej formie, z punktu widzenia Pythona to jest zwykły tekst. On sam z siebie nie wie w jakim formacie jest zapisany ten numer wersji, ani że w ogóle są tutaj jakiekolwiek cyfry.

Porównując dwa ciągi tekstowe operatorami <, >, =, porównywane są wartości ASCII poszczególnych znaków. Przykładowo, w Twoim przypadku porównując "6.10" i "6.7.1":
6 == 6
"." == "."
1

[bachus]

@yojo2: dziękuję za wyjaśnienie dobry człowieku!

[murdoc]

@bachus: chyba nie ja będę w pierwszej kolejności zastąpiony AI xD

[JebawczanRysionu]

@bachus: off topic, nie jestem specem, ale zamiast próbować dostosować apkę pod każdą wersję Pythona jednocześnie, lepiej załączyć do projektu requirements.txt gdzie określisz wymaganą wersję do odpalenia

[bachus]

@JebawczanRysionu wykomentowalem if-a, skrypt zadziałał

[kot_gagarina]

wykomentowalem if-a, skrypt zadziałał

@bachus: Świetny pomysł. Do momentu przeniesienia na produkcję, gdzie będzie np. wersja 6.7.0. :-))
Ale co tam, niech ktoś inny się męczy.
Pamiętaj, żeby nigdy, przenigdy nie zostawiać w gicie śladu po sobie jak dodasz taki kod.
Dam ci jeszcze jedną radę: nie baw się w rządną obsługę wyjątków, tu masz przykład dobrych

[bachus]

@kot_gagarina ja tego gòwna nie pisałem tylko Nutanix.