항목 번호에 대한 UPC를 계산하기 위해 Windows 양식을 작성 중입니다.
한 번에 하나의 품목 번호 / 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, "", ""));
}이것이 목록을 사용하여 그것에 관한 올바른 방법입니까, 아니면 다른 방법을보고해야합니까?
목록을 사용하는 것이 좋습니다. 추가하는 동안 목록을 반복하는 것은 코드를 폭파시키는 확실한 방법이며 논리 (또는 코드 작성)에 문제가 있음을 나타냅니다. 또한 이것은 버그이며 향후 방문자를 도울 가능성이 없습니다. 마감 투표.
—
Telastyn
부수적으로, (귀하의
—
Konrad Morawski
Code클래스에서) 이러한 모든 개인 필드 는 중복되며 실제로 소음 { get; private set; }만으로 충분합니다.
이것에 대한 질문 제목이 정말 정확합니까? 이것은 실제로 구현 질문을 개선 하는 방법 만큼 목록 대 배열 질문 처럼 보이지 않습니다 . 즉, 요소를 추가 하거나 제거 하는 경우 목록 (또는 다른 유연한 데이터 구조)을 원합니다. 배열은 처음에 필요한 요소 수를 정확히 알고있을 때만 유용합니다.
—
KChaloux
@ KChaloux, 그것은 내가 알고 싶었던 것입니다. 목록이 이것에 관한 올바른 방법입니까, 아니면 이것을 추구하는 다른 방법을 보았어야합니까? 목록이 좋은 방법 인 것 같습니다. 논리를 조정하면됩니다.
—
campagnolo_1 '12
@Telastyn, 나는 내가하려는 일을 보여주기 위해 내 코드를 개선하도록 요구하지 않았습니다.
—
campagnolo_1 '12