How do I display the status paid after encode the amount when the current status is not paid

Question

0.00/5 (No votes)

See more:

my problem is that when after I encode the pay of the tenant will display the status that is paid but the display still is not paid.

What I have tried:

C#

private void cbName_TextChanged(object sender, EventArgs e)
        {
            try
            {
                dataGridViewPayment.Columns.Clear();

                int cbid = Int32.Parse(cbName.SelectedValue.ToString());
                con.Close();
                cmd = new MySqlCommand("select * from tenant  WHERE id =  " + cbid, con.con);
                con.Open();
                reader = cmd.ExecuteReader();

                if (reader.Read())
                {
                    tbBusinessName.Text = reader["BusinessName"].ToString();
                }



                //fill gridview payment ledger
                con.Close();
                cmd = new MySqlCommand("SELECT a.id,a.Contractid,a.Schedule,a.Amort,IFNULL(Penalty, 0 ) AS penalty,a.Status FROM ledgercard AS a JOIN contract AS b  ON a.contractid = b.id WHERE b.TenantID ='" + cbName.SelectedValue.ToString() + "'", con.con);
                con.Open();
                reader = cmd.ExecuteReader();

                DataTable ledger = new DataTable();
                ledger.Columns.Add("Schedule", typeof(DateTime));
                ledger.Columns.Add("Amort", typeof(string));
                ledger.Columns.Add("Penalty", typeof(string));
                ledger.Columns.Add("Status", typeof(string));

                string Amort = "0.00";
                string Penalty = "0.00";
                string Stat = "";

                while (reader.Read())
                {
                    contractid = Int32.Parse(reader["Contractid"].ToString());
                 


                    // for amort
                    if (totalamort > 0)
                    {
                        if (totalamort >= Decimal.Parse(reader["Amort"].ToString()))
                        {
                            totalamort -= Decimal.Parse(reader["Amort"].ToString());
                        }
                        else
                        {
                            Decimal paybal = Decimal.Parse(reader["Amort"].ToString()) - totalamort;
                            Amort = paybal.ToString();
                            totalamort = 0;
                        }
                    }
                    else
                    {
                        Amort = reader["Amort"].ToString();
                    }


                    // for penalty
                    if (totalpenalty > 0)
                    {
                        if (totalpenalty >= Decimal.Parse(reader["Penalty"].ToString()))
                        {
                            totalpenalty -= Decimal.Parse(reader["Penalty"].ToString());
                        }
                        else
                        {
                            Decimal paybal = Decimal.Parse(reader["Penalty"].ToString()) - totalpenalty;
                            Penalty = paybal.ToString();
                            totalpenalty = 0;
                        }
                    }
                    else
                    {
                        Penalty = reader["Penalty"].ToString();
                    }


                    // for stats
                    if (totalamort == decimal.Parse("0.00") && totalpenalty == decimal.Parse("0.00"))
                    {
                        Stat = "PAID";
                    }
                    else
                    { 
                        Stat = "NOT PAID";
                    }
                       
                   

                    // gridview row entries
                    ledger.Rows.Add(reader["Schedule"].ToString(),
                        Amort.ToString(),
                        Penalty.ToString(),
                        Stat);
                    

                    dataGridViewPayment.DataSource = ledger;
                }
                reader.Close();

                //------------------------------------------------------------

                decimal totalpen = 0;
                decimal totalAmrt = 0;
                decimal totalBal = 0;

                foreach(DataGridViewRow rw in dataGridViewPayment.Rows)
                {
                    totalpen += Convert.ToDecimal(rw.Cells[2].Value.ToString());
                    totalAmrt += Convert.ToDecimal(rw.Cells[1].Value.ToString());
                    totalBal = totalpen + totalAmrt;
                }        
                tbPenalty.Text = totalpen.ToString();
                tbAmortization.Text = totalAmrt.ToString();
                tbTotalBalance.Text = totalBal.ToString();

Posted 22-Sep-22 22:29pm

Marc Chester Ferrer

Updated 22-Sep-22 23:48pm

OriginalGriff

v3

Add a Solution

Comments

Richard MacCutchan 23-Sep-22 4:53am

That code does very little and gives no clue as to your problem. Please use the Improve question link above, and add complete details of what is not working.

Marc Chester Ferrer 23-Sep-22 4:56am

I did improve my question

Richard MacCutchan 23-Sep-22 5:16am

You also need to explain exactly what the problem is, and which part of that unformatted code causes it. To make your code readable please add <pre></pre> tags around it.

OriginalGriff 23-Sep-22 5:40am

I just did that for him.

Richard MacCutchan 23-Sep-22 5:46am

You are too kind (but I already knew that).

Member 15627495 23-Sep-22 5:15am

in your "for each" , put a if statement like do "paid / not paid"

you know you can dynamically fill your datagrid , instead of using 'datasource' , within your for each.

for each custom_row in ledger ][datagrid.rows.addrange(custom_row) ][next

Richard Deeming 23-Sep-22 8:10am

cmd = new MySqlCommand("SELECT a.id,a.Contractid,a.Schedule,a.Amort,IFNULL(Penalty, 0 ) AS penalty,a.Status FROM ledgercard AS a JOIN contract AS b  ON a.contractid = b.id WHERE b.TenantID ='" + cbName.SelectedValue.ToString() + "'", con.con);

Don't do it like that! Your code is vulnerable to SQL Injection[^]. NEVER use string concatenation/interpolation to build a SQL query. ALWAYS use a parameterized query.

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

OriginalGriff · Accepted Answer · 2022-09-22T23:48:00

Solution 1

We can't tell: we have no idea what totalAmort or totalpenalty are, or even why type they are. And I'm not convinced you do either, given code like this:

C#

if (totalamort == decimal.Parse("0.00") && totalpenalty == decimal.Parse("0.00"))

If they are decimal values, why not use the decimal value literal indicator: Floating-point numeric types - C# reference | Microsoft Learn[^]

C#

if (totalamort == 0.00m && totalpenalty == 0.00m)

Is a whole load more readable ...

Use the debugger to find out exactly what is actually going on, and what values your variables actually contain.

Posted 22-Sep-22 23:48pm

OriginalGriff

Comments

Marc Chester Ferrer 23-Sep-22 21:01pm

totalamort that is the total of amortization and the totalpenalty that is the total penalty
and their type is decimal but I convert it to string

OriginalGriff 24-Sep-22 1:01am

Did you look at what I said, or just read the first sentence?

Marc Chester Ferrer 24-Sep-22 1:49am

what I mean is if the tenant pays his/her due first of the month that month will show the status paid then next month will still not be paid in terms of 24 months. the output is the number of months will minus the remaining months 23. but thank you so much for helping me with my problem