Since your already using scope for your variables (private/protected/public) you might as well use the constructor method though that's more out of preference and it just looks cleaner to me. Also preg_match is preferred over eregi