VBA Rubberducking

Duga

12:01 AM

RELOAD!

[Cardshifter/cardshifter.github.io] 4 commits. 1 closed issue. 3 issue comments.

[retailcoder/Rubberduck] 7 issue comments.

[rolfl/MicroBench] 5 commits.

[Vogel612/BloggingPlaform] 1 commit.

[Vogel612/BloggingPlatform] 2 commits.

RubberDuck

12:43 AM

So... Uh... F this guy.

.@Rubberduck203 While #VBA is good for macros, it's pretty much useless for anything else. The .NET world makes things so much better ;-)

Tweeted by bvukas on April 3, 2015 at 12:34 AM in reply to Rubberduck203

sᴉɔuɐɹɥԀ

@RubberDuck He makes it sound like there are no contexts where you need VBA, or where VBA is the better tool. Dick.

(and of course contexts where you're just stuck with VBA)

RubberDuck

Right?

Ya know, because VSTO really caught on, didn't it?

Mat's Mug

2:11 AM

@Rubberduck203 @bvukas AFAIK #VBA is still the only way the average office guy can write a CRUD app without bothering IT with an IDE install

Tweeted by rubberduckvba on April 3, 2015 at 2:05 AM in reply to Rubberduck203

^^ exactly 140 characters

Stack Exchange

twitter.com/rubberduckvba/status/583812482818568192

Mat's Mug

What do you want me to do? LEAVE? Then they'll keep being wrong!

sᴉɔuɐɹɥԀ

That comic never gets old :)

@RubberDuck I told you there's someone, somewhere, who writes good PHP. I think I found one! (they also came to CR from SO, thanks @Duga!)

1

Q: For a login portal, what security measures are needed to prevent unauthorized access?

I'm designing a login portal that has one angularjs page that displays/processes data queried from a database. I'm relying on a few php pages (a loginpage.php [verifies credentials/loads session variables], a check.php [page that is included in all php pages, verifies session variables are 'okay'...

Mat's Mug

hood php? lol

sᴉɔuɐɹɥԀ

Huh? ;)

Mat's Mug

2:27 AM

$_YELLCASE locals?

sᴉɔuɐɹɥԀ

Hm. Pretty minor, but good catch. Guessing session variables are not global eh?

Actually, I checked the manual and that's standard case

Creating New Session
==========================
<?php
session_start();
/*session is started if you don't write this line can't use $_Session  global variable*/
$_SESSION["newsession"]=$value;
?>

It's a bit sad that most of the comments are actually very appropriate because PHP is so weird

Mat's Mug

> You know nothing, ~~John Snow~~ Mug

2

is your Game of Thrones all up to date?

14 days until season 5 starts on HBO

...and I'll have to wait until it's out on blu-ray

sᴉɔuɐɹɥԀ

2:45 AM

Haven't really watched that show; watched a season or so. Pretty ladies but I didn't find it "my thing"

Been watching Top Gear (UK), for some reason they don't have anything before season 17 on NetFlix, but it's super funny and I love the cars!

You ever have an hour to burn, go to season 17 and watch the last episode, where they go to India to "promote Britain" it's hilarious!

Mat's Mug

I'm getting dragged into Outlander, seen that one?

sᴉɔuɐɹɥԀ

3:06 AM

Sure haven't

Duga

4:24 AM

[retailcoder/Rubberduck] retailcoder added label bug to issue #358: Renaming "ThisWorkbook" isn't done right.

[retailcoder/Rubberduck] retailcoder opened issue #358: Renaming "ThisWorkbook" isn't done right.

> Per this Stack Overflow answer I stumbled on (thanks vba4all!), the way we're renaming document components isn't done right; we need to set the [_CodeName] hidden property, not the Name.

[retailcoder/Rubberduck] retailcoder added label refactorings to issue #358: Renaming "ThisWorkbook" isn't done right.

Duga

4:40 AM

[retailcoder/Rubberduck] retailcoder commented on issue #357: "Procedure not used" is a problem for native interface implementations

> Well, there's another problem. There's no easy way of knowing the name of the type of a Document component - telling a Workbook from a Worksheet is much harder than it appears to be... but then in our case we also need to tell a Workbook (Excel) from a Document (Word) and a Report (Access), a Presentation (PowerPoint) or a Session (Outlook)... and who knows what else. I think we're going to have to assume that no one in their right minds would ever create a method...

starting with Workbook_* in a worksheet or report code module, so we can "reserve" a number of known document component method prefixes and call it a day (e.g. Workbook_, Worksheet_, Document_, Report_, Form_, ...any others?).

sᴉɔuɐɹɥԀ

5:52 AM

People write VBA for... PowerPoint!? :o

RubberDuck

11:37 AM

@Mat'sMug I'm not up to date. I'm a season behind, so no spoilers please. =)

Duga

[retailcoder/Rubberduck] ckuhn203 commented on issue #357: "Procedure not used" is a problem for native interface implementations

> Access is actually pretty easy. All form component names are in the form of Form_[formName]. Likewise with reports. If have to confirm it, but I'm pretty sure trying to rename them outside of that default format actually breaks Access.

Mat's Mug

Looks like the host app is going to start needing more visibility throughout RD

@RubberDuck so you know nothing ~~John Snow~~ ducky!

RubberDuck

12:09 PM

Aye. I know nothing @Mat'sMug.

@Mat'sMug have you seen this? You need to check this out. fcell.io

Mat's Mug

@RubberDuck Interesting. Expensive though.

RubberDuck

Very, but it's pretty cool.

Giving away RD may have been a dumb decision. lol

I mean, look at how much they charged for that thing!

Mat's Mug

...

RubberDuck

Yeah. Yeah. I know. Just saying.

Mat's Mug

12:24 PM

I wonder if R# works in their IDE

RubberDuck

lol

Probably not.

It looks like a pretty basic text editor.

Mat's Mug

VBA isn't dead yet!

RubberDuck

12 hours ago, by RubberDuck

Ya know, because VSTO really caught on, didn't it?

Mat's Mug

12:45 PM

If the calculator wins, I'm making an ANTLR grammar for the parsing of user input! :)

RubberDuck

I was thinking the same thing.

I think I might actually do it anyway just to learn a bit about ANTLR.

Mat's Mug

1:15 PM

@RubberDuck Good idea!

Duga

[retailcoder/Rubberduck] mwolfe02 opened issue #359: Code Inspection Idea: Identify "Magic Numbers"

> Good programming practice is to assign numbers to constants instead of using them directly in code. To support this I would recommend the following new code inspection:

Name: Use of "Magic Number"
Type: MaintainabilityAndReadabilityIssues
Severity: Suggestion

Algorithm:

0. Identify all number tokens (e.g., `3`, `1.5`, `255`, `&hFF`)
0. Ignore tokens that are the target of a `Const` statement (e.g., `Const DaysInWeek = 7`)
0. Ignore `0` and `1` tokens

Mat's Mug

Ooh nice! More false positives!

Lol kidding. Good idea :)

Duga

[retailcoder/Rubberduck] ckuhn203 commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> I'd like to amend the last one.

3. Ignore `0` and `1` tokens where they occur in a loop declaration.

For i = 0 To Blah + 1
' some code
Next i

Is this your intent @mwolfe02 , or do you think it's better to exclude all occurrences of `0` and `1`?

[retailcoder/Rubberduck] ckuhn203 added label code-inspections to issue #359: Code Inspection Idea: Identify "Magic Numbers"

Mat's Mug

That can get very complicated very fast :)

RubberDuck

I'm sure it can..

Mat's Mug

1:22 PM

The number literal would be in a ValueStmt rule.... y'know, that recursive one?

Not dismissing the suggestion, but just saying... refactor/rename is simpler!

RubberDuck

Eh, well. It's on the list. There's a lot on the list

Mat's Mug

Yeah. #TheList

Duga

[retailcoder/Rubberduck] mwolfe02 commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> I think it's better to include all occurrences of 0 and 1. Consider the following:

If DCount("*", "SomeTable") > 0 Then

FName = Dir("C:\*")
Do While Len(FName) > 0
FName = Dir()
Loop

IsPlural = (MyCollection.Count > 1)

Mat's Mug

Might not be all that complicated after all

Duga

[retailcoder/Rubberduck] ckuhn203 commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> Point taken. I was just refrerenced a section I recently read in Code Complete and it would appear Steve McConnel is in agreement with you. That's good enough for me.

Duga

2:19 PM

[retailcoder/Rubberduck] retailcoder commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> This might be simpler than I initially thought it would be, although it implies walking the tree and listening for ValueStmt rules; if the value statement does not have a ConstSubStmtContext parent then the magic numbers are any number literal greater than 1. Right?

RubberDuck

@Duga ummmmm...... I think so.

But @Mat'sMug would know better than me.

Oh.... wait....

Duga

[retailcoder/Rubberduck] retailcoder commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> The main problem is that ValueStmt is all over the grammar, and all over the code - this inspection may cause performance issues whenever its severity is set greater than DoNotShow.

Mat's Mug

And it's a recursive rule

@RubberDuck Haha

Duga

3:20 PM

[retailcoder/Rubberduck] mwolfe02 commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> @retailcoder Actually, any number literal greater than 1 or less than 0.

[retailcoder/Rubberduck] ckuhn203 commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> :+1: Good catch.

Duga

3:50 PM

[retailcoder/Rubberduck] Zomis commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> Related: Is 100 a magic number?

2

sᴉɔuɐɹɥԀ

4:28 PM

Today I learned about Turing completeness.

    Yes, that Tom. Did you know Tom likes the number 256?

    Yes, I mean that Discord. Discord did letters of
    "++++++++[>++++[>++>+++>+++>+<<<<-]>+>->+>>+[<]<-]>>.>>---.+++++++..+++.>.<<-.>.+++.------.--------.>+.>++.+++." .
    Yes, that Sweetie Belle. Today I found a book named Sweetie Belle, because she is the memory.

    Did you know Pinkie Pie likes the number one, because she is the memory pointer?
    Did you know Rarity likes Pinkie Pie, because she is the instruction pointer?

2

I think FiM++ is even funnier than LOLCODE :D

RubberDuck

4:45 PM

@sᴉɔuɐɹɥԀ That may be the most disturbing thing I've ever laid my eyes on. Congrats. You've won the internet today.

sᴉɔuɐɹɥԀ

What, a Brainfuck interpreter written in FiM++? ;-)

RubberDuck

@sᴉɔuɐɹɥԀ Troll level: Guru

2

sᴉɔuɐɹɥԀ

trolololol

The repo has a FiM++ interpreter that translates into Java/Scala, pretty cool. This is just an example on the repo.

Duga

[retailcoder/Rubberduck] retailcoder commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> @Zomis @ckuhn203 should we have a configurable "list of acceptable magic numbers"?

[retailcoder/Rubberduck] ckuhn203 commented on issue #359: Code Inspection Idea: Identify "Magic Numbers"

> Yes @retailcoder. We should. We can put 0 & 1 in there as defaults, but it should be configurable. It will be very similar to the code for the configurable TodoMarkers.

Mat's Mug

5:03 PM

That inspection is starting to sound like a performance bottleneck....

RubberDuck

Maybe, but what makes you say that now? The configurable magic numbers?

Mat's Mug

@sᴉɔuɐɹɥԀ If you learn enough FiM++ to put up a fizzbuzz for review, you could very well beat the LOLCODE fizzbuzz score...

sᴉɔuɐɹɥԀ

in The 2nd Monitor, yesterday, by sᴉɔuɐɹɥԀ

Oh and before you ask, No; I'm not making a FizzBuzz in FiM++ lol.

Mat's Mug

LOL!

OMG my phone has learned to write LOLCODE when I type all-caps "LO"

2

sᴉɔuɐɹɥԀ

LMAO

RubberDuck

5:10 PM

@Mat'sMug epic.

Mat's Mug

@RubberDuck The tree listener alone is a bottleneck actually.. IDK, we'll have to try and see what we get, but running that inspection on a large code base is sure to be painful

RubberDuck

Maybe. We'll see. We get to it when we get to it.

And if it takes too long to run, we nix it.

I'm trying to relax a bit about the project.

Mat's Mug

Yeah. I have other priorities anyway cough false positives cough

RubberDuck

Things get done when they get done and we do what we can do.

Mat's Mug

@RubberDuck Good call. No need to pressure ourselves, remember why we're doing this! #fun

RubberDuck

5:14 PM

Yeah man. I have enough stress going on as it is... I need to make sure I'm enjoying what little time I have to dedicate to RD getting source control integration finished.

Duga

5:55 PM

[ckuhn203/VBEX] ckuhn203 commented on pull request #19: Added Tree data structure

> It's not a fantastic implementation, but it can be safely merged.

[ckuhn203/VBEX] ckuhn203 merged pull request #19: Added Tree data structure from ckuhn203-Tree into master

[ckuhn203/VBEX] ckuhn203 pushed 4 commits to master

[ckuhn203/VBEX] ckuhn203 pushed commit 44c0d42f to master

> Merge pull request #19 from ckuhn203/ckuhn203-Tree

Added Tree data structure

[ckuhn203/VBEX] ckuhn203 deleted branch ckuhn203-Tree

sᴉɔuɐɹɥԀ

6:13 PM

So hard to write code with a French keyboard -.-