Yes, getAttribute() ska funka bättre.
Problemet med egna attribut är att de kan pajja i framtiden. Vem vet, det kanske kommer en spec där just keyvalue används och då blir det kanske riktigt knasigt på dina sidor om du har otur. Så det kanske är bäst att vara på säkra sidan? HTML 5 föreslår att man prefixar sina egna attribut med"data-".