@MathieuGuindon I know there are multiple solutions, but I recall having seen SyntaxHighLighter used to syntax highlight VBA. The parser/approach is likely not as robust as Rubberduck's, but it might be useful for the blog. It's on GitHub (So we should be able to make a custom brush if required) and WordPress (so you should be able to add it easily).
In fact, now that I think about it, the [Code] blocks you mentioned are probably designed to use it?
My car, interestingly enough, actually tries to predict your actions. So it can guess if I'm going to be slowing down and cut the gas before I take my foot off.
> Yeah, I could reword it a bit to clarify the purpose.
RE: reg file, I did mull about that as well, thinking that it could benefit having a shortcut in the start menu to "register" Rubberduck add-in, so when all-user install is run, everyone at least gets that shortcut.
OTOH, it's easy enough to configure the installer above to run in "register-only" mode by detecting that it's already installed, skip the install and simply execute the VBE addin registration.
@ThunderFrame didn't @IvenBach already included that in the copy to clipboard functionality?
@MathieuGuindon I would like your opinion. Generally, we prefer self-documenting code over comment and I'm a solid fan of it. But.... freaking pascal.
I'm seriously thinking I have to comment a fair more because next year, even if I'm here, I'll be all "WTF this guy smoking"
I'm thinking of commenting stuff like how this global variable should e used when it should be mutated, read, etc. that kind of information that is not readily available.
#2, Either use #if DEBUG or remove the build debug check. TBH, I'd find it less confusing if it was done with #if DEBUG and I had to enable it for release builds.
Rubberduck version: Version 2.1.6648.34736
Operating System: Microsoft Windows NT 10.0.16299.0, x64
Host Product: Microsoft Office 2016 x86
Host Version: 16.0.9029.2253
Host Executable: EXCEL.EXE
but that's a interesting thought - I have to think - what'd be the best way to communicate to the descendants that this code was never meant to multithreading? Obviously an attribute isn't going cut it.
I learned another useless programmning language. Maybe I should go for COBOL next.
well, in fact, I wished I had learnt COBOL when I was a kid. I'd be making lot of dough by now. :D Maybe I'll get lucky and VBA'll be the next COBOL. unlikely :p
@this Fixing broken/legacy VBA pays pretty well.... "We have this critical business function, and then 2 buses came along at once, and obliterated our support team. It runs daily, but it's now taking 14 hours, but our SLA requires 12 hours".
VBA Professional Rubberduck steps in, and reduces the runtime to 16 seconds.
@this I think I used Turbo Pascal in my final year of high school.... I remember using Records... Also wrote a TLB in Delphi in about 2000 (I'm told it's still being used today), but that was surprisingly drag-and-drop + wizards.
I run VirtualBox VMs instead of MS VMs, because I love Oracle VirtualBox lets me run Mac OS (and you can'r run VirtualBox while MS' HyperVisor is enabled).... My hardware will likely melt from having Oracle software on it before it fails because of Inno Setup.
Waiting for 300 MS updates to apply - testing is probably better done once those are done, and is likely to be a while...
I still remember the first time I saw bootup screenshots of a VM, and being amazed. Went and bought a copy of VirtualPC (before MS bought Connectix), which you could run Linux on, until MS bought it and removed that feature bug.
> Consider the following code, where `Sheet1` is passed as a `Worksheet` object: ```vb Public Sub Foo() Debug.Print GetQuotedWorksheetName(Sheet1) End Sub
Private Function GetQuotedWorksheetName(ByRef myWorksheet As Worksheet) As String GetQuotedWorksheetName = """" & myWorksheet.Name & """" End Function ```
`Worksheet.Name` is the only property accessed within the `GetQuotedWorksheetName` function, so the function's parameter could be changed to `String`. That makes the fun
> As mentioned in #3834, the rewriter leaves some redundand whitespace after removing pieces of code. The quick fix tests for the linked PR expect that whitespace, but it just doesn't feel right.
@Hosch250 one more question for you: I like applying my skills where I have strengths, but I also enjoy learning new things and taking on new challenges. What opportunities can MS/this role offer me to leverage my current and future skillsets, so that I can be most productive and grow at the same time?
And if you're feeling brave/cocky/brazen: I've always found school/college to be quite easy, and I feel like I outperform in my current role. What challenges and opportunities can Microsoft offer me if I can prove my abilities and exceed expectations?
^ That second one might require some finessing, and may or may not suit your personal interview style. It's a fine line between unchecked arrogance and unbridled ambition....
@this - I love trashing COBOL as much as the next guy (maybe even more so!). However, I had to take a COBOL class in college (yes it was that long ago) and it was taught by the only instructor in the CS dept who'd ever actually coded for a living. Her approach was totally different - "Sure, every test is open book - nobody memorizes all this stuff. However, you have to know where in the book to look, because there's never enough time to look everything up - on the test or at work."
It also led to my first post-college job, so I guess it's got that going for it, too.
@FreeMan Ha, that's funny - I know a former Access MVP who also had similar classes like your -- all tests were open book but you better know where to find it. She argued that it helped made her a better programmer because it taught the importance of reading and using the documentation... as opposed to randomly bumbling around.
OK, possibly dumb question: Thank heavens for rubberducking! as I was typing up the potentially embarrassingly dumb question, the answer hit me. Thanks guys! :)
> I see that the RequiredLibrary attribute is now used. Great! However, I don't see an additional attribute or flag to indicate that the inspection should only run only when Excel is the host (and not on any other hosts)? This will generate broken code on non-Excel hosts that references Excel.
Dim downloadStatus As WebDownloaderStatus
Set downloadStatus = New WebDownloaderStatus
Dim Downloader As IWebDownloader
Set Downloader = SnapSurveyWebDownloader.Create(Chrome, downloadStatus.downloadStatus)
^^^^^^ Seems somewhat repetative.
Naming stuff is hard
'Why yes, I did stutter, why do you ask?
Is it reasonable to have status codes of, say negative values = all is good, continue and positive values = something's gone wrong, back off? The have Public Property Get okToContinue as Boolean that returns If Status < 0 then True Else False?
I'm good with either way... is it reasonable for the FYI section to be an indicator of where in the process I am, so that if I hit an error, I can log the current value of status then interpret the tea leaves later? So, not really a warning status of >0, but more of an informational you are here.
> @bclothier Yes, that is not implemented yet. I'll probably need to spend some time figuring this out. I don't think this functionality is avaliable to us at the moment. I guess we can either postpone merging this PR until it is ready or add such functionality in a future PR.
if it's any consolation, the powershell script is indeed used to help drive the building of the "semi-real" installer.
TBH it's a bit of Rube goldsberg machine, if you ask me.... :\
we have what.... a powerscript shell that runs in post-build event of a project that executes a DLL built by the same project to generate more files which AV then runs iscc and the iss script #include the autogenerated file into the main .iss script, then builds it.
TFW you get a bug ticket and spend 2 hours to find the data is wrong because someone had A{rest-of-name} FooBar instead of A{some-other-name} FooBar in their data...
And they were checking the data against the second, but it was seeing the first.
@Chronocidal remember that InStr has a Compare optional parameter that defaults to vbBinaryCompare. Instead of LCase-ing the two strings, why not just provide a vbTextCompare parameter and make 1 function call instead of 3? — Mathieu Guindon46 secs ago
> In this example, the code wouldn't work, because the runCount would be reset each time. Of course, the simplest solution would be to declare a global variable instead, but in some cases, I want to avoid that for the sake of simplicity, encapsulation or other reasons.
So, is there any way to have the local variable persist after the procedure has finished running?
You'd think if they know enough to recognise there is a different between persisting a local variable and a global variable... they would be able to use google with the right search terms
yeah, but take the fact that searching for VBA often turns up VB.NET results, then the fact that static in VBA doesn't mean what static means in every other language, ...
...you get a legit useful question to put up on SO
Do not use Static (imo).
Use Public on module level instead of is more preferable.
But much more preferable will be to pass counter to function ByRef.
.
@CallumDA it makes a field or member belong to the type (class), rather than an instance of it.
so if you have public static void DoSomething() on Class1, then elsewhere you can invoke it like Class1.DoSomething();, without needing an instance of Class1
and a static field would have a "shared value" in all instances of the type
Do not use Static (imo).
Use Public on module level instead of is more preferable.
But much more preferable will be to pass counter to function ByRef.
.
VBA Rubberducking
VBA Rubberducking