C# Discussion Boards - CodeProject

shwaguy28-Dec-07 6:38

28-Dec-07 6:38

I seem to get my feathers up when people reply, with the first thought that comes into their head, without actually reading the original post; it tends to destroy a thread.

Your post was not so bad, simple mistake...but 2 replies in a row (one not yours) that went down the same path is a bit frustrating.

Usually when a thread gets to 2 like this I never get an answer to the question......so I will ask later.

thanks

shwaguy

Not Active28-Dec-07 6:40

28-Dec-07 6:40

shwaguy wrote:
so I will ask later.

I'll remember to ignore you then.

only two letters away from being an asset

shwaguy28-Dec-07 7:32

28-Dec-07 7:32

thanks for letting me know you will ignore my threads...it was very useful on topic information (sc).

I understand that people make mistakes, but, when I get 3 knee jerk responses (responses that have NO input to the question) in a row I tend to figure noone will want to read the thread.

I twice stated that there was no issue with the existing code, but, you seemed to miss that. Your ego must be easily bruised if to be hurt by those exclamation marks.

-If a response is not the solution thats all good.
-If a response is correct solution thats all good.
-If the response is off topic becuase the poster did not bother to read the post....I think "I wish that the poster could delete the response, its a waste of reading time and clutters the original post"

You take offense way too easily. Just because you have 3500 posts does not mean I am kissing your butt for very poor, of topic, replies. I don't have anything against you either, I think you may have helped a lot of threads.

shwaguy

Not Active28-Dec-07 7:51

28-Dec-07 7:51

shwaguy wrote:
I twice stated that there was no issue with the existing code,

Then why did you post it?

Go away, son, you bore me.

only two letters away from being an asset

shwaguy28-Dec-07 8:37

Re: directorysearcher findone() ---still seeking solutio

28-Dec-07 8:37

You just have to feel like your right on this don't you? Here is the news, you are wrong again.

The code although correct did not suit my purpose. If you bothered to read the posting you would have noted that I commented out (with comments) the items that would not work, but wanted to work.

as previously stated: beat it.

KaptinKrunch28-Dec-07 6:33

KaptinKrunch

28-Dec-07 6:33

I did a little more testing for you. The reason the result is null is because AD does not contain an entry for the userName provided. Try using a different username for the userName variable.

Confirmed here: http://msdn.microsoft.com/library/default.asp?url=/library/en-us/sds/sds/directoryservices_directorysearcher_findone.asp[^]

If more than one entry is found during the search, only the first entry is returned. If no entries are found to match the search criteria, a null reference (Nothing in Visual Basic) is returned.

Just because we can; does not mean we should.

Re: directorysearcher findone() ---still seeking solutio

shwaguy28-Dec-07 6:47

28-Dec-07 6:47

Thanks, but the userName is valid, it was the first thing I checked: The variable (and query) I used for the search filter worked fine with "FindAll()"

The thread is closed now....thanks for the help.

Paul Conrad28-Dec-07 6:56

Paul Conrad

28-Dec-07 6:56

It is proper to keep a thread even if you already found the solution. Someone later on might have the same type of problem and this thread could help them.

"The clue train passed his station without stopping." - John Simmons / outlaw programmer

shwaguy28-Dec-07 7:07

28-Dec-07 7:07

Hi Paul,

I agree. In this case no solution was found for the original question and the thread turned into an unreadable mess that would only serve to waste the time of someone seeking a solution.

Usually, I get good or great responses from this forum. People usually read, think then answer (even if the correct solution is not given). Maybe I just asked on a bad day.

shwaguy

Not Active28-Dec-07 7:25

28-Dec-07 7:25

shwaguy wrote:
Maybe I just asked on a bad day.

Or, more likely, you asked a poorly defined question and are now upset and pouting. Cry | :((

only two letters away from being an asset

shwaguy28-Dec-07 7:34

28-Dec-07 7:34

beatit fool....read the code comments of my original code and the posting. Your wrong, stop flaming.

I am starting to think taht a large portion of your 3500 posts were garbage posts.

Paul Conrad28-Dec-07 7:41

Paul Conrad

28-Dec-07 7:41

shwaguy wrote:
the original question

It did sound rather vague.

"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon

shwaguy28-Dec-07 8:32

28-Dec-07 8:32

Putting aside the 2 replies I got being totally off topic:

- "what error message did you get?" - uh...none...I got a null value as stated.

- replace code that was stated as working search.Filter = "samaccountname=" + userName; with this other filter that works equally well too (blah, blah, blah).

Whats vague about the post?:

//this filter works
search.Filter = "samaccountname=" + userName;

//This below formats for the search filter result in result=null

//search.Filter = String.Format("(cn={0})", userName); //code project filter
//search.Filter = "cn=" + userName;

SearchResult result = search.FindOne();

shwaguy

Generating Random Strings from an array and placing them into a textbox....

Paul Conrad28-Dec-07 9:18

Paul Conrad

28-Dec-07 9:18

The original post was rather vague. Not the ones that followed.

"Real programmers just throw a bunch of 1s and 0s at the computer to see what sticks" - Pete O'Hanlon

DSdragondude28-Dec-07 4:03

DSdragondude

28-Dec-07 4:03

I've wanted to do this and come up with this code:

private void generatebut_Click(object sender, EventArgs e)<br />
        {<br />
            string[] strRandomIdeas;<br />
            strRandomIdeas = new string[10];<br />
<br />
            strRandomIdeas[0] = "Bus";<br />
            strRandomIdeas[1] = "Animal";<br />
            strRandomIdeas[2] = "Surfing";<br />
            strRandomIdeas[3] = "Ride";<br />
            strRandomIdeas[4] = "Vehicle";<br />
            strRandomIdeas[5] = "House";<br />
            strRandomIdeas[6] = "Gaming";<br />
            strRandomIdeas[7] = "Flying";<br />
            strRandomIdeas[8] = "Entertainment";<br />
            strRandomIdeas[9] = "Music";<br />
<br />
            Random RandomIdea = new Random(strRandomIdeas);<br />
            txb1.Text = RandomIdea.Next();<br />
            txb2.Text = RandomIdea.Next();<br />
<br />
            if (txb1.Text = txb2.Text)<br />
            {<br />
                txb2.Text = RandomIdea.Next();<br />
            }

However, I get numerous build errors and it basically doesn't work, can someone help me??

Re: Generating Random Strings from an array and placing them into a textbox....

KaptinKrunch28-Dec-07 4:12

KaptinKrunch

28-Dec-07 4:12

Well if you look at what Random.Next() returns, that would be your first spot to check, as its returning a random number and from your code that random number is going to be all over the place.

Read this for generating random numbers:
http://msdn2.microsoft.com/en-us/library/system.random.aspx[^]

Then adjust your code accordingly.

Once you have your random number, use that as your index for your strRandomIdeas array.

Also when comparing string, you should use the string.compare method.

if(string.compare(txb1.Text, txb2.Text) == 0)<br />
{<br />
//strings are equal<br />
}<br />
else<br />
{<br />
//strings not equal<br />
}

Just because we can; does not mean we should.

Re: Generating Random Strings from an array and placing them into a textbox....

DSdragondude28-Dec-07 4:27

DSdragondude

28-Dec-07 4:27

Thankyou for the help, but the problematic code is this:

Random RandomIdea = new Random(strRandomIdeas);<br />
            txb1.Text = RandomIdea.Next();<br />
            txb2.Text = RandomIdea.Next();

Random commands are a bit confusing and this just expands the problem, I've checked the msdn site, but it didn't help, is there any way I can debug this?

Re: Generating Random Strings from an array and placing them into a textbox.... [modified]

KaptinKrunch28-Dec-07 4:42

KaptinKrunch

28-Dec-07 4:42

Rewrite your code as follows:

private void generatebut_Click(object sender, EventArgs e)<br />
{<br />
            string[] strRandomIdeas;<br />
            strRandomIdeas = new string[10];<br />
<br />
            strRandomIdeas[0] = "Bus";<br />
            strRandomIdeas[1] = "Animal";<br />
            strRandomIdeas[2] = "Surfing";<br />
            strRandomIdeas[3] = "Ride";<br />
            strRandomIdeas[4] = "Vehicle";<br />
            strRandomIdeas[5] = "House";<br />
            strRandomIdeas[6] = "Gaming";<br />
            strRandomIdeas[7] = "Flying";<br />
            strRandomIdeas[8] = "Entertainment";<br />
            strRandomIdeas[9] = "Music";<br />
<br />
            //create a random object<br />
            Random RandomIdea = new Random();<br />
<br />
            //generate a random number within the strRandomIdeas array bounds<br />
            int indexID = RandomIdea.Next(0,strRandomIdeas.Length -1);<br />
<br />
            //set the text for the first random string<br />
            string Text1 = strRandomIdeas[indexID];<br />
<br />
            //repeat for random string number 2<br />
            indexID = RandomIdea.Next(0, strRandomIdeas.Length - 1);<br />
            string Text2 = strRandomIdeas[indexID];<br />
<br />
            //compare strings<br />
            if (string.Compare(Text1, Text2) == 0)<br />
            {<br />
                indexID = RandomIdea.Next(0, strRandomIdeas.length -1);<br />
                Text2 = strRandomIdeas[indexID];<br />
            }<br />
}

This code compiles and returns your random strings.

Just because we can; does not mean we should.

modified on Friday, December 28, 2007 11:54:43 AM

Re: Generating Random Strings from an array and placing them into a textbox....

Luc Pattyn28-Dec-07 7:20

Luc Pattyn

28-Dec-07 7:20

DSdragondude wrote:
Random RandomIdea = new Random(strRandomIdeas);

The documentation clearly shows:
1. Random constructor takes either no argument or an integer.
2. Random.Next() always returns an integer.

Hence, use the array bounds to limit the range Random.Next() will use, and appply its
return value as an index in your array.

Smile | :)

Luc Pattyn [Forum Guidelines] [My Articles]

Happy Holidays!

Disposing of objects

Gareth H28-Dec-07 2:49

Gareth H

28-Dec-07 2:49

Hi,

This is more of a theory question i guess. I've recently been looking at some old code and trying to improve it. I've always been told you should dispose of objects once you've finished with them, but never have as i've only been working on small projects.

However, its came time for me to start improving on this. In one section of my code i create a List<string> and pass it to my sql statement builder. My understanding is that now i have 2 Lists in memory, one used in the sql statements method and one in the method i used to create it.

Would it be better to pass it to the sql statements method as a ref and then clear its list at the end of the method i created it in? or just pass it normally and clear it in both methods?

Regards,
Gareth.

Not Active28-Dec-07 3:35

28-Dec-07 3:35

gareth111 wrote:
create a List and pass it to my sql statement builder

If you're really interested in building better code, it would be wiser to use Stored Procedures and avoid other messes also.

only two letters away from being an asset

Gareth H28-Dec-07 3:50

Gareth H

28-Dec-07 3:50

Mark,

I have used stored procs in other projects. The reason im not using them is because i think its far easier to use a sql statement class that has 5 or so methods than to use stored procs that has 20 or so.

Regards,
Gareth

Colin Angus Mackay28-Dec-07 3:56

Colin Angus Mackay

28-Dec-07 3:56

gareth111 wrote:
The reason im not using them is because i think its far easier to use a sql statement class that has 5 or so methods than to use stored procs that has 20 or so.

What do the number of methods have to do with it?

Upcoming FREE developer events:
* Developer Day Scotland

Recent blog posts:
* Follow up on hiring a software developer
* Aarrgghh!! Accidentally pressing the Caps Lock key

My website | blog

Gareth H28-Dec-07 4:09

Gareth H

28-Dec-07 4:09

Because its easier to maintain.

Example: My project can connect to 5 different databases, which have 5 tables, so i need to create 25 stored procs just to do some simple select * from. With the sql builder class, i have 1 method.

It might just be me, but that seems like in the long run that will save me time. Their might be advantages of splitting it up so the data layer is actually handled by the db server, i am not saying there isn't.

Regards,
Gareth.