/prog/ - necro weed posting considered harmful

Name: Anonymous 2010-06-02 13:12

on the other hand /prog/ actually was good once

Name: Anonymous 2010-06-02 13:27

This is what greasemonkey is for Don't see their posts!

Name: Anonymous 2010-06-02 13:29

Dear 2012 self,

Be reminded that /prog/ was never good.
Also, I'm really sorry about the election results.

Name: Anonymous 2010-06-02 14:48

Hey /prog/, I tried whipping up a quick greasemonkey script to hide the necrothreads, but I can't see why this doesn't remove all threads not created this year. Any chance you can tell me where I've went wrong?

var dateregex = /^2010/;

var threads = document.getElementsByClassName('thread');

for (i=0; i<threads.length; i++) {

  var thread = threads[i];

  var opdate =  thread.getElementsByClassName('posterdate')[0].firstChild.nodeValue;

  if (!dateregex.test(opdate)) {

    var parent = thread.parentNode;

    var grandparent = parent.parentNode;

    grandparent.removeChild(parent);

  }

}

Name: Anonymous 2010-06-02 14:54

>>4
stop tryig to reinvent the wheel

Name: Anonymous 2010-06-02 16:07

>>5
Yeah, thanks bro. I am well aware that we have an anti-necrobump script already, I do remember last year. I was writing the script for a reason, in this instance to try and reacquaint myself with Javascript and the DOM. But assuming that I hadn't heard of it, and was new here, it's not like we've advertised the anti-necro script much this year, as there has been less need for it. How would a newbie know?

For those that care, the problem was retardedly simple. I forgot that when the child was removed from the DOM, it was also removed from the threads array. And therefore, the call to threads.length was different every time. The solution is below

var dateregex = /^2010/;

var threads = document.getElementsByClassName('thread');

var len = threads.length

for (i=0; i < len;) {

  var thread = threads[i];

  var opdate =  thread.getElementsByClassName('posterdate')[0].firstChild.nodeValue;

  if (!dateregex.test(opdate)) {

    var parent = thread.parentNode;

    var grandparent = parent.parentNode;

    grandparent.removeChild(parent);

  } else {

  i++;

  }

}

Name: Anonymous 2010-06-02 16:18

>>6
If you don't also decrement len you'll run out of bounds.

How unfortunate is that Javascript's for-each operator is so retarded.

Name: air max shoes 2010-07-23 11:00

Name: Anonymous 2011-02-03 6:25

Don't change these.

Name:		Email:

Entire Thread Thread List

necro weed posting considered harmful

1 Name: Anonymous 2010-06-02 13:12

2 Name: Anonymous 2010-06-02 13:27

3 Name: Anonymous 2010-06-02 13:29

4 Name: Anonymous 2010-06-02 14:48

5 Name: Anonymous 2010-06-02 14:54

6 Name: Anonymous 2010-06-02 16:07

7 Name: Anonymous 2010-06-02 16:18

8 Name: air max shoes 2010-07-23 11:00

9 Name: Anonymous 2011-02-03 6:25