VBA Rubberducking - 2018-05-01 (page 2 of 3)

« first day (1425 days earlier) ← previous day next day → last day (1755 days later) »

5:09 PM

~~I believe so. It was mentioned that the rubberduckvba.com could be replaced by an ducky icon, but I don't think that should hold up the PR.~~

user image

Any issue with Win7 border?

I'm editing the width so the full OS can be shown.

it's really a nitpick on my part but I really don't like how scrollbars don't align to the box - it should have its own box.

Mathieu Guindon

Might be just me but I think I miss the white background here

but I'm fine w/ merging and fixing it in the next

FWIW, I don't miss the white background.

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #3971: Update about control

[rubberduck-vba/Rubberduck] [codecov[bot]](github.com/apps/codecov) commented on pull request #3971: Update about control

> # [Codecov](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3971?src=pr&el=h1) Report
> Merging [#3971](https://codecov.io/gh/rubberduck-vba/Rubberduck/pull/3971?src=pr&el=desc) into [next](https://codecov.io/gh/rubberduck-vba/Rubberduck/commit/1b1fe5cb0a9f3f17e4016128b4808fda16af7982?src=pr&el=desc) will **increase** coverage by `0.03%`.
> The diff coverage is `76.92%`.

```diff
@@ Coverage Diff @@
## next #3971 +/- ##
=======================

Getting resource strings in now. Forgot about those.

Mathieu Guindon

5:21 PM

@IvenBach don't

Ok.

Mathieu Guindon

Gah nevermind it's going to conflict anyway... Sure bring it =)

@IvenBach the only nitpick I have is that I'd like a 5dip border all around, so that the labels don't look like they're right on the edge

Nitpick away. It's how I'm going to learn.

Duck check: Why are my newly added resource strings showing # in front of them in the design view?

Doesn't see them yet.

We use a library for that.

[rubberduck-vba/Rubberduck] daFreeMan commented on issue #3978: Explorer placement/grouping

> Primary on the right. Shoots that theory... 😉

5:29 PM

Restarting VS solved it.

Mathieu Guindon

clean/rebuild would have been less drastic, but sure :)

I did that and it still didn't fix.

Just don't worry about it.

It works at runtime, but it has some caching at design time.

I have OCD sometimes about why does ...

user image

[rubberduck-vba/Rubberduck] build for commit f7e593c9 on unknown branch: AppVeyor build succeeded

5:34 PM

> a COM add-in that extends a famously dreaded legacy Win32 IDE

Lol @MathieuGuindon

[rubberduck-vba/Rubberduck] Vogel612 added label PR-Status: Conflicting to pull request #3934: Code metrics moving and redesign (WIP)

@Duga I'll drop some code behind onto this to get the UI working...

at least that way it can be merged without stalling

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #3971: Update about control

@Duga @MathieuGuindon Unless I misunderstood your border request, that should be good.

@IvenBach I introduced an additional string there :(

"You can also find us at:"

"Report an Issue" should also be localized.

Mathieu Guindon

5:45 PM

@IvenBach last one: I'd rather have "Rubberduck" spelled out in RD itself.. "RD" is mostly used among ourselves

[rubberduck-vba/Rubberduck] Vogel612 commented on Rubberduck.Core/UI/About/AboutControl.xaml of pull request #3971: Update about control

@MathieuGuindon unfortunately, i must use 4.6

Mathieu Guindon

aw

yeah. won't even play with 4.5.2

Mathieu Guindon

mind making that a reply to Ryan's comment?

5:47 PM

at least 4.6 can run on Vista.

not yet -still building

but will do!

Mathieu Guindon

so we retarget everything to 4.6?

I'll post a comment and if it works I can turn it into an answer. According to the docs you should target .NET 4.6 so it's compatible with .NET Standard 1.3. And according to here Windows Vista supports up to .NET 4.6. Can you make all projects target 4.6? — Camilo Terevinto 2 hours ago

basically Camilo has a checkmark-worthy answer there

only if we can't get AV to like 4.5 for the other projects and 4.6 only for the analyzer

i'd rather that we leave everyone else at 4.5

huh, why?

Mathieu Guindon

pretty sure AV/MSBuild will complain about it

doesn't that only make it more complicated for us?

Mathieu Guindon

5:49 PM

^

I'd prefer all projects to target the same framework level to avoid inconsistencies and incompatibilities between them

Mathieu Guindon

me too

IDK. it seems just downright strange that a project that isn't even a part of the other projects has to be the same versions

belts'n'suspenders, right?

Mathieu Guindon

lol

5:50 PM

heh. true.

Mathieu Guindon

#WorthIt

[rubberduck-vba/Rubberduck] IvenBach synchronized pull request #3971: Update about control

@Duga About window last bit of tidying, for now.

[rubberduck-vba/Rubberduck] build for commit 91f8ade9 on unknown branch: AppVeyor build succeeded

gosh, they do not make it easy to remove/add and update the dependencies. way too much clicking for each project.

I dont' get this.

> 13>CSC : warning CS8032: An instance of analyzer RubberduckCodeAnalysis.ComVisibleTypeAnalyzer cannot be created from C:\GitHub\Rubberduck\Rubberduck\RubberduckCodeAnalysis\bin\Release\RubberduckCod‌eAnalysis.dll : Method 'get_SupportedDiagnostics' in type 'RubberduckCodeAnalysis.ComVisibleTypeAnalyzer' from assembly 'RubberduckCodeAnalysis, Version=1.0.0.0, Culture=neutral, PublicKeyToken=null' does not have an implementation..

Yet the analyzer is working; I get the expected results....

wait, that's a different message

NM, then. Earlier it was saying it couldn't find System.Collections.Immutable

uh guys, Rubberduck.Deployment project has been at 4.6.1 while everyone else was at 4.5.....

6:14 PM

yaaayyy

Mathieu Guindon

@this if it doesn't hurt anything to retarget it to 4.5, might as well :)

@CamiloTerevinto retargeting to 4.6 is in progress, you'll want to post that answer =) — Mathieu Guindon 1 min ago

deployment is also an "internal" project

Mathieu Guindon

...that impacts CI pre-releases :)

[rubberduck-vba/Rubberduck] build for commit 8aae3f4e on unknown branch: AppVeyor build succeeded

but we've been shipping them for a while already

with no problems apparently....

Mathieu Guindon

6:17 PM

it's just simpler if everything is targeting the same runtime

for the final product, absolutely.

so, I have 990 errors.

3

Mathieu Guindon

oh shit

what are they like?

i think i fixed it - forgot to retarget the test project

cleaned and aow rebuilding

Mathieu Guindon

:+1:

and that succeeds.

i need to restore the tests from the analyzer project then we can see if AV'll be OK with 4.6 for everyone

blargh this PR keeps getting bigger.

Mathieu Guindon

6:26 PM

it's ready to merge once CI passes, right?

no i still have few todos.... it's in the first post of the PR

need a factory for the AddIn object and the installer has to be updated, too.

@this C# 6 was released alongside .NET 4.6. C# 6 is the first one to use Roslyn.

It makes sense that you have to target it.

ah I see.

the weird thing is that it was working fine when they were 4.5

Really?

just not up in the AV for whatever reasons.

to be clear, the analzyer project was standard 1.3

6:32 PM

Oh, the files were all in the right spot or something.

but I can reference the project from all 4.5 projects

if I did not add an explicit dependency on the analyzer project, then it would fail to build after a clean

Did I tell you how I managed to get some files checked in at work, but not referenced in the project (old .csproj structure).

because they'd start building before the analzyer project finishes

So, it worked locally for everything because they were just .cshtml files--they were on the disk where it was looking for it.

by adding project dependencies, they would always build even after a clean but just not AV.

No, do tell me.

6:33 PM

Threw exceptions on the web server because they don't get copied unless they are referenced.

#fun times.

so it blew up in testing environment?

Yes.

i love those "works on my machine" symptoms

IKR?

Especially when it would work on all the dev machines...

17

Q: Once implicit/inferred typing is introduced, how can I impress upon students the importance of not abusing it?

I typically have the same students two semesters in a row, for programming I (VB.NET) and programming II (C#). In our early, impressionable time together, I stress to these students (high-schoolers) and show them by example (I write lots of code for them) the importance of writing clear, readable...

I'm ready to explode.

Man, RRRRRRRRRRRRRRRRRRRRRRRRRRRRRR.

@Frax, VB is unavoidable because it is the required language for progarmming I in my state. — Java Jive 7 hours ago

The state has ABSOLUTELY NO BUSINESS DOING THAT.

@Hosch250 note that that's on Uni Education AFAICT

6:42 PM

"[A]nd since it is my classroom after all". OH YEAH? IT'S THESE STUDENTS' CAREERS YOU ARE RUINING.

@Vogel612 Exactly.

the state usually has at least some influence on the curriculum

No it doesn't.

Only on public universities, that is.

And even then, the state has 100% no business doing that.

The politicians no nothing about anything except lying. How would they know anything about what should be taught?

and I think most are more motivated to get more students enrolled in, so that means dumbing down => higher attendance => let's do it!

And how are they going to make sure it keeps up to date?

I'm almost ready to found my own "university" that teaches programming well and industry-oriented.

@Hosch250 true that

well ... some politicians actually get stuff done

6:45 PM

No time limit or semesters. You either complete the work or you don't.

@Vogel612 True that too.

but most of them are on the very local level

Precisely.

And even then, we can't rely on them to be around forever.

the higher up in the gov't, the further removed from the actual day to day issues

So they need to leave this stuff to the university board.

Which then has the discretion to leave it to the teachers.

which then proceed to choose something like java for all courses, which is a stupid idea

or c++ for the beginner courses, which is also a stupid AF idea

6:47 PM

Depends which book you use.

programming and CS education is somewhat broken since a few decades

And I don't mind using one language for all classes--it allows the students to progress without learning a million new languages, which is difficult for beginners.

Java sucks, because it badly attempts to hide pointers

C++ sucks because you have to deal with pointers and memory management

every language has something it sucks at. badly

Yes, but if you have a really good book that teaches that properly, though, and guides you from step one, you are good.

and then you go write an algorithm to concat strings and it's slow AF and you don't know that you wrote a shlemiel the painter algo

6:49 PM

LOL.

As far as I'm concerned, the whole premise of uni is backward.

It's not about teachers teaching. It's about students learning.

@Hosch250 You got a guinnea ~~pig~~ duck right here.

And they need to have the motivation to learn on their own with the teachers basically being able to guide them a bit in showing them how to find what has been tried or thought of by others.

@IvenBach You are already pretty advanced. Now you just need a couple years experience.

I had some concepts figured out in my head. No way to articulate them or what to search for.

That was my bane for so long...

@IvenBach That's where the teachers are helpful. They can give you some jargon to search for, and how to find more jargon.

yay for jargon

6:55 PM

I must've been lucky at my college. All my professors were good and had industry experience.

Talking about that, do you realize it took me a whole day to figure out about compilers and IDE's and stuff from just knowing I wanted to start programming and knowing about Python, Java, and JavaScript?

~gracefully skips the opportunity to rant about Serializable

Most of my knowledge at this point is A) a bunch of jargon, B) C#, and C) SQL.

And then the abominable JS.

fwiw, I do think students have to take a bit of C and work upwards. Working downwards is not that helpful; you'll never see the painter algo that way.

@this then again pointers are usually something that leads to a lot of confusion

6:58 PM

Painter algo is 75 units of work done day 1. 30 day 2. 15 day 3. With failing diminishing returns thereafter?

C is great and all, but c++ is not that much higher level

@IvenBach exactly

i'm not sure one can get very far if they don't at least know about something about how pointers work.

Meaning the farther you get out it slows down because you have to always go back to the beginning.

@this this.

^ Hey an echo.

I"m not saying they should write a OS in C but they should start out writing few simple library functions in a DLL project, then expand on it.

@this Actually, I disagree.

7:00 PM

even better if they are then asked to use the same C DLL they wrote in their Python project (for example) so they can see the level of abstraction for themselves.

Great way to lose motivation never seeing your code run.

they have to understand the abstraction, or it's going to be blindly groping in darkness.

@this what do you think of Logo?

You need to learn the basic structures before you start learning abstraction and libraries and stuff, IMO.

"move the turtle" and all?

7:00 PM

FWIW, it's much easier to run a DLL than a program.

I know nothing about Logo.

@this for you maybe

@this Actually, no.

Most IDE's have a "Run" button.

see but it hides all the magic

Logo is a dead simple interpreted language for drawing lines on a canvas

When you get into DLL's, you need to make a second project that references the DLL and calls the member.

7:01 PM

there's "FORWARD(int), LEFT(double), RIGHT(double), UP(), DOWN()" and some trigonometric functions

UP ends drawing, Down starts drawing

there is an implicit current position

@Vogel612 Why the type differences?

FWD is theoretically in pixels, LEFT and RIGHT in degrees

I'd just realized that.

I thought it was 1,2,3,4 for 90 degrees at first.

something simplistic like that which very much limits the "problem domain" can be extremely useful to teach basics like control structures

there even is recursion in the language

and you don't have to deal with the messy underlying pointers and canvases and colors and all that mess

also you get directly visible results

IDK - IMO, the control structure aren't that hard to understand in any language.

Mathieu Guindon

7:09 PM

just got +100 and joined that site just to post this:

If you declare an Int32 as Object, you've just boxed a value type, which is terrible sloppy coding with real performance implications. If you declare an Int32 as var, all you've done is shift the reader's focus on the parts of the code that actually matter, as opposed to making your C# read like Java. FWIW you shouldn't even need to mention var until you're ready to get into anonymous types and LINQ. Also, hover the mouse over var, the IDE tells you what the compiler type is. var has NOTHING to do with Object whatsoever. And it's NOT dangerous, even less "sloppy". — Mathieu Guindon 15 secs ago

what is more hard to understand is why you are getting obscure error about a variable (that's right there!) cannot be converted to some type (and it is that type!) or something like that.

If they don't teach you how to diagnose and understand why it's doing that, then you're going to be forever mashing the run button.

Says the smith to the carpenter: "I also always think that people should be able to make their own knives and plowblades. It's so easy."

2

I love that quote!

Just keep in mind I have no formal comp. sci education... sooo.

blogs.windows.com/windowsexperience/2018/04/30/…

This is great.

@this It's easy to learn about that with C#'s excellent error messages. Just paste the error into the browser and get half a dozen excellent blog/SO posts that tell you more about that then you realized was possible.

@this that's not really relevant. Formal CS education is IME rather disjoint from the "everyday life of a programmer"

7:18 PM

On that we agree, Vogel. A good education should be more aligned to what they'll encounter in the everyday life of a programmer. I've seen few "experienced"/"educated" programmers who couldn't work it out that the error they had were due to a missing reference.... :\

@Hosch250 While I do agree that C# has good error messaging there are still few that can be quite opaque without the necessary background. I can't give an example but I know that there has been a number and sometime the error message is completely unrelated to what the fix should be because compiler only knows what it knows; it can't tell you what is the right thing to do in all cases.

@Vogel612 Extremely disjoint.

Mathieu Guindon

IMO CS educators should stick to the theory/curriculum and leave their opinions at home. If my C# teacher has a Java background and dislikes var and calls my code "sloppy", that's his opinion, nothing worth grading.

@this That's why I feel that C# is an excellent beginner language. It makes it easy to do right without trying to second-guess you like VB.NET.

@MathieuGuindon Easy to argue that about truly sloppy code, though.

Mathieu Guindon

however if they go and say "in the real world, you'll have to code using the conventions that are in place in the team you're in. In this course, the convention is to use explicit typing everywhere", then I'm fine with and respect that.

@this FWIW formal CS doesn't aim to educate people that can write programs

7:21 PM

They shouldn't teach a language unless they know the standard style conventions.

Mathieu Guindon

^

Also, I agree that absolute new beginners should use explicit typings.

Mathieu Guindon

too bad the question is protected (and assoc.bonus doesn't count)

@Vogel612 then what? isn't the whole point of CS to be a programmer or at least a derivative of one?

it aims to educate people about algorithms, complexity theory and generally the rather mathsy parts of programming

7:22 PM

@MathieuGuindon Something doesn't sit well with me on that. If they are the instructor teaching the course, shouldn't they have an idea about boxing value types?

It reduces cognitive load when you are a beginner.

Mathieu Guindon

@IvenBach they should, yes

@IvenBach Man, how many people do know about it?

I should set up a poll at work.

Mathieu Guindon

you're teaching data types in C# and don't even mention ValueType and boxing/unboxing? get another job, you're not entitled an opinion about var.

I retort with: How many people should know about that.

7:23 PM

@IvenBach Every C# developer ever.

If you're working with something you should know, even vaguely, how it does whatever magic it does do.

@IvenBach To a certain extent, I'd agree. You don't need to know about the C# compiler details, even though you should to write optimal programs.

Mathieu Guindon

I just remembered how much I hated school.

@MathieuGuindon Guess how I survived college?

Mathieu Guindon

pot?

7:25 PM

harsh

No.... You should know me better than that :)

Mathieu Guindon

lol

worked for me anyway :D

I learned programming.

Mathieu Guindon

@Vogel612 /s/r/c/

@Vogel612 those aren't bad things to know though. Understanding O(n) problem for instance seems useful. And if they know what the painter algo looks like, then good. But I'll grant that this is something for later years. In the early years, I'd like them to know how to do research and understand the context. That matters way more than they'd realize.

7:26 PM

Kept me mentally stimulated.

@Hosch250 what do you have against pottery?

Mathieu Guindon

pffft

@this Screws with your mind. I'd no way be as good as I am if I was high all the time.

@MathieuGuindon doesn't do anything because of the leading space

Also, as I've said before, my family has a history of bad drug/alcohol use. I'm staying away from them.

7:28 PM

#HumorFail

@this Teaching others the basics so they can teach themselves. VBA.Interaction.MsgBox != Excel.Application.InputBox.

@Hosch250 As does mine. I'm an odd enough duck without anything helping it along.

@MathieuGuindon welp, isn't doing pot a felony?

Not unless you're in right place

or wrong place

@Vogel612 No, just a misdemeanor in most states.

it's legalized in Washington and Colorado.

7:31 PM

And legal in ^

California

@IvenBach for recreational uses, too?

Also, I believe Maine too, or one of those eastern states.

They tried it in MN. Didn't pass yet.

if I map the terms correctly, it's theoretically a felony in Germany.

AIUI, WA & CO legal for all types of uses -- there are more states that allow but only as a medicine or something like that

7:32 PM

mostly ignored for truly recreational use

@this Pretty sure. It was talked about a lot on the radio. I don't pay enough attention to it though.

And apparently because it's "pot" they can get away with smoking it in public buildings and stuff, so if you are in a city in CO, you are liable to get high without smoking.

:\

that'd annoy me.

IKR. Really pisses me off.

Especially since CO was one of the places I was thinking about moving to. Lots of tech jobs out there.

Be responsible. It applies to everything.

7:33 PM

ATM, I'm thinking Utah maybe.

Beware of Utah, loads of Utards be living there.

Mathieu Guindon

@Vogel612 biggest hypocrisy in Occident, yes.

I don't get that joke.

@MathieuGuindon after the move of secularization

Mathieu Guindon

ever been to a rock concert?

7:35 PM

@Hosch250 Play on words combining Retard+Utah. Lots of highly sheltered people live there.

@MathieuGuindon nope. well not a big one anyways. Not much of a crowds person

@MathieuGuindon Why would I want to? Bunch of losers being paid to make an execrable noise and give people seizures with their flashing lights.

Mathieu Guindon

lol

Me: Dad, what's that?
Old Man: That's pot son.
Me: The shouldn't be doing that.
Old Man: You'll learn once you get older boy.

^ When I went with my dad to a concert long ago.

Must've been 12-13.

Mathieu Guindon

#LifeThings

7:37 PM

More seriously, I like a very limited selection of rock. Mostly southern rock.

Not enough to go to a concert, even if I didn't get severe headaches from flashing lights.

@Hosch250 Lynyrd Skynyrd, Kid Rock and maybe Brian Adams?

@Vogel612 Yeah.

Gimme Three Steps, All I Can Do is Write About It, Summer of '69, a Kid Rock I can't remember the name of, and a few others.

@Hosch250 Pink Floyd, Jethro Tull, Cream, ZZ Tops.

Selects of Marshall Tucker (they draw heavily on Classical style), not even sure they are rock. I've heard of Brian Adams, but can't think of any off-hand.

Big into folk, classical (especially dance and opera), and marches (I think it's the emphasis on the rhythm along with the tune).

Not very into gospel, although I like certain recordings, mostly by folk/country artists or classical groups.

Mathieu Guindon

#FunFact: Brian Adams is Canadian

7:46 PM

@MathieuGuindon So is Gordon Lightfoot.

Mathieu Guindon

except one of the two is known :p

Yeah. Pity Brian Adams career flopped like that.

@MathieuGuindon get a potted plant to snark at ;)

Mathieu Guindon

goes back to writing unreadable C# code riddled with var

right... I wanted to write code as well ...

being a mod has it's perks. and then you get drawn in to drama around the network and it's three hours later and you're chatting about how a certain chatroom got nuked

7:50 PM

Another one just went?

TBH, they should have starting being stricter years ago, and there wouldn't be this drama.

I could never be a mod. I salute those that help by being one.

I think I could get elected, but I'd be involuntarily removed for snark or something pretty quickly, so I won't try.

@Hosch250 nah

Which is why I'm glad there are people who are qualified to do it.

@Hosch250 mumble mumble

7:58 PM

Test Name:	MemberNotOnInterface_DoesNotReturnResult_BangNotation
Test FullName:	RubberduckTests.Inspections.MemberNotOnInterfaceInspectionTests.MemberNotOnInterface_DoesNotReturnResult_BangNotation
Test Source:	C:\GitHub\Rubberduck\Rubberduck\RubberduckTests\Inspections\MemberNotOnInterfaceInspectionTests.cs : line 189
Test Outcome:	Failed
Test Duration:	0:00:00.254

Result StackTrace:
at System.IO.__Error.WinIOError(Int32 errorCode, String maybeFullPath)
   at System.IO.Directory.DeleteHelper(String fullPath, String userPath, Boolean recursive, Boolean throwOnTopLevelDirectoryNotFound)

pls don't tell me we are actually writing to a real directory?!?

:)

Yes, we do that in a couple places in the tests, IIRC, LOL.

:(

« first day (1425 days earlier) ← previous day next day → last day (1755 days later) »

Transcript for

May '181

VBA Rubberducking

This chat has moved to Discord: discord.gg/MYX9RECenJ

antlr com-interop parsing rubberduck vb6 vba whiskey-tango-foxtrot

join this room
about this room

00:00

06:00

12:00

18:00

all times are UTC

site design / logo © 2024 Stack Exchange Inc; legal

mobile