C# Discussion Boards - CodeProject

Is my program idea possible?

30-Nov-20 6:20

If this is a WPF question, then it should be asked in the appropriate forum: Silverlight / WPF Discussion Boards[^] - this is a "Generic C#" forum.

"I have no idea what I did, but I'm taking full credit for it." - ThisOldTony
"Common sense is so rare these days, it should be classified as a super power" - Random T-shirt
AntiTwitter: @DalekDave is now a follower!

Re: Differences

michaelbarb30-Nov-20 6:39

michaelbarb

30-Nov-20 6:39

I reposted it under the forum you suggested. I almost put it there in the first place but some references showed it was part of C#.

So many years of programming I have forgotten more languages than I know.

Dioxazine27-Nov-20 20:09

Dioxazine

27-Nov-20 20:09

I have an idea for a program using the picture viewer control. I would like to have a label at the top of the form that says "Choose a photo of the musicians listed below to see a photo."

Then below that I would make four option buttons. And next to each option button put a musician's name.

Then below that on the bottom have two buttons. One buttons says "Show photo." The other button says "Clear image."

Problem is all the Youtube videos I've seen that demonstrate the picture box have you using open file dialog and it looks like you have to manually browse into the folder on your drive to select the photo.

I don't want to do that. I want my program to be automatic. No browsing for a photo. You select an option and then hit the button "Show photo" and the program shows the photo in the viewer.

But I don't know if this is possible so let me know please.

OriginalGriff27-Nov-20 20:20

27-Nov-20 20:20

Yes, it's possible: you do not need to use a OpenFileDialog to access the image, that just provides you with a path to the file.

If you want to load the file from a specific location, then you can just specify that:

byte[] data = File.ReadAllBytes(@"D:\Temp\MyImage.bmp");
MemoryStream ms = new MemoryStream(data);
Image im = Image.FromStream(ms);

Or you can load from the application EXE folder:

byte[] data = File.ReadAllBytes(@".\MyImage.bmp");
MemoryStream ms = new MemoryStream(data);
Image im = Image.FromStream(ms);

This has disadvantages in that you can't modify the image reliably as the folder is likely to be read only in production.
Or you can add the image as a resource and embed it in your application:

Image im = Properties.Resources.MyImage;

And now you can't change the image from run to run of your app at all because it is part of your application EXE file.

Dioxazine28-Nov-20 5:54

Dioxazine

28-Nov-20 5:54

Thanks. But I wish I understood what those lines of code mean. The only thing I understand is where you added the path to the image. That makes sense. But byte[] data and File.ReadAllBytes I don't know the why of that.

And I don't know what MemoryStream ms is either.

If you could perhaps explain what these lines do I would appreciate it.

OriginalGriff28-Nov-20 6:17

28-Nov-20 6:17

Oh come on! Are you trying to run before you can walk? This is pretty basic stuff!

You know what a byte[] is, don't you? It's an array of bytes, is all. So what do you think that File.ReadAllBytes does? And if you don't understand what a function does, look it up in the official documentation: File.ReadAllBytes(String) Method (System.IO) | Microsoft Docs[^]

The next two are there for a specific reason: while you can load an image directly from a file, that locks the file until the image is Disposed - so if you want to write a modified version back to the file in your app, you can't. Converting the data to a Stream gets round that, by locking the Stream instead of the file. And if you don't understand what a Stream is then how on earth are you doing any file input / output? Laugh | :laugh:

So those three lines do this:
1) Read the disk-based image file into an array of bytes.
2) Construct a memory-based Stream from the array.
3) Create an Image from the Stream.

Seriously, if you don't understand stuff like this, then you need to re-read your course notes from the beginning, because you have missed a heck of a lot of useful stuff!

Dioxazine28-Nov-20 8:09

Dioxazine

28-Nov-20 8:09

Come on now. I only program as a hobby. I've only done console programs so far. There is a lot of stuff I have not done. You programmers need to have some more understanding of what it's like not to be a full time developer.

If you came to me for guitar lessons I too could slice you to ribbons for all that you don't know. But that is a very poor way of teaching guitar.

Why would I need a byte array anyway. Just load the photo into the viewer. My photos will not be changing.

OriginalGriff28-Nov-20 8:22

28-Nov-20 8:22

Dioxazine wrote:
If you came to me for guitar lessons I too could slice you to ribbons for all that you don't know.

Yes, because I know nothing.

Dioxazine wrote:
But that is a very poor way of teaching guitar.

And so is what you are doing.

Suppose I came to you for help with my guitar playing, and told you that I was learning on my own by guessing what to do and wondering why I can't play as fast as others - but the forks and spoons I jam under the frets to select the notes to strum take me too long to pull out and move?

Clearly, I don't know enough to play the guitar because I've spent so long guessing what to do instead of sitting down and actually learning the right way!

And that's where you are with development. You've found code that does some of what you want, but you don't understand how it works - or even exactly what it is doing - so you can't change it to do something else yourself. And for the same reasons as me with my guitar spoons - they work for me, but not very well - I need to be taken back to the beginning and learn how to play in a structured way that introduces chords and so on in a logical progression.

Seriously, even as a hobby this is a complicated subject and it really benefits from learning the basics thoroughly before you try to rush ahead to "more interesting" projects.

Liktor Janos11-Dec-20 7:42

Liktor Janos

11-Dec-20 7:42

Why do you need stream to load picture into viewer?
Just imagine this:
You(stream) are necessary to play(load) a song(picture) at your guitar(viewer).

Error in SQL Light Compact 4

Gerry Schmitz28-Nov-20 7:51

Gerry Schmitz

28-Nov-20 7:51

You should just use a "picture viewer". Something like MS Photo. It illustrates concepts such as files and folders.

It was only in wine that he laid down no limit for himself, but he did not allow himself to be confused by it.
― Confucian Analects: Rules of Confucius about his food

Alex Dunlop23-Nov-20 3:47

23-Nov-20 3:47

Hi,
I want to save all of my DataGridView content directly into a SQL server. I'm Using SQL Light Compact 4.0
All codes related to save function are in a button:

<pre> private void btnSave_Click(object sender, EventArgs e)
        {
            SqlCeConnection connection = new SqlCeConnection();
            connection.ConnectionString = @"Data Source=C:\Users\Dell\Desktop\DataGridView\PMinfo.sdf";
            SqlCeCommand myCommand = new SqlCeCommand();
            myCommand.Connection = connection;


            for (int i = 0; i < dataGridView1.Rows.Count; i++)
            {
                myCommand.CommandText = @"INSERT into [PMinfo] (Number, CostCenter, pType, ServiceType, Receiver, WorkCenter, Start, End, Remaining, Status, ReceiveDate) values(@Number, @CostCenter, @pType, @ServiceType, @Receiver, @WorkCenter, @Start, @End, @Remaining, @Status, @ReceiveDate)";
                myCommand.Parameters.AddWithValue("@Number", dataGridView1.Rows[i].Cells[0].Value.ToString());
                myCommand.Parameters.AddWithValue("@CostCenter", dataGridView1.Rows[i].Cells[1].Value.ToString());
                myCommand.Parameters.AddWithValue("@pType", dataGridView1.Rows[i].Cells[2].Value.ToString());
                myCommand.Parameters.AddWithValue("@ServiceType", dataGridView1.Rows[i].Cells[3].Value.ToString());
                myCommand.Parameters.AddWithValue("@Receiver", dataGridView1.Rows[i].Cells[4].Value.ToString());
                myCommand.Parameters.AddWithValue("@WorkCenter", dataGridView1.Rows[i].Cells[5].Value.ToString());
                myCommand.Parameters.AddWithValue("@Start", dataGridView1.Rows[i].Cells[6].Value.ToString());
                myCommand.Parameters.AddWithValue("@End", dataGridView1.Rows[i].Cells[7].Value.ToString());
                myCommand.Parameters.AddWithValue("@Remaining", dataGridView1.Rows[i].Cells[8].Value.ToString());
                myCommand.Parameters.AddWithValue("@Status", dataGridView1.Rows[i].Cells[9].Value.ToString());
                myCommand.Parameters.AddWithValue("@ReceiveDate", dataGridView1.Rows[i].Cells[10].Value.ToString());

                connection.Open();
                myCommand.ExecuteNonQuery();
                connection.Close();
            }
        }

When I press the button, compiler gives error in line

myCommand.ExecuteNonQuery();

'There was an error parsing the query. [ Token line number = 1,Token line offset = 92,Token in error = End ]'

What is wrong with my code?

Victor Nijegorodov23-Nov-20 3:56

Victor Nijegorodov

23-Nov-20 3:56

Alex Dunlop wrote:
'There was an error parsing the query. [ Token line number = 1,Token line offset = 92,Token in error = End ]'

It seems that the parser doesn't like the field name "End".
Try

myCommand.CommandText = @"INSERT into [PMinfo] (Number, CostCenter, pType, ServiceType, Receiver, WorkCenter, Start, [End], Remaining, Status, ReceiveDate) values(@Number, @CostCenter, @pType, @ServiceType, @Receiver, @WorkCenter, @Start, @End, @Remaining, @Status, @ReceiveDate)";

Alex Dunlop23-Nov-20 4:04

23-Nov-20 4:04

Thank you. solved.
When I keep the last column in DataGridView null, compiler give the following error in line

System.NullReferenceException: 'Object reference not set to an instance of an object.'

System.NullReferenceException: 'Object reference not set to an instance of an object.'

I have check marked null properties for this column in SQL server.

Richard Deeming23-Nov-20 4:18

Richard Deeming

23-Nov-20 4:18

Quote:

dataGridView1.Rows[i].Cells[10].Value.ToString()

If the cell's Value is null, what do you think the result of calling the ToString method on the value will be? Smile | :)

Remove those .ToString() calls, and pass the cell values directly to the parameters.

myCommand.Parameters.AddWithValue("@Number", dataGridView1.Rows[i].Cells[0].Value);
myCommand.Parameters.AddWithValue("@CostCenter", dataGridView1.Rows[i].Cells[1].Value);
myCommand.Parameters.AddWithValue("@pType", dataGridView1.Rows[i].Cells[2].Value);
myCommand.Parameters.AddWithValue("@ServiceType", dataGridView1.Rows[i].Cells[3].Value);
myCommand.Parameters.AddWithValue("@Receiver", dataGridView1.Rows[i].Cells[4].Value);
myCommand.Parameters.AddWithValue("@WorkCenter", dataGridView1.Rows[i].Cells[5].Value);
myCommand.Parameters.AddWithValue("@Start", dataGridView1.Rows[i].Cells[6].Value);
myCommand.Parameters.AddWithValue("@End", dataGridView1.Rows[i].Cells[7].Value);
myCommand.Parameters.AddWithValue("@Remaining", dataGridView1.Rows[i].Cells[8].Value);
myCommand.Parameters.AddWithValue("@Status", dataGridView1.Rows[i].Cells[9].Value);
myCommand.Parameters.AddWithValue("@ReceiveDate", dataGridView1.Rows[i].Cells[10].Value);

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

OriginalGriff23-Nov-20 4:29

23-Nov-20 4:29

That's not an SQL error - that's a C# error:

myCommand.Parameters.AddWithValue("@ReceiveDate", dataGridView1.Rows[i].Cells[10].Value.ToString());

If the cell value is null, you can't call ToString (or any other property / method) on it.

You could try

myCommand.Parameters.AddWithValue("@ReceiveDate", dataGridView1.Rows[i].Cells[10].Value?.ToString());

Depending on your C# version.

Alex Dunlop23-Nov-20 7:38

23-Nov-20 7:38

I added a line for clearing sql table before saving new data and then add new data. There are two problems:

private void btnSave_Click(object sender, EventArgs e)
      {
          update.PerformClick();
          SqlCeConnection connection = new SqlCeConnection();
          connection.ConnectionString = @"Data Source=C:\Users\Dell\Desktop\DataGridView\PMinfo.sdf";
          SqlCeCommand myCommand = new SqlCeCommand();
          myCommand.Connection = connection;

          connection.Open();
          //Clearing data table
          myCommand.CommandText = "DELETE FROM PMinfo";
          connection.Close();

          myCommand.CommandText = @"INSERT into [PMinfo] (Number, CostCenter, pType, ServiceType, Receiver, WorkCenter, Start, [End], Remaining, Status, ReceiveDate) values(@Number, @CostCenter, @pType, @ServiceType, @Receiver, @WorkCenter, @Start, @End, @Remaining, @Status, @ReceiveDate)";
          connection.Open();
          for (int i = 0; i < dataGridView1.Rows.Count; i++)
          {
              myCommand.Parameters.AddWithValue("@Number", dataGridView1.Rows[i].Cells[0].Value);
              myCommand.Parameters.AddWithValue("@CostCenter", dataGridView1.Rows[i].Cells[1].Value);
              myCommand.Parameters.AddWithValue("@pType", dataGridView1.Rows[i].Cells[2].Value);
              myCommand.Parameters.AddWithValue("@ServiceType", dataGridView1.Rows[i].Cells[3].Value);
              myCommand.Parameters.AddWithValue("@Receiver", dataGridView1.Rows[i].Cells[4].Value);
              myCommand.Parameters.AddWithValue("@WorkCenter", dataGridView1.Rows[i].Cells[5].Value);
              myCommand.Parameters.AddWithValue("@Start", dataGridView1.Rows[i].Cells[6].Value);
              myCommand.Parameters.AddWithValue("@End", dataGridView1.Rows[i].Cells[7].Value);
              myCommand.Parameters.AddWithValue("@Remaining", dataGridView1.Rows[i].Cells[8].Value);
              myCommand.Parameters.AddWithValue("@Status", dataGridView1.Rows[i].Cells[9].Value);
              myCommand.Parameters.AddWithValue("@ReceiveDate", dataGridView1.Rows[i].Cells[10].Value);


              myCommand.ExecuteNonQuery();
          }
          connection.Close();
      }

1- Clearing table doesn't work.
2- When I can more than 1 rowsin DataGridView, compliler gives following error:

System.ArgumentException: 'The SqlCeParameter with this name is already contained by this SqlCeParameterCollection.'

I reviewed the loop but I couldn't find out the reason.

OriginalGriff23-Nov-20 7:53

23-Nov-20 7:53

Alex Dunlop wrote:
I reviewed the loop but I couldn't find out the reason

Hint: What do you suppose the "Add" part of "AddWithValue" might mean?

Alex Dunlop23-Nov-20 8:10

23-Nov-20 8:10

It adds a value to the end of the SqlParameterCollection.

OriginalGriff23-Nov-20 8:19

23-Nov-20 8:19

And the error message is telling you "The SqlCeParameter with this name is already contained by this SqlCeParameterCollection"

Put the two together, and ...

Alex Dunlop23-Nov-20 8:28

23-Nov-20 8:28

I think I need to put open and close parameters of sql inside the loop so in each iteration I open the sql write a row into sql table and close it and open it again and loop....
Am I right? Maybe I'd give it a try tomorrow.

modified 23-Nov-20 14:54pm.

Richard Deeming23-Nov-20 17:43

Richard Deeming

23-Nov-20 17:43

See Dave's reply[^] below.

"These people looked deep within my soul and assigned me a number based on the order in which I joined."
- Homer

Alex Dunlop24-Nov-20 6:06

24-Nov-20 6:06

I solved it by adding

<pre lang="c#">myCommand.Parametes.Clear();

The final code:

private void btnSave_Click(object sender, EventArgs e)
        {
            update.PerformClick();
            SqlCeConnection connection = new SqlCeConnection();
            connection.ConnectionString = @"Data Source=C:\Users\Dell\Desktop\DataGridView\PMinfo.sdf";
            SqlCeCommand myCommand = new SqlCeCommand();
            myCommand.Connection = connection;

            connection.Open();
            myCommand.CommandText = "Delete From PMinfo";
            myCommand.ExecuteNonQuery();
            connection.Close();

           myCommand.CommandText = @"INSERT into [PMinfo] (Number, CostCenter, pType, ServiceType, Receiver, WorkCenter, Start, [End], Remaining, Status, ReceiveDate) values(@Number, @CostCenter, @pType, @ServiceType, @Receiver, @WorkCenter, @Start, @End, @Remaining, @Status, @ReceiveDate)";
            connection.Open();
            for (int i = 0; i < dataGridView1.Rows.Count; i++)
            {
                myCommand.Parameters.Clear();
                myCommand.Parameters.AddWithValue("@Number", dataGridView1.Rows[i].Cells[0].Value);
                myCommand.Parameters.AddWithValue("@CostCenter", dataGridView1.Rows[i].Cells[1].Value);
                myCommand.Parameters.AddWithValue("@pType", dataGridView1.Rows[i].Cells[2].Value);
                myCommand.Parameters.AddWithValue("@ServiceType", dataGridView1.Rows[i].Cells[3].Value);
                myCommand.Parameters.AddWithValue("@Receiver", dataGridView1.Rows[i].Cells[4].Value);
                myCommand.Parameters.AddWithValue("@WorkCenter", dataGridView1.Rows[i].Cells[5].Value);
                myCommand.Parameters.AddWithValue("@Start", dataGridView1.Rows[i].Cells[6].Value);
                myCommand.Parameters.AddWithValue("@End", dataGridView1.Rows[i].Cells[7].Value);
                myCommand.Parameters.AddWithValue("@Remaining", dataGridView1.Rows[i].Cells[8].Value);
                myCommand.Parameters.AddWithValue("@Status", dataGridView1.Rows[i].Cells[9].Value);
                myCommand.Parameters.AddWithValue("@ReceiveDate", dataGridView1.Rows[i].Cells[10].Value);


                myCommand.ExecuteNonQuery();
            }
           
            connection.Close();
        }

Dave Kreskowiak24-Nov-20 8:52

Dave Kreskowiak

24-Nov-20 8:52

That's one way to do it, but it's the least performant and increases heap fragmentation.

Creating one set of parameter objects and reusing them is a much better method.

Asking questions is a skill
CodeProject Forum Guidelines
Google: C# How to debug code
Seriously, go read these articles.

Dave Kreskowiak

modified 25-Nov-20 0:59am.