Primitive Obsession

Penjelasan Smell

Smell ini terjadi ketika ada field atau parameter yang memakai tipe data primitif, padahal behavior-nya lebih daripada tipe data yang ia gunakan.

Hal ini tidak terjadi hanya di tipe data primitif int, float, dan sebagainya saja. Hal ini juga bisa terjadi di tipe data lain di level abstraksi yang lebih tinggi. Misalnya String. Intinya adalah, bila programmer menyimpan data dengan sebuah tipe data, padahal behavior dari data itu lebih dari tipe data yang ia gunakan, berarti sebenarnya harus diganti atau dibuatkan class baru.

Perhatikan contoh Mahasiswa.java. Terdapat field name menggunakan String dan tanggal lahir yang dipecah-pecah menjadi integer. Padahal name memiliki validasi sesuai dengan regex dalam code, dan tanggal lahir memiliki validasi tanggal harus valid.

java

public class Mahasiswa {
	private String name;
	private int dayOfBirth;
	private int monthOfBirth;
	private int yearOfBirth;

	//reguler | global
	private String type;

	public Mahasiswa(String name, String type, int dayOfBirth, int monthOfBirth, int yearOfBirth) {
		if(!isValidName(name)) {
			throw new IllegalArgumentException("name is not valid");
		}

		type = type.toLowerCase();
		if(!type.equals("reguler") && !type.equals("global")) {
			throw new IllegalArgumentException("type is not valid");
		}

		if(!isValidDate(dayOfBirth, monthOfBirth, yearOfBirth)) {
			throw new IllegalArgumentException("dob is not valid");
		}

		this.name = name;
		this.type = type;
		this.dayOfBirth = dayOfBirth;
		this.monthOfBirth = monthOfBirth;
		this.yearOfBirth = yearOfBirth;
	}

	private boolean isValidName(String name) {
		String regex = "^([A-Z\\'][a-z\\']*((\\s)))*[A-Z\\'][a-z\\']*$";
		Pattern pattern = Pattern.compile(regex);
		return pattern.matcher(name).matches();
	}

	private boolean isValidDate(int dayOfBirth, int monthOfBirth, int yearOfBirth) {
		Calendar cal = Calendar.getInstance();
		cal.setLenient(false);
		cal.set(yearOfBirth, monthOfBirth-1, dayOfBirth);
		try {
			cal.getTime();
		} catch (Exception e) {
			return false;
		}
		return true;
	}

	public String getType() {
		return type;
	}

	public String getName() {
		return name;
	}
}

public class Mahasiswa {
	private String name;
	private int dayOfBirth;
	private int monthOfBirth;
	private int yearOfBirth;

	//reguler | global
	private String type;

	public Mahasiswa(String name, String type, int dayOfBirth, int monthOfBirth, int yearOfBirth) {
		if(!isValidName(name)) {
			throw new IllegalArgumentException("name is not valid");
		}

		type = type.toLowerCase();
		if(!type.equals("reguler") && !type.equals("global")) {
			throw new IllegalArgumentException("type is not valid");
		}

		if(!isValidDate(dayOfBirth, monthOfBirth, yearOfBirth)) {
			throw new IllegalArgumentException("dob is not valid");
		}

		this.name = name;
		this.type = type;
		this.dayOfBirth = dayOfBirth;
		this.monthOfBirth = monthOfBirth;
		this.yearOfBirth = yearOfBirth;
	}

	private boolean isValidName(String name) {
		String regex = "^([A-Z\\'][a-z\\']*((\\s)))*[A-Z\\'][a-z\\']*$";
		Pattern pattern = Pattern.compile(regex);
		return pattern.matcher(name).matches();
	}

	private boolean isValidDate(int dayOfBirth, int monthOfBirth, int yearOfBirth) {
		Calendar cal = Calendar.getInstance();
		cal.setLenient(false);
		cal.set(yearOfBirth, monthOfBirth-1, dayOfBirth);
		try {
			cal.getTime();
		} catch (Exception e) {
			return false;
		}
		return true;
	}

	public String getType() {
		return type;
	}

	public String getName() {
		return name;
	}
}

Selain itu, terdapat juga field type menggunakan String. Padahal value type hanya boleh Reguler atau Global. Selain itu, type juga digunakan pada MahasiswaGreeter yang di dalam sini terdapat smell switch statement.

Penyelesaian

Dilakukan Extract Class pada name, dibuatkan class baru bernama FullName. Kemudian replace data value with object pada field name di class Mahasiswa.

Untuk integer dayOfBirth, monthOfBirth, dan yearOfBirth, kita ganti menggunakan class java.util.Date bawaan dari Java.

Untuk logic Greeter, kita gunakan replace type code with state/strategy.

Primitive Obsession ​

Penjelasan Smell ​

Penyelesaian ​

Primitive Obsession

Penjelasan Smell

Penyelesaian