Tôi đang làm việc trên một biểu mẫu cửa sổ để tính UPC cho số mục.
Tôi tạo thành công một cái sẽ xử lý một số mục / UPC tại một thời điểm, bây giờ tôi muốn mở rộng và thực hiện nó cho nhiều số mục / UPC.
Tôi đã bắt đầu và thử sử dụng một danh sách, nhưng tôi cứ bị kẹt. Tôi đã tạo một lớp trợ giúp:
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; }
}
}Sau đó, tôi đã bắt đầu sử dụng mã của mình, nhưng vấn đề là quá trình này tăng dần, nghĩa là tôi lấy số thứ tự từ một khung nhìn lưới thông qua các hộp kiểm và đưa chúng vào danh sách. Sau đó, tôi nhận được UPC cuối cùng từ cơ sở dữ liệu, tách mã kiểm tra, sau đó tăng số lượng lên một và đưa nó vào danh sách. Sau đó, tôi tính toán số kiểm tra cho số mới và đưa nó vào danh sách. Và ở đây tôi đã có được Ngoại lệ bộ nhớ. Đây là mã tôi có cho đến nay:
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, "", ""));
}Đây có phải là cách đúng đắn để sử dụng nó, sử dụng một danh sách, hay tôi nên nhìn theo một cách khác?
Sử dụng một danh sách là tốt. Lặp lại danh sách trong khi thêm vào đó là một cách chắc chắn để làm nổ mã của bạn và chỉ ra một vấn đề trong logic của bạn (hoặc viết mã). Ngoài ra, đây là lỗi của bạn và không có khả năng giúp khách truy cập trong tương lai. Bỏ phiếu để đóng.
—
Telastyn
Như một lưu ý phụ, tất cả các trường riêng này (trong
—
Konrad Morawski
Codelớp của bạn ) đều dư thừa và không có gì ngoài tiếng ồn thực sự, { get; private set; }sẽ là đủ.
Là tiêu đề câu hỏi cho điều này thực sự chính xác? Điều này thực sự không giống như một danh sách so với câu hỏi mảng , nhiều như là Làm thế nào tôi có thể cải thiện câu hỏi thực hiện của mình . Điều đó đang được nói, nếu bạn thêm hoặc xóa các thành phần, bạn muốn có một danh sách (hoặc cấu trúc dữ liệu linh hoạt khác). Mảng chỉ thực sự tốt khi bạn biết chính xác có bao nhiêu yếu tố bạn cần khi bắt đầu.
—
KChaloux
@KChaloux, đó là những gì tôi muốn biết. Là một danh sách đúng cách để đi về điều này, hay tôi nên xem xét một cách khác để theo đuổi điều này? Vì vậy, có vẻ như một danh sách là một cách tốt, tôi chỉ cần điều chỉnh logic của tôi.
—
campagnolo_1
@Telastyn, tôi không yêu cầu cải thiện mã của mình để thể hiện những gì tôi đang cố gắng làm.
—
campagnolo_1