Игорь_Vasinsky, ну можно наворотить со всеми проверками , вот только зачем, если ТС не может и в этих трех скромных строчках разобраться.

Не правильный подход, возврат должен быть null, но не false

	return isset($this->ar_data[$key]) ? $this->ar_data[$key] : null;

Посмотрите в сторону интерфейса ArrayAccess

public function setArData($key, $data, $remove=false) {

метод setXXX не должен возвращать результат

icedfox
Почитайте об идее setters, они созданы для того чтобы задавать значение но не считывать результат - если произошла ошибка, то выкидываете exception

Dezigo, да я знаю всю эту идею геттеров/сеттеров. Вы предложили в данном случае ей следовать, ну так подкрепите свои слова оптимальным кодом, который бы соответствовал вашим словам

icedfox
Нечего особо не меняется что я написал выше, это стандартное поведение.

Вы хотите оборотиться к свойству

public function getValue($key) {
    return isset($this->data[$key]) ? $this->data[$key] : null;
 // например в hashMap, значение возвращается null, если элемента нету., этого хорошо для phpunit, и значение может быть false
}

// запись
public function setValue($key, $value) {
    $this->data[$key] = $value;
 // нужно ли вам проверять на наличие уже записанного свойства? если да, то кидаем исключение
  return $this;
}

Понимаете, Вам нужно определиться что вы хотите достичь.
Уже давно всё придумано, берём пример любой и смотрим

Dezigo, а для проверки наличия свойства вызываем getValue ?
Итого получаем кучу строчек лишнего дублирующегося когда, только ради идеологии ?
Я вас понял.

icedfox
Если вам нужно проверить есть ли ключ то, containsKey($key);

Какой лишний код?
возвращать false, это ужас.

if ($class->containsKey($key)) {
   echo $class->getValue($key);
}