Maybe I missed something but are you sure this code works?

`for (uint i = 0; i < guardians.length; i++) {

if (guardians[i]) {

count++;

}

}`

The mapping is made on addresses and you are using indexes here.