Well I'm working some problems in one of my C# texts and I'm practicing redoing an exercise without looking at the source code.
It's a windows form program where guys exchange cash with themselves and a bank.
When I wrote the button code for the two guys to exchange cash it came out different from what the book's solution is. I know there are more than one way's to get a result. My code, I believe, works exactly as the books.
I'd appreciate any comments on whether my function call is 'preferred' over the book's method or vice versa. I put the the suggested book's solution in //comments. The call is at the bottom of the form "bob.Cash where the result is Joe gives $10 to Bob.
using System;
using System.Windows.Forms;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
namespace WindowsFormsApplication1
{
public partial class Form1 : Form
{
guy joe;
guy bob;
int bank=100;
public Form1()
{
InitializeComponent();
joe = new guy();
joe.Cash=50;
joe.name="Joe";
bob = new guy();
bob.Cash = 50;
bob.name = "Bob";
UpdateForm();
}
public void UpdateForm()
{
joescash.Text = joescash.Name + "has $" + joe.Cash;
bobscash.Text = bobscash.Name + "has $" + bob.Cash;
bankcash.Text = bankcash.Name + "has $" + bank;
}
private void Form1_Load(object sender, EventArgs e)
{
}
private void button1_Click(object sender, EventArgs e)
{
if (bank >= 10)
{
bank -= joe.ReceiveCash(10);
UpdateForm();
}
else
{
MessageBox.Show("The bank is low on money");
}
}
private void button2_Click(object sender, EventArgs e)
{
bank += bob.GiveCash(5);
UpdateForm();
}
private void joescash_Click(object sender, EventArgs e)
{
}
private void bobscash_Click(object sender, EventArgs e)
{
}
private void bankcash_Click(object sender, EventArgs e)
{
}
private void button3_Click(object sender, EventArgs e)
{
bob.Cash += joe.GiveCash(10);
UpdateForm();
//could also write:
//bob.ReceiveCash(joe.GiveCash(10))
//UpdateForm();
}
private void button4_Click(object sender, EventArgs e)
{
joe.Cash += bob.GiveCash(5);
UpdateForm();
}
}
}
Note the form calls the guy class functions from this class form
using System;
using System.Collections.Generic;
using System.Linq;
using System.Text;
using System.Windows.Forms;
namespace WindowsFormsApplication1
{
public class guy
{
public int Cash;
public string name;
public int GiveCash(int amount)
{
if (amount <= Cash && amount > 0)
{
Cash -= amount;
return amount;
}
else
{
MessageBox.Show("I don't have enough cash to give you " + amount);
return 0;
}
}
public int ReceiveCash(int amount)
{
if (amount > 0)
{
Cash += amount;
return amount;
}
else
{
MessageBox.Show(amount + "isn't an amount I'll take", name + "says");
return 0;
}
}
}
}
Appreciate any feedback, as I said I want to make sure I don't develop sloppy coding habits. Thanks kindly
