tracks/csharp/exercises/space-age/Example.cs in trackler-2.0.8.14 vs tracks/csharp/exercises/space-age/Example.cs in trackler-2.0.8.15
- old
+ new
@@ -6,70 +6,70 @@
private enum Planet
{
Earth, Mercury, Venus, Mars, Jupiter, Saturn, Uranus, Neptune
}
- public long Seconds { get; private set; }
+ private readonly long seconds;
- private readonly Dictionary<Planet, decimal> earthYearToPlanetPeriod = new Dictionary<Planet, decimal>
+ private readonly Dictionary<Planet, double> earthYearToPlanetPeriod = new Dictionary<Planet, double>
{
{ Planet.Earth, 1 },
- { Planet.Mercury, 0.2408467m },
- { Planet.Venus, 0.61519726m },
- { Planet.Mars, 1.8808158m },
- { Planet.Jupiter, 11.862615m },
- { Planet.Saturn, 29.447498m },
- { Planet.Uranus, 84.016846m },
- { Planet.Neptune, 164.79132m },
+ { Planet.Mercury, 0.2408467 },
+ { Planet.Venus, 0.61519726 },
+ { Planet.Mars, 1.8808158 },
+ { Planet.Jupiter, 11.862615 },
+ { Planet.Saturn, 29.447498 },
+ { Planet.Uranus, 84.016846 },
+ { Planet.Neptune, 164.79132 },
};
public SpaceAge(long seconds)
{
- Seconds = seconds;
+ this.seconds = seconds;
}
- private decimal CalculateAge(decimal periodInEarthYears)
+ private double CalculateAge(double periodInEarthYears)
{
- const decimal EARTH_ORBIT_IN_SECONDS = 31557600;
- return Math.Round(Seconds / (EARTH_ORBIT_IN_SECONDS * periodInEarthYears), 2);
+ const double EARTH_ORBIT_IN_SECONDS = 31557600;
+ return Math.Round(seconds / (EARTH_ORBIT_IN_SECONDS * periodInEarthYears), 2);
}
- public decimal OnEarth()
+ public double OnEarth()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Earth]);
}
- public decimal OnMercury()
+ public double OnMercury()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Mercury]);
}
- public decimal OnVenus()
+ public double OnVenus()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Venus]);
}
- public decimal OnMars()
+ public double OnMars()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Mars]);
}
- public decimal OnJupiter()
+ public double OnJupiter()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Jupiter]);
}
- public decimal OnSaturn()
+ public double OnSaturn()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Saturn]);
}
- public decimal OnUranus()
+ public double OnUranus()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Uranus]);
}
- public decimal OnNeptune()
+ public double OnNeptune()
{
return CalculateAge(earthYearToPlanetPeriod[Planet.Neptune]);
}
}
\ No newline at end of file