Ибрахим, Ну начнем по порядку.
1) ereg - данная функция уже не актуальна. *_replace() пришла ей на замену.
2) Нет проверки на существование логина уже в базе.
3) Код вырезан откуда-то, причем код старый и никому не нужный.