Funktion ausführen

[Bernd]

Guten Tag,

ich möchte gerne eine Funktion zweimal aufrufen bzw. wenn man das Dropdown Menü klickt sollte etwas passieren:

var updateSelect = function () {
        	$("#location").load("inc/auslesen.php", { location: $('#ort').val()});
    	};

    	<?php 
    	if(isset($_GET['edit'])) { ?>
			
			$("#ort").val(updateSelect);
		
		<?php } else { ?>
			
			$("#ort").change(updateSelect);
		
		<?php  } ?>

Soweit funktioniert auch alles, bin ich nicht im EDIT-Modus kann ich die Location wechseln, die Adresse wird richtig aufgerufen.

Rufe ich die Seite im EDIT-Modus auf wird die Adresse ordnungsgemäß ausgelesen. Nur wenn ich jetzt im Dropdown die Adresse wechsel, dann passiert nicht mehr. Eigentlich auch klar, wenn ich habe kein change mehr. Kann ich im EDIT-Bereich irgendwie sagen, zeig mir die Adresse und wenn ich eine andere wähle, dann nimm das change?

[Bernd]

Nachdem ich den Beitrag online hatte habe ich etwas versucht und es klappt

var updateSelect = function () {
        	$("#location").load("inc/auslesen.php", { location: $('#ort').val()});
    	};

    	<?php 
    	if(isset($_GET['edit'])) { ?>
			
			$("#ort").val(updateSelect);
      $("#ort").change(updateSelect);
		
		<?php } else { ?>
			
			$("#ort").change(updateSelect);
		
		<?php  } ?>

Ich weiß nur nicht ob man es besser man kann / soll?

[Gunnar Bittersmann]

@@Bernd

Nachdem ich den Beitrag online hatte habe ich etwas versucht und es klappt

var updateSelect = function () {
        	$("#location").load("inc/auslesen.php", { location: $('#ort').val()});
    	};

    	<?php 
    	if(isset($_GET['edit'])) { ?>
			
			$("#ort").val(updateSelect);
      $("#ort").change(updateSelect);
		
		<?php } else { ?>
			
			$("#ort").change(updateSelect);
		
		<?php  } ?>

Ich weiß nur nicht ob man es besser man kann / soll?

Wenn identischer Code im THEN- und im ELSE-Zweig auftaucht, kann man es besser machen:

var updateSelect = function () {
        	$("#location").load("inc/auslesen.php", { location: $('#ort').val()});
    	};

    <?php if(isset($_GET['edit'])): ?>
			
			$("#ort").val(updateSelect);
		
		<?php endif; ?>
			
			$("#ort").change(updateSelect);

LLAP 🖖

[Gunnar Bittersmann]

@@Bernd

BTW: Die geschweiften Klammern sind suboptimal. S.a. dort verlinkte Diskussion.

Dein Quelltext sähe mit der alternativen Syntax für Kontrollstrukturen besser so aus:

var updateSelect = function () {
        	$("#location").load("inc/auslesen.php", { location: $('#ort').val()});
    	};

    	<?php if(isset($_GET['edit'])): ?>
			
			$("#ort").val(updateSelect);
		
		<?php else: ?>
			
			$("#ort").change(updateSelect);
		
		<?php endif; ?>

LLAP 🖖

[Bernd]

Hallo,

ich persönlich finde es schwer zu lesen wenn nur

<?php else: ?>

steht. Hat es Vor- bzw. Nachteile wenn ich deine bzw. meine Lösung nehme?

[Gunnar Bittersmann]

@@Bernd

Hat es Vor- bzw. Nachteile wenn ich deine bzw. meine Lösung nehme?

Meine Diskussion mit Dem Martin hast du schon gelesen?

LLAP 🖖

[Bernd]

Ja, wenn du dieses meinst: https://forum.selfhtml.org/self/2014/jan/18/code-etwas-schoener-schreiben/1599975#m1599975

Es geht nur Geschmacksache. Von daher bleibe ich bei {} wie sie mir von Sublime Text und Visual Studio anbieten.

[Gunnar Bittersmann]

@@Bernd

Ja, wenn du dieses meinst: https://forum.selfhtml.org/self/2014/jan/18/code-etwas-schoener-schreiben/1599975#m1599975

Und jenes: https://forum.selfhtml.org/self/2014/jan/18/code-etwas-schoener-schreiben/1599979#m1599979

Es geht nur Geschmacksache.

Lesbarkeit von Code ist nicht unbedingt Geschmacksache.

Aus Sicht des Parsers ist natürlich eins wie das andere.

LLAP 🖖