优化此.NET爬网程序算法的提示

时间:2015-02-07 17:26:24

标签: sql .net sql-server performance algorithm

我正在编写类似网络抓取工具的东西,它的引擎遵循以下步骤:

  1. 阅读Rss链接(参数)
  2. 定义(Rss项目)列表
  3. 通过单独的查询检查数据库(SQL SERVER)中的每个链接是否存在
  4. 如果链接是新链接,它将通过单独的查询将字段插入到数据库

    Public Sub MyTickHandler()
        Dim NewItems As New List(Of Structures.RSSItem)
        Dim founded As Boolean = False
    
        NewItems = RssReader.ParseRssFile(RssURL)
    
        Dim connString = Configs.NewsDBConnection
        Dim myConnection As SqlConnection = New SqlConnection("Server=localhost;Database=db;Integrated Security=SSPI;;Connection Timeout=45;Max Pool Size= 300")
        myConnection.Open()
    
        For Each item In NewItems
            Dim cmdString As String = "SELECT id FROM posts  with (nolock) WHERE link LIKE '" & item.link.Trim.ToLower & "'"
            Dim TheCommand As SqlCommand = New SqlCommand(cmdString, myConnection)
            Dim result = TheCommand.ExecuteScalar()
            If result Is Nothing Then
                TheCommand = New SqlCommand("INSERT INTO posts (link) VALUES ('" & item.link.ToLower.Trim & "')")
                TheCommand.Connection = myConnection
                TheCommand.ExecuteNonQuery()
    
                TheCommand = New SqlCommand("INSERT INTO queue (link,descrip,site,title,category) VALUES ('" & item.link.ToLower.Trim & "','" & StringToBase64(item.description) & "','" & RssSite & "','" & StringToBase64(item.title) & "','" & RssCategory & "')")
                TheCommand.Connection = myConnection
                TheCommand.ExecuteNonQuery()
            End If
            TheCommand.Dispose()
        Next
    
        myConnection.Close()
        myConnection.Dispose()
        SqlConnection.ClearPool(myConnection)
    
     End Sub
    
  5. 这适用于单次通话。
    但我有一些关于 150 Rss链接的东西,我应该通过线程每2分钟检查一次,所以通过增加SQL查询的安装,这个过程以及sql server没有响应和应用程序崩溃!

    我尝试了一些提示,例如增加sql server响应超时,但它根本没有帮助。

    这个过程有更好的方法或提示吗? 感谢

2 个答案:

答案 0 :(得分:1)

  • 在for-each-loops之外只进行一次提取:

SELECT id, link FROM posts with (nolock) WHERE link in (@listOfLowerCaseLinks)

Dim myListOfLinks As New List(Of String)
...
TheCommand.Parameters.AddWithValue("@listOfLowerCaseLinks", myListOfLinks)
  • 将插入的整个操作(整个for-each-loop)包装到sql事务中。这样,数据库就不必在中间提交。

答案 1 :(得分:1)

我建议您将表值参数传递给此任务的存储过程。这将允许整个列表插入一次调用。下面是一个示例,您可以调整实际的列长度。在posts表的link列上有一个索引很重要。我假设链接在这个例子中是唯一的。

T-SQL创建表类型和proc:

CREATE TYPE dbo.linkInfo AS TABLE(
     link varchar(255) NOT NULL PRIMARY KEY
    ,descrip varchar(255)
    ,title varchar(255)
    );
GO

ALTER PROC dbo.usp_InsertRssItems
     @site varchar(255)
    ,@category varchar(255)
    ,@linkInfo dbo.linkInfo READONLY
AS

SET NOCOUNT ON;

DECLARE @InsertedPosts TABLE(link varchar(255));

INSERT INTO dbo.posts(link)
OUTPUT inserted.link INTO @InsertedPosts
SELECT link
FROM @linkInfo AS li
WHERE NOT EXISTS(
    SELECT *
    FROM dbo.posts AS p
    WHERE p.link = li.link
    );

INSERT INTO dbo.queue(link,descrip,site,title,category)
SELECT li.link, li.descrip, @site,li. title, @category
FROM @linkInfo AS li
WHERE EXISTS(
    SELECT *
    FROM @InsertedPosts AS ip
    WHERE ip.link = li.link
    );
GO

示例VB.NET代码:

Sub MyTickHandler()

    Dim NewItems As New List(Of Structures.RssItem)
    Dim founded As Boolean = False

    NewItems = RssReader.ParseRssFile(RssURL)

    Dim dt = getNewRssItemDataTable(NewItems)

    Dim connString = Configs.NewsDBConnection
    Dim myConnection As SqlConnection = New SqlConnection("Server=localhost;Database=db;Integrated Security=SSPI;;Connection Timeout=45;Max Pool Size= 300")
    Dim TheCommand As SqlCommand = New SqlCommand("dbo.usp_InsertRssItems", myConnection)
    TheCommand.Parameters.Add(New SqlParameter("@site", SqlDbType.VarChar, 255)).Value = "z"
    TheCommand.Parameters.Add(New SqlParameter("@category", SqlDbType.VarChar, 255)).Value = "z"
    TheCommand.Parameters.Add(New SqlParameter("@linkInfo", SqlDbType.Structured)).Value = dt
    TheCommand.CommandType = CommandType.StoredProcedure

    myConnection.Open()
    TheCommand.ExecuteNonQuery()

    myConnection.Close()
    myConnection.Dispose()

End Sub

Private Function getNewRssItemDataTable(NewRssItems As List(Of Structures.RssItem)) As DataTable

    Dim dt As New DataTable
    dt.Columns.Add("link", GetType(String)).MaxLength = 255
    dt.Columns.Add("descrip", GetType(String)).MaxLength = 255
    dt.Columns.Add("title", GetType(String)).MaxLength = 255

    For Each NewRssItem In NewRssItems
        Dim row = dt.NewRow
        dt.Rows.Add(row)
        row(0) = NewRssItem.link
        row(1) = NewRssItem.description
        row(2) = NewRssItem.title

    Next NewRssItem

    Return dt

End Function

修改

我看到你提到你想要一个SqlBulkCopy示例。如果插入是无条件的,则可以使用此技术:

Sub executeBulkInsert(connectionString As String, site As String, category As String, NewRssItems As List(Of Structures.RssItem))

    Dim dt As New DataTable

    dt.Columns.Add("link", GetType(String)).MaxLength = 255
    dt.Columns.Add("descrip", GetType(String)).MaxLength = 255
    dt.Columns.Add("site", GetType(String)).MaxLength = 255
    dt.Columns.Add("title", GetType(String)).MaxLength = 255
    dt.Columns.Add("category", GetType(String)).MaxLength = 255

    For Each NewRssItem In NewRssItems
        Dim row = dt.NewRow
        dt.Rows.Add(row)
        row(0) = site
        row(1) = category
        row(2) = NewRssItem.link
        row(3) = NewRssItem.description
        row(4) = NewRssItem.title

    Next NewRssItem

    Dim bcp = New SqlBulkCopy(connectionString)
    bcp.DestinationTableName = "dbo.queue"

    bcp.WriteToServer(dt)

End Sub