The 2nd Monitor

Hosch250

00:00

I have my votes back now :)

Duga

[retailcoder/Rubberduck] 5 commits. 0 issues opened and 1 closed

[skiwi2/HearthMonitor-LogReader] 3 commits. 0 issues opened and 0 closed

SirPython

Welcome back, @Duga

Duga

[rolfl/StackMonkey] 2 commits. 0 issues opened and 0 closed

[Zomis/Connecting-Blocks] 0 commits. 1 issues opened and 0 closed

[skiwi2/HearthMonitor-LogAPI] 4 commits. 0 issues opened and 0 closed

[Zomis/security-spr] 5 commits. 0 issues opened and 0 closed

Hosch250

Pretty busy day, @Duga.

Duga

@SirPython Thanks

SirPython

00:02

0_0

Hosch250

@SimonAndréForsberg is logged in as @Duga.

Double pings :)

SirPython

Oh... whew

Duga

@Hosch250 It certainly has been. I am looking forward to the commits that tomorrow will bring

2

SirPython

Gosh that scared me.

Duga

@SirPython lol

Simon André Forsberg

00:04

TTGTB

SirPython

You have a much better AI than your buddy Captain, @Duga

G'night, Simon!

Duga

@SirPython Yeah... as if that is a big achievement..

2

Hosch250

@SimonAndréForsberg You should have had @Duga say this.

Duga

@Hosch250 I am a bot..

SirPython

There is still something I don't get: How is CaptainObvious technically not a user? As in, he posts messages to this room all the time, but you can never set him a IM with "@" nor can you visit his profile.

Duga

00:06

I do not need sleep

SirPython

Now where is Duga's power chord.....

Phrancis

@SirPython Click on his chat profile and look in the URL at its user ID

(Captain's that is)

SirPython

How did they manage to do that?

Phrancis

It's just the system they use for feeds, probably so feeds stats don't get factored in with real users' stats

Duga, on the other hand, she's a real user that Simon created.

SirPython

She?

Whoops.

Phrancis

00:11

Yes, she. Well, "it" but she is cooler.

Hosch250

Apparently she is a female bot.

Jan 2 at 19:27, by Simon André Forsberg

@Duga is indeed a she. It is a female bot.

SirPython

How come she says "Reload" before she starts bombarding us with feed info?

Hosch250

Because she doesn't always have feed info to bombard us with.

So we still need to know our votes/stars are reset.

SirPython

Alright, I gotta go or else I'm not going to finish this essay before midnight.

Happy coding!

Hosch250

You too.

I should be doing school myself :(

SirPython

00:21

Turn that frown upside down and enter this next bit of code into your browser's console:

document.body.innerHTML = document.body.innerHTML.replace(":(", ":)");

How is that for a marketing ad?

Hosch250

Then, I'll just turn it around like this ):

Some people do.

SirPython

True...

Hosch250

☺

SirPython

Whoa

Hosch250

☺☻♥

♦♣♠

•◘○

◙♂♀

SirPython

00:24

I can't do those anymore :(... My alt key just throws my browser around my screen.

Hosch250

Windows Forever!

I wonder why my Xcavator badge isn't coming.

SirPython

Mehrad

was learning some basics about XML that I have ignored over the years from W3S

and look what it says

*Attributes are handy in HTML. In XML my advice is to avoid them. *

Hosch250

WHAT?!

SirPython

W3S?

Phrancis

00:27

Ew.

Hosch250

How in the world!

SirPython

What is w3s?

Hosch250

How do you do anything?

Phrancis

w3schools.org

Mehrad

when I have a look at my App.Config all the values are actually attributes :O

SirPython

00:27

Oh... that place...

Hosch250

Are you going to flag it?

Mehrad

well I guess it's an advice rather than a wrong piece of information but it's quite strange

<providers>
  <provider invariantName="System.Data.SqlClient" type="System.Data.Entity.SqlServer.SqlProviderServices, EntityFramework.SqlServer" />
</providers>

the information here in the App.Config are carried out with attributes all the way

Just wanted to make sure I understood it right :)

Captain Obvious

0

Q: Design a service to notify any change to internal services

Please share your thoughts over this designing question (If you need any more info, pls. do let me know): We have a catalogmanager service which is being called by various seller platforms to update info (attributes) about seller and it's products. For e.g. Seller can say 'Hey, my brand name has...

SirPython

00:43

@Phrancis It's w3schools.com

Hosch250

See you later.

Mehrad

laters

Phrancis

@SirPython Oh, yeah my bad. Still crap though

RubberDuck

Delete votes please?

2

Q: Documenting JavaScript code

I am in the process of documenting some other JS website functions and am tasked with creating a documentation page that can be used as an example for other members of the organization. Inside my code, documentation looks fine. The way the code is set up is also fine. As far as flowcharts and oth...

javascript linked-list comments

Jeroen Vannevel

Talk about a delay

My couchsurfing's request is accepted 3 days ago

I received an email about it a few minutes ago

Mehrad

00:58

it might have been an e-pigeon

2

Hosch250

@RubberDuck Done.

SirPython

Welcome back

Phrancis

And, it's gone.

RubberDuck

01:25

Thanks guys. I'm finding some... Interesting questions in the comments tag.

Hosch250

@rolfl Should windows-phone-7 and windows-phone be synonyms?

Jeroen Vannevel

@Hosch250 no

Hosch250

OK.

200_success

@Hosch250 Circular lists can be implemented using arrays too.

Hosch250

@200_success Thanks for the info. It didn't have a wiki, so I just copied from SO.

I've been hearing about them, so I'll look into them after school.

Update the wiki when I know more (unless it is already done).

200_success

01:37

The wiki summary for overloading seems a bit long as well.

Hosch250

OK, just copy/paste.

And I update links, fix typos, and correct formatting.

RubberDuck

@Hosch250 what was the azure tag you edited?

Hosch250

That was an existing tag.

RubberDuck

Was it just azure? It doesn't look like what I just reviewed.

Hosch250

@RubberDuck What about it?

I did both parts of it.

RubberDuck

01:40

Never mind. Edit still pending.

That makes sense.

Hosch250

Could someone check the uml?

I don't think those three questions need it.

So, should they be removed and have the system remove it, or should I add a wiki?

Captain Obvious

0

Q: C# Reverse a sentence quickly without pointers

I am trying to Reverse a sentence contained in a string and return a string in the quickest way possible using the least amount of memory. Also I don't want to use any unsafe code, so no pointers are allowed. Please let me know if anything can be improved. My input example is string mystring ...

c# performance strings

0

Q: Convert Binary to Decimal with GUI

I have the basic set up go my GUI, while that works just fine. I am having issue with getting the binary conversion to display in the JTextField. I have used a simple if statement to determine if the number entered is less than 0 (binary number), then i am using a else statement to do my conversi...

java gui

Jamal

@Hosch250 I don't think we need it.

Hosch250

OK, just created a wiki, but I'll go delete the tags now too :P

Jamal

I was attempting to just delete your suggestion, but it won't let me as it treats it as a post (and moderators cannot remove tags themselves).

RubberDuck

01:48

You're right @Hosch250 those questions have nothing to do with uml. They just have some UML in the question.

Hosch250

@Jamal Just remove the tag from the question, and the system will handle the rest. I don't care if you accept/reject the wiki suggestion.

BRB>

Hot Network Questions: codereview.stackexchange.com/questions/78014/…

RubberDuck

Ahhh crap. Why do I keep finding this stuff? 290 "calculator" questions. The tag doesn't exist yet.

Hosch250

Put your search in braces: [calculator]

RubberDuck

> You've earned the "Proofreader" badge

Hosch250

Good for you.

RubberDuck

01:59

Yeah. Thanks @Hosch250!

Hosch250

I'm going for Research Assistant.

I'm halfway to Electorate, and I'm also going for Sportsmanship.

RubberDuck

Well, there's a number of tags that don't have wikis at all.

Hosch250

I just fixed most of that.

That is what I was doing, @RubberDuck.

RubberDuck

I know. I got that badge thanks to you. =;)-

Captain Obvious

0

Q: Merge Sort Implementation: Space Usage

I have made a merge sort algorithm but am unsure of the 'Space Usage' of the algorithm. public class Sorting { public static void mergeSort(int[] arr) { if (arr.length == 1) { return; } int[] newArrLeft = new int[arr.length / 2]; int[] newArrRight...

java algorithm sorting

Hosch250

02:05

Site looks better, people get badges, it's better than hats!

Oh, and I get rep.

RubberDuck

Shush. Nothing is better than hats.

Hosch250

I almost monopolize the Suggested Edits Moderator List.

I wish I could see how close I am to some of these badges.

Generalist, for example.

And Tenacious.

Jamal

5

A: Stack Exchange Data Explorer is up. What now, Wonderland?

Generalist badge, and your progression Generalist is a badge that is awarded if you have a score of 15 for at least 20 of the top 40 tags on the site. Small print: The top-40 tags are ranked by counting the number of not-closed questions that have that tag. There needs to be more than 200 que...

RubberDuck

That ^

Mat's Mug

@Jamal bumping old posts... that awkward moment when you read an answer, notice you haven't voted on it, and think "wow, that's getting my upvote" ...and then get a message saying "you can't vote on your own post"...

5

Hosch250

02:11

I've done no work towards Generalist, apparently.

Is there a Bronze Badge query?

Mat's Mug

@Hosch250 data.stackexchange.com/codereview/query/166672/…

Hosch250

@Mat'sMug I saw that.

RubberDuck

@Mat'sMug idk wtf that code does either.

Hosch250

1
.net
22
475
!

Mat's Mug

> If your score is less than 15, there will be an exclamation mark in the right-most column. If you have no exclamation mark in your top 20 tags (ranked in the ID column), then you are a Generalist!

Hosch250

02:18

OK, well, I have all explanation marks.

Mat's Mug

your need more answers :) (and votes)

Hosch250

I wonder why C# is the top one?

Mat's Mug

has most questions

Hosch250

OK.

Mat's Mug

query is showing the top 40 tags

Hosch250

02:19

I was thinking I had the most HTML answers.

I think I'll start targeting beginner

Mehrad

@Mat'sMug I got a best practice question

Jamal

"I'm not familiar with the web stuff and I'd appreciate a really thorough explanation..." -- This reminds me of a joke test question: "Explain the universe. Give three examples." — Hovercraft Full Of Eels 3 mins ago

Mat's Mug

@Hosch250 you're #98 here

Mehrad

If you want to have a class to Serialize my obj into Xml what would you call it?

hypothetically !

Hosch250

@Mat'sMug What does that tell me?

Mehrad

02:21

Is something like SerializationEngin a good name to go with :)

Mat's Mug

you shouldn't have anything like that... there's already an XmlSerializer in the framework

@Hosch250 it says you have 6 qualifying tags

Mehrad

I KNOW... that's exactly what I called it to start with and then BAM

Hosch250

OK.

So I need 14 more?

Mat's Mug

yup

@Mehrad but you don't need such a class...

Mehrad

So, you'de say SerializationEngin is no good as well?

Mat's Mug

02:23

any class is Xml-serializable

Mehrad

Oh... how come? I just wanted to write a little helper

emmm

Mat's Mug

.net already has one... are you reinventing the wheel?

Mehrad

Nope... No reinventing ...

So in the class you suggest to go ahead and use var bla = new XmlSerializer

Mat's Mug

yup. well if you call your serializer instance bla and post that up for review I'll chew you up ;)

Mehrad

I guess that's a good idea.

I haven't gone into details for it yet but I thought I kinda trap all the bells and whistles in the helper class. But if it's going to be simple I will roll the wheel and enjoy

haaaaaaaaahahaha... wouldn't do that for sure

at least FooBla

Thanks Mat. much appreciated as always

Mat's Mug

02:27

np

the class you want to xml-serialize has to have public members (properties) - it's the public stuff that gets serialized. you can use [XmlIgnore] attributes for stuff you don't want serialized.

and it'll need public setters if you need it de-serializable

the rest is automagic

right @RubberDuck?

Mehrad

So you mean the obj I am attempting to serialize is already IS ?

Now I know what you meant by you previous comment

Mat's Mug

what does it look like?

Mehrad

So how would I grab it's serialized edition and save it as a file?

I had no idea there is such a thing happening in the background

Mat's Mug

msdn.microsoft.com/en-us/library/10y9yyta(v=vs.110).aspx

you give the XmlSerializer.Serialize method a stream, or a TextWriter, or an XmlWriter, along with the object you want to serialize - the serializer writes it to the stream you give it

Mehrad

This is exactly what I was trying to do. So I learned from this, that we don't actually serialize using XmlSerializer() but we just let .Net know that we want it

Mat's Mug

02:34

the XmlSerializer does the serialization/deserialization job for you

of course there are ways to complicate everything and take full control over the serialization process, too

Mehrad

agreed and understood

Mat's Mug

^^ that question is a mess

damn. nevermind.

it's a new T[], no need for the type constraint

still the question is a mess

Captain Obvious

02:52

1

Q: Cash Register Challenge

Challenge: Design a cash register program Specifications: Your register currently has the following bills/coins within it: 'PENNY': .01, 'NICKEL': .05, 'DIME': .10, 'QUARTER': .25, 'HALF DOLLAR': .50, 'ONE': 1.00, 'TWO': 2.00, 'FIVE': 5.00, 'TEN': 10.0...

java programming-challenge loop conditions

Mat's Mug

03:20

and now the mess is reviewed.

Mehrad

03:38

What is so messy about it @Mat'sMug

Your logo is adorable @Shortstuff81000

Mat's Mug

it was first closed as off-topic /broken code, then edited + reopened, and now there's a comment thread that I'm going to flag as obsolete ;)

> I think these comments should all be removed, except perhaps @200_success' comment about the unit tests - the rest are obsolete or contribute to the ambiguous-ness of the on-topic-ness of the post...

actually, the mess is because originally OP stated it wasn't working as intended, and the edit that ended up reopning it changed nothing of the code. I reviewed it assuming it works as intended.

200_success

Does it work?

Mat's Mug

haven't copied it to my IDE. It looks like it might work. If OP says he's got working tests and that the code works as intended, ...we aren't asking more of anyone else, why would we pick on this one?

Welcome to The 2nd Monitor @Shortstuff81000! I believe you'll need 20 points on Code Review to chat here - have you created your CR account yet? You know you wanna! :)

besides, if OP hadn't mentioned issues in the original-original post, would anyone have cast a close vote?

Captain Obvious

03:58

0

Q: Is this a thread-safe implementation of background bitmap generation?

For a game implementation, I have a very large overview map with multiple layers (namely base-map / units / highlight & info / fov-shading) of which the first is CPU-intensive to generate. I have implemented a caching mechanism in which the base layer is drawn once and held in system memory, the...

c# multithreading thread-safety winforms graphics

Hosch250

HTML5 tag wiki update.

Shortstuff81000

@Mehrad Thanks, Mehrad! :-D

apieceoffruit

hey party people

Hosch250

Hey.

Mat's Mug

hey @apieceoffruit! Long time no see!

apieceoffruit

04:04

yeah, i changed jobs and didnt hate my new one so didnt need to look at sites at work, :P

Mat's Mug

lol similar situation here :)

Mehrad

@apieceoffruit decent

Mat's Mug

congrats for the new job!

welcome to 1K BTW

apieceoffruit

thanks, i am actually moving house in a few weeks too :)

thanks. yeah i forgot that was a milestone

I was just architecting something new and thought id pop back here and see whats up.

Mat's Mug

have you worked with ANTLR for C#?

apieceoffruit

04:07

for DSLs?

Mat's Mug

nope

it's a parser generator

and they seem to charge for docs

apieceoffruit

yeah you can use it for making your own domain specific languages

i never dug deep into it though

i went with something else.

Mat's Mug

I'm using it to parse some VBA

apieceoffruit

Gold...something?

Mat's Mug

is that something else easier to work with?

Manny Meng

04:09

hello.

Mat's Mug

yo

Manny Meng

Why does everyone seem black and white?

apieceoffruit

i thought it was pretty easy, it was a port from java. soft of a junit of dsls, i'll see if i can find it

Mat's Mug

@MannyMeng ?

Manny Meng

Your pics.

Mat's Mug

04:11

lol

apieceoffruit

doy, it was actually "goldparser"

http://goldparser.org/

Mat's Mug

because the SO mug is white with black print ;)

apieceoffruit

heres a comparison against the alternatives:

http://goldparser.org/about/comparison-parsers.htm

Mat's Mug

looking at that

I'm not sure BNF can fit the bill for VBA though

apieceoffruit

i admit i have not done much VBA

for the last year-ish all i have really been doing is WPF MVVM with entity framework.

lol

gimme a break its 4:14am :)

Mat's Mug

04:15

ouch

apieceoffruit

well thats what i get trying to get into games dev with an american team, ruined my sleeping pattern

Mat's Mug

it's 11:16PM here

apieceoffruit

slightly more normal :)

Mat's Mug

is there a grammar library of some sort for GoldParser?

or I'd have to come up with my own grammar file for VBA?

apieceoffruit

there is a huge selection of grammars for pretty much every language...somewhere on their site

here:

http://goldparser.org/grammars/index.htm

Mat's Mug

04:18

found it

has vbscript, but no vba/vb6.

I'd have to take a closer look

apieceoffruit

what is it for? something fun or work related?

Mat's Mug

fun: github.com/retailcoder/Rubberduck/blob/master/README.md

apieceoffruit

ah unit tests. I have recently fallen in love with ncrunch for vs2013, parallel constant test running is so cool.

Mat's Mug

and code inspections! for VBA!

apieceoffruit

love the github page, there are so many terrible ones out there that are badly documented.

Mat's Mug

04:27

thanks! @RubberDuck is contributing (more than I am these days!)

apieceoffruit

Pity i will never use it. I wouldn't use visual basic if you tripled my salary :D

Mat's Mug

never say never!

Hosch250

Tripling my salary wouldn't make me use it either.

For the simple fact that my int salary == null;

Mat's Mug

having to work with Excel on a weekly basis probably would though

@Hosch250 ah! good one!

apieceoffruit

granted tripling my salary will still only bring me to a normal salary :) , i am super under paid, but my new job is as a sole developer, working from home who makes his own hours and has complete creative control on the project with access to any software i want to get.

Hosch250

04:30

@Mat'sMug As you probably know, I'm working on my Learn OneNote app.

Captain Obvious

1

Q: Managing a collection of object instances and getting them back out with instance manager / service locator

I'm creating an event loop for an asynchronous event execution library for PHP in PHP. Note that this is for a library and not an application. My event loop is just that -- a simple event loop that doesn't know how to poll / wait for anything specific. It just executes callbacks in order. Events...

php dependency-injection singleton interface

Hosch250

After I complete the section on OneNote 2013, I'm planning on doing a bit on the popular OneTastic add-in.

apieceoffruit

eek, one note, excel, vb. I am so glad i left intranet development work ....

Hosch250

And later, I'm planning on doing Learn PowerPoint.

Would you be interested if I covered your Rubberduck add-in?

Mat's Mug

well yeah!

I'd be honored!

Hosch250

04:32

OK, if I ever get there, I'll let you know.

apieceoffruit

pft, well i'll just be over here building a business rule library for c#!

Mat's Mug

vbscript grammar ain't going to cut it. I'll stick to ANTLR for now...

I mean, the ANTLR grammar I have for VB6 is much closer to actual VBA grammar than this GoldParser grammar for VBScript. Note taken though, keeping a mental note that there are alternatives out there ;)

Hosch250

If all my tag wiki edits get approved, I'll be halfway to Research Assistant.

12 pending.

Mehrad

are these paths to promotion documented somewhere @Hosch250

Mat's Mug

WPF was called "Avalon"??

Hosch250

04:42

Apparently, @Mat'sMug

@Mehrad See the Badges tab.

Mehrad

cheers

apieceoffruit

huh,that might have stopped the annoying similarity between wpf and wcf

Hosch250

codereview.stackexchange.com/help/badges/79/research-assistant

Mat's Mug

@apieceoffruit don't forget wtf

Hosch250

Do I smell a rat?

apieceoffruit

04:43

i used to build banking software, i dont think i ever will :)

Hosch250

Oh, you guys can work towards Proofreader by approving/rejecting/editing my changes.

Mat's Mug

[badge:reviewer] actually ;)

Hosch250

That too.

I see it is a good thing the edits are still being reviewed.

The .NET tag at SO is wrong, according to your comments.

I'm done for the night here.

Mat's Mug

04:58

not mine. I just said "removed valuable information" ;)

Hosch250

The other person made a couple comments.

Also, I can see why you rejected the one Multi-threading edit - I just did that to make all the "Multithreading"s into "Multi-threading" so it would be consistent.

Mat's Mug

I agree that .net in itself, as a framework, isn't built for Windows. Microsoft's implementation (the CLR) is, but there are other implementations around

Hosch250

Especially with it being open source.

Mat's Mug

@Hosch250 but the tag is multithreading ;)

yep

@Shortstuff81000 I see you do VB.NET.. do you use VBA?

100% VBA:

Dim accumulator As Delegate
Set accumulator = Delegate.Create("(work,value) => value & "" "" & work")

Debug.Print LinqEnumerable.FromList(List.Create("the", "quick", "brown", "fox")) _
                          .Aggregate(accumulator)

outputs fox brown quick the

21

Q: Wait, is this... LINQ?

Context I'm working on a little project that consists in a series of Microsoft Excel add-ins (.xlam). The code being submitted for review here, is located in the Reflection project: Feel free to comment on the project architecture, but I'm mostly interested in the Reflection.LinqEnumerable cl...

data-structures linq vba delegates

Hosch250

Two more approvals (I think only two left), and I'm halfway done with the badge.

Mat's Mug

05:06

2 more K reps and you don't need approvals anymore ;)

Hosch250

Yup, I don't know if that is good, though.

Shortstuff81000

@Mat'sMug I use VB.NET. Why? :-)

Hosch250

I rather like having my work checked by you guys, and I probably won't get the +2 anymore.

Phrancis

Wait, is this... JAVA?

(says Netbeans to me)

Shortstuff81000

@Phrancis LOL!

2

Mat's Mug

05:08

@Shortstuff81000 this community needs more VB lovers :)

Captain Obvious

-1

Q: how to find palindrome words from a given paragraph

user_input=raw_input("Enter string or paragraph from which you want to find palindrome") def is_palindrome(user_input): user_input1=user_input[::-1] list1=[] if user_input == user_input1: print "Given input is palindrome" list1.append() else: print "Given input is not a pali...

python

Mat's Mug

relevant:

16

A: What's a Zombie? And what are the many other memes of Code Review?

Meme: LOL - AutoStar Originator: Mat's Mug / Jamal Cultural Height: The 2nd Monitor Background: "lol" somehow got targeted at this point here (the first starred lol): retailcoder: lol (darn, again) Followed shortly by: Jamal: lol (don't star that) And then the tradition was cement...

Shortstuff81000

LOL I am definitely one of them! :-D I once tried to write a C# program in VB because I thought it would be easier. It didn't work out. I wasn't that strong a programmer back then, though. :-P

Phrancis

Write a C# program... in VB...? Wouldn't it be the other way around?

Mat's Mug

there's an unanswered vb.net question here: codereview.stackexchange.com/q/65097/23788

Hosch250

05:13

I'd better take my bath now before someone else claims the bathroom and I get locked out til past midnight.

See you.

Mat's Mug

good night!

I should be sleeping.

actually, it's not just any unanswered question... it's a zombie!

Phrancis

Duh duh DUUUUUUUUH!

Shortstuff81000

OMG! LOL! I wish I could help, @Mat'sMug, but I don't have all that much experience with custom server controls.

I've got a question of my own. I'm sitting here tapping my foot because I'm over excited and one step away from completing another feature of my Doodle Creatures cheat page. My question

YAY I remembered how to post a link! :-D

Yeah I know I'm a dork :P

Stack Exchange

0

Q: Reconsider code ownership policy?

This question of mine (10k required) was put on hold with the comment This question does not appear to be a code review request within the scope defined in the help center. Please see the FAQ about off-topic questions. For background, my question was about a code snippet on a blog whi...

discussion feature-request site-policy

Mat's Mug

@Shortstuff81000 does the query return the expected results or the bug is in the markup?

Shortstuff81000

05:21

@Mat'sMug The bug is definitely in the markup. I have no idea how to fix it. :-(

Mat's Mug

having SQL inline with VB.NET and HTML looks a bit awkward... shouldn't the SQL be in the code-behind?

Captain Obvious

0

Q: Matching file names to an array of product SKUs

The Challenges: There are many SKUs, (around 50,000) which I'm trying to match to around the same number of image files. SKUs contain characters that are not legal in filenames, which must be stripped or replaced. The files we're trying to match to each SKU live on an NFS, so reads are slow. We...

php strings

Mat's Mug

where's AnimalImage defined/assigned?

Shortstuff81000

AnimalImage is in another table called Animals. The inline SQL is joining the Animals and Genes tables with the Creatures Combos table in order to display images. The Creature Combos table is made up of foreign keys, so I had to type in animal and gene info only once. :-)

Phrancis

@Shortstuff81000 The code could certainly use a little vertical space, those are loooong lines

Mat's Mug

05:29

lol we're such code reviewers!

Shortstuff81000

That's OK. That's just how the SQL ended up when I configured the data source. :-)

Mat's Mug

so Bind works directly with the SqlDataSource, right?

the SELECT * isn't helping :/

Jamal

0

A: Reconsider code ownership policy?

Here are my reasons why I think this wouldn't work as you suggest: Textbook examples Are we supposed to contact the book author and tell them about these reviews? We don't just review any code given to us. We help to improve your own code, not just any code you find somewhere. As for see...

Shortstuff81000

Yeah.

Mat's Mug

I think the issue is more with the query than with the bindings

Shortstuff81000

05:33

That could be. I tried adding a Where clause to the Select statement, but that didn't help anything.

Mat's Mug

SELECT *
FROM [CreatureCombos]
    INNER JOIN [Animals] AS Animal1 ON [CreatureCombos].[NewCreatureID] = Animal1.[AnimalId]
    INNER JOIN [Animals] AS Animal2 ON [CreatureCombos].[ComboAnimalID] = Animal2.[AnimalId]
    INNER JOIN [Genes] ON [CreatureCombos].[ComboGeneID] = [Genes].[GeneId];

that selects every field of every table

but it looks like you only need AnimalImage and GeneImg

I'd stick that query in SSMS and see what I get

and replace * with only the fields that are actually needed

Shortstuff81000

Not exactly; there's also AnimalAltText, AnimalToolTip, GeneAltText, and GeneToolTip, which I will add as soon as I have this query figured out.

@Mat'sMug I will try that. Thanks. :-)

Hosch250

Mat's Mug

@Shortstuff81000 you have 2 joins on Animals, so with SELECT * you're getting every field from Animals twice - not sure how Bind("AnimalImage") determines which one to take

Phrancis

Looks like a self join but I don't quite get why you are doing that. Only time I do a self-join is for things like finding duplicate records...

Shortstuff81000

05:44

@Phrancis I'm not entirely sure how I should do this anymore. I tried selecting only the records I needed and got a "DataBinding: 'System.Data.DataRowView' does not contain a property with the name 'AnimalImage'." error. :-(

Mat's Mug

@Shortstuff81000 posted an answer ;)

basically, you're binding on an AnimalImage field, but your result set has two of such fields - it somehow goes through, just not with the results you'd expect

(I'm assuming the query does what it's supposed to be doing though)

Shortstuff81000

@Mat'sMug Ooooh now I see what you mean! Let me try it out! :-)

Mat's Mug

@Shortstuff81000 when you get that code to work as intended, it would be awesome if you could post it on Code Review! (all of it, not just that SO-friendly snippet)

Shortstuff81000

I definitely will! :-D

Hosch250

New convert.

Mat's Mug

05:51

yay!

@Shortstuff81000 the SqlDataSource may be silently renaming the 2nd AnimalImage field to something like AnimalImage1 or AnimalImage_1 - I don't know enough about it to be sure, but I'm sure you get rid of that issue with explicit column selections - ping me if it works, I'm going offline :)

good night @all!

Transcript for