Probleem met if/else

rdgweb · 5 Juni 2013

Hallo,

Ik heb een probleem met een if else die het niet doet.
Ik zal het even uitleggen..
Ik heb een overzicht met project en zodra er op meer informatie wordt geklikt krijg je meerdere informatie over het project (dat is dus dit script) en daarin zit een bijlage die je kan bekijken, alleen heb je niet voor elke project een bijlage, dus die hebben dus een id dat 0 is. ik heb dus een if else in mijn script als je bij id boven de 0 heb in je database, dan krijg je de link voor de bijlage, zo niet dan komt de link niet.
Alleen het probleem is als ik op meer informatie klik waarbij het id van bijlage op 0 staat dat ik dan helemaal niks gecho'ed krijg op mijn applicatie, bij projecten waar wel een bijlage bij zit doet die het wel gewoon.
wie weet de fout?

dit is code:

PHP

<?php
include("connect.php");
session_start();
 
$projecten_id = $_GET["projecten_id"];
$_SESSION["projecten_id"] = $projecten_id;
 
$projecten     = mysql_query("SELECT p.*, a.project_afbeeldingen, b.project_bestand FROM `projecten` AS p, `project_afbeeldingen` AS a, `project_bestanden` AS b WHERE `projecten_id` = $projecten_id AND a.item_id = p.item_id AND p.bestand_id = b.bestand_id");
if( !$projecten ) { echo sprintf( "Query failed; %s" , mysql_error()); }
$rij = mysql_fetch_array($projecten);
		$p_titel = $rij["project_titel"];
		$p_afbeelding = $rij["project_afbeelding"];
		$p_langeomschrijving = $rij["project_langeomschrijving"];
		$p_bestand_id = $rij['bestand_id'];
		$p_bestand = $rij["project_bestand"];


    echo $p_titel."<br>";
    echo '<img src="' . $p_afbeelding . '" height="250" width="325"><br>';
    echo $p_langeomschrijving."<br>"; 


	if ( $p_bestand_id => 0 ){
		echo "<a href =" . $p_bestand . ">Download bijlage"."<br>"; 
	}
	else {


	};
	
?>

Toon Meer

wimmpie · 5 Juni 2013

moet het niet zijn:
if($p_bestand_id > 0)

L.Kok · 5 Juni 2013

Volgens mij is het inderdaad je groter dan of gelijk aan in de if zoals wimmpie het zei.

Het moet niet zijn: => maar >=

Dit zeggen we in het Nederlands toch ook zo?

wimmpie · 5 Juni 2013

Ik dacht al dat er iets niet klopte aan die =>, maar ik wist niet wat :p
maar als ik mij niet vergis moet het volgens zijn uitleg gewoon groter dan zijn

rdgweb · 5 Juni 2013

Ik heb het aangepast.
Ik krijg alleen nog steeds niks te zien zolang het bestand_id op 0 staat..

WHMCSAddons · 5 Juni 2013

rdgweb
Dat klopt, je zegt nu "Is gelijk aan 0 of hoger dan 0".

PHP

if ($p_bestand_id != 0) {
     //Downloaden mag
} else {
    //Geen download bekend
}

rdgweb · 5 Juni 2013

Ook geprobeerd.
Maar het probleem is dat denk niet persee..
Want anders had ik wel project titel, afbeelding en omschrijving gezien, maar deze worden ook niet weeergeven:S

WHMCSAddons · 5 Juni 2013

Voer je query eens uit via phpmyadmin, wat is daarvan het resultaat ?

Stefan.J · 5 Juni 2013

Wat Fils zegt is zeker een goed idee.

Ik heb je script in ieder geval even opgeknapt, en daarbij ook die syntax error in de if-lus opgelost:

PHP

<?php


include("connect.php");
session_start();


$_SESSION["projecten_id"] = (int) $_GET["projecten_id"];


$projecten = mysql_query("SELECT p.*, a.project_afbeeldingen, b.project_bestand FROM `projecten` AS p, `project_afbeeldingen` AS a, `project_bestanden` AS b WHERE `projecten_id` = ". $_SESSION["projecten_id"] ." AND a.item_id = p.item_id AND p.bestand_id = b.bestand_id");
if (!$projecten) {
    echo sprintf("Query failed; %s", mysql_error());
}


$project = mysql_fetch_array($projecten);


?>
<?php echo $project['project_titel']; ?><br>
<img src="<?php echo $project['project_afbeelding'] ?>" height="250" width="325"><br>
<?php echo $project['project_langeomschrijving'] ?><br>


<?php if ($project['bestand_id'] > 0): ?>
    <a href ="<?php echo $project['project_bestand'] ?>">Download bijlage<br>
<?php endif ?>

Toon Meer

De query heb ik verder niet naar gekeken en een beetje mijn twijfels bij. Ik zou deze in ieder geval omschrijven naar een duidelijke query met joins.

M.Beers · 5 Juni 2013

Check je output even met var_dump() ... het kan voorkomen dat de database een STRING ipv INT stuurd dan moet je het tussen quotes zetten

PHP

<?php echo "<pre>"; var_dump($project); echo "</pre>"; ?>

p.s. mysql is deprecated ... ik zou werken aan andere technieken ik citeer:

Citaat

This extension is deprecated as of PHP 5.5.0, and will be removed in the future. Instead, the MySQLi or PDO_MySQL extension should be used.

EternalFlamez · 6 Juni 2013

Toch denk ik ook dat het aan de database ligt. Kan je niet een test-query uitvoeren om te kijken wat eruit komt?
En dan beter zoiets als de volgende query:

PHP

mysql_query("SELECT
		p.*, 
		a.project_afbeeldingen, 
		b.project_bestand 
	FROM 
		`projecten` AS `p`
	LEFT JOIN `project_afbeeldingen` AS `a`
		ON `p`.`item_id`=`a`.`item_id`
	LEFT JOIN `project_bestanden` AS `b`
		ON `p`.`bestand_id`=`b`.`bestand_id`
	WHERE `projecten_id`='". mysql_real_escape_string($projecten_id) ."'");

Toon Meer

Oh en Stefan.J
Als je Alleen een echo tussen je php tags hebt staan kan je het verkorten door simpelweg dit te doen:

PHP

//Oud;
<?php echo "een text"; ?>


//Nieuw
<?= "een text" ?>

EDIT:
Trouwens, ik zie dat je in je query "project_afbeeldingen" ophaalt maar later "project_afbeelding" ophaalt uit het mysql object.

rdgweb · 6 Juni 2013

Is verholpen! bedankt iedereen

Victor · 6 Juni 2013

Mogen wij ook weten hoe?

rdgweb · 6 Juni 2013

SQL Code was fout..
Code ziet er nu zo uit:

PHP

<?php
include("connect.php");
session_start();
 
$projecten_id = $_GET["projecten_id"];
$_SESSION["projecten_id"] = $projecten_id;
 
$projecten     = mysql_query("SELECT p.*, a.project_afbeeldingen, b.project_bestand FROM `projecten` AS p
                              join `project_afbeeldingen` AS a
                              on a.item_id = p.item_id
                              left join `project_bestanden` AS b
                              on p.bestand_id = b.bestand_id
                              WHERE `projecten_id` = $projecten_id");
if( !$projecten ) { echo sprintf( "Query failed; %s" , mysql_error()); }
while ($print = mysql_fetch_assoc($projecten)) {
    $bestand_id        = $row_projecten['bestand_id'];


    echo $print["project_titel"]."<br>";
    echo '<img src="' . $print["project_afbeeldingen"] . '" height="250" width="325"><br>';
    echo $print["project_langeomschrijving"]."<br>";


	if ( $print["bestand_id"] != "0" )
	    echo "<a href =" . $print["project_bestand"] . ">Download bijlage"."<br>";
}
?>

Toon Meer

Thx to vincent

eeyk · 7 Juni 2013

Voortaan even deze gebruiken :

Code

error_reporting(E_ALL);

dat bespaart iedereen heel veel moeite.

Partner Gezocht om meerdere NFT Collecties op Open Sea te Plaatsen

Plesk installer

cpanel / whm themes

PHP Point of Sale

Probleem omtrent Tinymce

Kunnen jullie mij helpen met deze enquete

een eigen cloud sever

Probleem C++ en CL

193 Nieuwe domeinnamen April 2025

228 Nieuwe domeinnamen Maart 2025

285 Nieuwe domeinnamen Februari 2025

Probleem met if/else

Participate now!

Fijne feestdagen

Kritieke update voor Really Simple Security-plug-in

ING Nederland streeft naar ondersteuning van Google Pay tegen eind februari

Functioneel ontwerp

Access Control List implementatie in PHP/MySQL - deel 1/2

Access Control List implementatie in PHP/MySQL - deel 2/2

Delen