Удаление нескольких строк из базы данных доступа

Question

Я хочу удалить несколько записей из базы данных доступа с помощью массива. Массив динамически загружается из имен файлов.

Затем я запрашиваю базу данных и вижу, соответствуют ли значения столбца базы данных значениям массива, а затем удаляют их, если совпадения не удаляются.

проблема в том, что: Ниже приведен код, который удаляет все записи независимо от того, где в состоянии.

arrays = Directory.GetFiles(sdira, "*", SearchOption.AllDirectories).Select(x => Path.GetFileName(x)).ToArray(); 
    fnames.AddRange(arrays); 

    here I have use also for loop but that also didnt help me out :(like for(int u = 0; u < arrays.length; u++) { oledbcommand sqlcmd = new oledbcommand ("delete from table1 where name not in ("'+arrays[u]+"')",sqlconnection); 
    I am using this one currently foreach(string name in arrays) 
    { 
     OleDbCommand sqlcmd = new OleDbCommand("delete from table1 where name not in ('" + name + "')", sqlconnection); 
     sqlcmd.ExecuteNonQuery();         }` 

Answer 1

Одна из проблем заключается в том, что ваш код запутан.

string [] a = {"" 'a.jpg', 'b.jpg', 'c.jpg' "} 

Во-первых, у вас есть двойной "в начале, должен быть только один.

string [] a = {" 'a.jpg', 'b.jpg', 'c.jpg' "} 

Тогда это создает массив строк с одним элементом,

a[0] = "'a.jpg', 'b.jpg', 'c.jpg'"; 

Тогда вы делаете foreach на этом, который natuarly ony выполняет один раз в результате этого запроса:

delete from table1  where name not in ('a.jpg', 'b.jpg', 'c.jpg') 

Но при загрузке массива динамически вы, вероятно, получите этого массив

a[0] = 'a.jpg'; 
a[1] = 'b.jpg'; 
a[1] = 'c.jpg'; 

, который будет выполняться 3 раза в Еогеаспе приводит к следующим 3 запросам

delete from table1  where name not in ('a.jpg') 
delete from table1  where name not in ('b.jpg') 
delete from table1  where name not in ('c.jpg') 

После второй таблицы будет пустой.

Вы должны попробовать это вместо:

string[] names = { "a.jpg", "b.jpg","c.jpg","j.jpg" }; 
string allNames = "'" + String.Join("','", names) + "'"; 

OleDbCommand sqlcmd = new OleDbCommand("delete from table1 where name not in (" + allNames + ")", sqlconnection); 
sqlcmd.ExecuteNonQuery(); 

Где имена создается динамически ofcause, и это приведет в следующем запросе соответствие тест:

delete from table1  where name not in ('a.jpg', 'b.jpg', 'c.jpg') 

Мой предпочтительный способ для динамического заполнения массива заключается в том, чтобы использовать список, поскольку чистый массив фиксирован по размеру, и любые изменения должны создавать новый массив.

Вы можете перебрать список как eacy, как массив.

List<string> names = new List<string>(); 
//or user var keyword 
var names = new List<string>(); 

Затем просто добавьте метод добавления элементов, зациклируйте его по мере необходимости.

names.Add(filename); 

Тогда для конкатенации:

string allNames = "'" + String.Join("','", names.ToArray()) + "'"; 

И вы сделали.

Или вы могли бы использовать

string[] filePaths = Directory.GetFiles(@"c:\MyDir\", "*.jpg"); 
string[] names = filePaths.ToList().ConvertAll(n => n.Substring(n.LastIndexOf(@"\") + 1)).ToArray(); 

Answer 2

Похоже, что вы либо не читаете файл правильно, либо файл пуст. Вы должны проверить массив, чтобы убедиться, что он не пуст, прежде чем запускать операцию против базы данных. Если массив пуст, ему ДОЛЖНО удалять все в базе данных, поскольку совпадений нет.

Answer 3

размещения мой комментарий как ответ

ваша строка не читает в 4 записей, чтение ее одну запись

string names = " 'a.jpg', 'b.jpg','c.jpg','j.jpg' "; 

должно быть

string[] names = { "a.jpg", "b.jpg","c.jpg","j.jpg" }; 

перед вашим для каждого было счет 1 теперь должен иметь счет 4 с фактическими значениями

Изменить: Не много усилий в этом решении я должен признать, но если вы хотите динамический вход может сделать что-то вроде:

string name = " 'a.jpg', 'b.jpg','c.jpg','j.jpg' "; 
    string[] names = name.Split(',').Select(x => x.Trim(' ').Trim('\'')).ToArray(); 

будет обновляться позже, если я получаю изменение как урезает не хорошо атм

для наполнения его, если вы хотите, как перечислим пример может быть что-то вроде

IEnumerable<string> filelocations = Directory.GetFiles(sourceDirectory, "*", SearchOption.AllDirectories).Select(x => Path.GetFileName(x)); 

или массива строк

string [] lok = Directory.GetFiles(sourceDirectory, "*", SearchOption.AllDirectories).Select(x => Path.GetFileName(x)).ToArray(); 

Answer 4

Вам нужно определить массив так:

string[] names = { "a.jpg", "b.jpg", "c.jpg", "j.jpg" }; 

, как вы определяете массив он содержит только одно значение:

" 'a.jpg', 'b.jpg','c.jpg','j.jpg' " 

Answer 5

код в ваш комментарий будет удалять любые файлы, не соответствующие первый файл. это были бы все файлы без 'a.jpg'. Следующая итерация удалит все файлы, которые не соответствуют «b.jpg», которые будут «a.jpg». Это приводит к пустой таблице. Редактирование: Объявление массива, которое вы создаете, хорошее предложение IN, когда вы делаете это вручную, но когда вы получаете список имен файлов, вы не создаете эту же строку.

Вам необходимо выполнить объединение в массиве, чтобы сгенерировать одну строку для предложения where таким образом, что предложение where включает в себя все файлы.Ваша конечная где положение должно выглядеть следующим образом:

where name not in ('a.jpg','b.jpg','c.jpg','d.jpg') 

Прямо сейчас у вас есть:

where name not in ('a.jpg') 

... Следующая итерация

where name not in ('b.jpg') 

Кроме того, следует помнить, что IN операции являются дорогостоящими и чем больше массив, тем быстрее растет запрос.

Answer 6

Попробуйте использовать список, если вы не хотите, чтобы создать массив размера статического

List<string> names = new List<string>(); 
names.Add("a.jpg"); 
names.Add("b.jpg"); 
names.Add("c.jpg"); 


foreach (string name in names) 
{ 
    OleDbCommand sqlcmd = new OleDbCommand("delete from table1 
    where name not in (" + name + ")", 
    sqlconnection); 
    sqlcmd.ExecuteNonQuery(); 
}