Apilar error de desbordamiento cuando se utiliza jQuery / ASP.NET para el chat sencilla ajax

StackOverflow https://stackoverflow.com/questions/726580

Pregunta

Estoy tratando de crear un sencillo de chat Ajax con jQuery y ASP.NET. Mi código funciona así:

  1. Cuando se carga la página se actualiza la div 'chatbox' con una solicitud a la página messages.aspx, que se ocupa de conseguir nuevos mensajes desde la base de datos y se inicia una actualización automática con el setTimeout ().
  2. Cada vez que el usuario hace clic en el botón de enviar, añade el mensaje a la base de datos dentro del código messages.aspx Load.

Estoy recibiendo un error de desbordamiento de pila desde el principio, cuando se inicia el tiempo de espera, y no estoy seguro de lo que podría causar esto? Podría ser el almacenamiento en caché? Tal vez el código en messages.aspx no puede completar en funcionamiento dentro de los 5 segundos? ¡Cualquier ayuda sería apreciada!

Además, no me preocupo por ataques de inyección SQL todavía b / c que sólo estaba tratando de conseguir que funcione con código simple.

Aquí está mi código:

Client-side: 

<%@ Page Language="C#" AutoEventWireup="true"  CodeFile="Default.aspx.cs" Inherits="_Default" %>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">

    <html xmlns="http://www.w3.org/1999/xhtml" >
    <head runat="server">
        <title>Untitled Page</title>
        <script src="jquery.js" type="text/javascript"></script>
        <script type="text/javascript">
            $(document).ready(function() {
                refreshChat();
                $("#btnSend").click(function() {
                    addMessage();               
                });
                return false;
            });

            function refreshChat()
            {
                $.get("messages.aspx", function(data) {  
                    $("#chatbox").empty();
                    $("#chatbox").prepend(data);
                });
                setTimeout(refreshChat(), 5000);
            }

            function addMessage()
            {
                $.get("messages.aspx", {usr: $("#usr").val(), msg: $("#msg").val()} );
            }
        </script>
    </head>
    <body>
        <form id="form1" runat="server">

            <div id="input">
                username: <input type="text" name="usr" id="usr" /><br />
                message:<br />
                <textarea id="msg" name="msg" rows="5" cols="30"></textarea><br /><br />
                <input type="button" id="btnSend" name="btnSend" value="Send" />
            </div>

            <div id="chatbox"></div>

        </form>
    </body>
    </html>

lado del servidor: 

using System;
using System.Data.SqlClient;
using System.Configuration;
using System.Collections;
using System.IO;
using System.Web;
using System.Web.Security;
using System.Web.UI;
using System.Web.UI.WebControls;
using System.Web.UI.WebControls.WebParts;
using System.Web.UI.HtmlControls;

public partial class messages : System.Web.UI.Page
{
    protected void Page_Load(object sender, EventArgs e)
    {     
        SqlConnection conn = 
            new SqlConnection("Data Source=.\\SQLEXPRESS;AttachDbFilename=F:\\Chatter\\App_Data\\messages.mdf;Integrated Security=True;User Instance=True");
        conn.Open();

        string sql;
        SqlCommand comm;

        if (Request["usr"] != null)
        {
            string user = Request["usr"].ToString();
            string message = Request["msg"].ToString();
            sql = "insert into messages (usr, msg, [date]) values ('"
                + user + "', '" + message + "', '" + DateTime.Now + "')";
            comm = new SqlCommand(sql, conn);
            comm.ExecuteNonQuery();
        }

        sql = "select top 5 usr, msg from messages order by [date] desc";
        comm = new SqlCommand(sql, conn);
        SqlDataReader dr = comm.ExecuteReader();

        while (dr.Read())
        {
            Response.Write(dr["usr"].ToString() + ": <br/>" + dr["msg"] + "<br/><br/>");
        }
        dr.Close();

        conn.Close();
    }
}
¿Fue útil?

Solución

Su Javascript refreshChat función recursiva se llama a sí mismo. Cambiar el código para: 

        function refreshChat()
        {
            $.get("messages.aspx", function(data) {  
                $("#chatbox").empty();
                $("#chatbox").prepend(data);
            });
            setTimeout(refreshChat, 5000);
        }

Otros consejos

Es necesario para envolver la llamada de función en su setTimeout entre comillas, de lo contrario es evaluado inmediatamente, causando la recursividad infinita y un desbordamiento de pila: 

setTimeout("refreshChat()", 5000);
Licenciado bajo: CC-BY-SA con atribución
No afiliado a StackOverflow
scroll top