php - PM hjælp - Udvikleren.dk - Programmering, webdesign og grafik

Tags: php

Bruger #9674 @ 10.06.06 20:16

Hej

Jeg har et meget irreterende problem med mi PM (Privat Message) system

Jeg har lavet det sådan så man kan have nogle fortrukne brugere, og det fungere udemærket.

Problemmet kommer så når jeg skal have vist brugere som har send en besked til fx mig og ikke er på ens foretrukne liste.

Her er koden til Hele Listen:

Kode

<?php
	# Overskrift
	echo "<center><p><u>Foretrukne:</u></p></center>";
	# Henter brugerens elevnr
	$elevnr = $_SESSION['elevnr1'];

	# Henter elevens foretrukne brugere
	$sql = "SELECT * FROM elevliste WHERE elevnr = '$elevnr'";
	tjek_sql($sql);
	opret_forbindelse();
		$x = mysql_query($sql);
		$c = mysql_fetch_array($x);
	luk_forbindelse();
	
	# Splitter resultatet op og laver et array med de foretruknes ID nr
	$fortrukne = split(" ", "$c[fav]");
	# Sortere de foretrukne
	sort($fortrukne);
	$i = 0;
	# Så længe arrayet er true
	while($fortrukne[$i]) {
		# Hvis den skulle være tom
		if(!empty($fortrukne[$i])) {
			# Henter information om de foretrukne elever
			$sql = "SELECT * FROM elevliste WHERE id='$fortrukne[$i]'";
			tjek_sql($sql);
			opret_forbindelse();
				$a = mysql_query($sql);
				$b = mysql_fetch_array($a);
			luk_forbindelse();
			
			# SQL til at hente antallet af beskeder fra den enkelte elev
			$sql = "SELECT * FROM pm_message WHERE til = '$c[id]' && fra = '$b[id]'";
			opret_forbindelse();
				$a = mysql_query($sql);
				$antal = mysql_num_rows($a);
			luk_forbindelse();
			# Udskriver eleven på listen
			echo "<p><a href='?action=skriv&id=$b[id]&e=$_SESSION[elevnr1]' title='Skriv besked'>
						<img src='img/ny.gif' border='0' id='pm_skriv'></a>
					<a href='?action=laes&id=$b[id]&e=$_SESSION[elevnr1]' title='Modtagende'>
						<img src='img/modtagne.gif' border='0' id='pm_mod'></a>
					<a href='action.php?action=delete&id=$b[id]&e=$_SESSION[elevnr1]' title='Slet kontakperson'>
						<img src='img/delete.gif' border='0' id='pm_del'></a>
						($antal) $b[fornavn] $b[efternavn]</p>";
			# $i + 1 for at komme videre i arrayet
			$i++;
		}
	}
	
	# Overskrift
	echo "<center><p><u>Andre Kontakter:</u></p></center>";
	
	# Henter brugerens Fortrukne og hans ID
	# Deler fortrukne op ved alle mellemrum " "
	# og smider det i et array
	$sql = "SELECT * FROM elevliste WHERE elevnr='$elevnr'";
	opret_forbindelse();
		$x = mysql_query($sql);
		$c = mysql_fetch_array($x);
	luk_forbindelse();
	$for = split(" ", $c[fav]);
	sort($for);
	
	# Variabel til at definere nummeret i arrayet
	$i = 0;

	# Henter alle beskeder til brugeren
	$sql = "SELECT * FROM pm_message WHERE `til` ='$c[id]'";
	opret_forbindelse();
		$a = mysql_query($sql);
	luk_forbindelse();
	
	# Begynder at skrive så længe der er beskeder til brugeren
	while($b = mysql_fetch_array($a)) {
		# Tjekker om beskeden er fra en bruger som eleven 
		# har på sin fortrukne liste hvis han har sker der ingen ting
		# ellers køre den videre
		if($for[$i] == $b[fra]) {
		} else {
			# Henter information om brugeren som har sendt beskeden
			$sql = "SELECT * FROM elevliste WHERE id='$b[fra]'";
			tjek_sql($sql);
			opret_forbindelse();
				$a = mysql_query($sql);
				$g = mysql_fetch_array($a);
			luk_forbindelse();
			
			# Tjeker hvor mange beskeder der er fra de enkelte brugere
			$sql = "SELECT * FROM pm_message WHERE til='$c[id]' AND fra ='$b[fra]'";
			tjek_sql($sql);
			opret_forbindelse();
				$a = mysql_query($sql);
				$antal = mysql_num_rows($a);
			luk_forbindelse();
			
			# Begynder udskrivningen			
			echo "<p><a href='?action=skriv&id=".$g[id]."&e=".$_SESSION[elevnr1]."' title='Skriv besked'>
						<img src='img/ny.gif' border='0' id='pm_skriv'></a>
					<a href='?action=laes&id=".$g[id]."&e=".$_SESSION[elevnr1]."' title='Modtagende'>
						<img src='img/modtagne.gif' border='0' id='pm_mod'></a>
						(".$antal.") ".$g[fornavn]." ".$g[efternavn]."</p>";
			# $i + 1 for at komme videre i arrayet
			$i++;
		}
	}
?>

Her er der et billeder af hvad der sker sådan som det er nu:

Link til billede

Håber der er nogen som kan hjælpe da jeg er på bar bund.

Hvis i har spørgsmål så skriv endelig så skal jeg svare så godt jeg kan.

7 svar postet i denne tråd vises herunder
1 indlæg har modtaget i alt 4 karma

Sorter efter stemmer Sorter efter dato

Bruger #4575 @ 11.06.06 20:28

218

Det var så lidt...

Bruger #4575 @ 11.06.06 12:07

218

Hvad er det præcis problemet består i?

Men der er i hvert fald fejl i dit "if($for[$i] == $b[fra])". Du skal gennemgå hele $for-arrayet i denne test, da du ikke kan forvente at de to arrays følger hinanden 100% - i det tilfælde skulle alle "andre kontakter" være på listen under "foretrukne".

I stedet kan du lave noget ala:

Kode

$isFor = false; /* Som udgangspunkt er ingen andre kontakter på listen over foretrukne */

/* Gennemgå alle foretrukne */
foreach($for as $elev)
{
  /* Tjek om eleven er foretrukken */
  if($b[fra] == $elev)
  {
    /* Angiv at eleven er foretrukken */
    $isFor = true;

    /* Stop tjekket, da det er urelevant at gå videre (vi ved allerede, at eleven er foretrukken) */
    break;
  }
}

if(!$isFor)
{
  // Alt din kode her
}

Desuden henter du alle beskeder, som sendes til en given elev. Forestil dig at kasper har sendt 3 beskeder til martin: Kasper vil nu findes 3 gange i din liste over beskeder til Martin. Da du ikke tjekker om eleven tidligere er udskrevet, vil Kasper blive udskrevet 3 gange

Løser dette ikke dit problem, må du jo vende tilbage..

Bruger #4575 @ 11.06.06 12:10

218

Forresten overskriver du $a. Du gemmer listen over alle beskederne i $a, før du gør ind i din while-løkke.

Senere i denne sætter du $a = mysql_query($sql). Dette overskriver din liste over beskeder, hvorfor du vil ende i et uendeligt loop, hvor samme bruger konstant vil udskrives. Vælg i stedet $c, så vil du undgå at udskrive samme person hele tiden

Bruger #9674 @ 11.06.06 14:37

111

Ok, takker for dit svar... Tjekker lige op på det du har skrevet og hvis der opstår nogle problemer vender jeg lige tilbage, ellers vil du se at du har modtaget 100 point

Bruger #9674 @ 11.06.06 14:46

111

Hej Mads

Du skriver det med at jeg vil få 3 med den samme elev i din første besked.

Hvordan tjekker jeg at personen ikke allerede er udskrevet?

Bruger #4575 @ 11.06.06 18:29

218

Jeg vil foreslå dig at oprette et array $udskrevet = array();

Dette gøres før du går ind i dit loop, hvorefter du fylder op med de elevers id, som bliver udskrevet:

array_push($udskrevet, $b[fra]);

Dette tilføjes allersidst i dit loop (hvor du tidligere opdaterede i

)

Herefter tilføjer du følgende kode umiddelbart efter, at du tjekker om brugerne er på listen over foretrukne:

Kode

$isUdskrevet = false; /* Som udgangspunkt er ingen udskrevet i forvejen */

/* Gennemgå alle foretrukne */
foreach($udskrevet as $elev)
{
  /* Tjek om eleven er udskrevet */
  if($b[fra] == $elev)
  {
    /* Angiv at eleven er udskrevet */
    $isUdskrevet = true;

    /* Stop tjekket, da det er urelevant at gå videre (vi ved allerede, at eleven er udskrevet) */
    break;
  }
}

if(!$isFor && !$isUdskrevet)
{
  // Alt din kode her...
}

Har ikke testet, men det burde løse dine problemer. Held og lykke med dit projekt

Bruger #9674 @ 11.06.06 19:02

111

Tak, rigtig mange gange...

Har ikke tjekket men ser ud til at det hvirker, så du får point...

Takker for hjælpen...

Du skal lige skrive en kommentar med muglighed for point, ellers for du ikke de 100...

Indlæg senest redigeret d. 11.06.2006 19:03 af Bruger #9674

PM hjælp

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler