The 2nd Monitor

I am aware that this is not necessarily the purpose of this and may not justify the effort, but would it be possible to get a list of questions closed with a specific close reason, in particular the custom ones? I am asking because the list of custom close reasons on German Language SE revealed ...

Gareth Rees

8:16 PM

@200_success That's a great answer! I don't understand why it's been downvoted.

RubberDuck

It's kind of amazing. I spent an afternoon working through an MVC tutorial, and all of a sudden URLs I see all the time suddenly make sense.

http://codereview.stackexchange.com/tools/question-close-stats?daterange=last30‌days

I get it now.

?daterange=last30‌days is a parameter sent into some code that tells the database which information to return.

Eureka!

Mast

@RubberDuck It's called a query

RubberDuck

Yes. Yes it is.

Mast

@RubberDuck Did you know you can extend it?

RubberDuck

But so is this actually.

http://meta.stackexchange.com/questions/257462/can-i-advertise-stack-exchange-s‌ites-on-my-web-site

257462 must be it's dbkey

@Mast What do you mean?

Phrancis

8:20 PM

@RubberDuck It is

RubberDuck

@QPaysTaxes right. So, /questions/ is the controller.

Mast

@RubberDuck ?daterange=last30‌days&postedby=Rubberduck

It's pseudo, but you get the gist.

Simon André Forsberg

@RubberDuck Wow, you didn't know this before?

@RubberDuck or /q/

Mat's Mug

@RubberDuck Pro ASP.NET MVC, chapter 1 ;-)

2

Mast

@Mat'sMug MVC is not the only part you encounter it. When you start handling HTTP requests and parse urls, you get to know some slang

RubberDuck

8:22 PM

@SimonAndréForsberg No. The only web dev I've ever done was hacked together by using Ruby to modify raw HTML files.

@Mat'sMug I wish I had stars buddy.

Phrancis

@RubberDuck RSA

Mast

@RubberDuck RSA

@QPaysTaxes Now I'm out

Mat's Mug

@SimonAndréForsberg I presume /q is just another route to the QuestionsController :)

Simon André Forsberg

Well, I guess it's never to late to learn MVC. As long as you're not learning it in PHP...

Mat's Mug

asp.net-mvc-rocks

Mast

8:23 PM

@QPaysTaxes It's what you get with all those people yelling TS

Mat's Mug

@QPaysTaxes phpoop?

RubberDuck

@Mat'sMug ahem asp.net-mvc-is-magic

I don't think I would like all the auto-generated code once I got into it though. It's not very dry, and a bit hard to make changes to.

Mat's Mug

@RubberDuck global.asax.cs is your friend

Jeroen Vannevel

You shouldn't and won't use it either

Mat's Mug

@JeroenVannevel unless you want to change the controller factory and wire up Ninject

Jeroen Vannevel

8:27 PM

You don't need it either. Create a class that inherits from Controller, use [Route] attributes and a few others and you're done

Mat's Mug

@JeroenVannevel dirty dirty dirty

Jeroen Vannevel

Why would you create controllers through ninject?

Mat's Mug

because your composition root is in global.asax

so you can constructor-inject them with their dependencies

Jeroen Vannevel

You don't need to do anything special to your controllers for that

Mat's Mug

like, some IRepository with an instance-per-request lifetime

Jeroen Vannevel

8:28 PM

You only have to register their dependencies in ninject

nothing to do with controllers

Mat's Mug

if you make a parameterized constructor to a controller, the default controller factory won't know what to do with them

Jeroen Vannevel

yeah it does :P

Mat's Mug

well that changed...

I'm not very up-to-date with that, TBH

but Ninject does come with a NinjectControllerFactory

Mast

@Mat'sMug If you have no idea what it means, that must sound spaced out

Mat's Mug

IKR

Phrancis

8:31 PM

Does Ninject have anything to do with ninjas?

Mat's Mug

YES!

ninject.org

Jeroen Vannevel

It's been a year since I've used Ninject but Unity can resolve any constructor's dependencies perfectly fine, including controllers'

just make sure that every type registered as a dependency either has a parameterless constructor or takes a parameter that is registered itself

Mat's Mug

parameterless constructor -> not constructor injection == not ideal DI

Jeroen Vannevel

If it has a paramless constr it just means there's nothing to inject

that doesn't contradict DI

Mat's Mug

unless you're newing up stuff

Jeroen Vannevel

8:34 PM

yeah but that's just bad code

Mat's Mug

nothing to inject in a controller doesn't sound very likely, unless SRP is taking a serious beating

@JeroenVannevel that

Jeroen Vannevel

I'm just saying the general way for Unity to resolve dependencies: if the type has a paramless constr, it uses that. If the constr takes a dependency, it will try to resolve that

Mat's Mug

how does it know to resolve a controller's dependencies if it's not the container instantiating it?

I'm missing something here

Jeroen Vannevel

Take a look here: github.com/Vannevelj/Moviepicker/tree/master/MoviePicker/WebApi

AccountController: github.com/Vannevelj/Moviepicker/blob/master/MoviePicker/WebApi/…

Has a dependency on IUserRepository

Which is here: github.com/Vannevelj/Moviepicker/blob/master/MoviePicker/…

IUserRepository has a dependency on MoviepickerContext

Which is here: github.com/Vannevelj/Moviepicker/blob/master/MoviePicker/…

Notice the parameterless constructor

All of this comes together here: github.com/Vannevelj/Moviepicker/blob/master/MoviePicker/WebApi/…

RubberDuck

@QPaysTaxes I just bought it on VHS

Jeroen Vannevel

8:39 PM

StartUp.cs is the replacement of Global.asax.cs

container.RegisterType<IUserRepository, UserRepository>(new TransientLifetimeManager());
container.RegisterType<IMovieRepository, MovieRepository>(new TransientLifetimeManager());
container.RegisterInstance(new MoviepickerContext());
configuration.DependencyResolver = new UnityConfig(container);
app.UseWebApi(configuration);

Mat's Mug

vhs-is-magic

Jeroen Vannevel

And that's it. Now the framework will automatically try to resolve using these types

Mat's Mug

you never heard of betamax

yup

nobody does

Mast

@Mat'sMug And we all know why Betamax was doomed.

Mat's Mug

there once was a war between beta vs. vhs. kinda like hd-dvd vs. blu-ray

beta had better quality though

@JeroenVannevel hmm

@QPaysTaxes I'm your age.. bit-shifted 1 bit left

yeah

oh look, it's 4:44!

well, on my screen at least

yeah 16:45 ...I meant PM

I'm not up at 4:44AM.. not 16 anymore

@JeroenVannevel I can't help thinking that code would be much simpler with Ninject conventions and wiring-up in global.asax.cs

but I'm just sold to Ninject

and ReSharper

Jeroen Vannevel

8:48 PM

the global.asax was removed to work with Owin, that's separate from Unity

Mat's Mug

and C#

@JeroenVannevel well, that is how parameterless constructors work!

Jeroen Vannevel

hmm?

Mat's Mug

right? it's not using the default controller factory

it's Unity instantiating the controllers

wait..

public AccountController(IUserRepository userRepository)

I should have started with the first link lol

so yeah, you are constructor-injecting controller dependencies, and it is the IoC container creating the controller instance.

Mast

TTGTB

Mat's Mug

i.e. we're both saying the same thing, you in unity, me in ninject

Mast

8:52 PM

night all

Mat's Mug

'night!

RubberDuck

tsk tsk SE. Using tables for formatting...

Jeroen Vannevel

I think I'll get started with this issue github.com/Vannevelj/VSDiagnostics/issues/59

Get some more C# 6 goodies in there

Mat's Mug

@JeroenVannevel writing a little R# of your own?

Jeroen Vannevel

You haven't heard?!

Mat's Mug

8:55 PM

Jeroenizer

Jeroen Vannevel

There are.. 14 or 15 refactorings already

Take a look at the ReadMe

Mat's Mug

damn I want that little popup dropdown in Rubberduck

Jeroen Vannevel

For fancy gifs you can look at my blogpost: vannevel.net/2015/05/24/introducing-vsdiagnostics

There have been 6 new diagnostics since writing that

the little popup dropdown also shows an entire preview of the changes

but I cropped that out since it would make the text unreadable

Mat's Mug

@JeroenVannevel pretty neat!

@JeroenVannevel those would be quickfixes in RD-speak though ;-)

Jeroen Vannevel

Silly duckspeak

You can add it as a NuGet package, by the way

Just putting that out there

Get some of that VS 2015 & Jeroen goodness in there

Mat's Mug

8:58 PM

refactorings

Captain Obvious

0

Q: HTML/CSS MS Word style navigation ribbon

I made a MS Word style ribbon header, but it all seems very hacky, and I'm not really sure how to simplify it to something more reasonable. Rolling over one of the navigation icons changes the related z-index to positive revealing it. Then, on hover it changes its own z-index to remain positive k...

Duga

9:16 PM

If you add your shell script this might also be better suited for Code Review since you do have running code, just slow. — Marc K just now

Duga

9:37 PM

There's a code review Stack Exchange site. This would fit there better — Sami Kuhmonen 48 secs ago

Jeroen Vannevel

I really want to use this null propagation more often but not using it often just feels.. better

Duga

Please provide your code of what you have tried. Also, by the sound of it, this seems like a question more fit for codereview.stackexchange.com — Dom 30 secs ago

RubberDuck

I haven't used it, but I hear it is @QPaysTaxes

Ya know what I want to use, the new & improved auto-properties.

What did they call it, auto-property initializer @JeroenVannevel? It's bad ass. I like it. Moves all that junk from the constructor up to where it's defined.

Mat's Mug

9:54 PM

TTQW

Duga

Welcome to Stack Overflow. Please take the time to familiarize yourself with the Stack Overflow help file, which will help you understand what kinds of questions are appropriate for this site. This site is intended to help you obtain answers to specific programming questions, as opposed to providing tutorial, design or code review assistance. — MarsAtomic 7 secs ago

Captain Obvious

0

Q: Count keys in a file

Below classes are to count keys in a text file. The file has key, value pair like below examples John, 12 Sara, 2 Adam, 19 John, 1 Adam, 3 and the main class FileKeyCounter prints output as The total for John is 13. The total for Sara 2. The total for Adam is 22. It consists of 3 classe...

java

Jeroen Vannevel

@RubberDuck I think they scrapped that feature

wait.. No it's not!

I thought they removed that

creates issue

@RubberDuck: I think I misinterpreted your feature request on github then.

I thought you just meant that we now can omit private set; from an automatic property

Could we do that before?

No we could not!

Captain Obvious

0

Q: Packet Scout (show traffic on specified port)

I've just made a github repository for a simple packet scout in C#. It is my first time using C# (other than Neoaxis 3D engine logic) since I am more familiar with C#. Below is the main bulk of the code that I'd like to improve. You can see all of the code here. If you have any suggestions please...

c# beginner

Duga

10:18 PM

I think you should ask this on Code Review or Game Development. Here it is off topic. — Burkhard 51 secs ago

RubberDuck

@JeroenVannevel =;)-

Jeroen Vannevel

@RubberDuck: a property which is only set in the constructor can use the new readonly property syntax

which is basically: omit the set; part

That's different from an auto-property initializer

and it's also different from an expression-bodied property

there are surprisingly many changes to properties in C# 6

It'll be easier once I look into it more closely and define a shitload of test scenarios to pass

Mat's Mug

10:33 PM

Aaah, semantically immutable auto-properties! At long freakin' last!!

RubberDuck

So long as it hasn't changed since I last looked, this should be valid. I might be able to write a few test cases later, but I'd have to install 2015, which may take some time.

github.com/Vannevelj/VSDiagnostics/issues/…

Jeroen Vannevel

@RubberDuck: those are two distinct features combined though

That's what caused my confusion

If it's about the default value, you should have int SomeProp { get; private set; } = 100;

If it's about the readonly property, you should have int SomeProp { get; }

Jeroen Vannevel

10:55 PM

Ring the churchbells, children

RubberDuck

I don't know @JeroenVannevel. According to this they're very closely related. msdn.microsoft.com/en-us/magazine/dn802602.aspx

Jeroen Vannevel

Properties and methods have an analyzer + code fix that allow them to be turned into an expression-bodied member

@RubberDuck They're work together perfectly but they're still different features, no?

RubberDuck

So, if the read only property is using a literal, (or anything that's not being injected I imagine) you could SomeProp { get; } = 100;

Jeroen Vannevel

You don't even need literals

public IList<int> MyProperty { get; } = new List<int>();

That works as well (I tested it)

RubberDuck

Right.

Jeroen Vannevel

10:58 PM

yes

RubberDuck

I'm not sure if I'd call them separate features or not. It's so intertwined.

I guess, yes. They're different features, because you can still get a read only auto property without an initializer.

Yes. A lot like Ruby's shorthand.

What is it? :attr or something?

Might have that colon backwards.

class Customer

def initialize
@age = 18
end

attr_accessor :age
end

sigh. I'm on my phone.

Jeroen Vannevel

@RubberDuck Well, if they can be used without eachother then I consider them separate features

RubberDuck

Yeah. That ^

Jeroen Vannevel

Oh okay, you said that

I was lagging behind

RubberDuck

No worries.

> Downloading VS 2015

Mat's Mug

11:16 PM

@RubberDuck FYI I'm not updating for another couple of months, when I ~~convince my wife that I need to~~ get a new laptop

RubberDuck

It's not like I'm uninstalling 2013. I still need it for RD. =)

Jeroen Vannevel

110 passed tests, 1 failed test. That one bug that I haven't bothered to fix for weeks..

RubberDuck

Rainbow Dash!

Jeroen Vannevel

It's really starting to annoy me now

RubberDuck

(Shut up, I have daughters)

Jeroen Vannevel

11:21 PM

maybe-you-should-though

RubberDuck

That is annoying Jeroen.

QPays, my wife and I have found ourselves watching it for hours after the kids lost interest.

Mat's Mug

@QPaysTaxes Rubberduck

lol

Rubberduck != @RubberDuck

RubberDuck

True, but I could still be an AI. You'd have no idea one way or the other.

Mat's Mug

fail

Jeroen Vannevel

> Choose two branches to see what's changed or to start a new pull request.

Nowhere an option to start a new pull request

Mat's Mug

11:25 PM

Mwahaha

RubberDuck

@JeroenVannevel gotta compare two branches before you can do that.

Jeroen Vannevel

I have two branches compared. Now what?

Do I click on something?

How did I do this the other times?!

oh hey

must've been a visual bug

I changed from Unified view to Split view and now the open pull request dialog shows

github.com/Vannevelj/VSDiagnostics/pull/61/files?diff=unified

Quick, review me

It's C-girl all over again

RubberDuck

Tests look good. Couldn't say if they're thorough or not.

Jeroen Vannevel

Oh, I forgot an important one. A property with a getter and a setter

somewhat common

RubberDuck

Is there a reason these are all internal rather than private? github.com/Vannevelj/VSDiagnostics/pull/61/…

Jeroen Vannevel

11:35 PM

Yeah, I'm accessing them inside the CodeFix and the Tests (see the DiagnosticResults)

Though maybe the Rule one can be made private

I'll look into it

Phrancis

@QPaysTaxes Heard about Lisp? ;)

RubberDuck

@QPaysTaxes barely

Captain Obvious

0

Q: C++ Observer design pattern implementation

I'm using the Observer design pattern in order to manage events in the game I'm currently developping. I based myself on the implementation demonstrated at http://gameprogrammingpatterns.com/observer.html but improved it in order to ease it's use. Here it is : class Observer { public: virt...