From: David ‘Bombe’ Roden <bombe@pterodactylus.net>
Date: Sun, 15 Mar 2015 12:41:31 +0000 (+0100)
Subject: Throw dedicated exception when adding a field with an empty name
X-Git-Tag: 0.9-rc1^2~3^2~1
X-Git-Url: https://git.pterodactylus.net/?a=commitdiff_plain;h=8ab419f60829a14b5222affcc759ff4a6995d637;p=Sone.git

Throw dedicated exception when adding a field with an empty name
---

diff --git a/src/main/java/net/pterodactylus/sone/data/Profile.java b/src/main/java/net/pterodactylus/sone/data/Profile.java
index 9c7da48..4970cf9 100644
--- a/src/main/java/net/pterodactylus/sone/data/Profile.java
+++ b/src/main/java/net/pterodactylus/sone/data/Profile.java
@@ -325,7 +325,9 @@ public class Profile implements Fingerprintable {
 	 */
 	public Field addField(String fieldName) throws IllegalArgumentException {
 		checkNotNull(fieldName, "fieldName must not be null");
-		checkArgument(fieldName.length() > 0, "fieldName must not be empty");
+		if (fieldName.length() == 0) {
+			throw new EmptyFieldName();
+		}
 		if (getFieldByName(fieldName) != null) {
 			throw new DuplicateField();
 		}
@@ -556,6 +558,13 @@ public class Profile implements Fingerprintable {
 	}
 
 	/**
+	 * Exception that signals the addition of a field with an empty name.
+	 *
+	 * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>
+	 */
+	public static class EmptyFieldName extends IllegalArgumentException { }
+
+	/**
 	 * Exception that signals the addition of a field that already exists.
 	 *
 	 * @author <a href="mailto:bombe@pterodactylus.net">David ‘Bombe’ Roden</a>