Знакомый говорит что неправильно. Правильно ли?
Судя по тому, что вы сюда даже исходник не можете выложить в читаемом виде, то и весь ваш остальной код не лучше. Поддерживать его не возьмется ни один нормальный разработчик - гораздо быстрее будет написать в нуля правильно, чем разобраться в ваших if, include и пр.
Хотя на скрине скрыт кусок "select *", но могу поспорить, что через форму регистрации в Nick возможна SQL-инъекция. И через браузер посторонний человек одним запросом может слить себе все ваши данные, а вторым - удалить вашу БД.
Первое что бросается в глаза - это то как вы работаете с элементами массивов. Мало того, что вы ключи массива пишете не как строки, а как неинициализированные константы, так ещё и полученное значение в кавычки берёте. Зачем?
А вообще вопрос очень странный. Надо давать некоторое пояснение - что именно в этом коде, для вас или вашего знакомого, является неправильным.