Mitt diskussionstrådsutskick blir slöare med ökat medlemsantal

Jag skulle föreslå dig att skapa ett separat script för mailutskicken som du kör t.ex var 10:e minut (schemaläggning alt. cron job) som kollar vilka trådar som är uppdaterade senaste 10 minuterna och skickar mailen.

Tack för förslaget. Just nu håller jag på och förbereder för en migrering bort från den aktuella plattformen till vBulletin.

Går det inte att göra så att knappen blir "disabled" under tiden som tråden postas, så att användaren inte kan trycka på den en gång till? hur skulle det i så fall se ut rent kodmässigt sett?

Självklart går det, här finns ett exempel: http://www.netevolution.co.uk/scripts.asp?ID=39

The Hulk, tack för länken! Det var precis det jag sökte efter. Koden är inga problem att implementera. Dock har jag en liten följdfråga.

Så här ser koden ut som ska in i form-taggen:

<form onSubmit="return disableForm(this);">

Hur gör jag om jag redan har en onsubmit i form-taggen i mitt inläggsformulär?

<form action="newthread.php" method="post" name="newthread" onsubmit="return validateNewThread(this)" id="postMsg">

Kan jag ha dubbla "onsubmit" på något vis?

Tack på förhand!

Hmm, kanske går att göra så här:

<form action="newthread.php" method="post" name="newthread" onsubmit="return validateNewThread(this) && disableForm(this);" id="postMsg">

annars kan du skapa en ny funktion där du anropar de två andra funktionerna ifrån.
Vet inte hur du vill hantera retur-värdena.

<script>
function fn_onSubmit(form)
{
  return validateNewThread(form) && disableForm(form);
}
// eller
function fn_onSubmit(form)
{
  var b1 = validateNewThread(form)
  var b2 = disableForm(form);
  return b1 && b2;
}

<form action="newthread.php" method="post" name="newthread" onsubmit="return fn_onSubmit(this)" id="postMsg">
</script>

Jag väcker liv i den här tråden igen. Som rubriken beskriver så har trådutskicket till medlemmar blivit slöare i takt med att antal prenumeranter har ökat. Av prestandaskäl blev jag tvungen att ta bort den temporärt på grund av att användare trodde att skicka-processen hade hängt sig och försökte på nytt och på nytt. Detta resulterade i att mailutskicket gick iväg i flera kopior till prenumeranterna.

Som denna tråd visar bollade jag lite idéer hur man kunde disabla skicka-knappen tills mailutskicket och hela den processen var klar. Fick det inte helt att fungera. Nu klistrar jag in den kod som är relevant igen så får jag se om jag tillsammans med någon kan komma fram till varför den inte gör någon skillnad. Ursprungligen anropade jag bara en javaskriptfunktion i <form>-taggen. Nu när skicka-knappen ska disablas krävs det ytterligare en funktion (förutom validateNewThread(obj)). Jag fick tipset att baka ihop dessa två till en. Det är detta som jag inte upplever fungerar. Jag får inte något skriptfel när sidan laddas men jag får heller ingen kvittens på att det fungerar.

Någon som ser vad som kan vara fel?! :)

Kod i newthread.php (innehåller formulär för att skriva inlägg):

<script language="JavaScript" src="../inc/function.js"></script>

<script type="text/javascript">
function sessionKeepAlive(){
    var img = new Image();
    img.src = "/noop.php?foo="+Math.random();
}
function scheduleSessionKeepAlive(){
    var delay = 1000 * 60 * 1; // antal millisekunder att vänta innan anrop till noop.php, t.ex. 55 minuter
    setInterval(sessionKeepAlive, delay);
}
scheduleSessionKeepAlive();

function disableForm(theform) {
    if (document.all || document.getElementById) {
        for (i = 0; i < theform.length; i++) {
        var tempobj = theform.elements[i];
        if (tempobj.type.toLowerCase() == "submit")
            tempobj.disabled = true;
        }
		
    setTimeout('alert("Your form was submitted. Submit and reset buttons were disabled.")', 4000);
    return true;
    }
}

function fn_onSubmit(form)
{
  return validateNewThread(form) && disableForm(form);
}
</script>

....

<form action="newthread.php" method="post" name="newthread" onsubmit="return fn_onSubmit(this);" id="postMsg">

....

<tr style="background-color: #fff;">
	<td width="15%" style="padding-left: 5px; font-weight: bold;">&nbsp;</td>
	<td colspan="3" width="85%"><input name="createthread" type="submit" value="Skapa tråden" />&nbsp;<input name="previewthread" type="submit" value="Förhandsgranskning" /></td>
</tr>

function.js (länkas in i newthread.php):

function validateNewThread(obj) {

	var elms = obj.elements;

	if(trim(elms["threadsubject"].value).length == 0) {
		alert("Fyll i en rubrik!");
	   	elms["threadsubject"].focus();
	   	return false;
	}

	if(trim(elms["forummessage"].value).length == 0) {
		alert("Skriv ett meddelande!");
	   	elms["forummessage"].focus();
	   	return false;
	}
	
return true;
}

Har försökt att slå ihop valideringsfunktionen med den som ska disabla skicka-knappen. Dock får jag inte det resultat jag vill ha.

function validateNewThreadNew(obj) {

	var elms = obj.elements;

	if(trim(elms["threadsubject"].value).length == 0) {
		alert("Fyll i en rubrik!");
	   	elms["threadsubject"].focus();
	   	return false;
	} 
	
	if(trim(elms["forummessage"].value).length == 0) {
		alert("Skriv ett meddelande!");
	   	elms["forummessage"].focus();
	   	return false;
	}

	if(trim(elms["forummessage"].value).length == 1 AND trim(elms["threadsubject"].value).length == 1) 
	{
        for (i = 0; i < obj.length; i++) 
		{
			var tempobj = obj.elements[i];
			if (tempobj.type.toLowerCase() == "submit")
				tempobj.disabled = true;
		}
		
    setTimeout('alert("Your form was submitted. Submit and reset buttons were disabled.")', 4000);
    return true;
    }
return true;
}

De två första if-satserna kontrollerar om rubrik och content är tomma eller ej. Är de tomma returneras ett false-värde. Den sista if-satsen är den som ska inaktivera skicka-knappen i formuläret. Som jag sa får jag det inte att fungera. Har säkert gjort en tankevurpa någonstans. Om både första if-satserna returnerar 1 (sant) så ska forumtråden kunna skickas iväg och då ska knappen inaktiveras.

Hur får jag till detta?

Så här ser anropet av ovanstående funktion ut i <form>-taggen i HTML-koden:

<form action="newthread.php" method="post" name="newthread" onsubmit="return validateNewThreadNew(this);" id="postMsg">

När jag kör ursprungsfunktionen ensamt så fungerar det som det är tänkt med att formulärknapparna inaktiveras.

Koden ser ut som följer:

function disableForm(theform) {
   	
	if (document.all || document.getElementById) {
        
		for (i = 0; i < theform.length; i++) {
        var tempobj = theform.elements[i];
        if (tempobj.type.toLowerCase() == "submit" || tempobj.type.toLowerCase() == "reset")
		{
            tempobj.disabled = true;
        }
setTimeout('alert("Your form was submitted. Submit and reset buttons were disabled.")', 2000);
    return true;
    }
}

Koden som jag vill integrera i samma funktion ser ut så här:

var elms = obj.elements;

	if(trim(elms["threadsubject"].value).length == 0) {
		alert("Fyll i en rubrik!");
	   	elms["threadsubject"].focus();
	   	return false;
	}

	if(trim(elms["forummessage"].value).length == 0) {
		alert("Skriv ett meddelande!");
	   	elms["forummessage"].focus();
	   	return false;
	}

Jag tror att det som ställer till det lite är att det är två retur-värden som ska tillbaka till webbläsaren, eller?

if(trim(elms["forummessage"].value).length == 1 AND trim(elms["threadsubject"].value).length == 1)

Det här ser underligt ut på flera sätt... Vill du verkligen bara köra formulärinaktiveringen om både rubriken och meddelandet är exakt ett tecken vardera? Och AND borde nog vara && om du verkligen vill ha med detta vilkor.

Men det känns som att den sista if-satsen inte borde vara med alls eftersom du redan innan kollat så att det skrivits något i de båda fälten... Du borde dessutom kunna använda dig av din elms-variabel i din sista for-sats eftersom den variabeln redan innehåller formulärelementen. Typ:

for (var i = 0; i < elms.length; i++)
{
	var tempobj = elms[i];
	if (tempobj.type && tempobj.type.toLowerCase() == "submit")
	{
		tempobj.disabled = true;
	}
}

Den här sista koden respresenterar mitt dåliga ihopkok :)

Funktionerna jag vill ha är att formulärknapparna inaktiveras varje gång man trycker på dem för att minimera risken för dubbelposter. Därefter vill jag kontrollera så att båda formulärfälten är ifyllda.

Jag utgick ifrån din näst sista post (den som var näst sist när jag skrev förra gången alltså: post #8), för den posten såg trots allt ut att ligga närmast det du försöker göra.

Om du byter ut den sista if-satsen (som jag skrev var fel) och dess innehåll och klistrar in koden jag skrev i mitt förra inlägg bör den väl göra det du vill tycker jag...

jawbreaker skrev:

if(trim(elms["forummessage"].value).length == 1 AND trim(elms["threadsubject"].value).length == 1)

Det här ser underligt ut på flera sätt... Vill du verkligen bara köra formulärinaktiveringen om både rubriken och meddelandet är exakt ett tecken vardera? Och AND borde nog vara && om du verkligen vill ha med detta vilkor.

Det var jag som läste koden dåligt. Jag tolkade värdet 1 som boolskt. Men .lengt talar faktiskt om antal tecken i strängen. Såg det nu.

Nu har jag gjort om koden. Då ser den ut så här:

function validateNewThreadNew(obj) {

	var elms = obj.elements;

	if(trim(elms["threadsubject"].value).length == 0) {
		alert("Fyll i en rubrik!");
	   	elms["threadsubject"].focus();
	   	return false;
	} 
	
	if(trim(elms["forummessage"].value).length == 0) {
		alert("Skriv ett meddelande!");
	   	elms["forummessage"].focus();
	   	return false;
	}

    for (var i = 0; i < elms.length; i++)
	{
		var tempobj = elms[i];
	
		if (tempobj.type && tempobj.type.toLowerCase() == "submit")
		{
			tempobj.disabled = true;
			return true;
		}
	}
		
    //setTimeout('alert("Your form was submitted. Submit and reset buttons were disabled.")', 4000);
}

Nu fungerar varken validerings- eller inaktiveringsfunktionen. I den sista if-satsen tog jag bort "return true". Skickar jag tillbaka det blir det väl fel om någon av valideringssatserna samtidigt skickar tillbaka ett false-värde?

Funktionen returnerar alltid bara ett värde. Om någon av valideringarna misslyckas returneras false, så inget efter return false-raden kommer att köras i det fallet.

Du bör se till att funktionen alltid returnerar ett värde. Jag är inte säker på att det spelar någon roll - egentligen borde nog formuläret postas så länge funktionen inte returnerar false - så det _borde_ inte vara det som är fel i det här fallet, men man vet aldrig...

I övrigt ser jag inget uppenbart fel i funktionen, ditt fel kan nog ligga var som helst... Du får posta mer (eller helst _all_) kod om det ska gå att hitta något.

Jag skrev ihop ett exempel lite snabbt, med utgång från koden ovan, och det fungerar för mig:

<!doctype html>
<html lang="sv">
<head>
<meta charset="utf-8">

<title>Testformulär</title>

<script>
function trim(str) { return str.replace(/^\s+|\s+$/g, ''); }

function validateNewThreadNew(obj) {

	var elms = obj.elements;

	if (trim(elms["threadsubject"].value).length == 0) {
		alert("Fyll i en rubrik!");
		elms["threadsubject"].focus();
		return false;
	}

	if (trim(elms["forummessage"].value).length == 0) {
		alert("Skriv ett meddelande!");
		elms["forummessage"].focus();
		return false;
	}

	for (var i = 0; i < elms.length; i++)
	{
		var tempobj = elms[i];

		if (tempobj.type && tempobj.type.toLowerCase() == "submit")
		{
			tempobj.disabled = true;
			alert("knapp inaktiverad...");
		}
	}

	return true;
}
</script>
</head>

<body>
<form action="" method="post" onsubmit="return validateNewThreadNew(this);">
	<fieldset>
		<legend>Skriv forummeddelande</legend>

		<label for="threadsubject">Rubrik</label><br>
		<input type="text" id="threadsubject" name="threadsubject">
		<br>
		<label for="forummessage">Meddelande</label><br>
		<textarea id="forummessage" name="forummessage" rows="10" cols="40"></textarea>
		<br>
		<input type="submit" value="Skicka">
	</fieldset>
</form>
</body>
</html>

Nu har jag testat ditt exempel ovan och allting fungerar bra med validering och inaktivering av knappar tills det är dags för formulärinnehållet att processas i php-filen. När jag trycker på skicka-knappen kommer jag hela tiden tillbaka till sidan med formuläret och det innehåll som finns i formuläret.

Nja, när du trycker på Skicka kommer du till den sida som är angiven i action-attributet i form-taggen. Jag har inte angivit någon sida alls i mitt exempel, så därför hamnar man på samma sida igen... Du får skriva in rätt php-sida i action-attributet helt enkelt...

I mitt fall så är det samma sida som formuläret återfinns på. Jag kör med en testfil nu och där står det exakt lika som i den skarpa. Ska felsöka ytterligare och se om jag kan komma på vad det beror på.

När jag inaktiverade följande del av koden (den som inaktiverar knappen9 gick det bra igen att skapa tråden i databasen:

for (var i = 0; i < elms.length; i++)
	{
		var tempobj = elms[i];

		if (tempobj.type && tempobj.type.toLowerCase() == "submit")
		{
			tempobj.disabled = true;
		}
	}

Vad kan det vara som skapar denna blockering?

Inte lätt att veta utan att se hur koden (även i php-filen) ser ut... Men en gissning är att submit-knappen har ett namn (name="...") och att du kollar efter detta namn i din php-fil innan du sparar något till databasen. När knappen inaktiveras kommer dess namn inte skickas med till php-filen, så då får man trixa till det på annat sätt...

jawbreaker skrev:

Men en gissning är att submit-knappen har ett namn (name="...") och att du kollar efter detta namn i din php-fil innan du sparar något till databasen. När knappen inaktiveras kommer dess namn inte skickas med till php-filen, så då får man trixa till det på annat sätt...

Varför tänker inte jag på sånt där?! Tur att det finnas andra som har huvudet med sig :)

Din teori låter logisk. Jag har en if-sats som drar igång hela processen med att spara ner informationen till databasen. Jag kör på submit-knappens värde som är "Skapa tråd". Är knappen inte aktiverad så kommer väl aldrig namnet med, som du sa?

Vilken tror du är den enklaste work-arounden för detta? Finns det någon annan variabel som jag kan använda mig av om vi pratar generella programmeringstermer?