Critique for the group: Weathersite - SvenSrc/Programming-Habit GitHub Wiki

Critique

Link to the Project


Das Projekt Weathersite von Moritz und Jonathan, haben eine funktionelle Website erstellt, wo man die lokale Wetter Nachricht bekommt oder einen gewünschten Ort eintragen kann. Dies wurde gemacht durch die Verwendung von einer API, die die Wetterdaten ausgibt.

Diese Wetterdaten werden anhand der IP herausgefunden mithilfe der folgenden Funktion.

def get_location():
    response = requests.get('https://api64.ipify.org?format=json').json()   
    ip = response["ip"]
    loc = geocoder.ip(ip)
    location_data = loc.latlng
    location_data.append(loc.city)
    return location_data

So zeigen sie auch, dass sie sich mit der Python Syntax auch auskennen. So sind auch zum Beispiel if und else vorhanden.

        if response["list"][i]["weather"][0]["main"] == "Rain":
            sorted_forecast_list_in_list.append(response["list"][i]["rain"]["3h"])
        else:
            sorted_forecast_list_in_list.append(0)

Ihr Code ist übersichtlich gestaltet und ihre Variablen sind sinnvoll benannt. So haben sie für Ihre Liste jeweils Namen verwendet.

    temp_list = []
    rain_list = []
    date_list=[]
    weather_list =[]

Weiterhin zeigen sie auch, dass sie sich mit anderen Datentypen auskennen wie einem Dictionary.

    forecast={
        "date_list": date_list,
        "temp_list": temp_list,
        "weather_list" : weather_list,
        "rain_list": rain_list
    }

Ebenfalls, was auch sehr gut ist, dass die Gruppe sich nicht nur mit einem Fachgebiet der Informatik fokussiert hat. Sie haben nicht nur mit Python gearbeitet sondern auch mit HTML, bzw. mit Django. Dies kann man an ihrem Projekt erkennen.

Was vielleicht fehlt, sind genauere Beschreibungen ihres Codes. Es ist gut, dass Kommentare vorhanden sind, allerdings wäre es besser wenn sie mehr beschreiben, was dieser Teil vom Code genau gemacht.

    # Sorted forecast list

Vielleicht wäre es besser genauer zu erklären wozu der da ist.


Aber ansonsten gibt es keine wirklich großen Beschwerden, weil es zum einem funktioniert und ordentlichen gemacht wurde. Sehr Gut gemacht 👍