Talk:Aho–Corasick algorithm: Difference between revisions

Content deleted Content added
Java vs. C++: new section
Line 36:
 
There is a reference to a C++ implementation in the references. Someone added a reference to a Java implementation, but it was reverted. Is this a Java vs. C+ thing? Why one and not the other? I don't actually know about either of them. [[User:Gah4|Gah4]] ([[User talk:Gah4|talk]]) 15:33, 21 September 2016 (UTC)
 
: No, it is not a language thing. This article is about an algorithm; it is agnostic to programming issues. Generally, the external links section is for information that would be included in the article later or may be too detailed. External links that include readable content are better than links to just code. A link to an article about the algorithm (e.g., the lecture slides) would be much better than a code listing. It's poor practice to subject encyclopedia readers to digesting code, but we do it from time to time. Some code implementations also have substantial documentation in plain language.
: A week ago an IP inserted an EL to a GitHub-hosted Java implementation.[https://en.wikipedia.org/w/index.php?title=Aho%E2%80%93Corasick_algorithm&diff=739736735&oldid=727616397] I followed the link and found a project that appeared to have been posted a few days previously. That suggests some programmer has just done an implementation and posted it. That's the same as a blog; there's no independent review or assessment; it is not a reliable source. It's documentation is just a link to the 1975 paper that is already cited in this article. Then a link is placed on WP to advertise the implementation. Consequently, I reverted the addition with the edit comment "recent project? advert?"[https://en.wikipedia.org/w/index.php?title=Aho%E2%80%93Corasick_algorithm&diff=740451885&oldid=739736735]
: Although I recognized there was a GitHub C++ implementation, I did not research that link at the time and just left it (the WP article does not have an implementation). I just looked at that post now, and its documentation is a pointer to this WP article. I'm tempted to delete the C++ implementation link.
: [[User:Glrx|Glrx]] ([[User talk:Glrx|talk]]) 17:51, 21 September 2016 (UTC)