Когда ваши списки пусты, вы должны добавить первый элемент в оба списка, наименее и чаще всего заимствованные, но вы добавляете только к большинству.
Кроме того, вам не нужно каждый раз создавать элемент представления списка, вы уже создали его вне ifs. То же самое касается частоты, вам не нужно анализировать снова, поскольку у вас уже есть значение в переменной f
.
public void evaluate()
{
string bookTitle = txtBookTitle.Text;
string frequency = txtTimesBorrowed.Text;
int f = Int32.Parse(frequency);
string[] book = { bookTitle, frequency };
var lstVBookItems = new ListViewItem(book);
lstVBooks.Items.Add(lstVBookItems);
if(lstVFBooks.Items.Count == 0 && lstVLBooks.Items.Count == 0)
{
lstVFBooks.Items.Add(lstVBookItems);
lstVLBooks.Items.Add(lstVBookItems);
mostF = f;
leastF = f;
}
else if (f > mostF)
{
lstVFBooks.Items.Clear();
lstVFBooks.Items.Add(lstVBookItems);
mostF = f;
}
else if (f == mostF)
{
lstVFBooks.Items.Add(lstVBookItems);
mostF = f;
}
else if (f < mostF && f == leastF)
{
lstVLBooks.Items.Add(lstVBookItems);
leastF = f;
}
else if (f < mostF)
{
lstVLBooks.Items.Clear();
lstVLBooks.Items.Add(lstVBookItems);
leastF = f;
}
}