code refactor

related to #8 (closed):

• I kept changes per commit small, so the number of commits got kind of numerous...

src/icartt/dataset.py

@@ -594,8 +594,11 @@ class Dataset:
                 vstandardname += [standardname]
                 vlongname += [longname]
 
-            return { # TODO: refactor dict comp for readability?
-                shortname: Variable(
+            d = {}
+            for shortname, unit, standardname, longname, scale, miss in zip(
+                    vshortname, vunits, vstandardname, vlongname, vscale, vmiss
+                ):
+                d[shortname] = Variable(
                     shortname,
                     unit,
                     standardname,
@@ -604,10 +607,10 @@ class Dataset:
                     miss=miss,
                     vartype=vtype,
                 )
-                for shortname, unit, standardname, longname, scale, miss in zip(
-                    vshortname, vunits, vstandardname, vlongname, vscale, vmiss
-                )
-            }
+
+            return d
+
+
 
         self.dependentVariables = readVars(f, VariableType.DependentVariable)