VBA Rubberducking

Duga

00:00

RELOAD!

[Cardshifter/cardshifter.github.io] 1 commit. 1 issue comment.

Mat's Mug

(drumroll)

Duga

[retailcoder/Rubberduck] 14 commits. 4 opened issues. 9 closed issues. 12 issue comments.

2

Mat's Mug

woot-woot!

Duga

[Vincentyification/JavaBot‌] 2 issue comments.

[Vogel612/JavaBot] 1 commit.

[Zomis/GithubHookSEChatService] 1 opened issue.

Simon André Forsberg

Only defining endOfRange and not initializing it as False inside the for-loop causes the routine to generate the same Civ for each loop. I added "endOfRange = False" right after the definition to counter this. — Simon Karfunkel 57 mins ago

I really can't understand why that ^^ is happening.

RubberDuck

00:17

Because it gets set to true, but never back to false. All variables declared inside a procedure are procedure scoped. There's no "loop" scope.

Mat's Mug

which is great, because it tremendously simplifies scoping the damn variables when you're parsing them :)

RubberDuck

Aaannnddd everything is relative. Lol

Also, any day where more issues are closed than opened is a good day.

Duga

[retailcoder/Rubberduck] retailcoder pushed commit 95a4e9ac to 1.21

> fixed double-click flicker

Mat's Mug

00:37

@RubberDuck IDK. A day like when we released 1.2 and had feedback from users via issues, I call that an excellent day!

RubberDuck

That it was. =)

Duga

[retailcoder/Rubberduck] ckuhn203 commented on RetailCoder.VBE/UI/CodeExplorer/CodeExplorerWindow.cs of commit 95a4e9ac

> Why the dead code?

Mat's Mug

@Duga I'll remove it, was copied from the working commit ;)

RubberDuck

Ahh Gotchya. Was just curious.

Simon André Forsberg

00:53

@RubberDuck ah, gotcha. Of course.

RubberDuck

Yeah VBA sucks.

Mat's Mug

01:11

@RubberDuck and yet.....

lol

Duga

02:42

[retailcoder/Rubberduck] retailcoder pushed 28 commits to GrammarIsFun

Mat's Mug

03:15

ok, WTF. Grammar won't build.

as is

no changes whatsoever

oh wait...

I did change it

don't tell me...

(rubberducking)

Mat's Mug

05:14

ugh. I give up.

grammar encoding is borked, I don't get it.

Mat's Mug

12:14

wow, 3 new followers overnight!

RubberDuck

Monking @Mat'sMug.

Mat's Mug

12:29

Monking!

RubberDuck

ALL THE THINGS ARE BROKEN this morning.

Mat's Mug

@RubberDuck What??

RubberDuck

@Mat'sMug My dailies started failing with the most unhelpful error known to man.

BIDS runs the SSIS package fine, but the job fails.

For no reason what so ever.

Mat's Mug

Ah, thought I had borked RD

lol

Not using SSIS logging?

Oh wait, it's the SQL Agent job that fails?

RubberDuck

I am using logging.

There isn't one.

nhgrif

12:43

Boo.

RubberDuck

I use the default logging to the db and some custom logging in places.

It seems to be working now, but I don't know why.....

@nhgrif Yeah. Exactly.

nhgrif

My iPads aren't cooperating this morning.

RubberDuck

Sounds like a MONDAY.

2

nhgrif

Also, I just found out my boss can't make it to interview this morning, so I'm doing it by myself... I've never done an interview before...

Mat's Mug

lol

Good luck!

RubberDuck

12:50

Yikes. Good luck.

Mat's Mug

Could be worse though; you could be sitting in the other chair! ;)

nhgrif

I'd rather be. I have no lack of confidence in my ability as a candidate...

Duga

[retailcoder/Rubberduck] Ramoonus starred us

2

[retailcoder/Rubberduck] Ramoonus forked us into Ramoonus/Rubberduck

RubberDuck

13:07

@Duga That one actually has the potential to submit a PR.

Mat's Mug

13:34

@nhgrif You'll do great, cheer up!

Mat's Mug

13:58

done?

nhgrif

The candidate will be working on remotely on my desktop.

Having them write some SQL and some VB.Net

First interview is in one hour.

We have two one-hour interviews scheduled back to back.

Mat's Mug

lol I'd call it one 2-hour interview if it's the same candidate

nhgrif

No, different candidates.

Mat's Mug

ah

nhgrif

So ideally, 3min SQL, 30 min VB

Mat's Mug

14:01

I hope you find your VB guy today!

nhgrif

We'll see. My boss should be there for second interview.

Mat's Mug

@RubberDuck I know exactly how to change the grammar file to pick up module options anywhere in the declarations section without breaking the parser. Only problem is that I can't build the damn thing because of stupid encoding issues - ANTLR doesn't recognize some characters in the .g4 file, no matter how I encode it.

and then you have VS that continuously wants to change the encoding.

I'll create a Rubberduck.Grammar project tonight in the GrammarIsFun branch, and move all the generated code there.

RubberDuck

Okay. Sounds like a next kind of thing. 1.21

is starting to look pretty solid.

Mat's Mug

indeed

I need to build an installer :)

only thing that bothers me is that the Assert variable doesn't get picked up as not used when it should.

but I'm happy to announce that fields that are of a UDT-type no longer get flagged as never assigned

@RubberDuck remind me, when I take screenshots for the refactor/rename functionality, that the code in the screenshot must involve a local array Dim a(42) As Integer and a Function a(ByVal foo As Integer).

2

;)

RubberDuck

14:23

@Mat'sMug that should help.

Mat's Mug

lol

Mat's Mug

15:36

@ticker it's gimmeh-teh-codez day!

Mat's Mug

15:54

> Use of obsolete Rem keyword

0

Q: Excel VBA Select multiple sheet

I'm trying to make a selection of multiple sheets one workbook based on run macros. At the end I have string containing sheets for selection, but my program prompt errors ... Example: SheetPrintPDF=Split(SheetPrintPDFTemp, ",") Rem Debug show me that SheetPrintPDFTemp contains "Sheet1, Shee...

vba

nhgrif

16:40

First one didn't have any programming experience. Second one didn't show up.

Questions as basic as "list at least two ways of looping": completely no clue.

He was okay with sql

RubberDuck

@Mat'sMug Seriously?!

Mat's Mug

@nhgrif in VB? lol ..there's 7 different loop constructs IIRC

RubberDuck

OH wow man.

On all parts...

nhgrif

I mean, I expected, at a minimum, for and while. But I know there are lots of others... But he could answer any...

He couldn't answer any programming question, only SQL questions.

And the questions aren't even that specific to vb

Mostly general programming questions.

RubberDuck

Yeah... i would have at least expected a For and ForEach.

They're the first two that come to my mind. I honestly don't use while loops unless I really need to.

Mat's Mug

16:51

Just a side note: Rubberduck code inspections would flag "Use of obsolete Rem keyword" and suggest to replace it with an apostrophe ;) — Mat's Mug 5 secs ago

couldn't help it

nhgrif

Yeah, but in a programming class, you don't generally get taught foreach.

RubberDuck

Who ~~takes~~ pays for programming classes?

nhgrif

College students.

Mat's Mug

well there's For...Next, For Each...Next, Do...Loop, Do...While, Do...Until, Do [While|Until]...Loop, and While...Wend

RubberDuck

Good grief, there's nothing you can't learn from reading and doing.

nhgrif

16:53

Realistically, the question is an opportunity for someone who knows programming to say "how come just two?"

2

RubberDuck

good point

nhgrif

@Mat'sMug and I was willing to count recursion.

Mat's Mug

then there's GoTo, and if you want you can even .... ^^ that

nhgrif

Yup

RubberDuck

@nhgrif if he didn't know how to loop, there's no way he understood recursion.

nhgrif

16:55

Except... If someone mentions go to without any qualifications... ...

RubberDuck

I guess "He" is a bit presumptuous of me...

1

A: Getting rid of goto without duplication in Go

This doesn't look dirty to me, but perhaps that's because this is a common error handling pattern in VBA. However, if you're really determined to get rid of the GoTo, then you could try an Or, couldn't you? (I don't know Go, so I could be wrong in syntax, but the principal should work.) passwd, ...

Goto isn't evil... not entirely.

nhgrif

As I said, "without qualifications"

RubberDuck

Right. I was just making the point.

One of the things I hate about programmers is the dogmatism. "Always" is a word we should avoid unless we're damn sure that it's really always.

Mat's Mug

17:12

Or unless you're a sith

RubberDuck

> Only Sith deal in absolutes.

Mat's Mug

Just noticed, we're 7 stargazers short of matching the fim++ repository :)

RubberDuck

LOL

Mat's Mug

I cannot wait to implement refactor/rename.

Just to post a link on @wqw's thread

and say "the grammar isn't ready for this, indeed. But Rubberduck is" :)

cheezsteak

17:55

@Mat'sMug in VB or VB.Net? All I can think of are For, For Each, While and Do Loop.

never mind just needed to read further...

RubberDuck

18:19

@cheezsteak lol

cheezsteak

18:50

Christ this cat is cuddly

RubberDuck

19:04

@cheezsteak That just made my day. So happy my phone was muted.

cheezsteak

why were you happy your phone was muted

RubberDuck

I cracked up, actually LOL.

Department wide meeting... very happy I muted my phone.

Mat's Mug

20:05

> 29 downloads

Duga

[retailcoder/Rubberduck] retailcoder commented on issue #286: Code Explorer does not list all module members when module options are not specified first in that module, or when code uses numbered lines.

> Version 1.3 will see a grammar rebuild. This is definitely getting fixed.

[retailcoder/Rubberduck] retailcoder removed label status-deferred from issue #286: Code Explorer does not list all module members when module options are not specified first in that module, or when code uses numbered lines.

[retailcoder/Rubberduck] retailcoder removed label unit-testing from issue #184: Make Configuration Settings UI Sexier

Mat's Mug

20:21

@RubberDuck regarding #184 - the treeview needs icons, and then the right-side panel could use some margins/padding. I already made the treeview background white and added alternating colors to the CI config grid rows; other than that the buttons on the TODO markers config page needs a little cleanup, and then the form itself shouldn't be resizeable (i.e. make it a dialog), so that user can't wreck the layout.

also there needs to be a [Save] and a [Cancel] button, so that the user can make changes without saving them when the form closes.

RubberDuck

@Mat'sMug That part is definitely a good call.

Trying to find time (and focus) to code right now is hard.

I've been trying to hit all of the smaller issues, but I ran out.

Mat's Mug

lol

it's all right, we're more than on-track for 1.21

the worst that could happen is that I implement all the changes myself ;)

RubberDuck

Yeah, but that's not fair to me!

Mat's Mug

don't worry, there will be more issues!

RubberDuck

How long ago did we release 1.2? A little over a week ago?

Mat's Mug

20:27

March 4th

5-6 days, depending on how you count it. Average ~5 downloads / day :)

RubberDuck

@Mat'sMug Perhaps the only certainty in programming.

Mat's Mug

I need to set up google analytics for rubberduck-vba.com, I have no idea how much traffic it's getting

and it's annoying the Hell out of me!

@RubberDuck I haven't played much in the configuration code, ..is there a way to bring up the dialog with a specific page displayed?

because I'd like to have a little icon on the CI toolwindow's toolbar, to bring up the CI config. ditto for the todo explorer

RubberDuck

Not right now, but it could be modified to do that.

It brings up the todo settings by default just because that's what got built first.

Damn batteries in my keyboard died.

Mat's Mug

we need a "General" tab to bring up by default ;)

RubberDuck

ahhhh better

@Mat'sMug What would you put in it?

Because I thought of that, but there weren't really any "General" settings to put in there.

Mat's Mug

20:44

IDK, perhaps a root path config, for exporting code files (could serve as a root folder for SC stuff too). Can't think of anything else for now, but there's certainly a number of things we've hard-coded that could be made configurable.

unit test module and procedure stubs, and whether to early or late-bind the Rubberduck TLB.. although that would really belong under a "Unit Testing" tab

RubberDuck

That ^

Mat's Mug

hi @SimonKarfunkel!

RubberDuck

But yeah. A general tab would be nice.

Hola @SimonKarfunkel!

Simon Karfunkel

Hi/Hola to you too :)

Mat's Mug

have you tried Rubberduck yet?

Simon Karfunkel

20:53

Wanted to ask you experts of some functionality in VBA as i'm still a bit of a newbie at it

Mat's Mug

sure!

Simon Karfunkel

no i haven't... i certainly should

RubberDuck

Sure. What's up?

Simon Karfunkel

i would want to filter the table in my workbook and only consider the filtered rows when my macro is run. Is this possible at all?

cheezsteak

there is a way to select only visible cells of a range

I never used it though

RubberDuck

20:55

Not waiting for @StackExchange. The follow up will be much harder to pick on.

1

Q: Follow-up: Randomizing Civilization 5 team choice

This is a follow-up on my previous question about a VBA macro for randomizing a draw from a table. It can be found here: Randomizing Civilization 5 team choice The code has been improved with the help I got earlier and I now seek even further improvements. Public Enum CivilizationTableColumns ...

vba excel

Hmmmmm that's a good question @SimonKarfunkel.

Is there a visible property?

Simon Karfunkel

ok cheezsteak. how would you do that?

Mat's Mug

"goto special"

Stack Exchange

2

Q: Follow-up: Randomizing Civilization 5 team choice

This is a follow-up on my previous question about a VBA macro for randomizing a draw from a table. It can be found here: Randomizing Civilization 5 team choice The code has been improved with the help I got earlier and I now seek even further improvements. Public Enum CivilizationTableColumns ...

vba excel

RubberDuck

It's not "visible" it's hidden. msdn.microsoft.com/EN-US/library/office/ff834657.aspx

Simon Karfunkel

thank you rubberduck :)

RubberDuck

20:57

You could check that, but Mug might be onto something too.

Mat's Mug

nah, can't find it in the API

Simon André Forsberg

@Mat'sMug Did I just hear a goto!?

Mat's Mug

but maybe macro recorder can reveal something

lol

RubberDuck

So, I would look at this too. Special Cells

But I've gotta go meet our new pooch. Cya guys.

Mat's Mug

Selection.SpecialCells(xlCellTypeVisible).Select

RubberDuck

20:59

@Mat'sMug, special cells is what you want.

Mat's Mug

yup

later ducky!

Simon Karfunkel

okay... i will look into this. Thanks for the tips

cya

Mat's Mug

come back anytime!

(and check out rubberduck-vba.com)!

Simon Karfunkel

i have already downloaded it ;)

and installed. Only have to try it now.

Mat's Mug

oh nice!

> 30 downloads

Simon André Forsberg

21:03

I had a feeling @SimonKarfunkel would be a potential RubberDuck user :)

Mat's Mug

most issues of 1.2 are solved already, we're planning to release 1.21 around the end of next week

cheezsteak

@SimonKarfunkel sorry I didn't remember because I never used it. Special cells was what I was thinking about.

Mat's Mug

21:23

lol. talk about inflated head:

> So unless your head is 100 light years wide, there is little chance of any individual neutrino interacting with it.

17

A: Can the brain detect the passage of a neutrino?

The cross-section for neutrino interactions is energy dependent. For solar neutrinos at $\sim 0.4$ MeV, which would likely dominate any neutrinos likely to interact (the cosmic background neutrinos have way low energies) , the cross-sections are $\sigma \sim 10^{-48}$ m$^2$, for both leptonic pr...

Mat's Mug

21:48

@SimonKarfunkel answered! :)

Duga

[retailcoder/Rubberduck] retailcoder opened issue #304: Extract Method should insert the extracted method just below the calling one

> Extract Method refactoring currently inserts the new procedure at the beginning of the code module, which is slightly off-putting (i.e. downright annoying). The extracted method should be inserted immediately below the method it was extracted from.

[retailcoder/Rubberduck] retailcoder added label refactorings to issue #304: Extract Method should insert the extracted method just below the calling one

[retailcoder/Rubberduck] retailcoder added label enhancement to issue #304: Extract Method should insert the extracted method just below the calling one

Simon André Forsberg

22:16

@Mat'sMug Is it just me or....? I'm really impressed at how @SimonKarfunkel's code changed from the first question to the second. Wow.

RubberDuck

Not just you @SimonAndréForsberg. I am too.

Mat's Mug

22:56

@SimonAndréForsberg Yeah, night and day! :)

Duga

[retailcoder/Rubberduck] ckuhn203 commented on issue #304: Extract Method should insert the extracted method just below the calling one

> :+1:

Simon André Forsberg

23:08

@RubberDuck @Mat'sMug Does VBA has some kind of ArrayList object? Or List as you C#'ers call it?

Mat's Mug

23:23

Well there's a Collection ...and that's about it :)

Simon André Forsberg

@Mat'sMug well that's helpful... what's that?

Mat's Mug

A container that lets you add & remove items

Simon Karfunkel

The thing is that i have a filtered table in which i have managed to select only the visible cells... Afterwards i have tried to create a range out of the selection but that is not helpful whatsoever as it starts at the first filtered result and continues through the consecutive rowa, including the hidden ones

rows*

Transcript for