-
-
Notifications
You must be signed in to change notification settings - Fork 240
Replace use of 'join' function with 'implode' #480
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
If a student googles 'join' to learn more they'll just see that it's an alias of implode, and then have to read about implode. https://www.php.net/manual/en/function.join.php Better to just use implode so there's one less step.
FYI I wrote my own PHP tutorial a couple of years ago. I don't know how much interest your students have in PHP though. https://a-moderately-short-php-tutorial.com/
I can also recommend Jon Duckett's book, which is came out recently and covers modern PHP.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🎉
Thanks @bdsl
That was quick! Thanks @KimberleyCook .
If a student googles 'join' to learn more they'll just see that it's an alias of implode, and then have to read about implode.
https://www.php.net/manual/en/function.join.php
Better to just use implode so there's one less step.