awelZH / airquality

Systematic and comprehensive air quality data compilation, analysis and visualisation for the Canton of Zurich, based on public data.
https://awelzh.github.io/airquality/
GNU Affero General Public License v3.0
0 stars 0 forks source link

cols über Index abfragen #2

Closed CGRBZH closed 6 months ago

CGRBZH commented 6 months ago

Spalten werden über ihren Index aufgerufen. Dies hat Nachteile:

  1. Die Reihenfolge der Spalten kann sich ändern und der Code funktioniert trotzdem, da die Indizes noch existieren. Wenn das Skript dann erst 500 Zeilen später failed, wird das Debugging schnell mal mühsam. Und im schlechtesten Fall liefert das Skript falsche Ergebnisse.
  2. Es widerspricht dem Prinzip des Fail early ;)

Aufgefallen bspw. in «02_functions.R» in Zeilen: 15: data <- readr::read_delim(file, col_select = c(-(1:10)), delim =";", locale = locale) 157: header <- header[c(1, 4, 8, 7), ]

Empfehlung: Spalten immer mit ihrem Namen aufrufen. Wenn sie nicht mehr existieren oder ihre Namen geändert wurden, wird das Skript direkt an diesem Schritt scheitern.

Sintermann commented 6 months ago

siehe #7