C# C# Streams and Data Processing Parsing Data Working with Floating-Point Types

Richard Miles
Richard Miles
10,964 Points

I'm totally lost here.. Any ideas?

can't figure out what I'm doing wrong here, but I keep getting this error.

Program.cs
using System;
using System.IO;

namespace Treehouse.CodeChallenges
{
    public class Program
    {
        public static void Main(string[] arg)
        {
        }

       public static WeatherForecast ParseWeatherForecast(string[] values)
       {
            var weatherForecast = new WeatherForecast();
            weatherForecast.WeatherStationId = values[0];
            DateTime timeOfDay;
            if (DateTime.TryParse(values[1], out timeOfDay))
            {
                weatherForecast.TimeOfDay = timeOfDay;
            }
            int temperature;
            if (int.TryParse(values[3], out temperature))
            {
                weatherForecast.Temperature = temperature;
            }
           double PrecipitationChance;
           if (double.TryParse (values [4], out PrecipitationChance))
           {
               weatherForecast.Temperature = temperature;
           }
           double PrecipitationAmount;
           if (double.TryParse (values[5], out PrecipitationAmount))
           {
               weatherForecast.Temperature = temperature;
           }
            return weatherForecast;
        }
    }
}
WeatherForecast.cs
using System;

/* Sample CSV Data 

weather_station_id,time_of_day,condition,temperature,precipitation_chance,precipitation_amount
HGKL8Q,06/11/2016 0:00,Rain,53,0.3,0.03
HGKL8Q,06/11/2016 6:00,Cloudy,56,0.08,0.01
HGKL8Q,06/11/2016 12:00,PartlyCloudy,70,0,0
HGKL8Q,06/11/2016 18:00,Sunny,76,0,0
HGKL8Q,06/11/2016 19:00,Clear,74,0,0
*/

namespace Treehouse.CodeChallenges
{
    public class WeatherForecast
    {
        public string WeatherStationId { get; set; }
        public DateTime TimeOfDay { get; set; }
        public Condition Condition { get; set; }
        public int Temperature { get; set; }
        public double PrecipitationChance { get; set; }
        public double PrecipitationAmount { get; set; }
    }

    public enum Condition
    {
        Rain,
        Cloudy,
        PartlyCloudy,
        PartlySunny,
        Sunny,
        Clear
    }
}

2 Answers

andren
MOD
andren
Treehouse Moderator 28,323 Points

Looking at your code it's pretty clear that you have copied the second example and used that as a starting point for the two new variables. Which is fine, but you have forgotten to modify a pretty key part of the code you copied which is the body of the if statements. It is within those if statements that you actually assign the variables to the weatherForecast object.

If you change the bodies of those if statements as well like this:

double PrecipitationChance;
if (double.TryParse(values[4], out PrecipitationChance))
{
    weatherForecast.PrecipitationChance = PrecipitationChance;
}
double PrecipitationAmount;
if (double.TryParse(values[5], out PrecipitationAmount))
{
    weatherForecast.PrecipitationAmount = PrecipitationAmount;
}

Then your code will pass. Though one thing I'd like to mention is that in C# variables are usually not named in PascalCase (every word capitalized), but are instead named in camelCase (every word expect the first one capitalized). Following naming standards like that will make your code easier to read and understand by other coders.

Richard Miles
Richard Miles
10,964 Points

Thanks for the help! Couldn't figure out what I was doing wrong and that fixed it.