Refused Bequest ​

sourcemaking

Penjelasan Smell ​

Refused bequest arti harafiahnya adalah "menolak warisan". Dalam smell ini, sebuah class turunan tidak memakai seluruh method hasil extendsnya. Hal ini mengarah ke violasi LSP dan/atau ISP.

Dalam contoh kasus, terdapat class Stack.java yang melakukan extends terhadap java.util.Vector.

Di dalam class Stack, terdapat fungsi standar sebuah stack LIFO. yaitu: pop, push, dan peek.

public class Stack<E> extends Vector<E> {

	public void push(E data) {
		this.add(data);
	}

	public void pop() {
		this.removeElementAt(this.size()-1);
	}

	public E peek() {
		return this.elementAt(this.size()-1);
	}

	/*
	 * you cannot remove by index, use pop instead
	 */
	@Override
	public synchronized E remove(int index) {
		return null;
	}
}

public class Stack<E> extends Vector<E> {

	public void push(E data) {
		this.add(data);
	}

	public void pop() {
		this.removeElementAt(this.size()-1);
	}

	public E peek() {
		return this.elementAt(this.size()-1);
	}

	/*
	 * you cannot remove by index, use pop instead
	 */
	@Override
	public synchronized E remove(int index) {
		return null;
	}
}

Namun, terdapat satu masalah. java.util.Vector memiliki banyak fungsi yang memungkinkan class melakukan manipulasi data di dalam array (misalnya bisa hapus data menggunakan indeks). Tentu saja ini melanggar prinsip LIFO (Last-In First-Out).

Oleh karena itu, di class Stack, diakali dengan cara melakukan override pada masing-masing fungsi java.util.Vector yang tidak ingin digunakan, dan kita menghilangkan kinerjanya dengan cara menghapus pemanggilan super.

Sebelumnya seperti ini:

@Override
public synchronized E remove(int index) {
  return super.remove(index);
}

@Override
public synchronized E remove(int index) {
  return super.remove(index);
}

Return super kita ubah menjadi return null. Sehingga remove by index tidak terjadi.

/*
* you cannot remove by index, use pop instead
*/
@Override
public synchronized E remove(int index) {
  return null;
}

/*
* you cannot remove by index, use pop instead
*/
@Override
public synchronized E remove(int index) {
  return null;
}

Hal ini menjadi code smell Refused Bequest, karena class Stack menolak warisan dari class Vector.

Penyelesaian ​

Untuk contoh kasus ini, kita melakukan Replace Inheritance with Delegation.

Hubungan is-a tidak cocok untuk Stack dan Vector. Kita ubah hubungannya menjadi hubungan has-a.

class Stack menyimpan java.util.Vector sebagai field-nya. Pop, push, dan peek dilakukan dengan Vector ini.

public class Stack<E> {
	private Vector<E> vector = new Vector<>();

	public void push(E data) {
		vector.add(data);
	}

	public void pop() {
		vector.removeElementAt(vector.size()-1);
	}

	public E peek() {
		return vector.elementAt(vector.size()-1);
	}
}

public class Stack<E> {
	private Vector<E> vector = new Vector<>();

	public void push(E data) {
		vector.add(data);
	}

	public void pop() {
		vector.removeElementAt(vector.size()-1);
	}

	public E peek() {
		return vector.elementAt(vector.size()-1);
	}
}

java.util.Stack ​

Java sudah memiliki class Stack-nya sendiri di package java.util. Class Stack disini dibuat sendiri hanya untuk keperluan studi kasus. FYI, java.util.Stack adalah hasil inherit dari java.util.Vector. Anda dapat menghapus data di tengah-tengah Stack layaknya menggunakan Vector.