Lets provide stricter rules when instantiating Hashes and Frames instances.

Author: AddictedCS

src/SoundFingerprinting/Data/Frames.cs

@@ -12,23 +12,33 @@ public class Frames : IEnumerable<Frame>
         public Frames(IEnumerable<Frame> frames, string origin, int frameRate)
         {
             this.frames = frames.ToList();
+            if (this.frames.Any() && frameRate == 0)
+            {
+                throw new ArgumentException(nameof(frameRate));
+            }
+
             RelativeTo = DateTime.Now.AddSeconds((double) this.frames.Count / frameRate);
             FrameRate = frameRate;
             Origin = origin;
         }
 
         public Frames(IEnumerable<Frame> frames, string origin, int frameRate, DateTime relativeTo)
         {
-            RelativeTo = relativeTo;
             this.frames = frames.ToList();
+            if (this.frames.Any() && frameRate == 0)
+            {
+                throw new ArgumentException(nameof(frameRate));
+            }
+
+            RelativeTo = relativeTo;
             FrameRate = frameRate;
-            Origin = origin; 
+            Origin = origin;
         }
-        
+
         public DateTime RelativeTo { get; }
 
-        public double Duration => (double)frames.Count / FrameRate;
-        
+        public double Duration => (double) frames.Count / FrameRate;
+
         public string Origin { get; }
 
         public int FrameRate { get; }

src/SoundFingerprinting/Data/Hashes.cs

@@ -45,6 +45,11 @@ public Hashes(IEnumerable<HashedFingerprint> fingerprints, double durationInSeco
         public Hashes(IEnumerable<HashedFingerprint> fingerprints, double durationInSeconds, DateTime relativeTo, IEnumerable<string> origins, string streamId)
         {
             this.fingerprints = fingerprints.ToList();
+            if (this.fingerprints.Any() && durationInSeconds <= 0)
+            {
+                throw new ArgumentException(nameof(durationInSeconds));
+            }
+            
             DurationInSeconds = durationInSeconds;
             RelativeTo = relativeTo;
             Origins = origins;