añadir ip a base de datos

manguiti · #1 (**permalink**) 23/04/2010, 06:42

Hola, estoy desarrollando un script que busque en una tabla donde se guardan las ips de los usuarios que han votado un video, junto con el id de ese video, de tal manera que si esa ip ya ha votado ese video no pueda volver a hacerlo y si no es así, almacene su voto e ingrese esa ip en la tabla... sencillo.... pues bien, me da error al formatear la consulta para ingresar la ip, a ver si podeis echarme una mano...

Código:

'Primero comprobamos que el usuario no haya votado ya ese video

sql_ip ="SELECT * FROM Ip_videos WHERE Id_video ="& id_video&""

'abrimos la conexion y ejecutamos la consulta
Set con_ip = Server.CreateObject ("ADODB.connection")
con_ip.Open Application("sqlserver")
set rs_ip=con_ip.execute(sql_ip)



	Do while not rs_ip.eof 
		
		
		
		response.write("ip_guardada: "& rs_ip("Ip_voto")&"<br>")
		
		if rs_ip("Ip_voto") = ip_voto THEN
		
			'Si esa ip ya ha votado el video redirijimos nuevamente a la pagina sin sumar el voto
			ip_repetida = "si"
			exit Do
			
			else
				ip_repetida = "no"
		end if
	
	rs_ip.movenext
		
	loop
	
'Cerramos la conexion
rs_ip.Close 
set rs_ip = nothing 
con_ip.Close 
set con_ip = nothing
	
	if ip_repetida = "no" then
		
		' En caso de no haber votado establecemos los nuevos datos
					
		puntos_nuevo = ccur(puntos) + ccur(votacion)
		
		response.Write("puntos_nuevo: "& puntos_nuevo &"<br>")
		
		votos_nuevo = votos + 1
		
		'Actualizamos los datos de la votacion del video
		sql_video ="UPDATE Videos_Canalrh SET Puntos = "& puntos_nuevo &", Total_votos= "& votos_nuevo &"  WHERE Id ="& id_video &""
		
		'abrimos la conexion y ejecutamos la consulta
		Set con_videos = Server.CreateObject ("ADODB.connection")
		con_videos.Open Application("sqlserver")
		set rs_videos=con_videos.execute(sql_video)
		
		
		 
		
		'Insertamos el valor de la ip del usuario que ha votado el video
		sql_inserta_ip="INSERT INTO Ip_videos (Id_video, Ip_voto) VALUES (" + id_video + ", " + ip_voto + ")"
		
		response.write("cons: "& sql_inserta_ip& "<br>")
		'abrimos la conexion y ejecutamos la consulta
		Set con_inserta_ip = Server.CreateObject ("ADODB.connection")
		con_inserta_ip.Open Application("sqlserver")
		set rs_inserta_ip=con_inserta_ip.execute(sql_inserta_ip)
		
	
	end if

Estoy empezando con asp, por lo que puede haber errores en cuanto al tratamiento de cierre de conxiones y demas.... espero que podais solucionarme esta duda... os paso el error que me da junto con la salida de la consulta...

cons: INSERT INTO Ip_videos (Id_video, Ip_voto) VALUES (1, 80.24.93.167)

Microsoft OLE DB Provider for SQL Server error '80040e14'

Line 1: Incorrect syntax near '.93'.

procesa_votos.asp, line 85

Myakire · #2 (**permalink**) 23/04/2010, 10:17

Encierra entre apóstrofes la IP (el campo es string, ¿cierto?), al dejarla así la base de datos cree que es un número

sql_inserta_ip="INSERT INTO Ip_videos (Id_video, Ip_voto) VALUES (" + id_video + ", '" + ip_voto + "')"

manguiti · #3 (**permalink**) 25/04/2010, 05:20

Muchas gracias, era eso!!

manguiti · #4 (**permalink**) 26/04/2010, 00:27

Por cierto, ahora me veo en la tesitura de que cuando se vota desde distintos ordenadores conectados a la misma red, solo deja votar al 1º por tener la misma ip, como podría solucionar este problema?Alguna idea??

Myakire · #5 (**permalink**) 26/04/2010, 08:17

Pues no se que tanto sea un problema, lo que quieres es que solo una IP pueda votar y eso es lo que estas consiguiendo, si quieres que solo un usuario pueda emitir un voto necesitarías un esquema Usaer/Password para este efecto, si estas trabajando en una página que corre sobre Internet.
Si es una intranet, pues hay códigos para obtener la mac address, que es lo que necesitarías hacer para distinguir a los diferentes clientes.

En mis pruebas tengo un código que bajé de algún sitio que alguna vez pensé utilizar en la red interna, aunque no me terminó de convencer la limitante del browser aunque fuera una intranet, pruebalo si gustas para ver la idea:

Código Javascript:

Ver original<!DOCTYPE  HTML  PUBLIC  "-//W3C//DTD HTML 4.0 Transitional//EN">
<html>
      <head>
            <title></title>
            <meta name="GENERATOR" content="Microsoft Visual Studio .NET 7.1">
            <meta name="vs_targetSchema" content="http://schemas.
microsoft.com/intellisense/ie5">
            <script id="clientEventHandlersJS" language="javascript">
<!--
 
function Button1_onclick() {
      var locator = new ActiveXObject ("WbemScripting.SWbemLocator");
 
      var service = locator.ConnectServer(".");
      var properties = service.ExecQuery("SELECT * FROM Win32_NetworkAdapterConfiguration");
      var e = new Enumerator (properties);
      document.write("<table border=1>");
      dispHeading();
      for (;!e.atEnd();e.moveNext ())
      {
            var p = e.item ();
            document.write("<tr>");
            document.write("<td>" + p.Caption + "</td>");
            document.write("<td>" + p.IPFilterSecurityEnabled + "</td>");
            document.write("<td>" + p.IPPortSecurityEnabled + "</td>");
            document.write("<td>" + p.IPXAddress + "</td>");
            document.write("<td>" + p.IPXEnabled + "</td>");
            document.write("<td>" + p.IPXNetworkNumber + "</td>");
            document.write("<td>" + p.MACAddress + "</td>");
            document.write("<td>" + p.WINSPrimaryServer + "</td>");
            document.write("<td>" + p.WINSSecondaryServer + "</td>");
            document.write("</tr>");
      }
      document.write("</table>");
}
 
function dispHeading()
{
      document.write("<thead>");
      document.write("<td>Caption</td>");
      document.write("<td>IPFilterSecurityEnabled</td>");
        document.write("<td>IPPortSecurityEnabled</td>");
        document.write("<td>IPXAddress</td>");
      document.write("<td>IPXEnabled</td>");
      document.write("<td>IPXNetworkNumber</td>");
      document.write("<td>MACAddress</td>");
      document.write("<td>WINSPrimaryServer</td>");
      document.write("<td>WINSSecondaryServer</td>");
      document.write("</thead>");
}
 
//-->
            </script>
      </head>
      <body>
            <INPUT id="Button1" type="button" value="Button" 
name="Button1" language="javascript" onclick="return Button1_onclick()">
      </body>
</html>

manguiti · #6 (**permalink**) 27/04/2010, 01:07

Bueno, mirando otras aplicaciones web donde se votan videos o noticias... si puedo votar la misma desde distintos ordenadores conectados a la misma red, pero sólo una vez al día, entonces supongo que se hará mediante cookies o caché ... de todas maneras, esa solución de poder votarlo una vez al día aunque sea desde distintos ordenadores de la misma red tampoco me convence... creo que la que he implantado actualmente es mas realista a la hora de obtener unos resultados estadísticos correctos...

Myakire · #7 (**permalink**) 27/04/2010, 07:18

Cita:

creo que la que he implantado actualmente es mas realista a la hora de obtener unos resultados estadísticos correctos

Yo solo se de estadística lo que me enseñaron en la universidad hace años

, lo cual no es mucho, pero supongo que tendrás alguna fórmula para determinar qué quieres discriminar: si a los muchísimos usuarios diferentes que salen por una misma IP (empleados de una empresa, de una escuela, etc) o a aquellos, no se que tantos puedan ser, que por diversión pueden tomarse el tiempo de buscar diferentes ordenadores o browsers para emitir un voto.

No se que sea peor o mejor para obtener resultados estadísticos correctos.

manguiti · #8 (**permalink**) 28/04/2010, 03:22

Hombre, pues no se, pero si alguien vota un video y puede hacerlo todos los días, por mucho que no se esté discriminando el voto de todos los de su misma red, la muestra será menos fiel si tenemos el mismo voto repetido o al mismo votante repitiendo voto (que no votacion) que si restringimos el acceso por ip y solo puede votar una vez cada video... en una base de datos de 30.000 suscriptores más el tráfico que genera que no lo son, creo que habrá "suficientes ip´s" como para tomar una muestra lo suficientemente amplia para hacernos una idea de si el video gusta o no.... o no??

alfberto · #9 (**permalink**) 29/04/2010, 05:15

El problema de las estadísticas en las encuestas es el siguiente:
Si quieres estadísticas verídicas, la encuesta no será anónima, ya que siempre tendrás que controlar la IP del que vota o usar otros métodos de autenticación.
Si quieres encuestas anónimas olvídate de obtener estadísticas fiables, entonces te recomiendo que pongas en la balanza lo que más te conviene y tomes una decisión.

Saludos