Skip to main content

New answers tagged

2 votes

CSV handling class

It's not a class I would say the main problem here is CSV not really being a class. But rather a collection of functions just "namespaced" under the same class. One can even take write_csv(),...
Your Common Sense's user avatar
6 votes

CSV handling class

design of Public API I don't understand this: ...
J_H's user avatar
  • 42.2k
5 votes

CSV handling class

Bug The ValidateFilepath function is inadequate. If the file name does not contain a dot (has no extension), your code would fail with an "Undefined array key&...
Kate's user avatar
  • 8,313
4 votes

CSV handling class

While it isn't required, it would be good to follow a style guide. One popular set of style guides is the PHP Standards Recommendations - specifically PSR-1 and PSR-12. They have numerous ...
Sᴀᴍ Onᴇᴌᴀ's user avatar
4 votes

Laravel Wallet Service Backend

Firstly it would be a great idea to put all of your RESTful (store, update, show, index, destroy/delete) methods within a Service class, and name them, by convention, store, update, show, index, or ...
Arthur's user avatar
  • 41
3 votes

Drop-down list with 14 options of RSS feed dates

Review Bearing in mind that this code was posted nearly 11 years ago, perhaps much has been learned about PHP, RSS feeds and similar topics. Is the code still used today? As Greg alluded to, the ...
Sᴀᴍ Onᴇᴌᴀ's user avatar

Top 50 recent answers are included