Scripts samenvoegen?

jemajeroen

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

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';
  }
?>

Meer tonen

hoop dat ik je zo verder kan helpen.

jemajeroen

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>
';
  }
?>

Meer tonen

Stefan.J

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

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

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

jemajeroen

laatmaar jullie kunnen niks uitleggen zeker....

ik wil dit ook leren voor andere scripts...

SkyNet

@ Jemajeroen,

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

jemajeroen

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

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

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

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

jemajeroen

dag cp

RDMNL2010

Dag jemajeroen :cheer:

lett me guess... 12 jaar?

jemajeroen

jammer 17 ei

RDMNL2010

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

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

ik iig nog iets toevoegen als

PHP

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

ICTscripters

Dé plek voor IT

Dé plek voor IT

nieuwe game

Criminolz

[Beta] Ottomaan: gratis browserspel met een echte, levende stad. Feedback gezocht!

Port Father: beta van een browsergame in Antwerpen 1931, feedback welkom

Maffia Hotel: een browsergame waarin je van lobbygast naar spil van het hotel groeit

Crime Club volledig herschreven vanaf nul

Malavita (nieuwe maffia game, van 0)

Herkent deze Notion-structuurscan echte problemen?

[Aanbod] Stadskaart voor je browsergame: van menu naar klikbare stad, vanaf €200

[Verkoop] G4G (Gangsters 4 Gangsters): volledige broncode, nieuw gebouwd in PHP 8.3 | € 30 per domein

[TE KOOP] Malavita.be [Maffiaspel, gebaseerd op omicidio.nl, unieke code]

Scripts samenvoegen?

Registratie

PHP waarschuwt voor twee ernstige bugs die makkelijk uit te buiten lijken

Functioneel ontwerp

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

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

Gebruikers die dit topic bekijken