The 2nd Monitor

Mat's Mug

18:00

bwahahaha

Quill

The part that baffles me is that I'm not really a swift developer

Vogel612

It's funnier than the recruiter mail I'm getting :/

Phrancis

/// <summary>
/// Vendors the electronic ordering.
/// </summary>
/// <param name="svid">The svid.</param>
/// <param name="fileDropPath">The file drop path.</param>
/// <returns></returns>
public static string VendorElectronicOrdering(int svid)

"Vendors the electronic ordering."

Dan Lyons

that looks like a GhostDoc gem

it doesn't have anything more to go on than the names you used, so you get wonderfullly informative doc comments like that

Phrancis

Yeah. Pretty hot stuff right there.

Mat's Mug

18:06

@Phrancis such smarty, much pants

Phrancis

I guess the assumption is that as developers refactor or change the code, they will update the documentation (because developers love writing documentation... right)

Dan Lyons

OTOH, the method and its parameter names could be improved, too

Mat's Mug

"methods should start with a verb" ...especially if you're auto-generating your xml docs

Phrancis

I think IsOrderingElectronicallyForVendor would sound better

Still weird that it returns a string though, I'd expect a predicate to return a bool

Oh wait, it returns a file drop path

Dan Lyons

which was apparently a parameter at one point :P

maybe an out param

Vogel612

18:10

sheeesh compiling from source is slow as helll

Dan Lyons

what's the method doing? the static suggests it's stateless, but there's not much stateless work you can do taking in a single int and returning a string

Vogel612

probably accessing state from the static context :/

Dan Lyons

@Vogel612 kill it with fire :o

Vogel612

like... VendorRepository.getById(svid) or sth.

Phrancis

OK so it takes an svid int as param, makes a DB call to a proc, puts the results into a DataTable and then counts the rows, if more than 0 and is marked electronic ordering, it updates the file path string, then returns... or returns an empty string... which means, if the value in the table is marked as electronic, but the row has no value for filepath field... KABOOM

Captain Obvious

18:18

0

Q: Grabs 10 hex bytes and converts it into a string

I'm working on some code right now where the goal is to take in a byte stream and grab the data from the body of a message. The data is 10 bytes long and is meant to be translated into hex. Specifically, the 10 bytes coming in look like 01000200030004000500 or 01010201030104010501 (with implied 0...

0

Q: php FormBuilder class suggestions

I've taken some suggestions from PHP Form Builder Class and tried to build my own FormBuilder class. Its supposed to work like this: a) Creates an object of type form b) attaches objects of type FormElement to form c) FormElements are aggregates of Rules and InputTypes d) Rules: define...

form php5 dom

skiwi

@Vogel612 So what my algorithm now does is it first generates the simplest queue possible (if your goal is Metal Mine 10 it puts 10 metal mines in a row with metal and crystal storages if necessary)

Then for all buildings on all positions in the queue it tries to add that building, computes the time, stores the one that generates the least time: It uses that building, and goes on to the next iteration until no further improvement is detected

A variant on that is that it generates all (building, building) pairs on all possible (position, position) pairs to insert two buildings at once

It also works with building^3 triples on position^3 triples, but performance of that is... non-existant

Now the old version gives this output pastebin.com/0jTdijAq and the new version (with storages) gives this output pastebin.com/gFL5fD7Y (it may be a bit difficult to read though)

Though lookahead-2 might be most interesting as only there it clan plan a robotics factory as individually planning a robotics factory is pointless as there is no deut production and planning a deut mine is pointless as there is no deut consumption, so you have to plan them at the same point

Vogel612

the final results do include robotics factories and deut mines though...

skiwi

Yes, for lookahead-2 that's possible

Because that one can plan two buildings at same moment

Jeroen Vannevel

I have a .txt file with ideas for all kinds of apps and tools to make. It's starting to grow out of control

6

Hosch250

Want to share it? Maybe we could get some done for you.

Quill

18:29

I've been out of serious ideas for 2 weeks

Ideas are an undervalued commodity

skiwi

I'm working on 9999999999 ideas right now, 0 are finished

Dan Lyons

@Quill good ideas, perhaps, but there has never been a shortage of bad ideas

skiwi

There's nothing similar to RubberDuck and VSDiagnostics for Java, is there?

Vogel612

Depends...

Jeroen Vannevel

@Hosch250 Sure, pick one gist.github.com/Vannevelj/0cc8965d8af0ec108c67

Vogel612

18:32

IntelliJ and Eclipse as well as Netbeans have refactorings inbuilt

for analysis there's Sonar

as well as a handful of other tools doing something similar

Duga

@KoKsMAN If you have working code and want to get answers how to improve it ask at SE Code Review please. — πάντα ῥεῖ 30 secs ago

Quill

> Top questions in a StackOverflow or StackExchange tag per day/week/month
Option to subscribe to emails

Vogel612

VCS is always integrated in the IDE and there's no need to "export the code" to version it..

Quill

isn't that already a thing

Dan Pantry

skiwi

18:33

@Vogel612 How good is Sonar?

Dan Pantry

All sent to an alias e-mail that I use on SO.

skiwi

The code in question is very prototypepy, I'm wondering how bad it is

Vogel612

@skiwi check it out yourself

Quill

@DanPantry help vampires != job offers

skiwi

@Vogel612 Finding Nemo?

Dan Pantry

18:34

@Quill No, but those job offers have come from people emailing me through SO :p

Duga

possible answer invalidation by D3107 on question by D3107: codereview.stackexchange.com/posts/118437/revisions

Mast

@DanPantry They like you.

Dan Pantry

The one from Jack Selt is actually interesting

Duga

@πάνταῥεῖ He's looking for a solution for a very specific problem, so it's on-topic here. No point in moving it to Code Review. — Mast 35 secs ago

Quill

interesting video/talk/whatever

Mario Gonzalez: Realtime HTML5 Multiplayer Games with NodeJS

►

Duga

18:45

This is the wrong spot for these type of questions. You should post these here : codereview.stackexchange.com — Jonas W 12 secs ago

Vogel612

@Duga mostly just PoB...not really a good fit for CR as is

Duga

possible answer invalidation by Jamal on question by TheKitsuneWithATie: codereview.stackexchange.com/posts/118374/revisions

CommitStrip

19:05

http://www.commitstrip.com/en/2016/02/03/safe-is-better-than-sorry/
CommitStrip
Safe is better than sorry
CommitStrip
1454525704

Sumurai8

Monking

Captain Obvious

0

Q: split or not split this class?

I wonder if I should divide this class into two , since it contains two " ifs " with different messages . To break it would be in keeping with the " S" SOLID ? It would be usual or a waste of time ? internal class ValidDateApuration : IBusineeOfValidation { public MessageValidation Validar(D...

c# oop

Mat's Mug

@Vogel612 you were saying? ;-)

Vogel612

ermph...

so many downvotes ...

Duga

Cross-posted on Code Review — Mat's Mug 9 secs ago

Hosch250

19:23

@Zak Are you around?

Zak

@Hosch250 indeed I am

Hosch250

Could you check the unit tests for my newest refactorings sometime and try to find cases I missed?

Captain Obvious

0

Q: How do i implement the "file name" into this program? (word search finder)

I am currently working on a word search program for my java class, and i thought i would do some research on other similar programs before i got started (see how they work before i write my own version). i stumbled accrossed this program :Java Word Search Solver it looks very well written, but ...

java

Hosch250

No C# knowledge required, just check the inputs and outputs.

Vogel612

HRNGHHH! creating a working install from sources can't be that hard, can it?

3 hours...

ttgh

Captain Obvious

19:34

0

Q: Slice chunking in Go

I have a slice with ~2.1 million log message strings in it that I am parsing for regular expression matches. For any matches, I add it to a local slice and then return said slice. func Search(re *regexp.Regexp, logs []string) []string { var lwg sync.WaitGroup var lresults = make([]string, 0) Num...

performance array go

Vogel612

Hrnghhh why??? Why doe the monodevelop make rely on nuget???

And why does configure not check it's present??

Duga

For improvement of working code ask at SE Code Review please. — πάντα ῥεῖ 12 secs ago

Quill

> return message.parent().parent().children('.signature').children('.username').text().re‌place(/ /g, '');

Vogel612

19:55

Wat

Quill

horrible hacks in JavaScript

Quill

20:05

Just when you think you’ve removed McAfee AntiVirus from a relative’s PC, you realise you still have 7 more Horcruxes left to destroy.

Tweeted by iamdevloper on February 3, 2016 at 10:39 AM

3

Mat's Mug

lol

Quill

20:29

Captain Obvious

20:47

0

Q: Sending encrypted message through sms

Im making a project that encrypt a message and decrypt by the receiver. the program is completely running in the emulator but when I run it in my phone and send an encrypted message,the program will just finish and no SMS was sent public class EncDecSMSActivity extends AppCompatActivity { /** Ca...

java android

0

Q: How to elimate post back when saving file to database

I've been tasked with storing and retrieving images from the database. I've gotten it to work using a post back but that doesn't flow with the rest of the site. Is there a way to rewrite this code in order not to post back? Not terrible difficult but not efficient and also uses the post back. ...

javascript jquery ajax

-1

Q: Does my chen prime finder actually work?

So I'm trying to write a programme to find chen primes. For those who don't know a chen prime number is a prime number where when the prime number = p, (p + 2) is either also a prime number or has two prime factors. I think I've got it working and certainly the first few values are correct but I...

python python-2.7 primes

RoboSanta

Naruto answer; accepted non-selfie answer with 0 score: Writing a Counter object to a CSV-file in Python 3 python-3.x csv comparative-review dictionary

Ripe zombie; open question with answers, at least one answer having score 0, no answer having score > 0: Using as few queries as possible for many table associations performance ruby ruby-on-rails active-record

Duga

21:05

I'm voting to close this question as off-topic because it's a request for code review. — PiotrWolkowski 14 secs ago

If you decide to post on Code Review, please be sure to provide a self-contained code sample, not just snippets. Also, please explain more precisely what your inputs are and what outputs you expect. Actually, doing this would be good for Stack Overflow as well. Also, if you ask on Code Review, please delete the question here since cross-posting is frowned upon. — 5gon12eder 9 secs ago

200_success

@CaptainObvious Some programmers tend to express more self-doubt than others. I think it's a bad idea to be reflexively hostile to questions that express those doubts.

Phrancis

Finally found the issue I've been tracking down all day... one tiny little trailing space in a worker's first name that trickled all the way down to the EDI and caused a failure on the vendor's end... all because the user inputs were not trimmed before inserting in the table.

4

Caridorc

@200_success exactly, I upvoted that question from the start

Zak

@Phrancis I hate those. I've got a function in my Standard Methods collection purely for taking strings and removing every single damned non-display character

Mast

@200_success I didn't vote, but looking at the original I think the reflex was from the fact OP wasn't sure whether it was working.

A downvote isn't personal.

Phrancis

21:17

@Zak Agreed. I set up a cleanup SQL job today to run each morning in production now and fix those, in the interim to the actual code fixes being deployed

Caridorc

@Phrancis I would not worry because they were not trimming the input, I would worry because the a trailing space caused an EDI failure, is that normal?

Zak

I was surprised to learn just how many 8-bit characters *aren't* display characters

skiwi

@Phrancis Be ready for the day that an user creates a bug that they cannot use names starting with a whitespace as workaround to hide them from other systems anymore :D

Zak

chr(0) through chr(31), chr(127), chr(129), chr(143) and chr(144)

Duga

I believe this is a good question for the Code Review stack exchange site because you have working code and are just looking to improve performance. I would suggest removing this question and posting it there instead — Kevin Wells 15 secs ago

Phrancis

21:21

@Caridorc There is also a known defect on the vendor's end when they parse the XML files the EDI sends where it parses a field is formatted like <Clinician>FIRSTNAME LASTNAME, RN</Clinician> which right now looks for exactly one space between the names, but in these cases it had two spaces so it thought that the last name was empty

Captain Obvious

-1

Q: Scrapy and extracting items on one page, then follwing link to extract more data

I am trying to get this spider to go to the www.newslookup.com/Business and first extract the date/time item['time'] the item ['source'] and then go into the article link and get the url and name. I believe I am getting an error from the request.meta section as I get the following error: 2016-0...

python python-2.7 web-scraping scrapy

Caridorc

@Phrancis regex.match("\ +", xml) ?

Phrancis

@Caridorc Well, you wouldn't want to parse XML with regex ;p

quartata

@CaptainObvious broken code

Caridorc

@Phrancis true, but also a legit xml parser would not care about the number of spaces right?

quartata

21:24

We should maybe get Duga to post an alert whenever a question contains "error"

or something similar

Phrancis

@Caridorc Well, it appears that the XML gets parsed OK, but whatever code is handling the parsed data is wrong

Caridorc

@quartata sounds like a sensible idea (just avoid checking inside the code-blocks)

@Phrancis so use multi-space regex on the already parsed data :)

quartata

@SimonForsberg

Duga

Verilog in a day is not terrible. best way to learn though is tackle a simple problem, compile test and debug. Post here if you get stuck, or codereview.stackexchange.com if it works but want better code. — Morgan 56 secs ago

Phrancis

@Caridorc Right, or just a regular string parse in whatever tech they use (probably C#)

Vogel612

21:27

@quartata meh... we have captain to post each question...

Mast

@Duga You can't learn a language in one day. Especially HDL languages.

Vogel612

every single one of them usually gets one or more pair of eyes on them

Mast

@quartata We already pick them up manually, we're quite good at that.

Vogel612

errors wanting to be fixed don't need an alert..

quartata

@Vogel612 Sure but the RSS feed is really slow plus not everyone is going to pay attention to it

I mean this question right here hasn't been closed yet...

Vogel612

21:29

so be it

Mast

@CaptainObvious More VTC on that.

quartata

@Vogel612 We need five pairs of eyes.

Vogel612

you're doing premature optimization there @quartata

Mast

That ^^

The system is working as intended.

Vogel612

review queues are also used if we're not closing this within minutes of captain picking it up

Mast

21:30

And there are always people lurking in the queues.

Caridorc

@Vogel612 yup, and I also fell into it, premature optimization is always there

Vogel612

There is no such problem as that it would require Duga to ~~alert~~ annoy us :)

Mast

@CaptainObvious Hammered.

quartata

@Vogel612 I suppose Duga is already annoying enough

Phrancis

Yeah it's not fair to the other regulars who use the site if only the people in chat get to triage stuff

Zak

21:31

@quartata Usually, if that pair of eyes spots something that deserves to be closed, they'll say so in chat, and then you'll *easily* get your 5

Duga

Cross-posted here: codereview.stackexchange.com/questions/118795/… — jimhark just now

Zak

Assuming a mod doesn't get there first ^^

Mast

Our mods are lightning fast.

Which is good, because every once in a while there's a shitstorm going on and we can definitely need the fast handling at that point.

Vogel612

It also has the drawback of them usually nuking the clear-cut stuff, before we get to do anything and thus we're getting lazy...

but eh

Duga

It sounds like you're seeking code improvement. You might be interested in our Code Review site: codereview.stackexchange.com — Litty 48 secs ago

Captain Obvious

21:41

-1

Q: Trying to run a java file from R

i am using a (D/ drive) to hold my R programs and files. I have installed rjava however i have tried all commands to try and run it however it keeps saying it cannot find the path. The java file is stored in a example like "D: work" what commands would i use in the R command to get it so it will...

java r

Mast

@CaptainObvious No code.

Vogel612

aside from not even remotely working...

Duga

If this is working code that you think could be improved, see Code Review. If not, please clarify the issue. — jonrsharpe just now

Mat's Mug

21:56

woot, today I earned my 4th [badge:yearling] on Stack Overflow

Phrancis

Nice!

Duga

Are you asking for a code review? If so, we have a site for that. — Litty 57 secs ago

Mat's Mug

Stack Exchange. Got a question? There's a site for that.

Quill

22:14

How come I can't do Dictionary<string, Dictionary<string, string>>?

Mat's Mug

why can't you?

private Dictionary<string, Dictionary<string, string>> foo = new Dictionary<string, Dictionary<string, string>>();

compiles fine

Quill

seriously? wat

Mat's Mug

...

Quill

omg, ffs C#

yesterday it wouldn't compile, I had to do something like class SubDict : Dictionary<string, string>

Mat's Mug

yesterday you were missing an angle bracket

Captain Obvious

22:16

0

Q: Hacker rank Jesse and Cookies

I am trying to solve a problem in hacker rank and the question is as follows Jesse loves cookies. He wants the sweetness of all his cookies to be greater than value K. To do this, Jesse repeatedly mixes two cookies with the least sweetness. He creates a special combined cookie with: sweetness =...

java algorithm

Quill

I wasn't missing any angle rbackets

Mat's Mug

and the subclass is probably cleaner than a dictionary of dictionaries

C# compiler didn't update itself overnight ;-)

Quill

C# just hates me

skiwi

@Mat'sMug Nice, quite some time

Quill

still looks like spaghetti: gist.github.com/The-Quill/04c69fa435686b8f5882

Mat's Mug

22:18

yeah.. I remember actually creating my account :)

Quill

back when you were only a young mug

Phrancis

We have a nice newcomer here (go vote!)

Quill

oh good, more javascript

Mast

@Quill Back then he didn't have a mug.

Mat's Mug

^^

Quill

22:22

What else would you drink coffee out of?

Zak

Wait, there was a time when @Mat'sMug didn't have the mug? Mind = Blown.

Mat's Mug

I wasn't @Mat'sMug then either

was working in the retail world, name was @retailcoder

my github repo still has the scars

Quill

lol

Phrancis

@lol.upvote

Best name ever

Mat's Mug

that was... a momentary lapse of ...whatever

Phrancis

22:25

Wow I just got an email from CMS with 158 character long subject...

Duga

possible answer invalidation by Mast on question by user3324848: codereview.stackexchange.com/posts/118816/revisions

Phrancis

Quill

@Duga lol no

@Phrancis oh the verbosity

Phrancis

@Quill IKR

Zak

@Mat'sMug So *that's* your real name ^^

Mast

22:26

@Quill Uhm, have you checked what your edit made the question look like? Not sure whether this is better.

Mat's Mug

for some values of "real", yeah

Mast

Quill

oh, alright, one sec

hmm, a downvote. that's weird

also fixed @mast

Mast

At least it's consistent now.

@Quill Where? I don't see one on that question.

Quill

on one of my answers ;-;

Captain Obvious

22:32

0

Q: Creation of playlist to fake-split a full-album video

I created this snippet to "split" my full-album videos into its individual songs without the need of actually splitting the video file. I personally created this snippet because I enjoy reading all musics from a directory in shuffle mode and stumbling upon a full album kinda ruins the point. #!...

python

Hosch250

Hi, @epicone1200.

You'll need 15 reputation on any site to chat here, SO rep doesn't count because it is on a different server.

Zak

@Hosch250 SO rep doesn't count? TIL

Quill

@Zak SO and MSE rep count

it just works in reverse for chat.se and chat.mse

Hosch250

Oh, it does?

I thought SO rep didn't count to chat on this server?

Mast

It's all the same network IIRC.

Phrancis

22:45

kids don't try this at home

select DB.[server], DB.databasename, DB.displayname,
    [Query] = 'update TELEHEALTH_VITAL_SIGNS_MESSAGE_HEADER set thvsmh_fileheader = ''1971'', thvsmh_processed = ''N'' where thvsmh_id = ' + cast(MH.thvsmh_id as varchar(10)) + ';' + char(10) + 'go',

Mast

@Phrancis I don't do much databasing, but that's such a bad idea for so many reasons.

Quill

@Mast chat.se is network wide, chat.so and chat.mse are specific

Phrancis

Yeah I know. Temporary patching things up manually while waiting on a job to be deployed

Mast

@Quill Didn't know their chats had own domains.

Quill

yeah, they do

mse chatroom and so chatroom

Mast

22:50

Yea, I looked them up right after I typed that.

Phrancis

TTQW

Duga

Yes, a sorted TStringList would probably be better, and depending on the version of Delphi there are probably even better alternative. This code has many issues with style, at least. You should probably post it over on code review. One thing: You set HostID to the result of CompareText - is that right? — 500 - Internal Server Error 7 secs ago

Captain Obvious

23:28

0

Q: Create a lag in Pong game?

I am a high school student in Computer Science 2 coding a pong game. I need to create a lag of 3 seconds before the ball begins to move. I have an update method as well as a run() method, and I think doing this would use those two? Please help me implement this!

java

Phrancis

@CaptainObvious lol nice.

Hosch250

@Phrancis Nice?

Phrancis

</sarcasm>

nhgrif

23:46

@Pimgd Not pay the vendor?

3

And continue using the previous version of their binary.

nhgrif

23:59

Delegating work is a magical thing...

Transcript for