The 2nd Monitor

Dan Pantry

18:00

zalgo he coMES

Jeroen Vannevel

-> is a C# operator

the more you learn

Dan Pantry

@JeroenVannevel wait, what

Jeroen Vannevel

msdn.microsoft.com/en-us/library/s8bz4d5h.aspx

Dan Pantry

uh

isn't that just because it's unsafe

and thus needs a pointer deref operator? <:

Jeroen Vannevel

yeah it's only valid in unsafe context

still, it's C#

I think

I'm not 100% sure what that is called but I think that's still C# area

Duga

18:05

possible answer invalidation by thepace on question by thepace: codereview.stackexchange.com/posts/127889/revisions

200_success

@Phrancis We don't have to parse everything — we just need one offending line to trigger the warning.

Phrancis

Hm, maybe if we just looked for a curly brace inside <p>}</p> tags that would be simple enough?

It would cover most languages that use curly braces, which is a lot of them

Captain Obvious

0

Q: object.ErrorHandler.js - Hanldes erorrs on bubble and capture phase

Is this a good way to handle errors ? Right now I'm using it to capture image errors, but I want to expand it a bit. Can I catch run time errors this way? $A.Obj.create('private', { Name: 'ErrorHandler', init: function () { // image errors don't bubble - use capture phase ...

javascript

0

Q: binary search in python 3

I have this error and i do not know how to solve it I am very sure the logic is correct I am using Python 3 in binary_search if (input_array[mid] == value): IndexError: list index out of range def binary_search(input_array, value): low = 0 high = len(input_array) - 1 while (low <= high): ...

python algorithm python-3.x data-structures

200_success

Yes, I think that looking for } on its own line would catch 80% of the offenders.

@Duga A bit of a mess, but I think we should allow the edit.

Joe Wallis

I thought about that, but then thought about the tests for : and then...

Phrancis

18:18

@JoeWallis Yeah, the reason I suggested that is that it's pretty rare that someone would purposefully use a curly brace in a paragraph, as opposed to say, : and then

Captain Obvious

0

Q: Typelist with extractor

In which ways the code below can be improved? #ifndef TYPELIST_H #define TYPELIST_H #include <cstddef> struct NullType; template <typename Head, typename ... Tail> struct TypeList { using head = Head; using tail = typename TypeList<Tail...>; }; template <typename Single> struct TypeL...

c++ template-meta-programming

0

Q: object.Storage.js - creates a namespace for Storage

This basically creates a namespace for Storage among other things. I'm looking for feedback on the structure of the code. /******************************************************************************************************************(120) ** Storage ** - Dependencies - None ** - Basically c...

javascript

Quill

I'm afraid this question does not match what this site is about. Code Review is about improving existing, working code. Code Review is not the site to ask for help in fixing or changing what your code does. Once the code does what you want, we would love to help you do the same thing in a cleaner way! Please see our help center for more information. — Quill 15 secs ago

200_success

@200_success On closer inspection, it still does not compile.

Joe Wallis

@Phrancis I know, it's just a shame that :, and possibly then, are fairly common

Quill

1

Q: Lua and complex numbers

I try to use complex numbers in a lua program. I need to parse a table of complex numbers printing side by side the name of the complex number and its value. Can't get through it : require "complex" a = complex.new (0,1) b = complex.new (7,1) c = complex.new (1,6.4) t = {a="a", b="b", ...

lua lua-table

This looks like Broken Code

200_success

18:21

… which is why I suggested looking for def, which is is not a common English word.

Dan Pantry

@Quill It is not OP's code

Joe Wallis

I'll just stop digging

Quill

Dan Pantry

This is not your code. Original source. We only review code that people own; asking for help understanding code that you did not write is off-topic. — Dan Pantry 36 secs ago

I flagged it because I don't have VTC rights

Quill

I saw it

@DanPantry You need to answer some more... ;-; or you'll never get a pretty rubberduck

Dan Pantry

18:24

¯\_(ツ)_/¯

Ok but how to iterate all the table to print a yields 1i ... — Tarass 4 hours ago

This tipped me off about him not understanding it

Quill

codereview.stackexchange.com/review/first-posts/56428

Duga

If you have working code which merely needs improvements then you're probably in the wrong place with this post. Code Review is where they handle existing / working code and do their upmost to improve it in terms of speed, security, sustainability, and longevity. Give it a try. They're good! — Ralph 44 secs ago

Dan Pantry

thanks-santa

Captain Obvious

-1

Q: TicTacToe AI in Javascript

I'm having trouble implementing this TicTacToe AI that I found here. I'm relatively new to javascript so I'm sure I'm doing something wrong with variable scoping. choices = { 0:'#ul', 1:'#um', 2:'#ur', 3:'#ml', 4:'#mm', 5:'#mr', 6:'#ll', 7:'#lm', 8:'#lr' } function getGrid...

javascript

Dan Pantry

wtf git syntax is confusing sometimes

$ git grep  --exclude '*.csproj' --pattern System.Web ./

I would have expected this to be valid syntax

200_success

18:31

@JoeWallis Do you still want to take another crack at revising the close reasons this week?

Joe Wallis

@200_success Dunno, it'd just add to the confusion, I got to the wording of the off-topic reasons, but don't think it's that solid, I'm sure you'll find a fault in no time.

> How to classify a question that asks how to modify working code to add a feature?

Makes me wonder if I'm reading things wrong again. ):

Quill

Any input is good input, don't worry about adding to confusion

Dan Pantry

I just removed all System.Web directives in a DTO project.

 86 files changed

There are 86 DTOs.

Why on earth did every DTO have using System.Web and not use it?

Captain Obvious

0

Q: Can someone make my code less cluttered and more efficient? (Java)

This is my code below, how can I make it more presentable and more efficient? It's an encryption project I'm working on and could anyone maybe suggest on how to make it more complex but not to complex it's not decipherable? I'd appreciate it! Thank you! for (int loop = 0; loop < password.len...

java

Quill

@DanPantry You guys should get resharper

Dan Pantry

18:39

I AM using ReSharper

That's how I removed it all

Quill

;-;

Dan Pantry

It would be amazing to hve it in work but our computers simple are not powerful enough to use it on our work laptops

Quill

If you run VS2015, Jeroen's VSDiagnostics is a good alternative

Dan Pantry

interestingly enough, VS with ReSharper is only using 570MB of memory - it regularly reaches 800MB on my work PC w/o ReSharper.

Phrancis

@DanPantry sounds like copy-pasta code

Dan Pantry

18:41

@Phrancis Copy pasta? In our code? nah, that would never happen..

(go ahead and search for "copy pasted code" in the transcript with my name)

@Quill I may give it a go if it is more lightweight

Mat's Mug

@Quill will be, once it's packaged as an actual add-in and not a nuget package :)

Dan Pantry

our disk speeds are dog slow and I only have 12GB of RAM. SQL server, VS, WebStorm and Node have to share that

Quill

@Mat'sMug I know... I only have it in my projects at work ;-;

Dan Pantry

amongst the OS, the McAfee antivirus that likes to start scanning at random times which I can't force-kill..

Quill

@DanPantry chat.stackexchange.com/…

Dan Pantry

18:42

Mar 13 at 16:18, by Dan Pantry

Daily reminder that copy pasted code is evil

Phrancis

@DanPantry 11, +5 for "copy pasta" (not counting the one you just posted)

Dan Pantry

Hmm, maybe my self restraint is better than I thought :-)

But, yes, R# is too good.

If our laptops get better I'm going to ask our boss to get it for us. the license cost is peanuts for projects like the ones we work on

Mat's Mug

@DanPantry try renaming a namespace without it (and without crying)

Dan Pantry

@Mat'sMug I know, I just moved all files from Old.Project.Namespace to New.Project.Namespace - about ~120 files. NP, did it nearly instantly

though you could use sed to do that (maybe)

Mat's Mug

I renamed RetailCoder.VBE to Rubberduck about two months into the project.. I know :)

Dan Pantry

18:45

Ah yes, I forgot RetailCoder was your old name

Quill

Before he earned the Mug of Destiny

8

~ electric guitar plays in the background

Dan Pantry

@Quill There should totally be a band called "Tenacious VB" with that as their first album

5

Mat's Mug

lol

Quill

blog.marcgravell.com/2016/05/…

Dan Pantry

Oooooh, I pressed "Remove redundant code in solution". That was a mistake. Even my PC isn't good enough for that.

Joe Wallis

19:02

I was thinking of asking a CR question, but it may belong to my friend. They wrote it, and then I improved it, would that be allowed on CR? (Or is this a question for meta?)

Phrancis

@JoeWallis Are you an owner or maintainer of the code?

Joe Wallis

Don't really know tbh...

It's not used any more, so there's no maintainers. And I'd guess their the owner, as they made it.

Jeroen Vannevel

ugh, no nitpicking

just allow it

If you've actively worked on it then that's fine in my eyes

Hosch250

@DanPantry Must have a lot of redundant code.

Mat's Mug

@JoeWallis you improved it, and they know, therefore you maintain it. the "spirit" of the rule is to avoid things like "look at this piece of crap code my coworker wrote" and "please let me know if I should fire the employee that committed this".

Joe Wallis

19:09

@Mat'sMug Oh, didn't know that's what maintain meant. I'll ask them if I can post it anyway, just wanted to get my JS reviewed again.

200_success

The main concerns are copyright, deflecting criticism by disavowing the code, not knowing how the code works, and heaping judgement on someone else.

Mat's Mug

^

Joe Wallis

Oh, well I don't think I'll do any of those things, I changed it the best I know. By "deflecting criticism by disavowing the code" I don't go 'well that's not my code anyway'?

200_success

I'm sure you wouldn't do that. Go ahead and post.

Joe Wallis

Thanks all, I'll wait for them to get back, to check if it's alright with them, as in my eyes their the owner.

Dan Pantry

19:22

middleware + [Authorize] attribute = super elegant authorisation in Katana. I'm amazed :D

Duga

I am trying to create a post deployment script which needs to be formatted as specified to pass code review :( — Peekay 9 secs ago

Jeroen Vannevel

19:44

0

A: Why ExpandoObject doesn't work properly with Guid converted to string?

The GUID parsing will end up with the same string (value) as just using a string literal. The difference however is the way it is stored in the dictionary: it's of type Dictionary<string, object>. This means that the Object class its == operator will be used which does a reference equality chec...

First time using expression trees

got things cooking!

Captain Obvious

0

Q: TypeError: Cannot read property 'session' of undefined

I am working with the angular--fullstack generator, and everything has ran smoothly until now. I do not know what I changed, but now my server task is halted by this error. /Users/User/APP/server/config/express.js:28 secret: _environment2.default.secrets.session, ...

node.js angular.js express.js angular-bootstrap

RoboSanta

20:01

Naruto answer; accepted non-selfie answer with 0 score: Drills: A JavaScript web application that creates customizable workout routines and reads commands aloud while keeping track of time javascript jquery html css jquery-ui

Ripe zombie; open question with answers, at least one answer having score 0, no answer having score > 0: Displaying a Google map with multiple complex markers and unique info windows javascript ruby ruby-on-rails google-maps

Phrancis

@JeroenVannevel you got almost 30K rep on SO, way to go!

Jeroen Vannevel

big baller

Duga

20:35

post your Q in CodeReview codereview.stackexchange.com — Tasos 37 secs ago

If the code is working as intended and the author is looking for review & improvements, this would be OK on Code Review as suggested. — Phrancis 39 secs ago

Duga

20:50

I'm voting to close this question as off-topic because this is just a giant code dump and a request for review, see code review.stackexchange.com for code reviews. Otherwise ask a specific question about a specific block of code and never post huge code dumps like this and ask for discussion/opinions on improvement, that is off-topic. — Jarrod Roberson 37 secs ago

If the code fully works as intended, it could work on Code Review, but be sure to read the on-topic page there first. From the description it sounds like something is not quite right with the code, but it is unclear if that's the case or not — Phrancis 56 secs ago

Phrancis

21:05

Trailing spaces in XML nodes can probably break things right? like <FirstName>CARMEN </FirstName>

Phrancis

21:31

TTQW

Joe Wallis

Is that time to quit working?

Vogel612

yeap

Joe Wallis

Thanks )

@200_success The Discuss Close Reasons chat is frozen so I can't talk in it.

200_success

Oops, fixed.

Duga

21:48

thanks for the advice. It seems codereview would be better for my query. — Sarah 36 secs ago

Nothing better comes to mind immediately. This might be a better fit on codereview.stackexchange.com though. — markspace 39 secs ago

Captain Obvious

22:03

0

Q: my code generates jquery mobile "pages" dynamically

I'm doing a college assignment with JQuery Mobile and JSON. Best practice with Jquery mobile is my main concern. My html starts off with a main 'page' (data-role="page") inside a div with id="all-pages". I use Jquery's.getJSON method to connect to a local json file called profiles.json. prof...

jquery html json mobile

Duga

22:27

Since you have posted this question on Code Review, you should just delete this Stack Overflow question. — 200_success 1 min ago

Did you mean to post this on Code Review? — Lightness Races in Orbit 42 secs ago

Duga

22:48

Does the code work? Does it do what you want it to do, but you feel there could be a better way of doing it and need feedback? It may be better suited on Code Review. If there is a problem in your code, or it's not doing what you want, then you need to identify that problem and tell us, so we know how to help. Currently, the question doesn't make much sense, but you can always edit it! — Tas 23 secs ago

Captain Obvious

0

Q: ATM machine transfert

So I have a project in my computer programming class which basically stimulates an ATM machine. The program is in C language. I did most of the program but I am still stuck with two parts: the first is transferring money from your balance to another person's balance, and the second part is when t...

c

Eᴀsᴛᴇʀʟʏ Iʀᴋ

TIL Simon's SO rep is over 9000.

5

Duga

I'm voting to close as unclear what you are asking. Please do not post/migrate this to Code Review in its current state, as it would likely get closed there for the same reason. — Phrancis 12 secs ago

Phrancis

Looks like we've got a script kiddy-wanna be here, trying to figure out how to randomly portscan all of the Internet (or at least the IPv4 part of it). — Sam Varshavchik 10 mins ago

RubberDuck

23:36

cough ;-) — Mat's Mug 2 mins ago

Lol @Mat'sMug

Hosch250

Hey, @RubberDuck. You've not been in VBA Rubberducking so long we can't even ping you.

Duga

possible answer invalidation by thexiv on question by thexiv: codereview.stackexchange.com/posts/127792/revisions

RubberDuck

Here. Let me fix that @Hosch250

Captain Obvious

23:53

0

Q: Beginner - Unity(C#) | Wall-runner type game

I'm working on a Wall-runner reaction based game. As I know my way around a lot of the features Unity provides, I'm still in the beginner phase when it comes to working with several classes and functions simultaneously. I currently got the two following classes hard at work together: Obstacle c...

c# unity

Edward

Hello all!

Quill

Hey there @Edward!

Edward

Needed a break from work, so I poured a beer, turned on the stereo and opened 2nd Monitor.

8

RubberDuck

@Edward That's a good plan!

Edward

It's working for me so far...

RubberDuck

23:57

It's actually really nice to see you. It's been a while.

Edward

I've been working, vacationing, travelling and writing. And some coding.

Transcript for