Please geen errors moren, please

Author: Standaard-boos

ViewModel.Test/ViewModel_Decisiontree_Method_Interest.cs

@@ -106,5 +106,12 @@ public void CheckOfInterestAreClose_False()
             var result = Helpers.Decisiontree.Methodes.InterestCheck.CheckInterest(list1, list2);
             Assert.False(result);
         }
+        [Test]
+        public void CheckOfInterstCanBeNull()
+        {
+            List<Interest> list1 = new List<Interest>();
+            var result = Helpers.Decisiontree.Methodes.InterestCheck.CheckInterest(null, list1);
+            Assert.False(result);
+        }
     }
 }

ViewModel.Test/ViewModel_Decisiontree_Method_MoralCheck.cs

@@ -105,5 +105,12 @@ public void CheckOfMoralAreClose_False()
             var result = Helpers.Decisiontree.Methodes.MoralCheck.CheckMoral(list1, list2);
             Assert.False(result);
         }
+        [Test]
+        public void CheckOfMoralCanBeNull()
+        {
+            List<Moral> list2 = new List<Moral>();
+            var result = Helpers.Decisiontree.Methodes.MoralCheck.CheckMoral(null, list2);
+            Assert.False(result);
+        }
     }
 }

ViewModel.Test/ViewModel_Decisiontree_Method_SameSchool.cs

@@ -32,5 +32,13 @@ public void CheckOfSchoolAreSame_false()
             var result = Helpers.Decisiontree.Methodes.SameSchoolCheck.CheckSameSchool(school1, school2);
             Assert.False(result);
         }
+
+        [Test]
+        public void CheckOfSchoolCanBeNull()
+        {
+            var school1 = new School(1, "Windesheim", "Zwolle", "Verf");
+            var result = Helpers.Decisiontree.Methodes.SameSchoolCheck.CheckSameSchool(null, school1);
+            Assert.False(result);
+        }
     }
 }

ViewModel.Test/ViewModel_Decisiontree_method_CheckMoralPercentage.cs

@@ -46,5 +46,13 @@ public void CheckOfMoralAreClose_false()
             Assert.False(result);
         }
 
+        [Test]
+        public void CheckOfMoralCanBeNull()
+        {
+            List<Moral> list1 = new List<Moral>();
+            var result = Helpers.Decisiontree.Methodes.MoralPercentageCheck.CheckMoralPercentage(null, list1);
+            Assert.False(result);
+        }
+
     }
 }

ViewModel.Test/ViewModel_Method_Decisiontree_SameAge.cs

@@ -35,5 +35,14 @@ public void CheckOfAgeAreSame_false()
             var result = Helpers.Decisiontree.Methodes.SameAgeCheck.CheckSameAge(pf1, pf2);
             Assert.False(result);
         }
+
+        [Test]
+        public void CheckOfAgeCanBeNull()
+        {
+            var pf2 = new Profile() { Age = "12 jaar" };
+            var pf1 = new Profile();
+            var result = Helpers.Decisiontree.Methodes.SameAgeCheck.CheckSameAge(pf1, pf2);
+            Assert.False(result);
+        }
     }
 }

ViewModel.Test/ViewModel_Method_Decisiontree_SameCity.cs

@@ -35,5 +35,14 @@ public void CheckOfCityAreSame_false()
             var result = Helpers.Decisiontree.Methodes.SameCityCheck.CheckSameCity(pf1, pf2);
             Assert.False(result);
         }
+
+        [Test]
+        public void CheckOfCityCanBeNull()
+        {
+            var pf2 = new Profile() { City = "Rotterdam" };
+            var pf1 = new Profile();
+            var result = Helpers.Decisiontree.Methodes.SameCityCheck.CheckSameCity(pf1, pf2);
+            Assert.False(result);
+        }
     }
 }

ViewModel/Helpers/Decisiontree/Methodes/SameAgeCheck.cs

@@ -34,9 +34,9 @@ public static bool CheckSameAge(Profile pf, Profile liupf)
         /// <returns>true or false</returns>
         private static bool CheckAge(Profile pf, Profile liupf)
         {
-            if (pf.Age == null || pf.Age == "")
+            if (String.IsNullOrEmpty(pf.Age))
             {
-                pf.Age = "00";
+                return false;
             }
 
             try

ViewModel/Helpers/Decisiontree/Methodes/SameCityCheck.cs

@@ -15,8 +15,8 @@ public static class SameCityCheck
         /// <returns>true if they live is the same city and add 5 points to overall score, false if not in the same city</returns>
         public static bool CheckSameCity(Profile pf, Profile lipf)
         {
-
-            if (pf.City == null || pf.City == "")
+            
+            if (String.IsNullOrEmpty(pf.City))
             {
                 return false;
             }