Making code more flexible

How to be diplomatic in refusing to write code that breaches the privacy of our users

Is a roofing delivery truck likely to crack my driveway slab?

Is there a problem with hiding "forgot password" until it's needed?

Personal Teleportation as a Weapon

What is difference between behavior and behaviour

Is HostGator storing my password in plaintext?

Was the picture area of a CRT a parallelogram (instead of a true rectangle)?

Where in the Bible does the greeting ("Dominus Vobiscum") used at Mass come from?

Have I saved too much for retirement so far?

Increase performance creating Mandelbrot set in python

Is the destination of a commercial flight important for the pilot?

when is out of tune ok?

Is there any easy technique written in Bhagavad GITA to control lust?

Why are on-board computers allowed to change controls without notifying the pilots?

How do I keep an essay about "feeling flat" from feeling flat?

There is only s̶i̶x̶t̶y one place he can be

Can I Retrieve Email Addresses from BCC?

apt-get update is failing in debian

What will be the benefits of Brexit?

How was Earth single-handedly capable of creating 3 of the 4 gods of chaos?

Hostile work environment after whistle-blowing on coworker and our boss. What do I do?

Your magic is very sketchy

How do I rename a LINUX host without needing to reboot for the rename to take effect?

Student evaluations of teaching assistants



Making code more flexible














0












$begingroup$


Basically, I am loading some information about the selected vendor using mysql query. I made a method with a List as return type, then created another method to assign values coming from a list. I am wondering if there is a better way of doing things.



P.S. The vendorCrud object is a database helper.



 private List<string> GetVendorInformation(string vendorId)

crud vendorCrud = new crud();
List<string> columnData = new List<string>();

string query = String.Format("SELECT tin, email_add, fax_no, tel_no, contact.contact_person FROM lib_inv_vendor vendor " +
"LEFT JOIN lib_inv_vendor_contact_person contact ON vendor.vendor_id = contact.vendor_id " +
"WHERE vendor.vendor_id = 0 ORDER BY contact_default DESC LIMIT 1", vendorId);

vendorCrud.load_data(query, Modules.connection.filesetupDB1, CommandType.Text);

foreach (DataRow item in vendorCrud.table.Rows)

columnData.Add(item["tin"].ToString());
columnData.Add(item["email_add"].ToString());
columnData.Add(item["fax_no"].ToString());
columnData.Add(item["tel_no"].ToString());
columnData.Add(item["contact_person"].ToString());


return columnData;


private void SetVendorInformation(List<string> colData)

txtVendorTin.EditValue = colData[0];
txtEmail.EditValue = colData[1];
txtFaxNo.EditValue = colData[2];
txtTelNo.EditValue = colData[3];
txtAttention.EditValue = colData[4];


private void txtSupplier_EditValueChanged(object sender, EventArgs e)

SetVendorInformation(GetVendorInformation(txtSupplier.EditValue.ToString()));









share







New contributor




Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
Check out our Code of Conduct.







$endgroup$
















    0












    $begingroup$


    Basically, I am loading some information about the selected vendor using mysql query. I made a method with a List as return type, then created another method to assign values coming from a list. I am wondering if there is a better way of doing things.



    P.S. The vendorCrud object is a database helper.



     private List<string> GetVendorInformation(string vendorId)

    crud vendorCrud = new crud();
    List<string> columnData = new List<string>();

    string query = String.Format("SELECT tin, email_add, fax_no, tel_no, contact.contact_person FROM lib_inv_vendor vendor " +
    "LEFT JOIN lib_inv_vendor_contact_person contact ON vendor.vendor_id = contact.vendor_id " +
    "WHERE vendor.vendor_id = 0 ORDER BY contact_default DESC LIMIT 1", vendorId);

    vendorCrud.load_data(query, Modules.connection.filesetupDB1, CommandType.Text);

    foreach (DataRow item in vendorCrud.table.Rows)

    columnData.Add(item["tin"].ToString());
    columnData.Add(item["email_add"].ToString());
    columnData.Add(item["fax_no"].ToString());
    columnData.Add(item["tel_no"].ToString());
    columnData.Add(item["contact_person"].ToString());


    return columnData;


    private void SetVendorInformation(List<string> colData)

    txtVendorTin.EditValue = colData[0];
    txtEmail.EditValue = colData[1];
    txtFaxNo.EditValue = colData[2];
    txtTelNo.EditValue = colData[3];
    txtAttention.EditValue = colData[4];


    private void txtSupplier_EditValueChanged(object sender, EventArgs e)

    SetVendorInformation(GetVendorInformation(txtSupplier.EditValue.ToString()));









    share







    New contributor




    Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
    Check out our Code of Conduct.







    $endgroup$














      0












      0








      0





      $begingroup$


      Basically, I am loading some information about the selected vendor using mysql query. I made a method with a List as return type, then created another method to assign values coming from a list. I am wondering if there is a better way of doing things.



      P.S. The vendorCrud object is a database helper.



       private List<string> GetVendorInformation(string vendorId)

      crud vendorCrud = new crud();
      List<string> columnData = new List<string>();

      string query = String.Format("SELECT tin, email_add, fax_no, tel_no, contact.contact_person FROM lib_inv_vendor vendor " +
      "LEFT JOIN lib_inv_vendor_contact_person contact ON vendor.vendor_id = contact.vendor_id " +
      "WHERE vendor.vendor_id = 0 ORDER BY contact_default DESC LIMIT 1", vendorId);

      vendorCrud.load_data(query, Modules.connection.filesetupDB1, CommandType.Text);

      foreach (DataRow item in vendorCrud.table.Rows)

      columnData.Add(item["tin"].ToString());
      columnData.Add(item["email_add"].ToString());
      columnData.Add(item["fax_no"].ToString());
      columnData.Add(item["tel_no"].ToString());
      columnData.Add(item["contact_person"].ToString());


      return columnData;


      private void SetVendorInformation(List<string> colData)

      txtVendorTin.EditValue = colData[0];
      txtEmail.EditValue = colData[1];
      txtFaxNo.EditValue = colData[2];
      txtTelNo.EditValue = colData[3];
      txtAttention.EditValue = colData[4];


      private void txtSupplier_EditValueChanged(object sender, EventArgs e)

      SetVendorInformation(GetVendorInformation(txtSupplier.EditValue.ToString()));









      share







      New contributor




      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.







      $endgroup$




      Basically, I am loading some information about the selected vendor using mysql query. I made a method with a List as return type, then created another method to assign values coming from a list. I am wondering if there is a better way of doing things.



      P.S. The vendorCrud object is a database helper.



       private List<string> GetVendorInformation(string vendorId)

      crud vendorCrud = new crud();
      List<string> columnData = new List<string>();

      string query = String.Format("SELECT tin, email_add, fax_no, tel_no, contact.contact_person FROM lib_inv_vendor vendor " +
      "LEFT JOIN lib_inv_vendor_contact_person contact ON vendor.vendor_id = contact.vendor_id " +
      "WHERE vendor.vendor_id = 0 ORDER BY contact_default DESC LIMIT 1", vendorId);

      vendorCrud.load_data(query, Modules.connection.filesetupDB1, CommandType.Text);

      foreach (DataRow item in vendorCrud.table.Rows)

      columnData.Add(item["tin"].ToString());
      columnData.Add(item["email_add"].ToString());
      columnData.Add(item["fax_no"].ToString());
      columnData.Add(item["tel_no"].ToString());
      columnData.Add(item["contact_person"].ToString());


      return columnData;


      private void SetVendorInformation(List<string> colData)

      txtVendorTin.EditValue = colData[0];
      txtEmail.EditValue = colData[1];
      txtFaxNo.EditValue = colData[2];
      txtTelNo.EditValue = colData[3];
      txtAttention.EditValue = colData[4];


      private void txtSupplier_EditValueChanged(object sender, EventArgs e)

      SetVendorInformation(GetVendorInformation(txtSupplier.EditValue.ToString()));







      c# mysql





      share







      New contributor




      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.










      share







      New contributor




      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.








      share



      share






      New contributor




      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.









      asked 2 mins ago









      Vish GuerreroVish Guerrero

      11




      11




      New contributor




      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.





      New contributor





      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.






      Vish Guerrero is a new contributor to this site. Take care in asking for clarification, commenting, and answering.
      Check out our Code of Conduct.




















          0






          active

          oldest

          votes











          Your Answer





          StackExchange.ifUsing("editor", function ()
          return StackExchange.using("mathjaxEditing", function ()
          StackExchange.MarkdownEditor.creationCallbacks.add(function (editor, postfix)
          StackExchange.mathjaxEditing.prepareWmdForMathJax(editor, postfix, [["\$", "\$"]]);
          );
          );
          , "mathjax-editing");

          StackExchange.ifUsing("editor", function ()
          StackExchange.using("externalEditor", function ()
          StackExchange.using("snippets", function ()
          StackExchange.snippets.init();
          );
          );
          , "code-snippets");

          StackExchange.ready(function()
          var channelOptions =
          tags: "".split(" "),
          id: "196"
          ;
          initTagRenderer("".split(" "), "".split(" "), channelOptions);

          StackExchange.using("externalEditor", function()
          // Have to fire editor after snippets, if snippets enabled
          if (StackExchange.settings.snippets.snippetsEnabled)
          StackExchange.using("snippets", function()
          createEditor();
          );

          else
          createEditor();

          );

          function createEditor()
          StackExchange.prepareEditor(
          heartbeatType: 'answer',
          autoActivateHeartbeat: false,
          convertImagesToLinks: false,
          noModals: true,
          showLowRepImageUploadWarning: true,
          reputationToPostImages: null,
          bindNavPrevention: true,
          postfix: "",
          imageUploader:
          brandingHtml: "Powered by u003ca class="icon-imgur-white" href="https://imgur.com/"u003eu003c/au003e",
          contentPolicyHtml: "User contributions licensed under u003ca href="https://creativecommons.org/licenses/by-sa/3.0/"u003ecc by-sa 3.0 with attribution requiredu003c/au003e u003ca href="https://stackoverflow.com/legal/content-policy"u003e(content policy)u003c/au003e",
          allowUrls: true
          ,
          onDemand: true,
          discardSelector: ".discard-answer"
          ,immediatelyShowMarkdownHelp:true
          );



          );






          Vish Guerrero is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216306%2fmaking-code-more-flexible%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown

























          0






          active

          oldest

          votes








          0






          active

          oldest

          votes









          active

          oldest

          votes






          active

          oldest

          votes








          Vish Guerrero is a new contributor. Be nice, and check out our Code of Conduct.









          draft saved

          draft discarded


















          Vish Guerrero is a new contributor. Be nice, and check out our Code of Conduct.












          Vish Guerrero is a new contributor. Be nice, and check out our Code of Conduct.











          Vish Guerrero is a new contributor. Be nice, and check out our Code of Conduct.














          Thanks for contributing an answer to Code Review Stack Exchange!


          • Please be sure to answer the question. Provide details and share your research!

          But avoid


          • Asking for help, clarification, or responding to other answers.

          • Making statements based on opinion; back them up with references or personal experience.

          Use MathJax to format equations. MathJax reference.


          To learn more, see our tips on writing great answers.




          draft saved


          draft discarded














          StackExchange.ready(
          function ()
          StackExchange.openid.initPostLogin('.new-post-login', 'https%3a%2f%2fcodereview.stackexchange.com%2fquestions%2f216306%2fmaking-code-more-flexible%23new-answer', 'question_page');

          );

          Post as a guest















          Required, but never shown





















































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown

































          Required, but never shown














          Required, but never shown












          Required, but never shown







          Required, but never shown







          Popular posts from this blog

          瀋陽號驅逐艦 目录 接收與服役 配置反潛直升機 武進三型性能升級 歷史 除役 參考資料 外部連結 导航菜单Taiwan Air Power海疆老兵-陽字號驅逐艦沿革World Navies Today: Taiwan (Republic of China)DD-839 USS POWER

          波兰旗帜列表 目录 国旗 军旗 其他制服部门旗帜 特别国家机构船只 参考文献 外部链接 导航菜单Polskie flagi, chorągwie, bandery... [波兰旗帜、条幅、船旗等]原始内容Ustawa z dnia 31 stycznia 1980 r. o godle, barwach i hymnie Rzeczypospolitej Polskiej oraz o pieczęciach państwowychZarządzenie Ministra Obrony Narodowej z dnia 14 grudnia 2005 r. zmieniające zarządzenie w sprawie szczegółowych zasad używania znaków Sił Zbrojnych Rzeczypospolitej Polskiej oraz ustalenia innych znaków używanych w Siłach Zbrojnych Rzeczypospolitej PolskiejZarządzenie Ministra Obrony Narodowej z dnia 29 stycznia 1996 r. w sprawie szczegółowych zasad używania znaków Sił Zbrojnych Rzeczypospolitej Polskiej oraz ustalenia innych znaków używanych w Siłach Zbrojnych Rzeczypospolitej PolskiejUstawa z dnia 19 lutego 1993 r. o znakach Sił Zbrojnych Rzeczypospolitej PolskiejHistoria Marynarki Wojennej RP [波兰海军史]Rozporządzenie Ministra Spraw Wewnętrznych i Administracji z dnia 12 kwietnia 2002 r. w sprawie wzoru flagi oraz oznakowania jednostek pływających i statków powietrznych Straży GranicznejRozporządzenie Ministra Spraw Wewnętrznych i Administracji z dnia 18 kwietnia 2005 r. w sprawie wzoru flagi oraz oznakowania jednostek pływających i statków powietrznych PolicjiRozporządzenie Ministra Infrastruktury z dnia 21 października 2005 r. w sprawie wzorów flag dla statków morskich na oznaczenie pełnionej specjalnej służby państwowej oraz okoliczności i warunków ich podnoszenia波兰旗帜波兰

          Indenting and Dedenting ASP code with Python