Scripts samenvoegen?

jemajeroen · 3 Januari 2011

hallo allemaal ik heb nu 3 scripts on te trainen

1 voor het te maken als trainen?p=misdaad
2 voor misdaad
3een voor auto stelen

hoe kan ik dit het beste maken.

dit is script 1

PHP

<?php /* ------------------------- */


  $page					= $_GET['p'];
  if($page == "misdaad")
    include("trainingmisdaad.php");
  else if($page == "auto")
    include("trainingauto.php");
   
/* ------------------------- */ ?>

ik wil daar inpplaats van de include de scripts zelf hebben maar het lukt me niet

kan iemand dit even uitleggen?

NielsB · 3 Januari 2011

je moet werken met accolades ( { } )
vb:

PHP

if(blaat == blaat) 
{
echo 'asdas';
} 
elseif(blaat != blaat) 
{
echo 'blaat is niet gelijk aan blaat';
}

de accolades kunnen er voor zorgen dat alles wat tussne die dingen staat, kan worden uitgevoerd.
Voordeel ervan is dat je meer regels kan gebruiken dan als je zonder { } gebruikt ( die kunnen namelijk maar 1 regel in een if of else gemaakt worden. )

oplossing voor jouw code:

PHP

<?php
$page                    = $_GET['p']; 
  if($page == "misdaad")
  {
    // hier komt de code van het script trainings misdaad
    echo 'doe iets'; 
  }  
  else if($page == "auto") 
  {
    // hier komt de code van het script trainings auto
    echo 'doe iets';
  }
?>

Toon Meer

hoop dat ik je zo verder kan helpen.

jemajeroen · 3 Januari 2011

dus zo?
(heb alleen nog auto gedaan)

PHP

<?php
$page                    = $_GET['p']; 
  if($page == "misdaad")
  {
    // hier komt de code van het script trainings misdaad
    echo 'doe iets'; 
  }  
  else if($page == "auto") 
  {
    $boksen1           = mysql_query("SELECT *,UNIX_TIMESTAMP(`opdruktijd`) AS `opdruktijd`,0 FROM `[users]` WHERE `login`='$data->login'");
$boksen            = mysql_fetch_object($boksen1);
	
if($boksen->opdruktijd + $boksen->opdruktijd1 > time()){
	print "Je bent nog bezig met een Training!.";
	print "</tr></td></table>";
}
else{
	if(isset($_POST['B'])) {
		$T         = $_POST['T'];
		if($T == 1){
			print "Je gaat nu trainen, de training duurt 5 minuten</td></tr></table>";
			mysql_query("UPDATE `[users]` SET `training1`=`training1`+'2' WHERE `login`='$data->login'");
			mysql_query("UPDATE `[users]` SET `opdruktijd1`='300', `opdruktijd`=NOW() WHERE `login`='$data->login'");
		}
		elseif($T == 2){
			print "Je gaat nu trainen, de training duurt 10 minuten</td></tr></table>";
			mysql_query("UPDATE `[users]` SET `training1`=`training1`+'3' WHERE `login`='$data->login'");
			mysql_query("UPDATE `[users]` SET `opdruktijd1`='600', `opdruktijd`=NOW() WHERE `login`='$data->login'");
		}
		elseif($T == 3){
			print "Je gaat nu trainen, de training duurt 15 minuten</td></tr></table>";
			mysql_query("UPDATE `[users]` SET `training1`=`training1`+'4' WHERE `login`='$data->login'");
			mysql_query("UPDATE `[users]` SET `opdruktijd1`='900', `opdruktijd`=NOW() WHERE `login`='$data->login'");
		}
	}
    echo '		<form method="POST">
  <table border=0 cellspacing=0 cellpadding=2 width=90% align=center>
  <tr>
  <td class=subTitle align=center>Activiteit</td>
  <td class=subTitle align=center>Tijd</td>
  </tr>
  <tr>
  <td width=50% class=mainTxt><input type="radio" value="1" name="T" class="normalRadio"> Slot breken.</td>
  <td class=mainTxt>5 Min.</td>
  </tr>
  <tr>
  <td width=50% class=mainTxt><input type="radio" value="2" name="T" class="normalRadio"> Auto starten.</td>
  <td class=mainTxt>10 Min.</td>
  </tr>
  <tr>
  <td width=50% class=mainTxt><input type="radio" value="3" name="T" class="normalRadio"> Alarm uitschakelen.</td>
  <td class=mainTxt>15 Min.</td>
  </tr>
  <tr>
  <td class=mainTxt colspan=2 align=center><input type="submit" value=" Train " name="B"></td>
  </tr>
  </table>
  </form>
';
  }
?>

Toon Meer

Stefan.J · 3 Januari 2011

Waarom wil je dit? Het is toch echt niet overzichtelijker om één bestand te hebben van 300 regels dan drie van 100 regels?

Of zie ik hier nu iets over het hoofd?

jemajeroen · 3 Januari 2011

dit wil ik zodat ik meer overzicht heb in de scripts zo heb ik dalijk 15 train scripts terwel het ook in 1 kan

RDMNL2010 · 3 Januari 2011

kan je het beter onderverdelen in mappen..15 scripts in 1 script is erg onoverzichtelijk.

jemajeroen · 3 Januari 2011

laatmaar jullie kunnen niks uitleggen zeker....

ik wil dit ook leren voor andere scripts...

SkyNet · 3 Januari 2011

@ Jemajeroen,

Als je zo blijft reageren helpt niemand je straks meer hoor.

jemajeroen · 3 Januari 2011

nee maar luister het probleem zit hier dat mensen eerst zo veel opmerkingen maken zodat je geen hulp meer nodig hebt maar ik probeer te leren door het zelf te doen en uitleg te vragen tot veel anderen hier.

FrankY · 3 Januari 2011

Je kan misschien wel doen dat je maar 1 training per x kan doen. Dat scheelt dan ook weer 2 regels. Voor de rest staat het nu goed. En ik snap je point wel, waarom zoveel als je het in 1 kan doen plus je kan het altijd nog overzichtelijk maken.

RDMNL2010 · 3 Januari 2011

inderdaad jemajeroen.. Zoek het dan maar lekker zelf uit;)

Als jij lekker eigenwijs wilt doen moet je dat vooral doen. Als wij zeggen dat het niet overzichtelijk is dan wil meneer gewoon dat doen omdat hij dat wilt.. Je vraagt hier toch om hulp of zie ik dat verkeerd? En als je die hulp krijgt d.m.v. tips dan kan je hier wat mee doen maar niet zo achterlijk gaan reageren. Op die manier zegt iedereen straks: Dag jeroen!

SkyNet · 3 Januari 2011

De uitleg luidt : 'Onderverdeel je script in meerdere scripts'
Je komt er later weg achter hoe veel dat scheelt qua zoekwerk.

jemajeroen · 3 Januari 2011

dag cp

RDMNL2010 · 3 Januari 2011

Dag jemajeroen :cheer:

lett me guess... 12 jaar?

jemajeroen · 3 Januari 2011

jammer 17 ei

RDMNL2010 · 3 Januari 2011

ei? Ga eerst het forum eens doorlezen en probeer dan in te zien dat mensen je tips geven i.p.v. als een pubertje van 17 te reageren!

Stefan.J · 3 Januari 2011

Lijkt erop dat mijn reactie niet helemaal goed ontvangen is...

Iets proberen te leren is goed, maar een goede programmeur wordt je nooit als je je niet afvraagt waarom je eigenlijk iets wilt. Ik kan je best uitleggen hoe je met een if-structuurtje alles bij elkaar in kunt plakken, dat is zo spannend niet. Veel interessanter is het om een goede architectuur in je code je hebben. Het idee achter deze wijzigingen van jou is een verbeterde architectuur, dan mag ik toch aangeven dat ik denk dat het geen goed idee is?

knudde1 · 3 Januari 2011

ik iig nog iets toevoegen als

PHP

if(!isset($_GET['p'])){
  $page                    = "misdaad";
}else{
  $page                    = $_GET['p'];
}

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

Scripts samenvoegen?

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

Gebruikers die dit topic bekijken