I bet you have set your Service Category field to be a lookup field in your table design. If so, think how much time you would have saved if you hadn't done that.
Another tip to save time in the long run is to not have spaces in your field names - a) it is an extra 3+ characters to type each time, b) you forget and get errors and spend time trying to resolve, c) even with the square brackets, access will for no rational reason find the word 'date' (or other reserved words) upsetting and generate an ambiguous error when you least expect it
Also, nested iifs can become difficult to manage so recommend you add the 'choose' and 'switch' functions to your repertoire
your code would then be simplified to
Code:
Next Service Due: Choose([Service Category],"No Further Service Required",DateAdd("m",6,[Service Date]),DateAdd("m",12,[Service Date]), DateAdd("m",24,[Service Date]), DateAdd("m",42,[Service Date]),DateAdd("m",72,[Service Date]))