|
1.Yes. It is essentially the same as passing a value (a property is just syntactic sugar for a get and set method). Passing Values between Forms with C# and VB.NET examples[^]
I have studied above topic but I do not have to pass values.
2. Page? Is this a web application?
Its a windows based application.
3.The accessor levels of methods has nothing to do with security. If you want security you might want to look into areas such as CAS (Code Access Security). If someone wants to to call your method then they can do so quite easily using reflection. So, making something private, protected or internal has zero security associated with it.
I did not knew about that.So Is this mean that I have to use CAS to improve security of my application.
Please guide
Thanks
Regards
THE SK
|
|
|
|
|
Morning fellow coders;
I need to come up with a networkable sign in sign out board. Can somebody tell me if there are any samples to look at or give me the big picture of what I would need to do.
i.e.
Would this have to be ASP.net or can I do this with windows forms?
thanks in advance for any information
rafone
Statistics are like bikini's...
What they reveal is astonishing ...
But what they hide is vital ...
|
|
|
|
|
It can be done with either forms or asp. The difference being with forms, every sign in point needs a copy or a mapping to a copy of the executable, with asp you need a browser and the address of the page.
Which way you go really depends on what you're more comfortable with and how you expect it to be used. Do you need update notifications if someone hasn't signed in or out, do you only need current status or a tracking log of historical times.
The most simplistic version is to simply install an IM client on each persons machine if you just need to know if they're in the office. If you actually want to track their in/out times, then you'll have to make something. A simple database with two tables, employee name/id's and a recent activity table with ID/date/time/last activity, would handle the data. Then you just need to make an interface to update/read the tables as needed.
The true man wants two things: danger and play. For that reason he wants woman, as the most dangerous plaything.
|
|
|
|
|
Please bare with me I only started learning c# a few days back.
Anyway I am creating a split function (by line not file size) and thus far I have got it to create do the necessary math & create its directory. However I have a problem in my loop in which the error: Cannot write to a closed TextWriter. I just cant seem to solve this problem and need some help!
<br />
private void btnSplit_Click(object sender, EventArgs e)<br />
{<br />
string inputFile = this.txtFileToImport.Text;
FileStream fs = new FileStream(inputFile, FileMode.Open, FileAccess.Read);<br />
int numberOfFiles = (int)this.numericUpDown1.Value;<br />
int numberOfRows = (int)this.dataGridView_preView.RowCount - 1;<br />
decimal rowsPerFile = numberOfRows / numberOfFiles;<br />
int sizeOfEachFile = (int)Math.Ceiling((double)rowsPerFile);<br />
lblPray.Text = " Rows: " + sizeOfEachFile.ToString();<br />
<br />
string OutputFolder = inputFile + "_Batches";<br />
if (Directory.Exists(inputFile) == false)<br />
<br />
{<br />
Directory.CreateDirectory(OutputFolder);<br />
}<br />
<br />
System.IO.StreamReader reader = new System.IO.StreamReader(inputFile);<br />
string strHeader = reader.ReadLine();<br />
<br />
int FileIndex = 0;<br />
<br />
do<br />
{<br />
<br />
string PiecePath = OutputFolder + "\\" + Path.GetFileNameWithoutExtension(inputFile) + "_" + FileIndex + Path.GetExtension(inputFile);<br />
StreamWriter Writer = new StreamWriter(PiecePath, false);<br />
Writer.AutoFlush = false;<br />
Writer.WriteLine(strHeader);<br />
<br />
for (int i = 1; i <= sizeOfEachFile; i++)<br />
{<br />
<br />
string s = reader.ReadLine();<br />
if (s != null & _IsAbort == false)<br />
{<br />
Writer.WriteLine(s);<br />
}<br />
else<br />
{<br />
Writer.Flush();<br />
Writer.Close();<br />
}<br />
}<br />
<br />
Writer.Flush();<br />
Writer.Close();<br />
}<br />
while (true);<br />
} <br />
}<br />
}<br />
Ive probably been looking straight at the problem and just been sitting here far to long to realise... either that or its all wrong! Any help would be much appreciated.
Cheers 
|
|
|
|
|
Hi,
Before helping with your problem, let me recommend something to you born out of years of experience:
Don't write your own CSV parser![^]
Seriously, hell is awaiting you if you're trying to parse a CSV file by hand.
Your first thought is, "I need to handle commas inside of quotes."
Your next thought will be, "Oh, crap, I need to handle quotes inside of quotes. Escaped quotes. Double quotes. Single quotes..."
Instead of rolling your own, use an existing library that has done the work for you, such as the free FileHelper[^] library.
Religiously blogging on the intarwebs since the early 21st century: Kineti L'Tziyon
Judah Himango
|
|
|
|
|
It seems you are trying to flush the file after closing it.
Also, please make your code more readable by using the <pre> tag and correct indentation.
|
|
|
|
|
I'm not sure where your problem lies, but
do
{
} while (true);is an infinate loop - never a good idea, plus I can't see anywhere you attempt to break out of it.
DaveBTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)Visual Basic is not used by normal people so we're not covering it here. (Uncyclopedia)
|
|
|
|
|
Something like this ought to do it (assuming you are only splitting by lines and not parsing each line):
do
{
string PiecePath = OutputFolder + "\\" + Path.GetFileNameWithoutExtension(inputFile) + "_" + FileIndex + Path.GetExtension(inputFile);
using (StreamWriter Writer = new StreamWriter(PiecePath, false))
{
Writer.AutoFlush = false;
Writer.WriteLine(strHeader);
int linesOut = 0;
while( (linesOut < sizeOfEachFile) && ((s = reader.ReadLine()) != null) )
{
linesOut++;
Writer.WriteLine(s);
}
}
FileIndex++;
} while (s != null);
There might be the odd misplaced ( or ) - just done off the top of my head.
Have a look at your calculation for sizeOfEachFile - why two a decimal and a double? What do you get when you divide an int by an int and assign it to a decimal?
Regards
David R
|
|
|
|
|
Thanks you very much for all your help its now working properly and I have also taking all your advice under consideration and changed my code accordingly!
Cheers
|
|
|
|
|
You're welcome. 
Regards
David R
|
|
|
|
|
Hello,
I've got an easy on for you all.
Since I chose to have input of QueryTime possibly be a decimal (ex ".5"), I have kept it a decimal.
However, Timer.Interval is an int and I need to convert a decimal to an int.
public void StartTimer(decimal QueryTime)
{<br />
int shizz = Decimal.ToInt32(QueryTime);
int wtf = (Decimal.ToInt32(QueryTime) * 60) * 1000;
<br />
QueryTimer.Interval = ((int)QueryTime * 60)* 1000;<br />
QueryTimer.Enabled = true;<br />
<br />
}
Event when QueryTime = 0.5, both shizz and wtf are 0.
Does anyone know how to properly convert a decimal to an integer?
Thanks,
Matt
|
|
|
|
|
A decimal will not fit in an integer so you have to decide in code how you want it to fit.
Have a look at the Math class in particular its Round method with MidPointRoundingMode
DaveBTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)Visual Basic is not used by normal people so we're not covering it here. (Uncyclopedia)
|
|
|
|
|
Hello!
Thanks for your response.
I am giving the users the option to set the interval of the timer. I had specified "minutes." Instead, I can specify "seconds" and keep it int throughout.
However, I'll take a look at this for future reference.
Thanks!
Matt Brown
|
|
|
|
|
In that case, maybe a TimeSpan would be more appropriate?
I used some thing similar in my ApplicationIdle component article[^]
DaveBTW, in software, hope and pray is not a viable strategy. (Luc Pattyn)Visual Basic is not used by normal people so we're not covering it here. (Uncyclopedia)
|
|
|
|
|
Other idea:
int non_wtf = Decimal.ToInt32(QueryTime * 60 * 1000);
Btw, why a decimal? Why not, say, a float or double? Almost everything that uses non-integral values uses either floats or doubles, decimal is rare (and slow, and only meant for when the numbers are for human interaction since they expect 4 instead of 3.99999999 for example)
|
|
|
|
|
Thanks for the input.
I'd like to have users be able to input values of 10-.01 as a string.
I then would like to convert this string to a value.
I would then multiply this value by 60, then 1000.
Then I would like to take this value and change it into an integer.
What value should I be converting the string to perform the calculations?
Thanks,
Matt
|
|
|
|
|
Small tip, multiply by 60000 instead, bit faster.
Since you're turning it into an int anyway, and considering the range of expected values, you could use a float without trouble (several orders of magnitude faster than a decimal)
Something like:
int someFunctionName(string input)
{
return (int)(float.Parse(input) * 60000);
}
warning: untested
In RealLife(tm) you might want to use float.TryParse and do something clever in case it returns false.
|
|
|
|
|
harold aptroot wrote: multiply by 60000 instead, bit faster.
Actually QueryTime * 60 * 1000 is OK, the compiler optimizes that to *60000 while the reader can better see the intent.
Luc Pattyn [Forum Guidelines] [My Articles]
- before you ask a question here, search CodeProject, then Google
- the quality and detail of your question reflects on the effectiveness of the help you are likely to get
- use the code block button (PRE tags) to preserve formatting when showing multi-line code snippets
|
|
|
|
|
Depends on how you write it of course, exactly like "QueryTime * 60 * 1000" it would be ok but most other ways to write it wouldn't be
|
|
|
|
|
Hi Matt,
Everything you tried there, including ((int)QueryTime * 60)* 1000; is wrong; the heart of the problem is (int)x where x is float/double/decimal is rounding towards zero, so if x is between -1 and 1, it will be rounded to zero. You should perform the entire calculation in the highest precision that is present, and only then convert it to an integer.
Luc Pattyn [Forum Guidelines] [My Articles]
- before you ask a question here, search CodeProject, then Google
- the quality and detail of your question reflects on the effectiveness of the help you are likely to get
- use the code block button (PRE tags) to preserve formatting when showing multi-line code snippets
|
|
|
|
|
Hi I'm Tim and this is my first post. I just started coding in C# a few days ago. I'm developing a piece of software to manage some things at work. If anyone here has a few minutes, can you go over my code so far and let me know if it looks solid and please please let me know if I'm doing anything that throws up "warning flags" as far as coding style etc? Thanks in advance!
2 Forms so far.
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
using System.Collections;
namespace WindowsFormsApplication1
{
public partial class Form1 : Form
{
ArrayList vins = new ArrayList();
public Form1()
{
InitializeComponent();
}
public void CreateDataTable()
{
string s1 = "Date";
string s2 = "System.String";
DataTable dt = new DataTable();
dt.TableName = textBox1.Text;
while (s1!= "Stop")
{
DataColumn dc = new DataColumn();
dc.ColumnName = s1;
dc.Caption = dc.ColumnName;
dc.DataType = System.Type.GetType(s2);
dc.DefaultValue = null;
dt.Columns.Add(dc);
switch (s1)
{
case "Date":
s1 = "Member#";
s2 = "System.Int32";
break;
case "Member#":
s1 = "VIN#";
s2 = "System.Int32";
break;
case "VIN#":
s1 = "Location Name";
s2 = "System.String";
break;
case "Location Name":
s1 = "Address";
s2 = "System.String";
break;
case "Address":
s1 = "City";
s2 = "System.String";
break;
case "City":
s1 = "State";
s2 = "System.String";
break;
case "State":
s1 = "Zip";
s2 = "System.Int32";
break;
case "Zip":
s1 = "Phone";
s2 = "System.String";
break;
case "Phone":
s1 = "Fax";
s2 = "System.String";
break;
case "Fax":
s1 = "Email";
s2 = "System.String";
break;
case "Email":
s1 = "Stop";
break;
}
}
DataRow dr;
dr = CreateTableRow(dt);
}
public DataRow CreateTableRow(DataTable dtPass)
{
DataRow myRow;
myRow = dtPass.NewRow();
DateTime today = DateTime.Now;
string date;
int year = today.Year;
int month = today.Month;
int day = today.Day;
date = month + "\\" + day + "\\" + year;
myRow[0] = date;
myRow[1] = int.Parse(textBox1.Text);
myRow[2] = int.Parse(textBox2.Text);
return myRow;
}
private void label1_Click(object sender, EventArgs e)
{
}
private void AddTruck()
{
try
{
if (textBox2.Text != null)
{
long vinNumber = long.Parse(textBox2.Text);
vins.Add(vinNumber);
textBox2.Text = "";
label3.Text = label3.Text + vinNumber + Environment.NewLine;
}
}
catch (FormatException)
{
textBox2.Text = "";
}
}
private void button1_Click(object sender, EventArgs e)
{
Form2 f = new Form2(this);
AddTruck();
}
private void OpenContactInfo()
{
}
private void button2_Click(object sender, EventArgs e)
{
try
{
if (textBox1.Text != null)
{
CreateDataTable();
}
}
catch (FormatException)
{
textBox1.Text = "";
}
}
}
}
using System;
using System.Collections.Generic;
using System.ComponentModel;
using System.Data;
using System.Drawing;
using System.Linq;
using System.Text;
using System.Windows.Forms;
namespace WindowsFormsApplication1
{
public partial class Form2 : Form
{
Form1 f;
public Form2()
{
InitializeComponent();
}
public Form2(Form1 fr1)
{
InitializeComponent();
f = new Form1();
f = fr1;
}
private void Form2_Load(object sender, EventArgs e)
{
}
}
}
|
|
|
|
|
Overall, it looks pretty good, especially for a beginner. I've seen worse code in production.
Couple thoughts:
Don't use ArrayList. That's old .NET 1.0 junk that boxes every value type you put in it (thus making more memory allocations). It makes your code uglier because when pulling values out, you have to cast it to the correct type. It makes your code more prone to error because you can put other objects besides longs in it.
Since you're just putting VIN#s of type long in that list, use a List<long> instead. The compiler will make sure you don't put anything besides longs in it, it performs better than ArrayList, your code will be cleaner, no boxing required.
Another thought related to int.Parse. int.Parse is OK, however, you should be aware of TryParse alternative method. TryParse won't throw an exception, and will return a value indicating whether the parse was successful:
int value;
bool successfullyParsed = int.TryParse("123", out value);
if (successfullyParsed)
{
}
Since exceptions are costly, you may want to consider TryParse instead of Parse if that fits your code OK. If Parse fits better, feel free to continue using it.
Religiously blogging on the intarwebs since the early 21st century: Kineti L'Tziyon
Judah Himango
|
|
|
|
|
Thanks for the quick response and compliment Judah. I had no idea ArrayList was an outdated item. Goes to show that the internet is getting older.
I don't have time to delve into how List<long> works right now, but it sounds like a more solid option that I will look into. I would assume it behaves similar to ArrayList in that if you delete an item (ie myList[3]), that myList[4] would be copied into myList[3] and so on and so forth. Is this correct? If so, the change will be a no brainer.
|
|
|
|
|
stonebergftw wrote: I would assume it behaves similar to ArrayList in that if you delete an item (ie myList[3]), that myList[4] would be copied into myList[3] and so on and so forth. Is this correct?
Yep, that's right, same semantics as ArrayList.
Religiously blogging on the intarwebs since the early 21st century: Kineti L'Tziyon
Judah Himango
|
|
|
|
|
Welcome.
You should use more descriptive names for your variables and classes.
e.g consider renaming s1 to columnName and s2 to columnDataType, etc. Also, change the default names for your text boxes and buttons. You'll quickly forget what textBox1 & button1 are when you app grows. Same for forms, give them more descriptive names than Form1 and Form2 like DataEntryForm or whatever make sense for each one.
Keep it up 
Simon
|
|
|
|
General 
News 
Suggestion 
Question 
Bug 
Answer 
Joke 
Praise 
Rant 
Admin 
Submit an article or tip
