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

Popular posts from this blog

c# - Validate object ID from GET to POST -

node.js - Custom Model Validator SailsJS -

php - Find a regex to take part of Email -