|
You need to create object before assining it properties.
Replace first line in form1_load to this:
<br />
Patient PatientData = new Patient();<br />
Also you can turn your patient class into struct.
Best regards, Alexey.
|
|
|
|
|
Interestingly I just figured it out prior to your posting...HOWEVER....now, I'm having problems accessing the values... I'm checking the values in my immediate window and they are populating fine.... I have a function that I'm passing a parameter (the id) so I can produce the correct values in my testbox.... (see code below)....
However, I'm not getting an error....I'm just getting a 0 in my first field....any help is greatly appreciated....
<br />
private void ShowPatient(int id)<br />
{<br />
Patient PatientData = new Patient();<br />
txt1.Text = PatientData.id[id].ToString();<br />
txt2.Text = PatientData.Lname[id];<br />
txt3.Text = PatientData.Fname[id];<br />
txt4.Text = PatientData.Mname[id]; <br />
}<br />
Thank you again...
-Kay
|
|
|
|
|
Of course, you must initialize this variable first.
Please read some books, about C# and object oriented programming.
Best regards, Alexey.
|
|
|
|
|
lol..ok, is there anyone ELSE that can help me. I know I'm close and I got this far by using books....but I'm taking it beyond what the book example is and that's why I'm on this forum. If anyone can share the knowledge to address this basic problem, I'd greatly appreciate it. Thank you anyway Alex... I'll be sure to share the answer with you once I figure it out.
-Kay
|
|
|
|
|
got it...Here is the functioning code for all those who can benefit from the knowledge... Keep in mind it's just the basic functions and class.... I'm not including all the event... you'll have to fill in the holes...
<br />
private void Form1_Load(object sender, System.EventArgs e)<br />
{<br />
Patient.id[0] = 1;<br />
Patient.Lname[0] = "Smith";<br />
Patient.Fname[0] = "John";<br />
Patient.Mname[0] = "Allen";<br />
}<br />
<br />
public class Patient<br />
{<br />
public static int[] id = new int[5];<br />
public static string[] Lname = new string[5];<br />
public static string[] Fname = new string[5];<br />
public static string[] Mname = new string[5];<br />
}<br />
<br />
private void ShowPatient(int id)<br />
{<br />
txt1.Text = Patient.id[id].ToString();<br />
txt2.Text = Patient.Lname[id];<br />
txt3.Text = Patient.Fname[id];<br />
txt4.Text = Patient.Mname[id]; <br />
}<br />
-Kay
-- modified at 0:28 Friday 12th May, 2006
|
|
|
|
|
Hi!
The suggestion to read some more about OOP is a good idea IMHO.
For example, using static attributes for your Patient is plain wrong - you surely want to deal with more than 1 patient, don't you?
If you do not understand what the keyword static means or that you have to create new instances of objects before you can use them, you'll have a hard time figuring out why your code doesn't do what you want it to do.
Regards,
mav
--
Black holes are the places where god divided by 0...
|
|
|
|
|
Mav,
Do you understand what I've done? I now understand what static means(and it appears I understand it better than you) and it's not consistent with your statement. Allow me to enlighten you...
<br />
private void Form1_Load(object sender, System.EventArgs e)<br />
{<br />
Patient.id[0] = 1;<br />
Patient.Lname[0] = "Smith";<br />
Patient.Fname[0] = "John";<br />
Patient.Mname[0] = "Allen";<br />
}<br />
<br />
public class Patient<br />
{<br />
public static int[] id = new int[5];<br />
public static string[] Lname = new string[5];<br />
public static string[] Fname = new string[5];<br />
public static string[] Mname = new string[5];<br />
}<br />
This code means that I can have up to 5 values for each property in that class.
This means I can write:
<br />
Patient.id[1] = 1;<br />
Patient.Lname[1] = "Smith";<br />
Patient.Fname[1] = "John";<br />
Patient.Mname[1] = "Allen";<br />
<br />
Patient.id[2] = 1;<br />
Patient.Lname[2] = "Barker";<br />
Patient.Fname[2] = "Dave";<br />
Patient.Mname[2] = "Barry";<br />
<br />
Patient.id[3] = 1;<br />
Patient.Lname[3] = "James";<br />
Patient.Fname[3] = "Rick";<br />
Patient.Mname[3] = "Franklin";<br />
<br />
Patient.id[4] = 1;<br />
Patient.Lname[4] = "Jones";<br />
Patient.Fname[4] = "Paul";<br />
Patient.Mname[4] = "Alexander";<br />
And anytime I call my:
<br />
private void ShowPatient(int id)<br />
{<br />
txt1.Text = Patient.id[id].ToString();<br />
txt2.Text = Patient.Lname[id];<br />
txt3.Text = Patient.Fname[id];<br />
txt4.Text = Patient.Mname[id]; <br />
}<br />
It will populate the correct patient based on the id being passed in. Hope that helps you understand it better...
Also, Please don't HELP demean people by telling them to go read books. You don't know how many books that person has in front of them when they actually ask for help. You want to place an article...that's fine... but it appears that not everyone who says "Read Books" knows the answer. Truth is... I think "Read Books" is an easy way to not answer the question. This forum is for help... I respectfully ask that you save comments like that for yourself so that valueable messageboard threads aren't wasted on "Go Read a Book" replies...
However, I do appreciate your willingness to aid. This is why I'm sharing my findings with you...
-Kay
|
|
|
|
|
The root of your design problem is that you have declared a Patient class that holds an array of what would be properly conceived as individual patient instances. Your code has a severe readability issue and violates object-oriented design principles. If you're going to use that class to hold 5 static people, then a class called PatientList would indicate your intent more clearly. Save the obfuscation for later.
An approach that would be more reasonable from an OOD standpoint would be to create a class called Patient that encapsulates each individual patient's ID, first, middle, and last names, and then use the PatientList class to maintain an array of Patient objects.
Also, your use of static is highly questionable. Unless your goal is to ensure that there is only one list of patients--ever--then there's no reason not to use member fields and instantiate a single patient list.
Finally... yes, read books. I don't think the CP community is necessarily trying to insult or demean when making this suggestion. There's no reason to get started off in programming on the wrong foot. It's apparently obvious from the nature of the original question that you don't have a solid grasp on C#'s object fundamentals and object-oriented design.
These basics are going to be necessary if you ever plan on moving past "Hello, World!" and becoming a leader. Leaders are successful and secure. Followers are a dime a dozen and can easily be replaced by someone willing to do the work for half the price.
--
I've killed again, haven't I?
|
|
|
|
|
I have no problem being told to read books... but if that's all you put in the post. Then it has an insulting demeanor. If it's accompanied by help...it feels different. That's just my opinion...
As for your comments. I agree.... the Patient Class would've been better named as PatientList.
You go on to say create a class to maintain another class's array Information.
If I'm not mistaken the Code would looks something like PatientList.Patient.id[0]
Either way, I thought you might find this funny.....sometimes "Hello World" isn't such a bad thing. Sometimes it makes the most sense....
-Kay
|
|
|
|
|
If you use PatientList with an array of Patient objects, accessing the first patient's ID would be more along the lines of PatientList.Patients[0].id.
It would also be more readable and nimble when doing multiple things to a single patient:
Patient p0 = PatientList.Patients[0];<br />
p0.id = "blah";<br />
p0.middleName = "Blahblah";<br />
...
--
I've killed again, haven't I?
|
|
|
|
|
Just thought I'd show you the progress I made based on your suggestions.... It doesn't quite look like the code you suggested (it didn't compile).. but with some minor changes it worked just the same...
Thanks for the direction...
<br />
private void Form2_Load(object sender, System.EventArgs e)<br />
{<br />
ClientList C = new ClientList();<br />
C.Client[0] = new Client();<br />
C.Client[0].id = 1;<br />
C.Client[0].ClientNm = "John Allen";<br />
<br />
C.Client[1] = new Client();<br />
C.Client[1].id = 2;<br />
C.Client[1].ClientNm = "Michael Thomas";<br />
<br />
for (int i=0;i<2;i++)<br />
{<br />
MessageBox.Show (C.Client[i].id.ToString());<br />
MessageBox.Show (C.Client[i].ClientNm);<br />
}<br />
}<br />
<br />
public class Client:ClientList<br />
{<br />
public int id;<br />
public string ClientNm;<br />
}<br />
<br />
public class ClientList<br />
{<br />
public Client [] Client = new Client[5];<br />
}<br />
-Kay
-- modified at 19:51 Saturday 13th May, 2006
|
|
|
|
|
Good work; you're definitely off to a better start.
One little detail is in how you define the Client class:
public class Client:ClientList
{
public int id;
public string ClientNm;
}
The Client class should not inherit from ClientList. Inheritance supports the "is-a" relationship, but ClientList->Client is a "has-a" relationship (more, see "Inheritance"[^]). The way it's set up now, each individual Client also has its own array of Clients.
If you simply remove the :ClientList from that definition, it should behave just fine.
Also, if the array of Client objects defined in ClientList were called "Clients", it would follow common convention and have a more natural reading flow, since arrays imply an intent to hold more than just one object.
--
I've killed again, haven't I?
|
|
|
|
|
|
Thank you, you're right...that's an excellent article...
-Kay
|
|
|
|
|
The value assigned to the text box is 0 because it is 0. You must initialize you variable first. And I agree to alex, you really have to read some books. Such silly questions simply flood the forum.
I wish you luck,
Pavel
|
|
|
|
|
The variable is initialized and populated correctly. The value is 0 because the value is not carrying over through my application. Declaring it as static fixed this as my initilizing and value assigning was not only valid to the local procedure. Maybe I'm not the one who needs to read books here because I challenge you to make it work without using static.
P.S.--Don't waste your whole day on it... it's not that big a deal... I'm just trying to prove a point.
-Kay
|
|
|
|
|
And you call this a challenge. There's nothing more simple than that. You have to do 4 things:
1. Remove the keyword static
2. Declare an instance of your class in the global scope of the class where you want to use it
3. Initialize the instance
4. Use it
Really simple, isn't it 
P.S.:
Take it easy, pal. Every programmer has been a beginner 
|
|
|
|
|
hi key,
your code shows you didnt create instance of Patient class,only reference you declared which do not reserve any memory on jeap.simply solution is that
first create instance of Patient class theh use medhods. like this
private void Form1_Load(object sender, System.EventArgs e)
{
Patient PatientData = new Patient();
PatientData.id[0] = 0;
PatientData.Lname[0] = "Smith";
PatientData.Fname[0] = "John";
PatientData.Mname[0] = "Allen";
}
public class Patient
{
public int[] id = new int[5];
public string[] Lname = new string[5];
public string[] Fname = new string[5];
public string[] Mname = new string[5];
}
it will work fine.
nikesh
nikesh
|
|
|
|
|
Is there a way to create a loopback adapter using C#?
|
|
|
|
|
picasso2 wrote: Is there a way to create a loopback adapter using C#?
Yes there is.
What is a loopback adapter?
"What classes are you using ? You shouldn't call stuff if you have no idea what it does"
Christian Graus in the C# forum
led mike
|
|
|
|
|
Hehe, didnt quite have the same effect as the sudoko one
|
|
|
|
|
|
What do you mean by loopback adapter? Loopback would exist in the network stack, not in an applications space.
Matt Newman
Even the very best tools in the hands of an idiot will produce something of little or no value. - Chris Meech on Idiots
|
|
|
|
|
Sometimes it is easy to test some applicatins using Microsoft loopback adapter. It does not need a real network. I wanted to have this usility for that purpose specially if the end user is not computer-oriented or the application needs to be run on a laptop (no connection)
|
|
|
|
|
I understand why you want to use the loopback adapter, I just don't understand what you mean by creating one with C#
Matt Newman
Even the very best tools in the hands of an idiot will produce something of little or no value. - Chris Meech on Idiots
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
He actually has the nuts to ask us to be specific in our replies when he can't provide any meaningful information. 