>Based on price is fine -- when splits happen just adjust the weight by the split factor It's not fine since companies have varying counts of shares, nevermind

>Based on price is fine -- when splits happen just adjust the weight by the split factor It's not fine since companies have varying counts of shares, nevermind the stock splits. The proper solution is getSharePrice() * getShareCount(). Which is what most other indeces do.