Je zou het buiten de PHP code kunnen halen:

<?php
    public function showAllProducts()
    {
        global $mysqli;
    
        $stmt = $mysqli->query("SELECT * FROM `products`");
        while($x = $stmt->fetch_assoc())
        {
            
                ?><img src="<?php echo $x['image'] ?>" width="100" height="80">
                <form method="post">
                            <input type="hidden" name="id" value="<?php echo $x['id'] ?>">
                            <input style="width: 40px;" type="number" min="0" name="amount">
                            <input type="submit" value="Toevoegen!" name="submit">
                        </form>
               <?php
        }
        
    }

Je global zorgt er trouwens voor dat je code volledig on-onderhoudbaar wordt, en zou je eigenlijk anders moeten oplossen!

Daarbij kun je beter werken met views, en geen PHP en HTML code meer mixen.

Het voorbeeld gaat zo niet werken omdat je JavaScript-code syntaxisch niet klopt. Kijk eens naar de accolades, je sluit niet alle code blokken.

[qoute]Tenzij er veel bugs inzitten en jij die eruit haalt word het gedeeltelijk jou systeem.
Dan mag je er wel winst aan maken volgens mij. [/quote]

Dat is nergens op gebaseerd. Als jij software van iemand afneemt onder een bepaalde licentie, dan moet je je daar aan houden. Zodra je iets aanpast is het niet ineens van jou, en moet je nog steeds ervoor betalen als dat bij jou toepassing van het product de bedoeling is.

Ja. De PHP code hoort niet in het bestand met de JSON, dat werkt zo niet. Je zult het in het onderstaande bestand moeten zetten.

Ps. Hoezo is de JSON niet altijd hetzelfde?

De kwestie die je nu voorlegt is een er eentje tussen security en gebruikersgemak.

Ik hanteer meerdere wachtwoorden, en als ik ergens een tijdje niet ben geweest is voor mij dan een inlogpoging of 5 zeker niet weinig. Een bruteforce aanval gaat eerder over de 100 duizenden pogingen heen.

De je gebruikers een lol en geef ze wat ruimte om hun wachtwoord in te vullen. Tot 25 pogingen kan makkelijk.

Toch meer security? Neem dan iets wat echt invloed heeft, een goede wachtwoord policy. Minimaal 8 tekens met cijfers, speciale tekens, en letters bijvoorbeeld.

De volgende pakketten zijn hiervoor beschikbaar:

- Google Chart Tools (https://developers.google.com/chart/)
- Highcharts (http://www.highcharts.com/)

En nog tig anderen. Ik wil je Highcharts aanraden, dat is een hele mooie namelijk. Deze kost wel wat als je het commercieel wilt gebruiken.

M.Martens: JSON is geen taal en hij programmeert in PHP..

Waar heb je het PHP fragment wat je post neergezet? Want ik denk te begrijpen dat je een file x hebt met daarin het bovenstaande PHP fragment en daaronder JSON, en een ander PHP bestand dat met fopen file x opent en deze over het netwerk schrijft.

Dat kan ik verkeerd hebben. Maar het bovenstaande PHP fragment moet in het bestand komen te staan dat de JSON serveert.

Kun je anders alle code van toepassing eens posten?

Ik heb je code maar even geformat, want het was een zooitje, enters zijn niet verboden.

Een action meevoeren klinkt als een rare vraag.. Bedoel je dat je een GET parameter ($_GET) wilt gebruiken in een geïnclude script? Want dan is de oplossing eenvoudig, alle variabele zijn beschikbaar voor geïnclude files uit de betreffende context. Dus ook globals als $_GET en $_POST.

Ik heb het script even in de debugger gehangen. Het probleem zit hem erin dat wanneer er slechts een element bestaat met de naam price[], er geen array wordt gereturned, maar enkele dat element. Als er meerdere instaan, dan bevat elements['price[]'] wel een array.

Het beste lijkt me als je de price elementen anders selecteert.

Overigens zou ik je aanraden een framework als jQuery te gebruiken, en niet alles op de global scope te definiëren.

Zodra je functies gaat schrijven waarin je het global-keyword nodig hebt betekent dat eigenlijk altijd dat je architectuur niet correct is... Bedenk je dus eerst heel goed waarom je het global-keyword moet gebruiken, en of er geen betere oplossing is.

rdgweb: Je bent al wel begonnen met je stage zei je?

Ik heb al meerdere plugins geschreven voor verschillende systemen (voornamelijk Java), dus globale vragen mag je stellen, maar denk niet dat je er veel mee opschiet.

Wat voor opleiding doe je precies? Hoe groot is het bedrijf waar jij stage loopt?

Buitenom dat ik dit project geen groot kans van slagen geef (dit soort initiatieven zijn er vaker geweest) vind ik het positief dat het geprobeerd wordt. De meesten mensen hier hebben er hobby aan te programmeren maar zijn zeker geen volwaardige programmeurs. Maar dat is niet erg, van elkaar kun je leren en als je daarin gemotiveerd bent kun je een heel eind komen.

De deelnemers zou ik willen zeggen: organiseer goed, en respecteer elkaar en elkaars kunnen. Dan zou dit best eens goed kunnen aflopen.

scarface: Je stelt je erg negatief op tegenover de anderen in dit topic. Maar gezien jou berichten hier en wat ik in eerdere topics van jou zie mag je jezelf ook best onder de categorie beginners plaatsen.. Overigens is het niet JavaScript of jQuery, maar JavaScript met jQuery. Dan kun je er trouwens nog voor kiezen frameworks als AngularJS of Backbone te gebruiken.

Citaat van M.Martens

Ik verwacht al wat problemen.

Euhh, wat voor problemen?

Voor de gebruiker zal het niet veel invloed hebben. De snelheid van het laden van pagina's overigens ook niet hoor. Advertentiesystemen en gebruikersstatistieken zullen er vast onder leiden inderdaad.

Spreekwoorden en wel of niet Anonymous aanhangers kan ik me niet echt druk om maken, misschien moeten jullie dat ook maar niet meer doen.

Grote bedrijven zoals banken hebben vaak een heel applicatielandschap. In het verleden zijn er allerlei softwarepakketten aangeschaft of gerealiseerd voor specifieke taken binnen deze bedrijven. De software wordt vaak via allerlei lijntjes met elkaar verbonden. Echter, doordat softwarepakketten verouderen en soms de originele ontwikkelaars niet meer beschikbaar zijn kan het een langslepende en dure kwestie worden om software te vervangen waar veel kennis voor nodig is. Miljoenen regels zijn dan zeker niet ongebruikelijk.

Natuurlijk sluipen er in de software van banken ook bugs. Kijk eens hoeveel security patches Windows heeft gekregen? Het is wellicht niet te vergelijken, maar je kunt zeker niet verwachten dat er in de software van banken nooit iets mis zal gaan. Je kunt ze hooguit dwingen aan protocollen te voldoen, en dat gebeurt ook echt wel.

@Fils: De tweede 'oplossing' lijkt me niet zo verstandig. Je onderdrukt simpelweg de foutmelding, maar de header zal nooit worden meegestuurd in de response.

Waarschijnlijk staat er op de allereerste regel een witregel, zo aan het script te zien.

Overigens vind je via Google op 'headers already sent' miljoenen records..

Tim: Method overloading is toch niet zo bijzonder?

Maar aan het gebrek aan reacties te zien hoef ik dit dus niet nogmaals te doen?

Het is tijd op de oplossing prijs te geven. Ik zal beginnen met de oplossing van EternalFlamez. Deze oplossing geeft het juiste resultaat, en er is geen gebruik gemaakt van recursie.

<?php
$arr = array(7);


base();


function base() {
	global $arr;
	show();
	while(getLastNotOne() != -1) {
		doItAll();
		show();
	}
}


function doItAll() {
	global $arr;
	$c = count($arr);
	//Pakt achterste waarde die groter is dan 1
	$i = getLastNotOne();
	
	//Verkleint het getal gevonden op pos $i. (4 -> 3 + 1 // 3 -> 2 + 1 // 2 -> 1 + 1)
	array_splice($arr, $i, count($arr) - $i, decreaseAtPos($arr, $i));
	
	//Als er een getal groter is dan de 1e is het een duplicaat (4 + 2 == 2 + 4)
	while(isOneLargerThanPrev() != -1) {
		$k = isOneLargerThanPrev();
		//Als het een duplicaat is dan moet er gewoon doorgegaan worden.
		array_splice($arr, $k, count($arr) - $k, decreaseAtPos($arr, $k));
	}
}


//Er wordt een subarray gemaakt van $arr die begint op $pos. 
//Als alleen de 1e geen 1 is, worden ze samengevoegd. (Zie count_ones())
//Anders wordt de 1e waarde verlaagd, en een 1 toegevoegd aan het einde van de array.
//Bijv: $arr = array(4, 2, 1); De subarray wordt array(2, 1), die wordt dan door deze functie array(1, 1, 1).
//Dit wordt dan weer teruggevoegd in de originele array, waardoor het array(4, 1, 1, 1) wordt.
function decreaseAtPos($arr, $pos) {
	$arr = array_splice($arr, $pos);
	if(count($arr) > 1) {
		$c = count_ones();
		
		if($c == count($arr) - 1) {
			array_splice($arr, 1, count($arr), array($c + 1));
		}
		else {
			array_push($arr, 1);
		}
	}
	else {
		array_push($arr, 1);
	}
	
	$arr[0]--;
	return $arr;
}


//Displayt de array.
function show() {
	global $arr;
	$count = count($arr);
	$text = '';
	for($i = 0; $i < $count; $i++) {
		if($i != 0) {
			$text .= ' + ';
		}
		
		$text .= $arr[$i];
	}
	
	echo($text. '<br/>');
}


//Returnt achterste waarde die groter is dan 1.
function getLastNotOne() {
	global $arr;
	$c = count($arr);


	for($i = $c - 1; $i >= 0; $i--) {
		if($arr[$i] != 1) {
			return $i;
		}
	}
	
	return -1;
}


//Telt aantal enen om te mergen zodra het klaar is met een reeks.
//Bijv: (4 + 2 => 4 + 1 + 1 => 3 + 3)
function count_ones() {
	global $arr;
	$r = 0;
	$c = count($arr);
	for($i = 0; $i < $c; $i++) {
		if($arr[$i] == 1) {
			$r++;
		}
	}
	
	return $r;
}


//Kijkt of de waarde al langs is geweest. Returnt -1 als het niet zo is, anders getLastNotOne().
function isOneLargerThanPrev() {
	global $arr;
	$c = count($arr);
	$smallest = $arr[1];
	
	for($i = 1; $i < $c; $i++) {
		if($arr[$i] > $smallest || $arr[$i] > $arr[0]) {
			return getLastNotOne();
		}
		else {
			$smallest = $arr[$i];
		}
	}
	return -1;
}
?>

De oplossing van victor geeft op dit moment nog niet exact het juiste resultaat. Hij is er nog mee bezig en wil dat nog oplossen. Zijn oplossing geeft wel alle oplossingen maar enkele dubbele (2 + 1 + 1 en 1 + 2 + 1 bijvoorbeeld). Zijn oplossing is (tot nu toe) als volgt:

<?php


function get_numbers($getal)
{


    $oplossingen = array();


    if($getal == 1)
    {
    
        array_push($oplossingen, '1');


    }


    for($i = 1; $i < $getal ; $i++)
    {


        foreach(get_numbers($getal-$i) as $oplossing)
        {


            array_push($oplossingen, $oplossing.'+'.$i);
        
        }
    
    }
    
    array_push($oplossingen, $getal);
    $oplossingen = array_unique($oplossingen);


    return $oplossingen;


}


echo '<pre>';


print_r(get_numbers(4));


echo '</pre>';


?>

Dan als laatste nog mijn eigen oplossing, die wel gebruik maakt van recursie:

<?php


function getPartitions($number, $previous = -1, $prefix = "") {
	if($number <= $previous || $previous == -1) {
		echo $prefix . $number . "<br />\n";
	}
	for($i = 1; $i < $number; $i++) {
		if(($number - $i) <= $previous || $previous == -1) {
			getPartitions($i, ($number - $i), $prefix . ($number - $i) . " ");
		}
	}
}


echo getPartitions(7);

Wat vinden jullie van de oplossingen? En wat vinden jullie van de uitdaging? Is dit voor herhaling vatbaar? Zou je wel meedoen als de opdracht makkelijker is (mocht je nu niet hebben meegedaan)? Ik hoor graag feedback!

@Fils: Het stukje sarcasme leek me wel duidelijk, maar goed.

Ik heb inmiddels ook een mooie oplossing van victor gekregen. Met veel gepruts heb ik op de macbook van een vriend van me de code kunnen uitvoeren :P. De hoeveelheid code is lekker compact, maar de oplossing geeft helaas wel dubbele oplossingen (ook 1 + 1 + 3 bijvoorbeeld terwijl alleen 3 + 1 + 1 erin hoort te staan). Ik zal morgen nog eens iets beter naar de oplossing kijken, want hij ziet er wel interessant uit. In ieder geval ook voor deze oplossing mijn complimenten!

Het is gelukt! Ik heb de eerste werkende oplossing ontvangen van EternalFlamez. Hoewel zijn oplossing lang niet de meest efficiënte is (in regels code), ben ik wel positief verrast. Het ziet er netjes uit, en er is duidelijk tijd aan besteed, mijn complimenten!

Dus, iedereen die niet met een valide oplossing kan komen is wat mij betreft een mindere programmeur dan EternalFlamez.

Wie gaan het nog meer proberen?

Citaat van bramieboy100

Ik niet, het is een onmogelijk opdracht omdat er ontelbaar veel getallen zijn. Zelfs met recursie lukt dit niet.

Dat is zeker niet het geval. Als ik het afdrukken van de oplossing (wat verreweg het meeste tijd kost) achterwege laat, dan is het voor het getal 50 bijvoorbeeld in ongeveer een seconde uitgerekend. En dat is dan in PHP, een taal als Java of zelfs C is nog vele malen sneller.

Daarbij gaat het ook niet zozeer om de performance. Het is eigenlijk helemaal geen moeilijke opdracht!