• 0 Posts
  • 354 Comments
Joined 7 个月前
Cake day: 2025年5月10日
  • 6nk06@sh.itjust.workstoProgramming@programming.dev•Kinda proud of this python script I wrote
    1·
    10 小时前

    Yes (and I’m sorry for not having more time to study this but):

    logDateTime

    That thing should be automatically printed by the logger somehow (most of the time, timestamps and log levels are inside the logger itself). If it’s not, check the configuration of the logger. Anyway. logDateTime has got to go. IMHO a logger is a magical object that you don’t control. It’s the user of the application who should set the rules one way or the other, like:

    logger.info("qbittorrent listen port set to...

    IIRC the logger can be set from the command-line, which you don’t control anyway. It’s the same thing in C++ and with any library on the planet (trust me), someone sets up the logging system for you outside of the application (in the main function or on the command-line or with environment variables or in a library/DLL) and all you can do is call “log.debug()” or “log.warning()” and hope for the best. Some libraries have huge systems to handle logs (rotating logs, serial ports, files, network…) and you shouldn’t give yourself headaches while using those things. Think of a logger as a pre-configured system that does way more that you should and want to know.

    logger = logging.getLogger(…

    Put that outside of main, after the imports. You can prepend it with an underscore to show that it’s a “private or protected” object belonging to your module. If it’s in main, you can’t use it in the other functions. You should do something like:

    from qbittorrent import Client

_logger = logging.getLogger(...)

    This way your logger is independent on whether you’re using it from the command line (with main) or not (with an import).

    QB_PASSWORD

    Same thing with all the QB_ variables IF they don’t change, you can put them globally for such a module, like _QB_PASSWORD = ... and use it everywhere if your module were to change in the future. But only if the variables are constants across the usage of the script.

    a40124291102d

    Last but not least, use argparse if you can, it’s a simple module to handle command-line arguments. I guess that the container ID can change, you can either detect it with the subprocess module, or write def get_current_forwarded_port(container_id = "a40124291102d"): or use argparse. Feel free to ask if argparse if too weird, but it’s the best module out there and it’s built-in.

    TL;DR:

    • change logFilePath, what if I don’t have a filesystem or a hard drive?
    • change logDateTime and “message”, what if I (the user of your script) want another format?
    • change qbConfigUrl to be configurable, what if I use your script on a weird custom Linux or a BSD, or macOS or Windows?

    Anyway, great job, writing weird stuff for my own needs is how I started coding. Have fun.

  • 6nk06@sh.itjust.workstoProgramming@programming.dev•Kinda proud of this python script I wrote
    7·
    21 小时前

    Make the logger a global variable. This may be the only case where its acceptable to be global. And remove the log function. You can call the logger directly.

    Also wrap the main code (after BEGIN SCRIPT) in a main function.

    Last but not least log everything to the standard output (logging should do that by default), or configure the path in main(), it’s cleaner. If you log to stdout, the shell can redirect the logs to any path like python script.py > mylogs.txt

  • 6nk06@sh.itjust.workstoNo Stupid Questions@lemmy.world•to any EU federalist, why do you believe you can make a federation out of 27 different countries with different languages and cultures?
    3·
    3 天前

    English speaking

    Not in your constitution or your law. Nice try though.

    predominantly protestant

    With all the mormons, catholics, asians, and scientologists? Do you think we are THAT stupid?

    different countries … had to assimilate

    lol. All you achieved to do was destroy the existing culture of the native americans.

    a certain way of seeing the world

    Capitalism. That’s all what is left in your rotten brain.

    What percentage of Italians or Romanians can speak fluent English

    99% I guess, what is your point?

    Every American knows what CNN, WSJ and USA Today are

    TV is culture now? Fuck off.

    Maybe you can prove me wrong?

    You’re a nazi and/or russian troll, I don’t have to prove you wrong.

    Edit: you’re from .ml, I understand now why you have nazi ideas. Have a nice day. You can still grow up though and take care of people around you instead if licking Putin’s ass, but it will be hard work.