У меня есть cronjob, который запускает скрипт -> получение данных из разных хранилищ itunes. Иногда я получаю вышеупомянутое сообщение об ошибке.
Я думаю, что это из-за специальных букв или что-то вроде этого. Есть ли способ проверить, где именно проблема? Какой «специальный символ» отвечает за ошибку. Есть ли обходной путь, например, с оператором IF?
Я не могу воспроизвести ошибку, так как она появляется не всегда. Было бы здорово получить помощь по этому вопросу.
Вот код:
foreach ($Kategorien->entry as $item) {
$id = addslashes($item->id);
$title = utf8_decode(addslashes($item->title));
$preview = addslashes($item->link[1]["href"]);
$namespaces = $item->getNameSpaces(true);
$im = $item->children($namespaces['im']);
$track_title = utf8_decode(addslashes($im->name));
$track_artist = utf8_decode(addslashes($im->artist));
$track_amount = addslashes($im->price->attributes()->amount);
$track_currency = utf8_decode(addslashes($im->price->attributes()->currency));
$release_date = addslashes($im->releaseDate);
$image = addslashes($im->image[2]);
$entry_id['im'] = $item->id->attributes('im', TRUE);
$track_id = addslashes($entry_id['im']['id']);
$category_id['im'] = $item->category->attributes('im', TRUE);
$genre_id = addslashes($category_id['im']['id']);
$genre_cat = utf8_decode(addslashes($item->category->attributes()->term));
$insertSQL = sprintf("UPDATE track_itunes_countries_total SET modified = NOW(), modified_genre = '$genre_name' WHERE id = ".$row_select_country['id']."");
$Result1 = mysql_query($insertSQL, $con) or die(mysql_error());$insertSQL = sprintf("INSERT INTO track_itunes_".$cc."_total (id, title, preview, track_title, track_artist, track_amount, track_currency, release_date, image, track_id, genre_id, genre_cat, country, Online, Approved) VALUES ('$id', '$title', '$preview', '$track_title', '$track_artist', '$track_amount', '$track_currency', '$release_date', '$image', '$track_id', '$genre_id', '$genre_cat', '$cc', '1', '1') ON DUPLICATE KEY UPDATE title='$title',preview='$preview',track_title='$track_title',track_artist='$track_artist',track_amount='$track_amount',track_currency='$track_currency',release_date='$release_date',image='$image',track_id='$track_id',genre_id='$genre_id',genre_cat='$genre_cat',country='$cc'");
$Result1 = mysql_query($insertSQL, $con);}
Я отвечу косвенно, потому что вы используете библиотеку mysql_ *, которая устарела и вместо этого показывает PDO. Это либо а) решит вашу проблему, либо б) предоставит гораздо более информативное сообщение об ошибке, которое поможет вам при отладке.
foreach ($Kategorien->entry as $item) {
$id = $item->id;
$title = utf8_decode($item->title);
$preview = $item->link[1]["href"];
$namespaces = $item->getNameSpaces(true);
$im = $item->children($namespaces['im']);
$track_title = utf8_decode($im->name);
$track_artist = utf8_decode($im->artist);
$track_amount = $im->price->attributes()->amount;
$track_currency = utf8_decode($im->price->attributes()->currency);
$release_date = $im->releaseDate;
$image = $im->image[2];
$entry_id['im'] = $item->id->attributes('im', TRUE);
$track_id = $entry_id['im']['id'];
$category_id['im'] = $item->category->attributes('im', TRUE);
$genre_id = $category_id['im']['id'];
$genre_cat = utf8_decode($item->category->attributes()->term);
$insertSQL = "UPDATE track_itunes_countries_total
SET modified = NOW(), modified_genre = :genre_name
WHERE id = :row_id");
$stmt = $pdo->prepare($insertSQL);
$stmt->bindValue(':genre_name', $genre_name);
$stmt->bindValue(':row_id', $row_select_country['id']);
$success = $stmt->execute();
if(!$success){
//something bad happened
}
//use whitelist techniques to guarantee valid, non-malicious input
//with table names or column names. whitelistTableName is a function
//that YOU have to write.
$clean_table_name = whitelistTableName("track_itunes_{$cc}_total");
$insertSQL = "INSERT INTO {$clean_table_name}
(id, title, preview, track_title,
track_artist, track_amount, track_currency,
release_date, image, track_id, genre_id,
genre_cat, country, Online, Approved)
VALUES
(:id, :title, :preview, :track_title,
:track_artist, :track_amount, :track_currency,
:release_date, :image, :track_id, :genre_id,
:genre_cat, :country, :Online, :Approved)
ON DUPLICATE KEY UPDATE
title=:title_u,preview=:preview_u,track_title=:track_title_u,
track_artist=:track_artist_u,track_amount=:track_amount_u,
track_currency=:track_currency_u,release_date=:release_date_u,
image=:image_u,track_id=:track_id_u,genre_id=:genre_id_u,
genre_cat=:genre_cat_u,country=:cc_u");
$stmt = $pdo->prepare($insertSQL);
$stmt->bindValue(':id', $id);
$stmt->bindValue(':title', $title);
$stmt->bindValue(':preview', $preview);
$stmt->bindValue(':track_title', $track_title);
$stmt->bindValue(':track_artist', $track_artist);
$stmt->bindValue(':track_amount', $track_amount);
$stmt->bindValue(':track_currency', $track_currency);
$stmt->bindValue(':release_date', $release_date);
$stmt->bindValue(':image', $image);
$stmt->bindValue(':track_id', $track_id);
$stmt->bindValue(':genre_id', $genre_id);
$stmt->bindValue(':genre_cat', $genre_cat);
$stmt->bindValue(':country', $cc);
$stmt->bindValue(':Online', 1);
$stmt->bindValue(':Approved', 1);
//some drivers doesn't allow to have a named placeholder to appear more than once so we must duplicate those.
$stmt->bindValue(':id_u', $id);
$stmt->bindValue(':title_u', $title);
$stmt->bindValue(':preview_u', $preview);
$stmt->bindValue(':track_title_u', $track_title);
$stmt->bindValue(':track_artist_u', $track_artist);
$stmt->bindValue(':track_amount_u', $track_amount);
$stmt->bindValue(':track_currency_u', $track_currency);
$stmt->bindValue(':release_date_u', $release_date);
$stmt->bindValue(':image_u', $image);
$stmt->bindValue(':track_id_u', $track_id);
$stmt->bindValue(':genre_id_u', $genre_id);
$stmt->bindValue(':genre_cat_u', $genre_cat);
$stmt->bindValue(':country_u', $cc);
$stmt->bindValue(':Online_u', 1);
$stmt->bindValue(':Approved_u', 1);
$success = $stmt->execute();
if(!$success){
//something bad happened
}
}
больше читать здесь: http://php.net/manual/en/book.pdo.php Это довольно просто и намного лучше, чем библиотеки mysql_ *
Расширение на методы белых списков: есть несколько подходов для очистки пользовательских входов. Один из них ускользает: это делается, когда пользовательский ввод является «открытым», как текстовый ввод, где существует неограниченное количество возможностей. Подготовленные заявления идеально подходят для этого, как показано выше.
Другой возможностью является белый список, и он полезен, когда допустимые возможности для пользовательского ввода ограничены (например, радиокнопки, флажки, выбор опций и т. Д.), А любой недействительный ввод является либо ошибочным, либо вредоносным.
пример следует:
whitelistTableName($tablename){
$allowedTables = array('tbl1', 'tbl2', 'tbl3');
if(in_array($tablename, $allowedTables)){
return $tablename;
} else {
throw new Exception('Malicious attempt detected');
}
}
Это очень просто, но позволяет вам начать. Лучшим подходом было бы сделать запрос к вашей базе данных information_schema, чтобы получить каждое допустимое имя таблицы, а не жестко кодировать их вручную.
Это не то, как работает sprintf.
Sprintf означает строку printf — вы выполняете printf, которая возвращает строку вместо печати непосредственно в стандартный вывод.
printf работает, назначая заполнители в строку формата (первый аргумент) и связанные значения для заполнителей в качестве последующих аргументов.
например
$s = sprintf("SELECT * FROM %s WHERE id = %d", 'some_table', $id);
Это в некотором роде наивный способ очистки входных данных, поскольку вы заставляете переменные преобразовываться в определенные типы, используя форматы: в этом случае% s и% d для строка а также десятичный / цифра, соответственно. Во время выполнения они будут заменены на «some_table» и любой другой тип intval ($ id).
Причина, по которой вы получаете «слишком мало аргументов», заключается в том, что вы пропускаете связанные значения.
Прежде всего: вы должны использовать mysql_real_escape_string () или даже лучше использовать mysqli вместо функций mysql, так как mysql устарел.
А для сообщения об ошибке вы должны заглянуть в документацию sprintf, чтобы понять ошибку.
Или просто используйте правильную конкатенацию.
$string = "fooo='".$var."'";
вместо твоего ленивого обозначения
$string = "fooo='$var'";
Вот пример, включающий mysql_real_escape_string ():
$query = sprintf("SELECT * FROM users WHERE user='%s' AND password='%s'",
mysql_real_escape_string($user),
mysql_real_escape_string($password));
Вы также можете указать конкретные позиции для связанных переменных:
$query = sprintf("SELECT * FROM users WHERE user='%2$s' AND password='%1$s'",
mysql_real_escape_string($password),
mysql_real_escape_string($user)
);
А для mysql_query () вам не нужно указывать ссылку на соединение $ con, если вы не используете более 1 соединения с базой данных.
mysql_query ($ запроса);
сделает свое дело.