rorybot warning analytics test script [WIP]

[jeremyhansonfinger]

At the moment this script asks you what file type you want to analyze, recursively runs Rorybot on every file of that type in your current directory, and outputs the frequency of each warning to a CSV file that you can manipulate further with Google Sheets.

Closes #4

[jules2689]

input_file.read.scan(warning_message).map { |w| frequency[w] + 1 }

[jeremyhansonfinger]

Cool!

[jeremyhansonfinger]

I can't get this to add to frequency (even taking out read). If I try to print frequency it gives me {}

If I put it back to

input_file.scan(warning_message) do |word|
  frequency[word] = frequency[word] + 1

it does work, however.

[jules2689]

Ah, I think I misread this. That said, frequency[word] = frequency[word] + 1 should work as frequency[word] += 1

[jules2689]

This is a pretty nasty regex, what is it intending to match? I dont really wanna take the time to figure it out lol

[jeremyhansonfinger]

Ha, yeah, there's got to be a better way. I'm trying to match this:

http://rubular.com/r/ILo7BWe8ux

[jules2689]

^.*warning (.*)$

• ^ denotes the beginning of the line.
• .* matches everything up to warning
• warning matches warning
• matches the 2 spaces (could also be\s+)
• (.*)$ creates a capture group for the rest of the line because $ means "end of line"

[jeremyhansonfinger]

Got it. Waaaaaay cleaner. I don't know what I was thinking.

[jules2689]

Looks like my comments got suppressed. Can you go through all the outdated ones and comment on them?

[jules2689]

basically all these:

[jeremyhansonfinger]

Will do!
On May 18, 2016 11:29 AM, "Julian Nadeau" [email protected] wrote:

basically all these:
[image: image]
https://cloud.githubusercontent.com/assets/3074765/15364781/cccee5b6-1ceb-11e6-83b3-2b05af03d9d3.png

—
You are receiving this because you authored the thread.
Reply to this email directly or view it on GitHub
#17 (comment)

[jeremyhansonfinger]

@jules2689 I'm running into an error now on my most recent commit, on line 26:

input_file.scan(warning_message).map { |w| frequency[w] + 1 }

Any ideas?

[jeremyhansonfinger]

Regarding my error above, I did change your suggestion to

- input_file.read.scan(warning_message).map { |w| frequency[w] + 1 }
+ input_file.scan(warning_message).map { |w| frequency[w] + 1 }

because of the suggestion to change File.open to File.read, but leaving read in there resulted in a an undefined method error.

[jules2689]

Okay, so frequency[w] would be initialized as nil. Apparently when passing a value to Hash.new it does this:

Hashes have a default value that is returned when accessing keys that do not exist in the hash. If no default is set nil is used. You can set the default value by sending it as an argument to ::new:

TIL. Change the frequency = {} above back to frequency = Hash.new(0)

[jeremyhansonfinger]

No longer required.