Wie vermeide ich Scriptlets in meiner JSP-Seite?

Mir wurde gesagt, dass die Verwendung von scriptlets (<%= ... %>) in meiner JSP-Seiten ist nicht so eine tolle Idee.

Kann jemand mit ein bisschen mehr java - /jsp-Erfahrung, bitte geben Sie mir einige Hinweise, wie Sie zu ändern Sie diesen code, damit Ihr mehr 'best practice', was immer das sein mag?

Diese JSP ist eigentlich mein sitemesh main decorator Seite. Im Grunde ist mein web design hat eine Registerkarte, Streifen und ein Untermenü, und ich möchte irgendwie markieren Sie die aktuelle Registerkarte, und zeigen Sie das richtige Untermenü, indem man die aktuellen request-URI.

<%@ taglib uri="http://www.opensymphony.com/sitemesh/decorator" prefix="decorator" %>

<html>
<head>
  <title>My Events - <decorator:title /></title>
  <link href="<%= request.getContextPath() %>/assets/styles.css" rel="stylesheet" type="text/css" />
</head>
<body>

<div class="tabs">
  <a 
    <%= request.getRequestURI().contains("/events/") ? "class='selected'" : "" %>
    href='<%= request.getContextPath() %>/events/Listing.action'>Events</a>
  <a 
    <%= request.getRequestURI().contains("/people/") ? "class='selected'" : "" %>
    href='<%= request.getContextPath() %>/people/Listing.action'>People</a>
</div>

<div class="submenu">
  <% if(request.getRequestURI().contains("/events/")) { %>
    <a href="Listing.action">List of Events</a>
    |<a href="New.action">New Event</a>
  <% } %>
  <% if(request.getRequestURI().contains("/people/")) { %>
    <a href="Listing.action">List of People</a>
    |<a href="New.action">New Person</a>
  <% } %>  
  &nbsp;
</div>

<div class="body">
  <decorator:body />
</div>

</body>
</html>

Danke an alle

Kommentar zu dem Problem

Als ein beiseite, ist '<%= request.getContextPath() %> " eine akzeptable Verwendung von scriptlets, das ist nicht verpönt, so viel? Kommentarautor: Chris

Beginnen sollte man mit facelets für das templating. Zwingt Sie den code korrekt. Kommentarautor: Andrew Dyster

Sagen Sie, verwenden facelets statt sitemesh? Kommentarautor: Chris

mögliche Duplikate von stackoverflow.com/questions/3177733/... Kommentarautor: Bozho

InformationsquelleAutor der Frage Chris | 2010-02-02

Ich denke, es hilft mehr, wenn Sie es mit eigenen Augen sehen, dass es überhaupt gemacht werden kann, ganz ohne scriptlets.

Hier ist ein 1 auf 1 umschreiben, mit Hilfe von unter anderem JSTL (nur drop jstl-1.2.jar in /WEB-INF/lib) Kern und Funktionen taglib:

<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c" %>
<%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn" %>

<html>
<head>
  <title>My Events - <decorator:title /></title>
  <link href="${pageContext.request.contextPath}/assets/styles.css" rel="stylesheet" type="text/css" />
</head>
<body>

<div class="tabs">
  <a 
    ${fn:contains(pageContext.request.requestURI, '/events/') ? 'class="selected"' : ''}
    href="${pageContext.request.contextPath}/events/Listing.action">Events</a>
  <a 
    ${fn:contains(pageContext.request.requestURI, '/people/') ? 'class="selected"' : ''}
    href="${pageContext.request.contextPath}/people/Listing.action">People</a>
</div>

<div class="submenu">
  <c:if test="${fn:contains(pageContext.request.requestURI, '/events/')}">
    <a href="Listing.action">List of Events</a>
    |<a href="New.action">New Event</a>
  </c:if>
  <c:if test="${fn:contains(pageContext.request.requestURI, '/people/')}">
    <a href="Listing.action">List of People</a>
    |<a href="New.action">New Person</a>
  </c:if>
  &nbsp;
</div>

Hier eine etwas optimierte Fassung, beachten Sie, dass ich verwendet c:set "cache" - Ausdruck der Ergebnisse für die Wiederverwendung und das ich HTML benutzen <base> - tag, um zu vermeiden, dass Sie den context path (in jedem link (nur stellen Sie alle relativen URL ' s Ihrer Webseite relativ, um es-ohne den führenden slash!):

<%@ taglib uri="http://java.sun.com/jsp/jstl/core" prefix="c" %>
<%@ taglib uri="http://java.sun.com/jsp/jstl/functions" prefix="fn" %>

<c:set var="isEvents" value="${fn:contains(pageContext.request.requestURI, '/events/')}" />
<c:set var="isPeople" value="${fn:contains(pageContext.request.requestURI, '/people/')}" />

<html>
<head>
  <title>My Events - <decorator:title /></title>
  <base href="${pageContext.request.contextPath}">
  <link href="assets/styles.css" rel="stylesheet" type="text/css" />
</head>
<body>

<div class="tabs">
  <a ${isEvents ? 'class="selected"' : ''} href="events/Listing.action">Events</a>
  <a ${isPeople ? 'class="selected"' : ''} href="people/Listing.action">People</a>
</div>

<div class="submenu">
  <c:if test="${isEvents}">
    <a href="Listing.action">List of Events</a>|<a href="New.action">New Event</a>
  </c:if>
  <c:if test="${isPeople}">
    <a href="Listing.action">List of People</a>|<a href="New.action">New Person</a>
  </c:if>
  &nbsp;
</div>

Kann es tatsächlich optimiert werden mehr, wenn Sie sammeln diese "hardcoded" Werte wie events und people - und link-Texte in eine Map im Anwendungsbereich und verwenden Sie unter jeder der JSTL <c:forEach> zur Anzeige der Registerkarten.

Ihre tatsächlichen Frage, kann man deaktivieren scriptlets (und runtime-Fehlern, ihn zu benutzen), indem Sie den folgenden Eintrag in der webapp ist web.xml. Es kann helfen, zu erkennen, überwacht die scriptlets.

<jsp-config>
    <jsp-property-group>
        <url-pattern>*.jsp</url-pattern>
        <scripting-invalid>true</scripting-invalid>
    </jsp-property-group>
</jsp-config>

Erfahren Sie mehr über EL, überprüfen Sie die Java EE tutorial-Teil II-Kapitel 5. Implizite EL-Objekte, wie ${pageContext} beschrieben hier. Erfahren Sie mehr über JSTL, überprüfen Sie die Java EE tutorial-Teil II-Kapitel 7. Beachten Sie, dass JSTL und EL sind zwei getrennte Dinge. JSTL ist eine standard-taglib und EL ermöglicht lediglich einen Zugriff auf backend-Daten programmgesteuert. Obwohl es ist normalerweise verwendet in taglibs wie die JSTL, es können auch standalone genutzt werden im Vorlagen-text.

InformationsquelleAutor der Antwort BalusC

9

Nebenbei, ist <%= request.getContextPath() %> eine akzeptable Verwendung von scriptlets, das ist nicht verpönt, so viel?

Dies möglicherweise eine unpopuläre Meinung, aber wenn alles, was Sie tun, sind einfach Bedingungen und text Einfüge, kann ich nicht finden, viel Fehler in der Verwendung von scriptlets. (Hinweis: die wenn)

Ich würde wahrscheinlich die Verwendung von JSTL und der expression-Sprache, sondern vor allem, weil es sein kann, weniger Tippen, und IDE-Unterstützung könnte besser sein (aber eine gute JSP-IDE können auch finden, fehlende schließende Klammern und sowas).

Aber grundsätzlich (wie in "keep Logik der templates") ich kann nicht erkennen, keinen Unterschied zwischen
```
<% if(request.getRequestURI().contains("/events/")) { %>
```
und
```
${fn:contains(pageContext.request.requestURI, '/events/') 
```
InformationsquelleAutor der Antwort Thilo
6

Dies ist nicht eine direkte Antwort auf deine Frage (und es gibt bereits mehrere gute, so werde ich nicht versuchen, um es hinzuzufügen), aber Sie haben Sie erwähnt:

Kann jemand mit ein bisschen mehr java/jsp
Erfahrung bitte geben Sie mir einige
Zeiger, wie dies zu ändern code
so mehr 'best practice', was auch immer
das kann?

Meiner Meinung nach die beste Praxis in Bezug auf JSP ist, dass es verwendet werden soll, wird ausschließlich als eine Template-engine, und nicht mehr (d.h., keine business-Logik). Mit Hilfe von JSTL, wie viele darauf hingewiesen, auf jeden Fall hilft Sie dorthin zu gelangen, aber auch mit JSTL, es ist einfach zu viel in einer JSP.

Ich persönlich mag die Regeln festgelegt, die in Durchsetzung der Strikten Model-View-Trennung Templating Engines durch die Terence Parr bei der Entwicklung in JSP. Das Papier nennt den Zweck von Template-engines (Trennung von Modell und view), und die Merkmale einer guten Template-engine. Es braucht einen guten Blick auf JSP und weist darauf hin weisen, es ist nicht eine gute Template-engine. Nicht überraschend, JSP ist im Grunde viel zu mächtig und erlaubt es dem Entwickler, zu viel zu tun. Ich empfehle diesen Aufsatz zu Lesen, und es wird Ihnen helfen, beschränken Sie sich auf die "guten" Teile von JSP.

Wenn Sie Lesen nur einen Abschnitt in diesem Papier, Lesen Sie in Kapitel 7, welche die folgenden Regeln:
1. die Ansicht nicht ändern können das Modell entweder durch direkte änderung Modell
  Daten-Objekte oder durch aufrufen von Methoden auf
  das Modell, das Nebenwirkungen hervorrufen.
  Das heißt, dass ein template auf Daten zugreifen können
  aus dem Modell und Methoden aufgerufen werden, aber
  solche Verweise werden muss Nebeneffekt
  frei. Diese Regel ergibt sich teilweise
  da die Daten Referenzen müssen
  um-und Kleinschreibung. Siehe Abschnitt 7.1.
2. die Sicht nicht durchführen Berechnungen auf abhängige Daten
  Werte, weil die Berechnungen können
  ändern in der Zukunft, und Sie sollte
  sauber gekapselt in das Modell in
  jeden Fall. Zum Beispiel die Sicht nicht
  compute book-Verkauf Preise
  "$Preis*.90". Unabhängig zu sein von der
  Modell, die Sicht nicht machen
  Annahmen über die Bedeutung von Daten.
3. die Sicht nicht vergleichen abhängige Daten Werte, aber testen kann die
  Eigenschaften von Daten wie
  Anwesenheit/Abwesenheit bzw. die Länge des
  multi-valued data-Wert. Tests wie
  $Blutdruck<120 bewegt werden muss, um
  das Modell, wie ärzte gerne behalten
  die Reduzierung der max systolischen Druck
  auf uns. Ausdrücke in der Ansicht sein muss
  ersetzt durch einen test für das Vorhandensein eines
  Wert Simulation eines booleschen wie
  $bloodPressureOk!=null-Template Ausgabe
  kann bedingt auf Modell-Daten und
  Berechnungen, die nur bedingt
  berechnet werden in dem Modell. Auch
  einfache tests, die negative Werte
  rot sollte berechnet werden, in das Modell;
  die richtige Ebene der Abstraktion ist usu-
  ally etwas höheren Ebene, wie z.B.
  "Abteilung x ist, Geld zu verlieren."
4. die Sicht nicht machen Datentyp Annahmen. Einige Typ-Annahmen
  offensichtlich, wenn der Blick geht von einer Daten
  Wert ist ein Datum, zum Beispiel, aber mehr
  subtile Art Annahmen erscheinen: Wenn ein
  Vorlage übernimmt, $userID ist eine
  ganze Zahl, die der Programmierer nicht
  ändern Sie diesen Wert, um eine nicht-numerische
  im Modell ohne die
  - Vorlage. Diese Regel verbietet array
  Indizierung wie colorCode[$topic] und
  $name[$ID] wird Die Ansicht ist noch nicht
  aufrufen von Methoden mit Argumenten werden - Ursache
  (statisch oder dynamisch) es ist
  bei einer angenommenen argument geben, es sei denn, man
  garantieren konnte das Modell Methode
  nur behandelt Sie als Objekte.
  Neben Grafik-Designer sind nicht
  Programmierer; wir erwarten, dass Sie aufrufen
  Methoden und wissen, was passieren wird
  unrealistisch.
5. Daten aus dem Modell nicht enthalten display-oder layout-Informationen.
  Das Modell kann nicht passieren keine Anzeige
  Informationen zu den anzeigen, getarnt als
  Daten-Werte. Dies schließt nicht die Weitergabe
  der name einer Vorlage, um diese anzuwenden zu
  andere Daten Werte.
Übrigens, Terence hat seine eigene Template-engine namens String Template, die angeblich macht einen wirklich guten job, die Durchsetzung dieser Regeln. Ich habe keine persönliche Erfahrung mit, aber würde lieben, überprüfen Sie es heraus auf mein Nächstes Projekt.

InformationsquelleAutor der Antwort Jack Leow
6

Scriptlets sind nicht das Schlimmste auf der Welt. Ein wichtiger Aspekt dabei ist zu denken, wer sich um die Aufrechterhaltung der code. Wenn Ihre web-Designern, die nicht viel Java-Erfahrung, sind Sie wahrscheinlich besser dran, wenn man mit tag-Bibliotheken. Jedoch, wenn Sie Java-Entwickler sind dabei die Wartung, kann es einfacher sein, auf Sie zu gehen, um mit scriptlets.

Wenn Sie am Ende mit einer tag-Bibliothek und JSTL, Sie erwarten, dass die Betreuer auch zu lernen, die die tag-Bibliothek und wissen, JSTL. Einige Entwickler werden mit diesem feinen, wie es ist eine Fertigkeit, die Sie haben wollen oder bereits haben, aber für einige Entwickler, die nur zu tun haben mit JSPs alle paar Monate oder so, es kann sein, viel weniger schmerzhaft, mit zu arbeiten deutlich geschrieben scriptlets geschrieben in Nizza, der bekannten Java.

InformationsquelleAutor der Antwort Spike Williams
3

Möchten Sie vielleicht beginnen, indem Sie mithilfe von tag-Bibliotheken. Sie können die standard-tag-Bibliothek JSTL zu tun, die meisten der gemeinsamen Dinge, die Sie brauchen, scriplets. Es gibt viele andere die reicher tag-Bibliotheken, die verwendet werden, wie in der struts2-framework oder apache.

z.B.
```
  <c:if test="${your condition}">
       Your Content
  </c:if>
```
ersetzen würde deine if-Anweisungen.

InformationsquelleAutor der Antwort Vincent Ramdhanie
3

Die bevorzugte alternative zu scriptlets ist die JSTL expression language; hier's eine gute übersicht. Müssen Sie die taglib etwa so:
```
<%@ taglib uri='http://java.sun.com/jsp/jstl/core' prefix='c' %>
```
Als ein Beispiel, JSTL bietet eine Reihe von impliziten Objekte, geben Sie die Sachen, die Sie brauchen; die, die Sie wollen, ist pageContext.request.

So können Sie ersetzen <%request.getRequestURI%> mit ${pageContext.request.requestURI}.

Können Sie tun, Konditionalsätze mit <c:if> tags.

InformationsquelleAutor der Antwort Jacob Mattison
2

Müssen Sie einige web-framework. Oder zumindest einige praktische taglib. Oder eine Template-enginge wie FreeMarker.

Ad-frameworks:

Wenn Sie wie JSP Möglichkeit, die Codierung, dann würde ich vorschlagen, dass Struts 2.
```
<s:if test="%{false}">
    <div>Will Not Be Executed</div>
</s:if>
<s:elseif test="%{true}">
    <div>Will Be Executed</div>
</s:elseif>
<s:else>
    <div>Will Not Be Executed</div>
</s:else>
```
Dann gibt es Komponenten-orientierte JSF.

Wenn Sie wie OOP und Programmieren in Java, versuchen Apache Wicket (mein Favorit) oder Google Web Toolkit.

InformationsquelleAutor der Antwort Ondra Žižka

Schreibe einen Kommentar

Du musst angemeldet sein, um einen Kommentar abzugeben.