Re: ArrayList worked unexpectedly... - C# Discussion Boards

Does your dll use any 3rd party libraries? COM objects? Does it make P/Invoke calls into native dlls?

Tech, life, family, faith: Give me a visit.
From my latest post: "It's sobering to watch: the whole country stops for 2 minutes as a siren is blast to remember the 22,437 victims of terror and soldiers who died in defense of Israel..."
The apostle Paul, modernly speaking: Epistles of Paul

Judah Himango

Yes it does use third party libraries. I am new to c# so I do not know much about p/invoke but yes it uses p/invoke as well. I do not think it uses COM objects. Again it is working on on 64bit machine (not a vista one).

Can you post some of your P/Invoke calls? Most likely it's an issue with those.

Tech, life, family, faith: Give me a visit.
From my latest post: "It's sobering to watch: the whole country stops for 2 minutes as a siren is blast to remember the 22,437 victims of terror and soldiers who died in defense of Israel..."
The apostle Paul, modernly speaking: Epistles of Paul

Judah Himango

Here is the format of one such call.

[DllImport(dllName, ExactSpelling=true, SetLastError=false, CallingConvention=CallingConvention.Cdecl)]<br />
       internal static extern float a_function(int n, [In] float[] x, int incX);<br />

Thanks.

Yep, that looks problematic: the first value is an integer. In .NET, this will always be 4 bytes. In native code, however, this will be either 4 bytes on a 32-bit system, or 8 bytes on a 64 bit system (IIRC). Instead of an integer, pass in an IntPtr. Your function would look something like this:

[DllImport(dllName, ExactSpelling=true, SetLastError=false, CallingConvention=CallingConvention.Cdecl)]
       internal static extern float a_function(IntPtr n, [In] float[] x, IntPtr incX);

<br />

Tech, life, family, faith: Give me a visit.
From my latest post: "It's sobering to watch: the whole country stops for 2 minutes as a siren is blast to remember the 22,437 victims of terror and soldiers who died in defense of Israel..."
The apostle Paul, modernly speaking: Epistles of Paul

Judah Himango

I will try it with a new test program to see if it works. Now apparentely the code should work because i am getting error from DLL not from native code. The client program basically does not find the necessary signature or entry point inside DLL. It does not reach the native code part at all. The DLL is just a wrapper of the native code. I will give it a try though and will let you know what happens.

I'm dealing with my course project. This method handles ListView.SelectedIndexChanged, and al is an ArrayList.

<br />
        private void lvFiles_SelectedIndexChanged(object sender, EventArgs e)<br />
        {<br />
            al.Clear();<br />
            foreach (int i in lvFiles.SelectedIndices)<br />
                al.Add(exTree.SelectedPath+lvFiles.Items[i].Text);<br />
        }<br />

I want to collect the selected items in ListView when event SelectedIndexChanged is triggered. But I always get an ArrayList that contains two copies of selected items, just like this:
AAAAAA
BBBBBB
AAAAAA
BBBBBB
But what I want is:
AAAAAA
BBBBBB
Confused | :confused:

I'm totally confused....Can anyone explain this? Many thanx....

I'm not sure what could be wrong, the problem might be somewhere else. Try putting a break point on al.Clear() and step through the foreach loop checking the contents of al each time in the debugger. If you find that by some strange reasoning everything is getting added twice in the loop, then you can try a slightly different loop, like:

foreach(ListViewItem item in lvFile.SelectedItems)
   al.Add(...);

or something.

Its always possible that items are getting doubled up somewhere else, so use the debugger to find out whats going on.

My current favourite word is: Bacon!

-SK Genius

Hi,

I can't explain the doubles, every time you add or remove an item to the selection,
the event will fire but since you always clear the ArrayList that should not cause
wrong results.

However I do suggest you use SelectedItems instead of SelectedIndices, it will yield simpler code,
skipping all the index stuff.

Smile | :)

Luc Pattyn [Forum Guidelines] [My Articles]

I dislike the black-and-white voting system on questions/answers. Dead | X|

natsuyaki,

Add some Trace'ing so that you know how many items are selected, and then how many items are added to the arraylist and what those items are.

Also, if you're using .NET 2.0 upwards, i'd change the arraylist to a generic list, just makes life easier.

Regards,
Gareth.

Where do you create a new arraylist?

Is there a chance that you are creating a new copy each time hence having more than one instance of al?

Continuous effort - not strength or intelligence - is the key to unlocking our potential.(Winston Churchill)

How do you determine what the ArrayList contains?

Are you stuck with framework 1.1? Otherwise you should use a List<string> instead of an ArrayList.

Despite everything, the person most likely to be fooling you next is yourself.

Thx everyone...
al is all right.The problem is: the main thread print this list, and then another thread tried again.... Poke tongue | ;-P

Hey guys

i need to programaticly stop a sql server and then start it again

the reson for this is that i have to delete a database even if its in use, then create the database again...

or is there some other programatical way of "kicking" the connections to that database?

thanx

Harvey Saayman - South Africa
Junior Developer
.Net, C#, SQL
think BIG and kick ASS

you.suck = (you.passion != Programming)

You can try ClearAllPools() or ClearPool() to clear the ADO.NET connection pool. To restart SQL Server, you need to restart the service. Check this[^] to get an idea about starting/stopping services through code.

All C# applications should call Application.Quit(); in the beginning to avoid any .NET problems.- Unclyclopedia

How to use google | Ask smart questions

thanx

ill have a look at it

Harvey Saayman - South Africa
Junior Developer
.Net, C#, SQL
think BIG and kick ASS

you.suck = (you.passion != Programming)

I'd try this[^] and if that doesn't work for you then there's this[^]. Forgive me, both examples are in VB but converting them to C# shouldn't be too hard Smile | :)

Thanx man, it works like a charm Smile | :)

Harvey Saayman - South Africa
Junior Developer
.Net, C#, SQL
think BIG and kick ASS

you.suck = (you.passion != Programming)

i am sooo sorry i never knew how things are done here,this is my first time of askin a question,i am not in the habi....I AM SO SORRY. i am actually workin on a desktop application..
and nobody will understand my code but i want to implement it in this area
its over 5000 lines of code, i thot nobody will be interested in that sh*t..
I AM VERY SORRY IF I HAVE OFFENDED ANYONE WITH MY APPROACH..
I WILL BE VERY GLAD TO REMAIN HERE...

if (readval1.Read())
{
//assign these values into current sem sum variables
int TUT3, TUP3, COUT3;
decimal WS3, GPA3;
TUT3 = System.Convert.ToInt16(readval1.GetValue(3).ToString());
TUP3 = System.Convert.ToInt16(readval1.GetValue(4).ToString());
WS3 = System.Convert.ToDecimal(readval1.GetValue(5).ToString());
GPA3 = System.Convert.ToDecimal(readval1.GetValue(6).ToString());
COUT3 = System.Convert.ToInt16(readval1.GetValue(7).ToString());

MessageBox.Show("TUT = " + TUT3.ToString() + "TUP = " + TUP3.ToString() + "WS = " + WS3.ToString() + "GPA = " + GPA3.ToString() + "Outstanding = " + COUT3.ToString());

//add the values of previous sem to current semester
decimal CUT3, CUP3, AOUT3;
decimal CWS3, CGPA3;
CUT3 = (TUT3 + PTUT2);
CUP3 = (TUP3 + PTUP2);
CWS3 = (WS3 + PCWS2);
CGPA3 = Math.Round((CWS3 / CUT3), 2);
AOUT3 = System.Convert.ToInt32((CUT3 - CUP3));

MessageBox.Show("CUT =" + CUT3 + " CUP = " + CUP3 + " CWS =" + CWS3 + " CGPA =" + CGPA3 + " OUT =" + AOUT3);

//update All Semester Summary
string updalltab = "UPDATE AllSemesterSummary SET Levels = '200', Semester = '2', CumulativeUnitTaken = '" + CUT3 + "', CumulativeUnitPassed = '" + CUP3 + "', CumulativeWeightedScore = '" + CWS3 + "', CumulativeGradePointAverage = '" + CGPA3 + "', OutstandingUnits = '" + AOUT3 + "' WHERE MatricNumber = '" + matnumber + "' ";

Oluwayomi Ige
MCP MCAD.

Please ignore the duplicate post Hmmm | :|

Best Regards
-----------------
Abhijit Jana
"Success is Journey it's not a destination"

This is about as bad as code gets, plus you have not actually asked a question - you've just posted your crap repeatidly.

Stick to 419's!

Jamie is correct. This is some really nasty code and you haven't asked a question about it.

Oluwayomi wrote:
and nobody will understand my code

So, you're saying that you're writing code like this in an effort to build "job security" for yourself?? Thank God, 'cause I wouldn't want to be the poor bastard coming in behind you to support this code.

A guide to posting questions on CodeProject[^]

Dave Kreskowiak
Microsoft MVP
Visual Developer - Visual Basic
2006, 2007, 2008

i am sooo sorry i never knew how things are done here,this is my first time of askin a question,i am not in the habi....I AM SO SORRY. i am actually workin on a desktop application..
and nobody will understand my code but i want to implement it in this area
its over 5000 lines of code, i thot nobody will be interested in that sh*t..
I AM VERY SORRY IF I HAVE OFFENDED ANYONE WITH MY APPROACH..
I WILL BE VERY GLAD TO REMAIN HERE...

if (readval1.Read())
{
//assign these values into current sem sum variables
int TUT3, TUP3, COUT3;
decimal WS3, GPA3;
TUT3 = System.Convert.ToInt16(readval1.GetValue(3).ToString());
TUP3 = System.Convert.ToInt16(readval1.GetValue(4).ToString());
WS3 = System.Convert.ToDecimal(readval1.GetValue(5).ToString());
GPA3 = System.Convert.ToDecimal(readval1.GetValue(6).ToString());
COUT3 = System.Convert.ToInt16(readval1.GetValue(7).ToString());

MessageBox.Show("TUT = " + TUT3.ToString() + "TUP = " + TUP3.ToString() + "WS = " + WS3.ToString() + "GPA = " + GPA3.ToString() + "Outstanding = " + COUT3.ToString());

//add the values of previous sem to current semester
decimal CUT3, CUP3, AOUT3;
decimal CWS3, CGPA3;
CUT3 = (TUT3 + PTUT2);
CUP3 = (TUP3 + PTUP2);
CWS3 = (WS3 + PCWS2);
CGPA3 = Math.Round((CWS3 / CUT3), 2);
AOUT3 = System.Convert.ToInt32((CUT3 - CUP3));

MessageBox.Show("CUT =" + CUT3 + " CUP = " + CUP3 + " CWS =" + CWS3 + " CGPA =" + CGPA3 + " OUT =" + AOUT3);

//update All Semester Summary
string updalltab = "UPDATE AllSemesterSummary SET Levels = '200', Semester = '2', CumulativeUnitTaken = '" + CUT3 + "', CumulativeUnitPassed = '" + CUP3 + "', CumulativeWeightedScore = '" + CWS3 + "', CumulativeGradePointAverage = '" + CGPA3 + "', OutstandingUnits = '" + AOUT3 + "' WHERE MatricNumber = '" + matnumber + "' ";

Oluwayomi Ige
MCP MCAD.

Oluwayomi wrote:
I AM VERY SORRY IF I HAVE OFFENDED ANYONE WITH MY APPROACH..
I WILL BE VERY GLAD TO REMAIN HERE...

You didn't, and I apologise if you thought it was personal. There are some rules when posting in forums, and things can get a bit heated (read sarcastic) if these rules aren't adhered to. The first rule is that we don't do work for others. We'll help if they are stuck on a particular problem, but we won't write solutions. Secondly, we need decent descriptions of the problem - only then can we help. Third, we need to know that you've tried to solve the problem for yourself.

Tick all those boxes and you'll find your time on the boards a much more pleasant one.

Deja View - the feeling that you've seen this post before.

My blog | My articles

Oluwayomi wrote:
(To whom it may concern)SEE THIS ITS IMPORTANT

Try to put some useful information in the subject line, so that it says anything about the question. That subject line contains no useful information at all.

Oluwayomi wrote:
I AM VERY SORRY IF I HAVE OFFENDED ANYONE WITH MY APPROACH..
I WILL BE VERY GLAD TO REMAIN HERE...

You are very welcome to stay here, if you just STOP SCREAMING! Wink | ;)

Oluwayomi wrote:
nobody will understand my code

Perhaps if you said anything at all about what the code is supposed to do...?

Oluwayomi wrote:
int TUT3, TUP3, COUT3;

Don't be afraid to use variable names that are somewhat longer. If you can see from the variable name what the variable is supposed to be used for, it's easier to understand the code, and easier to find errors.

Oluwayomi wrote:
UT3 = System.Convert.ToInt16(readval1.GetValue(3).ToString());

An int is not an Int16, it's an Int32.

Why are you getting the value as a string, then convert it to a number? Do you store everything as text in the database?

Despite everything, the person most likely to be fooling you next is yourself.