Path: csiph.com!fu-berlin.de!uni-berlin.de!not-for-mail
From: Stefan Behnel <python-de@behnel.de>
Newsgroups: de.comp.lang.python
Subject: Re: [Python-de] f-string Formatanweisung
Date: Mon, 22 Jun 2020 14:55:40 +0200
Lines: 102
Message-ID: <mailman.252.1592830727.24731.python-de@python.org>
References: <rcq1j2$5b5$1@dont-email.me> <3c7649df-8e7a-6099-878c-a10882a90955@behnel.de>
Mime-Version: 1.0
Content-Type: text/plain; charset=utf-8
Content-Transfer-Encoding: 8bit
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101 Thunderbird/68.8.0
In-Reply-To: <rcq1j2$5b5$1@dont-email.me>
Content-Language: de-DE
Precedence: list
Xref: csiph.com de.comp.lang.python:5645

Hallo Martin,

schön, dass du deinen Code hier veröffentlichst. Lass es mich zum Anlass
nehmen, weiter unten noch ein paar Kommentare dazu zu schreiben.

Das hier

    s='f"'+(f"{{i:0{dl}d}} {{gray:0{fw}b}} {{bin:0{fw}b}}")+'"'

scheint mir ein Versuch zu sein, einen f-string innerhalb einer
Zeichenkette zu generieren. War das beabsichtigt? Zumindest wird er dann
nirgendwo verarbeitet. Wie Mike schon bemerkte, fehlt da irgendwas, aber
mir ist nicht so ganz klar, was der Zweck war und was dann die richtige
Korrektur wäre. Kannst du das ein bisschen erklären?


ruppert@hs-worms.de schrieb am 22.06.20 um 12:36:
> #!/usr/bin/python3
> import sys
> if(len(sys.argv)<2):
>     a=sys.argv[0].split('/')
>     a=a[len(a)-1]
>     print('usage:',a,'digits of GrayCode')
>     print('   eg.',a,'12')
>     exit(1)
> 
> def bintogray(bin):
>     return(bin^(bin>>1))
>     
> def graytobin(n,gray):
>     val=0
>     for i in range(n,-1,-1):
>         val = val|((((1<<(i+1))&val)>>1)^((1<<i)&gray))
>     return(val)
> 
> dbg=0
> if(len(sys.argv)>2):dbg=int(sys.argv[2])
> fl=int(sys.argv[1])
> fw=fl
> if(fw<4):fw=4
> dl=int(fl/3.321928+1) #dezimale Stellen
> if(dl<4):dl=4
> f="%%-%ds %%-%ds %%-%ds"%(dl,fw,fw)
> if(dbg>0):print("f=",f)
> h=f%("#val","gray","bin")
> f="{{:0{:1d}d}} {{:0{:1d}b}} {{:0{:1d}b}}".format(dl,fw,fw)
> #s wird in der for-Schleife unten nicht interpretiert,
> #sondern als string ausgegeben:
> s='f"'+str(f"{{i:0{dl}d}} {{gray:0{fw}b}} {{bin:0{fw}b}}")+'"'
> if(dbg>0):print("f=",f)
> if(dbg>0):print("s=",s)
> for i in range(2**fl):
>     if(i%24==0):print(h)
>     gray=bintogray(i)
>     bin=graytobin(fl,gray)
>     print(f.format(i,gray,bin))
>     s='f"'+(f"{{i:0{dl}d}} {{gray:0{fw}b}} {{bin:0{fw}b}}")+'"'
>     print(s)

Zusätzlich zu dem, was Mike schon gesagt hat: es wäre vermutlich auch für
dich einfacher gewesen, in diesem Code einen Fehler zu finden, wenn du den
Code mit mehr Abständen hingeschrieben hättest. Als jemand, der schon oft
genug kopfschüttelnd vor meinem eigenen Code von vor ein paar Jahren
gestanden hat, kann ich dir sagen: Code wird immer für Menschen
geschrieben, nicht für Maschinen.

Ich finden z.B. das hier

    if(fw<4):fw=4
    dl=int(fl/3.321928+1) #dezimale Stellen
    if(dl<4):dl=4
    f="%%-%ds %%-%ds %%-%ds"%(dl,fw,fw)

sehr schwer zu lesen, wohingegen ich das folgende viel klarer finde:

    if fw < 4:
        fw = 4

    dl = int(fl / 3.321928 + 1)  # dezimale Stellen
    if dl < 4:
        dl = 4

    f = "%%-%ds %%-%ds %%-%ds" % (
        dl, fw, fw)

Noch besser als das letzte Kommando finde ich in diesem Fall:

    f = f"%-{dl}s %-{fw}s %-{fw}s"

Und der Zahl "3.321928" würde ich auch noch einen Namen geben, damit klar
ist, woher die so plötzlich kommt und was für eine Bedeutung die hat, z.B.

    DEZIMALE_STELLEN_PRO_FRZLFIFF = 3.321928

    dezimale_stellen = int(fl / DEZIMALE_STELLEN_PRO_FRZLFIFF + 1)

Vielleicht ist das nicht auf den ersten Blick klar, aber es könnte
irgendwann ein Tag kommen, an dem du jemandem auf den Knien danken
möchtest, dass eine Variable im Code "dezimale_stellen" heißt und nicht
"dl". Im besten Fall dir selbst.

Stefan