Das deutsche Python-Forum

eject hat geschrieben:

Code: Alles auswählen

filename = 'mobile.number'
targetfile = open(filename, 'w')

So, jetzt funktioniert es auch als normale .py Datei. ^^

1. Benutze keine Sternchenimporte. Damit holst Du Dir alle Namen aus dem importierten Modul in den Namensraum Deines Modules. Und das kann unerwünschte Effekte haben. Zudem finde ich, dass es beim Lesen von Code von Vorteil ist, wenn über das Modulprefix ersichtlich ist, woher ein Name stammt → ``randint()`` ↔ ``random.randint()``
2. Verwende Namen, hinter denen wir, die wir Deinen Code nicht kennen und auch Du in einem halben Jahr noch verstehen, was damit gemeint ist. Stell' Dir mal ein etwas größeres Programm vor, das mit `a`, `w` und `z` hantiert. Mindestens ich müsste mir beim Lesen eine Notiz anlegen, um nicht immer wieder nachschauen zu müssen, auf was die Namen denn verweisen. Warum nicht `COUNTRY_CODE` an Stelle von `z`?
3. Verwende Funktionen, sie sind etwas wunderbares!
4. Öffne Dateien mit dem with statement. Damit stellst Du sicher, dass die Datei auch im Falle eines Fehlers wieder geschlossen wird.

import random

COUNTRY_CODE = '+49'

def write_to_file(filename, content):
    with open(filename, 'w') as file_:
        file_.write(content)

def main():
    while True:
        print("|--------------------------------|")
        print("|                                        |")
        print("|    IRGENDEINEN TEXT         |")
        print("|                                         |")
        print("|--------------------------------|")
        print("|VODAFONDE+++TELEKOM+++EPLUS+++O2|")
        print("| 152    |   151   |   157 | 159 |")
        print("| 162    |   160   |   163 | 176 |")
        print("| 172    |   170   |   177 | 179 |")
        print("| 173    |   171   |   178 |     |")
        print("| 174    |   175   |       |     |")
        print("|________________________________|")
        print("")
        print("")
        filename = input('Filename: ')
        prefix = input('Prefix: ')
        runs = int(input('How many mobile numbers to create? '))
        template = '{}{}{{phone_number}}\n'.format(COUNTRY_CODE, prefix)
        mobile_numbers = [template.format(
            phone_number=random.randint(11111111, 99999999)
        ) for _ in range(runs)]
        write_to_file(filename, ''.join(mobile_numbers))
        vote = input('Once more again (y/n)? ')
        if vote in 'nN':
            break

if __name__ == '__main__':
    main()

mutetella hat geschrieben:

eject hat geschrieben:

Code: Alles auswählen

filename = 'mobile.number'
targetfile = open(filename, 'w')

So, jetzt funktioniert es auch als normale .py Datei. ^^

Auch die erste Version funktioniert. Ich nehme mal an, Du führst das Modul via Python 2 aus. Dabei erzeugt das erste `input()` tatsächlich einen Fehler, weil die `input()` Funktion unter Python 2 etwas anderes macht als unter Python 3.
Oder aber ich verstehe nicht, was Du anfangs mit '... als normale .py Datei funktioniert es bei mir nicht. gemeint hast.

Zu Deinem Code:

1. Benutze keine Sternchenimporte. Damit holst Du Dir alle Namen aus dem importierten Modul in den Namensraum Deines Modules. Und das kann unerwünschte Effekte haben. Zudem finde ich, dass es beim Lesen von Code von Vorteil ist, wenn über das Modulprefix ersichtlich ist, woher ein Name stammt → ``randint()`` ↔ ``random.randint()``
2. Verwende Namen, hinter denen wir, die wir Deinen Code nicht kennen und auch Du in einem halben Jahr noch verstehen, was damit gemeint ist. Stell' Dir mal ein etwas größeres Programm vor, das mit `a`, `w` und `z` hantiert. Mindestens ich müsste mir beim Lesen eine Notiz anlegen, um nicht immer wieder nachschauen zu müssen, auf was die Namen denn verweisen. Warum nicht `COUNTRY_CODE` an Stelle von `z`?
3. Verwende Funktionen, sie sind etwas wunderbares!
4. Öffne Dateien mit dem with statement. Damit stellst Du sicher, dass die Datei auch im Falle eines Fehlers wieder geschlossen wird.

Hier mal eine Idee von mir:

Code: Alles auswählen

import random

COUNTRY_CODE = '+49'

def write_to_file(filename, content):
    with open(filename, 'w') as file_:
        file_.write(content)

def main():
    while True:
        print("|--------------------------------|")
        print("|                                        |")
        print("|    IRGENDEINEN TEXT         |")
        print("|                                         |")
        print("|--------------------------------|")
        print("|VODAFONDE+++TELEKOM+++EPLUS+++O2|")
        print("| 152    |   151   |   157 | 159 |")
        print("| 162    |   160   |   163 | 176 |")
        print("| 172    |   170   |   177 | 179 |")
        print("| 173    |   171   |   178 |     |")
        print("| 174    |   175   |       |     |")
        print("|________________________________|")
        print("")
        print("")
        filename = input('Filename: ')
        prefix = input('Prefix: ')
        runs = int(input('How many mobile numbers to create? '))
        template = '{}{}{{phone_number}}\n'.format(COUNTRY_CODE, prefix)
        mobile_numbers = [template.format(
            phone_number=random.randint(11111111, 99999999)
        ) for _ in range(runs)]
        write_to_file(filename, ''.join(mobile_numbers))
        vote = input('Once more again (y/n)? ')
        if vote in 'nN':
            break

if __name__ == '__main__':
    main()