Мне пришлось уйти от компьютера, когда я увидел первый ответ, и все еще огорчен тем, что он был одобрен многими людьми. Это ужасающий пример самого худшего вида ASP-кода, такого, который гарантирует, что ваш сайт будет SQL-инъекционным, и, если вы продолжите использовать этот код по всему сайту, его можно будет взломать в течение дюйма его жизни.
Это НЕ тот код, который вы должны давать кому-то, кто плохо знаком с ASP-кодированием, поскольку он будет думать, что это профессиональный способ кодирования на языке!
НИКОГДА не раскрывайте строку подключения в вашем коде, поскольку она содержит имя пользователя и пароль для вашей базы данных. Вместо этого используйте файл UDL или, по крайней мере, константу, которую можно объявить в другом месте и использовать на сайте.
Больше нет оправдания использованию встроенного SQL для какой-либо операции в веб-среде. Используйте хранимую процедуру - преимущества безопасности не могут быть подчеркнуты достаточно. Если вы действительно не можете этого сделать, тогда рассматривайте встроенные параметры как второй лучший вариант ... Встроенный SQL сделает ваш сайт широко открытым для внедрения SQL, внедрения вредоносных программ и всего остального.
Позднее объявление переменных может привести к небрежному кодированию. Используйте «option visible» и объявите переменные в верхней части функции. Это лучшая практика, а не настоящий WTF, но лучше начинать так, как вы хотите.
Нет никаких намеков на базу данных относительно того, что это за тип соединения - он только для чтения или пользователь будет обновлять записи? Соединение может быть оптимизировано, и база данных может очень эффективно обрабатывать блокировку, если эффективно сказать, чего ожидать.
Соединение с базой данных не закрывается после использования, и объект набора записей не полностью уничтожен.
ASP по-прежнему сильный язык, несмотря на то, что многие люди предлагают перейти на .NET - с хорошими практиками кодирования можно написать сайт ASP, который прост в обслуживании, масштабируем и быстр, но вы ДОЛЖНЫ убедиться, что используете каждый метод чтобы сделать ваш код эффективным, вы ДОЛЖНЫ поддерживать хорошие методы кодирования и немного обдумать. Хороший редактор тоже поможет, я предпочитаю PrimalScript, который я считаю более полезным для ASP-кодировщика, чем любой из последних продуктов MS, который кажется очень .NET-ориентированным.
Кроме того, откуда берется поле "MEMO"? Это номенклатура Access или, может быть, MySQL? Я спрашиваю, как такие поля назывались полями TEXT или NTEXT в MS-SQL в течение десятилетия.