Конфиги для модулей CMS

В своей CMS решил сделать так, чтобы модули легко настраивались путём легкой смены конфига конкретного модуля. Решил данные конфига хранить в серилизованном виде (выбирал м/у XML, json и сериализацией). Накатал маленький класс для работы с конфигом.

PHP:

1. <?php
2. class config {
3.     /* Пример работы
4.    
5.     $cnf = new config('config.cnf');                // Загрузка конфигурации из файла
6.    
7.     $cnf->addValue('param1',3,'');                  // Добавление параметра
8.     $cnf->setValue('param1',123,'комментарий');      // Изменение параметра
9.     $cnf->getValue('param1');                       // Получение параметра
10.     $cnf->delValue('param1');                       // Удаление параметра
11.    
12.     $cnf->saveConfig('config.cnf');                 // Сохранение конфигурации в файл
13.    
14.     */
15.    
16.     public $values = array();   // Массив конфигурации
17.    
18.     // Конструктор класса
19.     public function __construct($val = '') {
20.         if (is_array($val)) {
21.             $this->values = $val;
22.         } elseif (is_file($val)) {
23.             $this->values = unserialize(file_get_contents($val));
24.         }
25.     }
26.    
27.     // Сохранение конфига в файл
28.     public function saveConfig($file) {
29.         if (!file_exists($file) || is_writeable($file)) {
30.             @file_put_contents($file, serialize($this->values));
31.         } else
32.             return false;
33.     }
34.    
35.     // Добавление нового параметра
36.     public function addValue($name, $value = '', $comment = '') {
37.         if (isset($name) && !empty($name))
38.             $this->values[] = array('name' => $name, 'value' => $value, 'comment' => $comment);
39.     }
40.    
41.     // Удаление параметра по имени
42.     public function delValue($name) {
43.         foreach ($this->values as $k=>$v) {
44.             if ($this->values[$k]['name'] == $name)
45.                 unset($this->values[$k]);
46.         }
47.     }
48.    
49.     // Достаём значение параметра по имени
50.     public function getValue($name) {
51.         foreach ($this->values as $arr) {
52.             if ($arr['name'] == $name)
53.                 return $arr['value'];
54.         }
55.     }
56.    
57.     // Устанавливаем значение параметра по имени
58.     public function setValue($name, $value, $comment='') {
59.         foreach ($this->values as $k=>$v) {
60.             if ($this->values[$k]['name'] == $name) {
61.                 $this->values[$k]['value']      = $value;
62.                 $this->values[$k]['comment']    = $comment;
63.             }
64.         }
65.     }
66. }

в index.php перед инклюдингом модуля загружается его конфиг, затем в модуле соответственно юзаются параметры.

Собственно вопрос - нормальный подход или пойти убиться ап стену?
Второй вопрос - пнитев направлении правильного оформления кода.