c# - How Can I Shorten This Code -
private void button4_click(object sender, eventargs e) { textbox1.text = datagridview1.rows[0].cells[0].value.tostring() + "&" + datagridview1.rows[0].cells[1].value.tostring() + "&" + datagridview1.rows[0].cells[2].value.tostring() + "&" + datagridview1.rows[0].cells[3].value.tostring() + "%" + datagridview1.rows[1].cells[0].value.tostring() + "&" + datagridview1.rows[1].cells[1].value.tostring() + "&" + datagridview1.rows[1].cells[2].value.tostring() + "&" + datagridview1.rows[1].cells[3].value.tostring() + "%" + datagridview1.rows[2].cells[0].value.tostring() + "&" + datagridview1.rows[2].cells[1].value.tostring() + "&" + datagridview1.rows[2].cells[2].value.tostring() + "&" + datagridview1.rows[2].cells[3].value.tostring() + "%" + datagridview1.rows[3].cells[0].value.tostring() + "&" + datagridview1.rows[3].cells[1].value.tostring() + "&" + datagridview1.rows[3].cells[2].value.tostring() + "&" + datagridview1.rows[3].cells[3].value.tostring() + "%" + datagridview1.rows[4].cells[0].value.tostring() + "&" + datagridview1.rows[4].cells[1].value.tostring() + "&" + datagridview1.rows[4].cells[2].value.tostring() + "&" + datagridview1.rows[4].cells[3].value.tostring() + "%" + datagridview1.rows[5].cells[0].value.tostring() + "&" + datagridview1.rows[5].cells[1].value.tostring() + "&" + datagridview1.rows[5].cells[2].value.tostring() + "&" + datagridview1.rows[5].cells[3].value.tostring() + "%" + datagridview1.rows[6].cells[0].value.tostring() + "&" + datagridview1.rows[6].cells[1].value.tostring() + "&" + datagridview1.rows[6].cells[2].value.tostring() + "&" + datagridview1.rows[6].cells[3].value.tostring() + "%" + datagridview1.rows[7].cells[0].value.tostring() + "&" + datagridview1.rows[7].cells[1].value.tostring() + "&" + datagridview1.rows[7].cells[2].value.tostring() + "&" + datagridview1.rows[7].cells[3].value.tostring() + "%" + datagridview1.rows[8].cells[0].value.tostring() + "&" + datagridview1.rows[8].cells[1].value.tostring() + "&" + datagridview1.rows[8].cells[2].value.tostring() + "&" + datagridview1.rows[8].cells[3].value.tostring() + "%" + datagridview1.rows[9].cells[0].value.tostring() + "&" + datagridview1.rows[9].cells[1].value.tostring() + "&" + datagridview1.rows[9].cells[2].value.tostring() + "&" + datagridview1.rows[9].cells[3].value.tostring() + "%" + datagridview1.rows[10].cells[0].value.tostring() + "&" + datagridview1.rows[10].cells[1].value.tostring() + "&" + datagridview1.rows[10].cells[2].value.tostring() + "&" + datagridview1.rows[10].cells[3].value.tostring() + "%" + datagridview1.rows[11].cells[0].value.tostring() + "&" + datagridview1.rows[11].cells[1].value.tostring() + "&" + datagridview1.rows[11].cells[2].value.tostring() + "&" + datagridview1.rows[11].cells[3].value.tostring() + "%" ; }
maybe rows[11] [250].. or more than.. can shorten code. please me .. sorry bad english :(..
here's version stringbuilder
, works number of rows/columns:
private void button4_click(object sender, eventargs e) { var sb = new stringbuilder(); foreach (datagridviewrow row in datagridview1.rows) { foreach (datagridviewcell cell in row.cells) sb.append(cell.value).append('&'); // replace last '&' '%' if (sb.length != 0) sb[sb.length - 1] = '%'; } textbox1.text = sb.tostring(); }
stringbuilder
build string without having allocate lot of string
objects (which immutable) , copying string data on place.
side note: should really rename variables: button4
, datagridview1
, textbox1
etc really terrible names. use meaningful instead, it'll save headaches later on.
Comments
Post a Comment