How to save screen output into a text file

Question

So I need to create a script and save the info into a text file, and here is what I did:

import platform

def bios_version():
    uname_result = platform.uname()
    pl = {}
    pl['BIOS version'] = uname_result.version

    for a, b in pl.items():
        print(a, ":", b)

#The def is working perfectly but I don't know how to save the file.

print(bios_version())
save_path = 'D:/'
name_of_file = input("What is the name of the file: ")
completeName = os.path.join(save_path, name_of_file+".txt")

f = open(completeName, "w")
f.write(bios_version())
f.close

So it return:

f.write(bios_version())
TypeError: write() argument must be str, not None"

Akshay Jain · Accepted Answer · 2021-04-18 03:25:16Z

1

Try below code:

I don't understand why you are using loop and why to store that version detail in dictionary. If there is only single value then just store that in variable and directly return the value and just write into file.

Code

import platform
import os

def bios_version():
    uname_result = platform.uname()
    pl = f"BIOS version: {uname_result.version}"
    return pl

save_path = 'D:/'
name_of_file = input("What is the name of the file: ")
completeName = os.path.join(save_path, name_of_file+".txt")

f = open(completeName, "w")
f.write(bios_version())
f.close

Output:

BIOS version: <<Your version detail>>

answered Apr 18, 2021 at 3:25

Akshay Jain

7901 gold badge7 silver badges23 bronze badges

Sign up to request clarification or add additional context in comments.

7 Comments

KorD WLO Over a year ago

HI, thank you for the suggestion. Im a newbie in python so there are a lot of things i still confused with ;)

Akshay Jain Over a year ago

Is my solution, make sense to you or not @KorDWLO

Niloct Over a year ago

Good catch about the single value. But not so good if he needs more keys in the future.

Akshay Jain Over a year ago

Then he will change the code to loop, but for now this is only the best way. At this point he must need to understand when to use which data type and all the stuff.

KorD WLO Over a year ago

hi Jain, the suggestion abut def is good but it is not saving the content of the def

|

Niloct · Accepted Answer · 2021-04-18 03:52:08Z

0

Here's an alternative, considering other answers:

import platform
import os

def write_bios_version(file_name = None):
    pl = {}
    pl['BIOS version'] = platform.uname().version

    with open(file_name, 'w') as f:
        for a, b in pl.items():
            print(a, ":", b, file=f)

save_path = 'D:/'
name_of_file = input("What is the name of the file: ")
complete_name = os.path.join(save_path, name_of_file+".txt")
write_bios_version(complete_name)

edited Apr 18, 2021 at 3:52

answered Apr 18, 2021 at 3:09

Niloct

10.1k3 gold badges47 silver badges62 bronze badges

12 Comments

KorD WLO Over a year ago

For the def, it works perfectly fine for me. I just have trouble with saving the screen.

Niloct Over a year ago

write() argument must be str, not None". When you don't include a return statement in a function, by default it returns None. You have to change the function code.

Niloct Over a year ago

And saying that you want to "save the screen" is a bad term. You actually want to "save" your output (print statements), and if you return the pl variable from your function write will work.

Niloct Over a year ago

print(bios_version()) also prints nothing since you're not returning anything from the function.

KorD WLO Over a year ago

Hi, so i change it as u suggest and it tell me this "f.write(bios_version()) TypeError: write() argument must be str, not dict"

|

Collectives™ on Stack Overflow

How to save screen output into a text file

2 Answers 2

Code

Output:

7 Comments

12 Comments

Your Answer

Hot Network Questions

Collectives™ on Stack Overflow

2 Answers 2

Code

Output:

7 Comments

12 Comments

Your Answer

Sign up or log in

Post as a guest

Related