Click here to Skip to main content
16,020,298 members
Please Sign up or sign in to vote.
0.00/5 (No votes)
See more:
I want to open PDF-file on winforms axAcroPDF1.src (Adobe Reader).I made from a database get and created in HDD. But how to read by axAcroPDF1.src (Adobe Reader)


What I have tried:

private void dataGridView1_CellContentClick(object sender, DataGridViewCellEventArgs e)
        {
            if (e.ColumnIndex == 0)
            {
                string id;
                FileStream FS = null;
                byte[] dbbyte;

                //Get selected file ID field
                id = dataGridView1.Rows[e.RowIndex].Cells[1].Value.ToString();

                sqlcon.Open();
                sqlcmd = new SqlCommand("select * from PDFupload where ID='" + id + "'", sqlcon);
                da = new SqlDataAdapter(sqlcmd);
                dt = new DataTable();
                da.Fill(dt);
                sqlcon.Close();


                if (dt.Rows.Count > 0)
                {
                    //Get a stored PDF bytes
                    dbbyte = (byte[])dt.Rows[0]["fcontent"];

                    //store file Temporarily 
                    string filepath = "E:\\nusga.pdf";


                    //Assign File path create file
                    FS = new FileStream(filepath, System.IO.FileMode.Open);

                    //Write bytes to create file
                    FS.Write(dbbyte, 0, dbbyte.Length);

                    //Close FileStream instance
                    FS.Close();


                    // Open fila after write 

                    //Create instance for process class
                    Process Proc = new Process();

                    //assign file path for process
                    Proc.StartInfo.FileName = filepath;
                    Proc.Start();
                }
                sqlcon.Close();
            }
Posted
Updated 15-Feb-20 0:57am
v3

Couple of things:
0) Never use SELECT * FROM - it returns all the columns in a table in an "unknown" order which may be subject to change later. That's both wasteful as some of the columns may not be needed, and error-prone if you rely on the order in your code. Instead, list only the columns you are interested in, and use the column name as the index to retrieve the data:
C#
sqlcmd = new SqlCommand("SELECT fcontent FROM PDFupload WHERE ID=@ID", sqlcon);
sqlcmd.Parameters.AddWithValue("@ID", id);
...
dbbyte = (byte[])dt.Rows[0]["fcontent"];
And check the numbe of columns you get back as well!

1) Never concatenate strings to build a SQL command. It leaves you wide open to accidental or deliberate SQL Injection attack which can destroy your entire database. Always use Parameterized queries instead, as shown above.

When you concatenate strings, you cause problems because SQL receives commands like:
SQL
SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
The quote the user added terminates the string as far as SQL is concerned and you get problems. But it could be worse. If I come along and type this instead: "x';DROP TABLE MyTable;--" Then SQL receives a very different command:
SQL
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Which SQL sees as three separate commands:
SQL
SELECT * FROM MyTable WHERE StreetAddress = 'x';
A perfectly valid SELECT
SQL
DROP TABLE MyTable;
A perfectly valid "delete the table" command
SQL
--'
And everything else is a comment.
So it does: selects any matching rows, deletes the table from the DB, and ignores anything else.

So ALWAYS use parameterized queries! Or be prepared to restore your DB from backup frequently. You do take backups regularly, don't you?

2) Never store files in the root of any drive: for boot devices, that is restricted for security reasons, and it's likely to become more restricted over time, not less. Use a subdirectory instead. In this case, because it's a temporary file, you should use the Path.GetTempPath Method (System.IO) | Microsoft Docs[^] to find the TEMP folder, and then create a temporary file within that - I tend to use GUID values to generate the filename:
C#
string path = Path.GetTempPath();
string file = Guid.NewGuid().ToString("N") + ".pdf";
string filepath = Path.Combine(path, file);
Using the Combine method ensures that path dividers are inserted as required automatically.

And you can simplify your code a bit as well:
C#
string path = Path.GetTempPath();
string file = Guid.NewGuid().ToString("N") + ".pdf";
string filepath = Path.Combine(path, file);
File.WriteAllBytes(filepath, dbbyte);
Process p = new Process();
p.StartInfo.FileName = filepath;
p.Start();

When I place a valid PDF file into dbbyte, it opens using my default PDF reader.

So if you try that and it doesn't work it's most likely that you saved the wrong data into the DB, or saved it badly.
 
Share this answer
 
Comments
Member 14625523 15-Feb-20 4:11am    
I want to open PDF-file on winforms axAcroPDF1.src (Adobe Reader)
Use something like this:
ProcessStartInfo startInfo = new ProcessStartInfo();
startInfo.FileName = "AcroRd32.exe";
startInfo.Arguments = filepath;
Process.Start(startInfo);

Also see: https://www.dotnetperls.com/process[^]
And: c# - How to get Adobe Reader full path(including executable file name)? - Stack Overflow[^]

When using AxAcroPDF, see: Read PDF file from sql to AxAcroPDF c#[^]
 
Share this answer
 
v2
Comments
Member 14625523 15-Feb-20 2:22am    
no I want to open pdf file on winforms with axAcroPDF1.src (Adobe reader)
Patrice T 15-Feb-20 3:21am    
Use Improve question to update your question.
So that everyone can pay attention to this information.

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



CodeProject, 20 Bay Street, 11th Floor Toronto, Ontario, Canada M5J 2N8 +1 (416) 849-8900