Improve parsing for decimal values (#35)

viceroypenguin · Mar 30, 2023 · 9d1a419 · 9d1a419
1 parent ff0989a
commit 9d1a419
Show file tree

Hide file tree

Showing 2 changed files with 15 additions and 13 deletions.
diff --git a/Source/AlphaVantage.Net/Stocks/DailyAdjustedTimeSeriesResponse.cs b/Source/AlphaVantage.Net/Stocks/DailyAdjustedTimeSeriesResponse.cs
@@ -1,4 +1,5 @@
-using CsvHelper.Configuration.Attributes;
+using System.Globalization;
+using CsvHelper.Configuration.Attributes;
 
 namespace AlphaVantage.Stocks;
 
@@ -10,21 +11,21 @@ public class DailyAdjustedTimeSeriesResponse
 #pragma warning disable CS1591
 	[Name("timestamp")]
 	public required DateTime Timestamp { get; set; }
-	[Name("open"), NullValues("", "null")]
+	[Name("open"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? Open { get; set; }
-	[Name("high"), NullValues("", "null")]
+	[Name("high"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? High { get; set; }
-	[Name("low"), NullValues("", "null")]
+	[Name("low"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? Low { get; set; }
-	[Name("close"), NullValues("", "null")]
+	[Name("close"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? Close { get; set; }
-	[Name("adjusted_close"), NullValues("", "null")]
+	[Name("adjusted_close"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? AdjustedClose { get; set; }
 	[Name("volume"), NullValues("", "null")]
 	public required long? Volume { get; set; }
-	[Name("dividend_amount"), NullValues("", "null")]
+	[Name("dividend_amount"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? DividendAmount { get; set; }
-	[Name("split_coefficient"), NullValues("", "null")]
+	[Name("split_coefficient"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? SplitCoefficient { get; set; }
 #pragma warning restore CS1591
 }
diff --git a/Source/AlphaVantage.Net/Stocks/TimeSeriesResponse.cs b/Source/AlphaVantage.Net/Stocks/TimeSeriesResponse.cs
@@ -1,4 +1,5 @@
-using CsvHelper.Configuration.Attributes;
+using System.Globalization;
+using CsvHelper.Configuration.Attributes;
 
 namespace AlphaVantage.Stocks;
 
@@ -10,13 +11,13 @@ public class TimeSeriesResponse
 #pragma warning disable CS1591
 	[Name("timestamp", "time")]
 	public required DateTime Timestamp { get; set; }
-	[Name("open"), NullValues("", "null")]
+	[Name("open"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? Open { get; set; }
-	[Name("high"), NullValues("", "null")]
+	[Name("high"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? High { get; set; }
-	[Name("low"), NullValues("", "null")]
+	[Name("low"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? Low { get; set; }
-	[Name("close"), NullValues("", "null")]
+	[Name("close"), NullValues("", "null"), NumberStyles(NumberStyles.Float)]
 	public required decimal? Close { get; set; }
 	[Name("volume"), NullValues("", "null")]
 	public required long? Volume { get; set; }