Sorting Array of objects

Robert Rohde

BoneSoft wrote:

Array.Sort returns a new array

No it doesn't - it sorts the existing instance

Robert Rohde

As you sort a Terminology array also the Terminology class should implement IComparable and not the Form.

Rizwan Rathore

BoneSoft wrote:

word = Array.Sort(word, new WordComparer());

well i ve tried this already but it gives compile time error that cant covert type void to type TextMiner.Terminology[] :(

Rizwan Rathore

sir it also didnt work :( i ve written the same code in the CompareTo function which i ve written in TextMiner class but it made no difference again same exception :( is there any way out of this ?? looking forward for help Regards,

Robert Rohde

I assume some error is thrown within your CompareTo method. You should probably check if the are null values within your array or probably if sometimes term is null.

Rizwan Rathore

well the exception doesnt come during sorting it comes after the sorting the statement i m cofused about is the use of private string strValue = "term"; isnt there any error in this statement?? i mean term is the variable of terminology class and i m declaring it as a string value in TextMiner class how will this work??

Robert Rohde

Could you please post the important part of the Terminology class and the Sort call you are currently doing? Maybe I can help you then.

Rizwan Rathore

public class Terminology : IComparable
{
private string term;
private int df;
private int qTf;
private double qWeight;
private ArrayList tf = new ArrayList();
private ArrayList position= new ArrayList();
private ArrayList docID= new ArrayList();
private static int Count;

    public int CompareTo(object obj)
{
	Terminology w = (Terminology) obj;
	return( w.term.CompareTo( w.term ));
}

//only method in this class other than properties and constructors
public void incDF()
{
df++;
}
}

now moving to TextMiner class

public class TextMiner : System.Windows.Forms.Form , IComparable
{
const int WORDS = 20000;
public Terminology[] word = new Terminology[WORDS];
public void Indexer()
{
Array.Sort(word);
}
private string strValue = "term";
public int CompareTo(object obj)
{
Terminology w = (Terminology) obj;
return( strValue.CompareTo( w.term ));
}
}

it gives this exception An unhandled exception of type 'System.NullReferenceException' occurred in 2ndAssignment.exe Additional information: Object reference not set to an instance of an object. and on this statement

for(int j = 0; j < Terminology.count-1; j++)
{
sw.Write(j+1 +"\t"+word[j].term.PadRight(20,' '));
}

and the exceptions comes when the value of j is 0....m really confused :( looking forward for help Regards, -- modified at 5:43 Wednesday 24th May, 2006

Robert Rohde

Several things: 1. In the first snippet the CompareTo should look like the following:

public int CompareTo(object obj)
{
Terminology w = (Terminology) obj;
return( this.term.CompareTo( w.term ));
}

In your implementation you compared w.term with itself. 2. The following part isn't needed (as far as I can see from your snippets):

private string strValue = "term";
public int CompareTo(object obj)
{
Terminology w = (Terminology) obj;
return( strValue.CompareTo( w.term ));
}
}

3. If you are getting such an error when j is 0 than my previous assumption that there are null values in your array seems right. The first element of the word array seems to be null. You could verify this for example with the following code:

for(int j = 0; j < Terminology.count-1; j++)
{
if (word[j] == null)
MessageBox.Show("word array at index " + j + " is null");
else if (word[j].term == null)
MessageBox.Show("term property in word array at index " + j + " is null");
sw.Write(j+1 +"\t"+word[j].term.PadRight(20,' '));
}

Rizwan Rathore

Robert Rohde wrote:

if (word[j] == null) MessageBox.Show("word array at index " + j + " is null");

Sir it is showing entire array of word as null.....but it works perfectly fine without sorting..........how can all the values changed to null??

BoneSoft

Yeah, don't know what I was thinking... Visit BoneSoft.com