Я працюю над формою Windows, щоб обчислити UPC для номерів елементів.
Я успішно створюю той, який буде обробляти один номер елемента / UPC одночасно, тепер я хочу розширити і зробити це для кількох номерів елементів / UPC.
Я почав і спробував використовувати список, але я продовжую зациклюватися. Я створив клас помічників:
public class Codes
{
private string incrementedNumber;
private string checkDigit;
private string wholeNumber;
private string wholeCodeNumber;
private string itemNumber;
public Codes(string itemNumber, string incrementedNumber, string checkDigit, string wholeNumber, string wholeCodeNumber)
{
this.incrementedNumber = incrementedNumber;
this.checkDigit = checkDigit;
this.wholeNumber = wholeNumber;
this.wholeCodeNumber = wholeCodeNumber;
this.itemNumber = itemNumber;
}
public string ItemNumber
{
get { return itemNumber; }
set { itemNumber = value; }
}
public string IncrementedNumber
{
get { return incrementedNumber; }
set { incrementedNumber = value; }
}
public string CheckDigit
{
get { return checkDigit; }
set { checkDigit = value; }
}
public string WholeNumber
{
get { return wholeNumber; }
set { wholeNumber = value; }
}
public string WholeCodeNumber
{
get { return wholeCodeNumber; }
set { wholeCodeNumber = value; }
}
}
Тоді я розпочав свій код, але проблема полягає в тому, що процес поступовий, тобто я отримую номер елемента з gridview через прапорці і поміщую їх у список. Потім я дістаю останній UPC з бази даних, знімаю контрольну цифру, потім збільшую число на одне і ставлю його до списку. Тоді я обчислюю контрольну цифру для нового номера і заношу це у список. І ось я вже отримую виняток із пам'яті. Ось код, який у мене є:
List<Codes> ItemNumberList = new List<Codes>();
private void buttonSearch2_Click(object sender, EventArgs e)
{
//Fill the datasets
this.immasterTableAdapter.FillByWildcard(this.alereDataSet.immaster, (textBox5.Text));
this.upccodeTableAdapter.FillByWildcard(this.hangtagDataSet.upccode, (textBox5.Text));
this.uPCTableAdapter.Fill(this.uPCDataSet.UPC);
string searchFor = textBox5.Text;
int results = 0;
DataRow[] returnedRows;
returnedRows = uPCDataSet.Tables["UPC"].Select("ItemNumber = '" + searchFor + "2'");
results = returnedRows.Length;
if (results > 0)
{
MessageBox.Show("This item number already exists!");
textBox5.Clear();
//clearGrids();
}
else
{
//textBox4.Text = dataGridView1.Rows[0].Cells[1].Value.ToString();
MessageBox.Show("Item number is unique.");
}
}
public void checkMarks()
{
for (int i = 0; i < dataGridView7.Rows.Count; i++)
{
if ((bool)dataGridView7.Rows[i].Cells[3].FormattedValue)
{
{
ItemNumberList.Add(new Codes(dataGridView7.Rows[i].Cells[0].Value.ToString(), "", "", "", ""));
}
}
}
}
public void multiValue1()
{
_value = uPCDataSet.UPC.Rows[uPCDataSet.UPC.Rows.Count - 1]["UPCNumber"].ToString();//get last UPC from database
_UPCNumber = _value.Substring(0, 11);//strip out the check-digit
_UPCNumberInc = Convert.ToInt64(_UPCNumber);//convert the value to a number
for (int i = 0; i < ItemNumberList.Count; i++)
{
_UPCNumberInc = _UPCNumberInc + 1;
_UPCNumberIncrement = Convert.ToString(_UPCNumberInc);//assign the incremented value to a new variable
ItemNumberList.Add(new Codes("", _UPCNumberIncrement, "", "", ""));//**here I get the OutOfMemoreyException**
}
for (int i = 0; i < ItemNumberList.Count; i++)
{
long chkDigitOdd;
long chkDigitEven;
long chkDigitSubtotal;
chkDigitOdd = Convert.ToInt64(_UPCNumberIncrement.Substring(0, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(2, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(4, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(6, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(8, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(10, 1));
chkDigitOdd = (3 * chkDigitOdd);
chkDigitEven = Convert.ToInt64(_UPCNumberIncrement.Substring(1, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(3, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(5, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(7, 1)) + Convert.ToInt64(_UPCNumberIncrement.Substring(9, 1));
chkDigitSubtotal = (300 - (chkDigitEven + chkDigitOdd));
_chkDigit = chkDigitSubtotal.ToString();
_chkDigit = _chkDigit.Substring(_chkDigit.Length - 1, 1);
ItemNumberList.Add(new Codes("", "",_chkDigit, "", ""));
}
Це правильний шлях для вирішення цього питання, використовуючи список, чи я повинен дивитись на інший шлях?
Використання списку чудово. Ітерація списку під час додавання до нього є надійним способом підірвати ваш код і вказує на проблему у вашій логіці (або написанні коду). Крім того, це ваша помилка і, швидше за все, не допоможе майбутнім відвідувачам. Голосування про закриття.
—
Теластин
Зауважимо, що всі ці приватні поля (у вашому
—
Конрад Моравський
Codeкласі) є зайвими, і нічого, крім шуму, насправді { get; private set; }не вистачить.
Чи справді назва запитання для цього справді точна? Це насправді не схоже на питання щодо списку проти масиву , настільки, як як я можу покращити моє питання щодо реалізації . Це означає , що якщо ви додаєте або видаляєте елементи, вам потрібен список (або інша гнучка структура даних). Масиви справді хороші лише тоді, коли ви точно знаєте, скільки елементів вам потрібно на старті.
—
KChaloux
@KChaloux, це майже все, що я хотів знати. Чи є перелік правильним шляхом для цього, чи я повинен був розглянути інший спосіб досягти цього? Тож здається, що список є гарним способом, я просто повинен коригувати свою логіку.
—
kampgnolo_1
@Telastyn, я не так просив покращити свій код, як показати, що я намагаюся зробити.
—
kampgnolo_1