Знакомый говорит что неправильно. Правильно ли?

Question

[sniff1]

Правильно ли? Знакомый говорит что неправильно

Посмотрите на $tabl.

И дальше такой вывод

Ошибок нету и код нормально работает,просто спрашиваю.
Кодом: $tabl = array('test' => '12');
Вывод делаю такой:

$data = mysqli fetch assoc($query);
if( $data["$tabl[test]"] == '12' ) {
   echo "good!";
} else { echo "bad!"; }

Повторяюсь,ошибок нет,код работает.Просто спрашиваю,все ли верно?

Comments

[sim3x]

Код текстом в тег code
Ответ сервера текстом
Суть вопроса - текстом

[sniff1]

sim3x: Готово

[AlikDex]

вот это вот, что это такое?
$data["$tabl[test]"]
Чей светлый ум учил вас такому?

Answer 1

[Boris Köln]

Судя по тому, что вы сюда даже исходник не можете выложить в читаемом виде, то и весь ваш остальной код не лучше. Поддерживать его не возьмется ни один нормальный разработчик - гораздо быстрее будет написать в нуля правильно, чем разобраться в ваших if, include и пр.

Хотя на скрине скрыт кусок "select *", но могу поспорить, что через форму регистрации в Nick возможна SQL-инъекция. И через браузер посторонний человек одним запросом может слить себе все ваши данные, а вторым - удалить вашу БД.

Answer 2

[Иван Корюков]

Первое что бросается в глаза - это то как вы работаете с элементами массивов. Мало того, что вы ключи массива пишете не как строки, а как неинициализированные константы, так ещё и полученное значение в кавычки берёте. Зачем?
А вообще вопрос очень странный. Надо давать некоторое пояснение - что именно в этом коде, для вас или вашего знакомого, является неправильным.