Wie findet ihr diesen Python Code?

Ich habe ein kleines Programm in Python geschrieben die verschiedenen Temperaturen in andere umwandelt. Mich würde eure Meinung sehr interessieren.

Schon mal danke im Voraus für das Feedback.

Hier ist der Code:

https://github.com/tibor20319/tibor20319/blob/main/Temperatur%20Umwandler.py

Der Code ist verbesserungsfähig 100%

Der Code ist gut 0%

Der Code enthält Grundlegende Fehler 0%

8 Stimmen

5 Antworten

Vom Beitragsersteller als hilfreich ausgezeichnet

ralphdieter

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Programmiersprache, Python

13.01.2023, 19:09

Der Code ist verbesserungsfähig

Warum ist die Eingabe -300 für Fahrenheit erlaubt, aber nicht für Celsius und Kelvin? Wenn schon, würde ich überall auf xxx_to_kelvin(temp)>=0 testen.

Warum werden bei der Umrechnung von Fahrenheit so viele Nachkommastellen ausgegeben?

Warum muss man nach dem Beenden nochmal eine Taste drücken?

Warum muss man überhaupt so viele Tasten drücken? Eingaben der Form <Zahl><Einheit> (z.B. "273 K") wären viel bequemer, und ausgeben kannst Du immer alle drei Einheiten ("273.00 K = -0.15 °C = 31.73 °F"). Beendet wird das Programm mit einer leeren Eingabe.

Wieso treibst Du so viel Aufwand bei der Prüfung der Temperatureingabe, schlampst aber bei der Auswahloption? Und hast Du auch mal die Eingabe "nan" oder "-inf" getestet?

Und last not least: Dein Programm ist praktisch nicht erweiterbar. Baue noch zwei weitere Einheiten mit ein, dann wird nichts mehr funktionieren. Eine Alternative wäre, nur die Parameter (m, c) für die Umrechnung von und zu einer Basiseinheit zu speichern:

Einheiten = {
  C: ( 1, 0, "Celsius", "°C" ) # Basiseinheit
  F: ( 9/5, 32, "Fahrenheit", "°F" )
  K: ( 1, -273.15, "Kelvin", "K" )
}

Dann brauchst Du nur noch zwei Umrechnungsfunktionen

def toC ( temp, unit ) : return unit[0]*temp + unit[1]
def fromC ( temp, unit ) : return (temp-unit[1]) / unit[0]

und kannst damit alles erschlagen. Wenn jetzt noch weitere Einheiten dazukommen, brauchst Du im Idealfall nur eine neue Zeile im dict Einheiten.

PrinceSaid

13.01.2023, 17:41

Der Code ist verbesserungsfähig

Ich würde im Hauptteil der Anwendungen ein paar Verbesserungen vornehmen:

while true ist ein Code-Smell. Ersetze das durch eine geeignete Abbruchbedingung, damit sofort ersichtlich ist, wann die Schleife endet. Endlosschleifen sollte man vermeiden.
Der Code in der Schleife wiederholt sich sehr. Baue das etwas um.
Die Konstante mit dem absoluten Nullpunkt auch im oberen Teil verwenden

KarlRanseierIII

Von gutefrage auf Grund seines Wissens auf einem Fachgebiet ausgezeichneter Nutzer

Programmiersprache, Python, Python 3

13.01.2023, 22:47

Der Code ist verbesserungsfähig

Ich würde das mit structural matching machen wollen:

match(option):
   case 1:
     .....
   case 2:
     .....

Oder alternativ natürlich gleich ganz anders, Für die Umrechungen eine liste, die funktion als Lambda hinterlegen und Dein einfach über dne Infex aufrufen :-D.

Racooder

13.01.2023, 17:38

Der Code ist verbesserungsfähig

Grundsätzlich ein guter Code. Mich wundert nur das du oben den Absoluten Nullpunkt als Variable definierst und dann unten es trotzdem als Zahl schreibst.

Woher ich das weiß:Hobby – Hobby Programmierer von kleinen Spielen, Websites und Bots

nail745

13.01.2023, 17:33

Im allgemeinen ist er recht übersichtlich, würde dir aber empfehlen Verben zu benutzen.

Ähnliche Beiträge

Fehler "126" Permission Denied Python - Was tun?

Ich will einen Code auf einem Raspberry PI 4 ausführen und es kommt folgender Fehler:

/tmp/geany_run_script_TIHFZ1.sh: 7: ./Neu: Permission denied

------------------

(program exited with code: 126)

Press return to continue

Folgender Code soll ausgeführt werden, um mittels "L298" Modul einen Motor zu steuern:

https://github.com/jumejume1/pi-l298n-dc-motor/blob/master/l298n_dc.py

Weiß da jemand mehr? Danke im Voraus!

...zum Beitrag

Snake Game mit Python Programmieren?

Hey, ich habe ein kleines Problem. Und zwar funktioniert:

if getPixelColorStr == "black":

setPenColor("red")

bei mir nicht... hier der ganze Code:

https://github.com/Huhnicorn/Snake-Python/blob/master/Snake-Python.py

...zum Beitrag

Wie führt man dieses Programm aus? Wie bekomme ich es? Github, Python, Programmierung?

Hei, ich halte in Schulprojekt über Dijkstras Algorithmus und würde gerne dieses Programm der Klasse vorführen, doch wie bekomme ich es und wie führe ich es aus? LG xd

Programm: https://github.com/raphsenn/Python-Pathfinder-with-Dijkstras-Algorithm

...zum Beitrag

OpenWeatherMap Python?

import requests

API_KEY = "xyz"

city = "Berlin"

url = 'https://api.openweathermap.org/data/2.5/weather?q={city}&appid={API_KEY}&units=metric'

data = requests.get(url).json()

temp = ['main']['temp]-273.15

print('Temperatur: ' + temp )

Code gibt es so im Internet. Allerdings kommt bei mir folgender Error:

Traceback (most recent call last):

File "/xyz/wetter.py", line 9, in <module>

temp = data['main']['temp']-273.15

KeyError: 'main'

Irgendwelche Ideen an was das liegen kann?

...zum Beitrag

Wie funktioniert dieser Code?

Guten Tag,

ich habe hier ein Discord tokengrabber code und ich will lernen diesen zu verstehen. Aber ich habe echt gar keine Ahnung wo ich anfangen soll. Ich mein für was braucht man diese 3 Verschlüsselungsmethoden (base64, AES, win32crypt)? Kann mir jemand bitte diesen Code erklären?

LG ItsIdzA

P.S.: Ich werde diesen Code nie benutzen um jemand damit zu schaden, ich habe den rausgesucht, weil ich denke, dass ich mich damit weiterbilden kann.

https://github.com/AstraaDev/Discord-Token-Grabber-V2/blob/main/token_grabber.py

...zum Beitrag

eine .exe datei umcoden?

Guten Tag,
ich habe damals mal eine .exe dabei von einem Kollegen zugeschickt bekommen.
Sobald ich das Programm ausführe habe ich die Option mit "f4" meinen Autoclicker zu aktiveren oder mit "f8" du Destructen. Jedoch würde ich gerne die Taste "f4" umbinden und das geht soweit ich weiß nur wenn ich das Programm umcode, so stelle ich mir die Frage wie und mit welchem Tool.

Ich habe gerade den cpp Code gefunden,
kann mir jemand erklären wie ich den jetzt änder und wieder in eine exe Datei konvertiere https://github.com/itsnotjohn/vepe-clicker-fixed/blob/main/main.cpp

...zum Beitrag

Command 'python' not found, did you mean: command 'python3' from deb python3 command 'python' from deb python-is-python3?

Hallo,

ich bin dabei mein attinity85 zu programmieren und bin an dem tastaturen layout hängengeblieben. Wollte jetzt mithilfe von https://github.com/mame82/duck2spark/tree/master und https://github.com/mame82/duckencoder.py meinen code so verändern lassen, dass es passt. Wenn ich jetzt aber

cat text.txt | python duckencoder.py -r -l de > /dev/hidg0

eingebe, so wie es von mir verlangt wird bekomme ich die Fehlermeldung

Command 'python' not found, did you mean:

command 'python3' from deb python3

command 'python' from deb python-is-python3

ich habe beides installiert und es funktioniert trotzdem nicht.

Was mache ich falsch oder bin ich vielleicht sogar auf dem komplett falschen Weg? Ich werde erst morgen antworten können, bedanke mich aber für alle antworten

...zum Beitrag

Warum schreibt Python nicht in .txt file sobald ssh verbindung getrennt ist?

Hallo zusammen

Ich habe ein Python Programm mit Selenium code. Dieses Programm wird durch einen Cronjob für einige stunden in einem regelmässigen 15 minuten rhythmus ausgeführt. Um zu überprüfen, wie das Programm läuft, habe ich mir ein Log-File bereit gelegt. Python schreibt dort bei jedem Durchlauf einen Timestamp, plus alle Variablen die mich interessieren, rein. Nun habe ich heute Nacht gesehen, dass ab einer gewissen Zeit keine Einträge in das Log-File gemacht wurden. In den Cron-Logs habe ich gesehen, dass der Job genau wie erwünscht ausgeführt wurde. Hat jemand eine Idee woran das liegen könnte?

...zum Beitrag

Gut Programmiert(Python- DinoGame + KI)?

Hey Python Community:

Ich wollte mal fragen ob ich das Dino-Spiel von Google gut nachprogrammiert habe. Danke im Voraus, dass ist der Code:

https://github.com/NCode73/DinoGame-Chrome-AI-

...zum Beitrag

Python neue Zeile ohne Entertaste?

Ganz blöde Frage, aber wie komm ich in der Python interpreter app in die nächste Zeile? Wenn ich Enter drücke wird ja direkt der Code ausgeführt.

Ich fange gerade erst an, programmieren zu lernen. Um mein Problem konkreter zu schildern, ein Beispiel:

x = float(input("Bitte geben Sie die aktuelle Temperatur ein: "))

if x < 20 :

print("Es ist zu kalt.")

elif x > 25 :

print("Es ist zu heiß")

else :

print("Es ist angenehm warm.")

So, sobald ich die erste Zeile eingebe, wird sofort der Code ausgeführt. Ich möchte aber, dass der Code erst ausgeführt wird, wenn die ganzen if-Bedingungen auch dabei sind.

...zum Beitrag

Python if__main__?

Hallo zusammen,

ich bin gerade dabei in Python mithilfe eines Youtube Videos einen Voice Assistenten zu programmieren. Ich will gerade den Main Loop einfügen und dieser lautet: if__name__ == '__Main__':

Allerdings wenn ich nun den Code ausführe, bekomme ich eine Fehlermeldung, diese lautet:

File "C:\####\####\Desktop\Assistent\TestCode.py", line 41

if__name__ == '__TestCode__':

SyntaxError: invalid syntax

[Finished in 90ms]

Warum sollte das ein ungültiger Satzbau sein, wenn das doch die richtige Funktion ist?

...zum Beitrag

Satellitenprogramm Python image?

Hey ihr Süßen, hab einen Code für einen Sateliten der um verschiedene Objekte im All umkreist erstellt und will das ganze jetzt noch mit Bildern verschönern. Einen Sateliten habe ich schon als Turtle shape hinzugefügt und das funktioniert auch auf meinem Laptop (auf anderen Geräten mit denen ich den Code ausprobieren nicht). Jetzt will ich den Hintergrund ändern und speicher das Bild genau so ab wie den Sateliten aber das funktioniert nicht.

Wenn ihr noch mehr vom Code braucht, fragt einfach. lg

...zum Beitrag

Wo ist der Fehler in meinem Python Code (MQTT)?

Ich habe ein Python Script erstellt, damit später bei Nachrichten bestimmte befehle ausgeführt werden. Beim ersten Script(Openhab_Mqtt.py) habe ich beim Testen 3 Terminals aufgemacht und beim 2.(Openhab_Mqttv2.py) auch genau gleiche Sorgehensweise.

Im 1. Terminal habe ich das Python Script gestartet mit:

python Openhab_Mqttv2.py

Output:
Script 1: Connected to MQTT broker
Script 2: Connected to MQTT broker
Connected to MQTT broker

Im 2. Terminal habe ich die Nachricht ausgegeben bei Nachrichten mit:

mosquitto_sub -t "main/messager" -v -u xxx -P xxx

Output(Nach T3):
Script 1: main/messager test
Script 2: main/messager test

Und im 3. Terminal habe ich eine Nachricht gesendet mit:

mosquitto_pub -t "main/messager" -m "test" -u xxx-P xxx

Output:
Script 1: Payload: test und BLA...
Script 2:

Ich kann einfach nicht den Fehler finden, warum es bei Openhab_Mqtt.py funktioniert und bei Openhab_Mqttv2.py nicht funktioniert.

Openhab_Mqtt.py DATA:

import paho.mqtt.client as mqtt
import os
import time
import bme280_temp
import smbus2
import bme280
import threading

def on_subscribe(client, userdata, mid, granted_qos):
 print("Subscribed to topic : " + str(mid) +" with QoS" + str(granted_qos))


def on_connect(client, userdata, flags, rc):
 print("Connected to MQTT broker")


def on_message(client, userdata, msg): 
 print(msg.topic+" "+str(msg.qos)+" "+str(msg.payload))
 print("Payload: " + msg.payload.decode())
 if msg.payload.decode() == "send_signal":
  print("Calling script to send signal...")
  IR_Signal()
 if msg.payload.decode() == "temp_bme280_start":
  print("Calling script to for temperature start...")
  userdata["counter"] += 1
  print(str(userdata["counter"]))
  client.publish("frame/monitor", "START", qos=0, retain=True)
  Temp_sensor() 
 if msg.payload.decode() == "temp_bme280_stop":
  print("Calling script to for temperature stop...")
  userdata["counter"] -= 1
  print(str(userdata["counter"]))
  client.publish("frame/monitor", "STOP", qos=0, retain=True)
 if msg.payload.decode() == "test":
  print("BLA...")  
  print(str(["counter"])) 
  os.system("vcgencmd measure_temp") 
 else:
  pass    



#Bme280_basic Temperature
address = 0x76
bus = smbus2.SMBus(1)
calibration_params = bme280.load_calibration_params(bus, address)

def Temp_sensor():
 data = bme280.sample(bus, address, calibration_params)
 temperature_celsius = data.temperature      
 print("Temperature: {:.2f} °C".format(temperature_celsius))
 time.sleep(2)
    
 
def IR_Signal():
 os.system("irsend SEND_ONCE pioneer_vsx-301 KEY_POWER")
 print ("Signal Send!")


client = mqtt.Client()
client.on_message = on_message 
client.on_connect = on_connect
client.user_data_set({"counter": 0})
client.username_pw_set( "xx" , "xx" )
client.connect( "192.x.x.x", 1883, 60)
client.subscribe( "main/messager" , qos=0)


client.loop_forever()

Openhab_Mqttv2.py

import paho.mqtt.client as mqtt
import time
import smbus2
import bme280
import threading


# Getting information
def on_subscribe(client, userdata, mid, granted_qos):
 print("Subscribed to topic : " + str(mid) +" with QoS" + str(granted_qos))
                                                                 
def on_connect(client, userdata, flags, rc):
 print("Connected to MQTT broker")


# Damit loops nicht am anfang durchstarten 
TempLoop_running = False


# Checking for messages to execute code
def on_message(client, userdata, msg):
 print(msg.topic+" "+str(msg.qos)+" "+str(msg.payload))
 print("Payload: " + msg.payload.decode())
 if msg.payload.decode() == "Temp_Start":
  start_TempLoop()
 if msg.payload.decode() == "Temp_Stop":
  stop_TempLoop()
 if msg.payload.decode() == "test":
  print("TESTING...")
  client.publish("main/messager", "START", qos=0, retain=True)
 else:
  pass



# Execute things here ->


# Getting Temperature
address = 0x76
bus = smbus2.SMBus(1)
# Load calibration parameters
calibration_params = bme280.load_calibration_params(bus, address)
def Temperature_loop():
 global TempLoop_running
 while TempLoop_running:
  data = bme280.sample(bus, address, calibration_params)
  #Extract temperature, pressure, and humidity
  temperature_celsius = data.temperature      
  #Print the readings
  print("Temperature: {:.2f} °C".format(temperature_celsius))
  # Nachricht an das entsprechende Topic senden
  client.publish("Temp/Loop", "Temperature: {:.2f} °C".format(temperature_celsius))
  # Wartezeit, um die Ausgabe zu verlangsamen
  time.sleep(2)
# Funktion zum Starten der Temperatur Schleife
def start_TempLoop():
 global TempLoop_running
 TempLoop_running = True
 threading.Thread(target=Temperature_loop).start()
# Funktion zum Stoppen der Temperatur Schleife
def stop_TempLoop():
 global TempLoop_running
 TempLoop_running = False



# Set general data
client = mqtt.Client()
client.on_message = on_message 
client.on_connect = on_connect
client.connect( "192.x.x.x", 1883, 60)
client.username_pw_set( "xx", "xx" )
client.subscribe( "main/messager", qos=0)


# Loop forever
client.loop_forever()

...zum Beitrag

Output in einer JSON Datei speichern?

Ich brauch mal wieder Hilfe in Python xd

Weiß jemand wie ich denn output von Zeile 42, und 35 in einer JSON Datei speichern kann? Sorry für die dumme Frage xd

https://github.com/124-Darky/EvilScraper/blob/main/scraper.py

...zum Beitrag

Was möchtest Du wissen?

Deine Frage stellen