OriginalGriff
Пара вещей:
0) никогда не используйте SELECT * FROM
- он возвращает все столбцы в таблице в "неизвестном" порядке, который может быть изменен позже. Это одновременно расточительно, поскольку некоторые столбцы могут быть не нужны, и подвержено ошибкам, если вы полагаетесь на порядок в своем коде. Вместо этого перечислите только те столбцы, которые вас интересуют, и используйте имя столбца в качестве индекса для извлечения данных:
sqlcmd = new SqlCommand("SELECT fcontent FROM PDFupload WHERE ID=@ID", sqlcon);
sqlcmd.Parameters.AddWithValue("@ID", id);
...
dbbyte = (byte[])dt.Rows[0]["fcontent"];
И проверьте количество столбцов, которые вы получите обратно, а также!
1) Никогда не объединяйте строки для построения SQL-команды. Это оставляет вас широко открытыми для случайной или преднамеренной атаки SQL-инъекции, которая может уничтожить всю вашу базу данных. Вместо этого всегда используйте параметризованные запросы, как показано выше.
Когда вы объединяете строки, вы вызываете проблемы, потому что SQL получает такие команды, как:
SELECT * FROM MyTable WHERE StreetAddress = 'Baker's Wood'
Цитата, добавленная пользователем, завершает строку в том, что касается SQL, и вы получаете проблемы. Но могло быть и хуже. Если я приду и наберу вместо этого: "x';DROP TABLE MyTable;--", то SQL получит совсем другую команду:
SELECT * FROM MyTable WHERE StreetAddress = 'x';DROP TABLE MyTable;--'
Которые SQL видит как три отдельные команды:
SELECT * FROM MyTable WHERE StreetAddress = 'x';
Совершенно правильный выбор
DROP TABLE MyTable;
Вполне допустимая команда "удалить таблицу"
--'
А все остальное-это комментарии.
Так оно и происходит: выбирает любые совпадающие строки, удаляет таблицу из базы данных и игнорирует все остальное.
Поэтому всегда используйте параметризованные запросы! Или будьте готовы часто восстанавливать свою БД из резервной копии. Вы ведь регулярно делаете резервные копии, не так ли?
2) Никогда не храните файлы в корне любого диска: для загрузочных устройств это ограничено по соображениям безопасности, и со временем оно, скорее всего, станет более ограниченным, а не менее. Вместо этого используйте подкаталог. В этом случае, поскольку это временный файл, вы должны использовать
Метод Path.GetTempPath (System.IO) | Microsoft Docs[
^] чтобы найти временную папку, а затем создать в ней временный файл - я обычно использую значения GUID для генерации имени файла:
string path = Path.GetTempPath();
string file = Guid.NewGuid().ToString("N") + ".pdf";
string filepath = Path.Combine(path, file);
Использование метода комбинирования гарантирует, что разделители пути вставляются по мере необходимости автоматически.
И вы также можете немного упростить свой код:
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();
Когда я помещаю допустимый PDF-файл в dbbyte, он открывается с помощью моего стандартного PDF-ридера.
Поэтому, если вы попытаетесь это сделать, и это не сработает, скорее всего, вы сохранили неправильные данные в БД или сохранили их плохо.