Assign the count of messages to a variable.

richsage / RMSPushNotificationsBundle

NOT MAINTAINED! ⛔️ Push notifications/messages for mobile devices. Supports iOS, Android (C2DM, GCM), Blackberry and Windows Mobile (toast only). A Symfony2 bundle.

MIT License

321 stars 152 forks source link

Assign the count of messages to a variable. #121

Closed TheFox closed 8 years ago

TheFox commented 8 years ago

count() in a for() loop condition is always a bad idea because the count() will always be executed, even if the $messages array does not change.

richsage commented 8 years ago

Thanks :)