Работать будет. ;-) сейчас же работает.
Но можно не уследить в процессе правок кода и злоумышленник сможет выполнять у вас на сайте произвольный код. Потому и считается использование eval() плохим способом.
Да и логика теряется, сложно разбираться в том, что код делает.
И если сейчас вы помните, то например лет через 5 уже будет проблемно понят, что и зачем.
Кто мешает сохранить результаты в ещё один массив, вместо формирования "хитрой" строки,
а затем уже сравнивать его внутри цикла исходного массива. B-)