Write short units of code

• Limit the length of code units to 15 lines of code.
• Do this by not writing units that are longer than 15 lines of code in the first place, or by splitting long units into multiple smaller units until each unit has at most 15 lines of code.
• This improves maintainability because small units are easy to understand, easy to test, and easy to reuse. ##

Tijdens de zoektocht naar een juiste implementatie voor het berekenen van de horoscopen kwam ik verschillende opties tegen. Een die een voorbeeld gaf van een te lange code is een stuk code die ik hier vond:

        public class Example {
            public static void main (String[] args) {
               int day = 7;
               String month = "August";
               String sign="";
               if (month == "January") {
                  if (day < 20)
                     sign = "Capricorn";
                  else
                     sign = "Aquarius";
               }
               else if (month == "February") {
                  if (day < 19)
                     sign = "Aquarius";
                  else
                     sign = "Pisces";
               }
               else if(month == "March") {
                  if (day < 21)
                     sign = "Pisces";
                  else
                     sign = "Aries";
               }
               else if (month == "April") {
                  if (day < 20)
                     sign = "Aries";
                  else
                     sign = "Taurus";
               }
               else if (month == "May") {
                  if (day < 21)
                     sign = "Taurus";
                  else
                     sign = "Gemini";
               }
               else if( month == "June") {
                  if (day < 21)
                     sign = "Gemini";
                  else
                     sign = "Cancer";
               }
               else if (month == "July") {
                  if (day < 23)
                     sign = "Cancer";
                  else
                     sign = "Leo";
               }
               else if( month == "August") {
                  if (day < 23)
                     sign = "Leo";
                  else
                     sign = "Virgo";
               }
               else if (month == "September") {
                  if (day < 23)
                     sign = "Virgo";
                  else
                     sign = "Libra";
               }
               else if (month == "October") {
                  if (day < 23)
                     sign = "Libra";
                  else
                     sign = "Scorpio";
               }
               else if (month == "November") {
                  if (day < 22)
                     sign = "scorpio";
                  else
                     sign = "Sagittarius";
               }
               else if (month == "December") {
                  if (day < 22)
                     sign = "Sagittarius";
                  else
                     sign ="Capricorn";
               }
               System.out.println("The astrological sign for " + day + " " + month + " is " + sign );
            }
         }
        

Uiteindelijk hebben we dit in het project anders weten op te lossen, door middel van een ENUM te maken. Hierdoor worden er niet allerlei “if-else”-statements gebruikt binnen de code. Ook maakt het de code leesbaarder.