ilch Forum » Allgemein » Plauder Ecke » Spaghetti code

Geschlossen
  1. #1
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Halli hallo,

    bin gerade am Shop basteln, hab nen fertig-template und dieses grafisch geändert. Allerdings bin ich nun dabei, den Spaghetti code zu entfernen.

    Ich makiere einfach mal alle stellen wo ich denke das dies Spagheti code ist mit ???^^Kann natürlich sein, dass ich was übersehen habe^^

    <div align="center">
            <div id="body">
                    <div id="innerBox">
                    <!-- Logo + Navigation -->
                                    <div id="head">&nbsp;</div>
                                    <br style="???clear:left;???" />
                                    <div id="navigation">
                                            <div id="pathway">
                                                    {$navtrail}
                                            </div>
                                            <div id="account">
                                                    {if $account}{php} if (isset($_SESSION['customer_id'])) { {/php}
                                                    <a href="{$logoff}">{#link_logoff#}</a> | {php} } {/php}{/if}{if $account}<a href="{$account}">{#link_account#}</a> | {/if}<a href="{$cart}">{#link_cart#}</a> | <a href="{$checkout}">{#link_checkout#}</a>
                                            </div>
                                    </div>
                    <!-- Ende Logo + Navigation -->
                                    <br style="???clear:both???" />
                    <!-- Hauptteil -->
                                    <div id="main_body">
                                    <!-- Links -->
                                            <div id="nav_left" align="left" style="???float:left???">
                                                    {$box_CATEGORIES}
                                                    {$box_SEARCH}
                                                    {$box_SPECIALS}
                                                    {$box_CONTENT}
                                            </div>
                                    <!-- Links Ende -->
                                    <!-- Content -->
                                                    <div id="content" style="float:left; width:575px;">
                                                            {$main_content}
                                                    </div>
                                    <!-- Content Ende -->
                                    <!-- Rechts -->
                                            <div id="nav_right" align="left" style="float:right">
    
                                                    {$box_CART}
                                                    {$box_LOGIN}
                                                    {$box_ADMIN}
                                                    {$box_REVIEWS}
                                                    {$box_WHATSNEW}
                                            </div>
                                    <!-- Rechts Ende -->
    
                                            <br style="???clear:both???" /></div><!-- Ende Hauptteil -->{if $BANNER}<div id="banner">{$BANNER}</div>{/if}</div></div>


    Nun stell ich mal ne Noobfrage, wie kann ich es so machen, dass es eben kein Spaghetti-code mehr ist zwinker

    Falls mir Jemand hilft, bitte genau erklären, bin nen noob xD

    Achja, ist kein ilch, sondern xt_commerce^^

    Liebe Grüße


    Zuletzt modifiziert von Botox84 am 17.12.2010 - 19:44:35
    0 Mitglieder finden den Beitrag gut.
  2. #2
    User Pic
    oink Mitglied
    Registriert seit
    23.09.2010
    Beiträge
    740
    Beitragswertungen
    227 Beitragspunkte
    ZitatZitat geschrieben von Botox84
    Nun stell ich mal ne Noobfrage, wie kann ich es so machen, dass es eben kein Spaghetti-code mehr ist zwinker


    ZitatZitat geschrieben von Botox84
    Achja, ist kein ilch, sondern xt_commerce^^


    kein xtc verwenden reicht schon um um den spaghetticode rumzukommen
    before creation there must be destruction
    0 Mitglieder finden den Beitrag gut.
  3. #3
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    ZitatZitat geschrieben von oink

    ZitatZitat geschrieben von Botox84
    Nun stell ich mal ne Noobfrage, wie kann ich es so machen, dass es eben kein Spaghetti-code mehr ist zwinker


    ZitatZitat geschrieben von Botox84
    Achja, ist kein ilch, sondern xt_commerce^^


    kein xtc verwenden reicht schon um um den spaghetticode rumzukommen


    Keine besonders schlaue Antwort^^Mal im Ernst, ich verwende es nunmal und kanns nun auch nicht ändern^^
    0 Mitglieder finden den Beitrag gut.
  4. #4
    User Pic
    MaddinXx Mitglied
    Registriert seit
    15.02.2007
    Beiträge
    787
    Beitragswertungen
    40 Beitragspunkte
    Naja dann musst du xtc so abändern, dass es alles ohne Spaghetti ausgibt. und das ist eine menge arbeit zwinker
    0 Mitglieder finden den Beitrag gut.
  5. #5
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Das ist ja meine Frage, wie mach ich das?
    Haufen Arbeit? Kommt drauf an, was für nen Design man hat und bei mir sinds gerade mal 21 Spaghetti-Code-Vorkommen, also denke ich weniger, dass es nen haufen Arbeit ist^^Wobei die css nun mittlerweile schon valide ist, fehlt eben nur noch das html

    Also nen Spaghetti-Code ist ja z.B. nen <b></b> wenn man daraus ein <strong></strong> macht, hats den selben effekt, Text ist dick, aber es ist halt kein Spaghetti-Code mehr.
    0 Mitglieder finden den Beitrag gut.
  6. #6
    User Pic
    MaddinXx Mitglied
    Registriert seit
    15.02.2007
    Beiträge
    787
    Beitragswertungen
    40 Beitragspunkte
    Also wenn es auf allen Seiten vorkommt oder nur auf Unterseiten (also nicht auf index.php aber bspw. auf order.php dann weisst du wo suchen. Ansonsten such mal im xtc Forum welche Dateien es alles nimmt um den Content aufzubauen, dann weisst du wo suchen.
    0 Mitglieder finden den Beitrag gut.
  7. #7
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Ich habe doch oben schon den code gepostet und immer ??? gesetzt, wo spaghetti code verbaut ist^^
    0 Mitglieder finden den Beitrag gut.
  8. #8
    User Pic
    lordrepha Mitglied
    Registriert seit
    11.03.2006
    Beiträge
    1.362
    Beitragswertungen
    22 Beitragspunkte
    Also ich antworte dir mal auf deine Frage Botox.

    im Prinzip ist das was du Markiert hast Spagethie Code du kannst das ändern in dem du Klassen vergibst und eben die Style Atributte inne CSS Datei schreibst. Ich würde hier bewusst classen nehmen da du dann nicht in den ID Rumpfuschen musst und dir dann woanders vielleicht das Layout zerschiesst.

    Prinzipiell stellt sich aber die Frage nach dem Sinn deines Vorhabens. XT Commerce (und ich kenne XT Commerce) besteht nicht nur aus dem Template sondern aus vielen fast unzähligen Dateien in der eins immer ganz gewiss ist HTML und PHP und CSS stehen in 1 Datei. Ob es jetzt nun ein Modul ist was du einfügst oder ein Template es ist immer irgendwie "Spagehtie Code " zu finden.

    Normaler weise entfernt man den Code weil man sich Perfomrance verbesserungen in der Abaarbeitung der Scripte wünscht. Ich denke du hast einfach auf das Falsche Shopsystem gesetzt.

    Da ich mal nicht so sein will lasse ich dich nun mal an meinem wissen in bezug auf Shopsystem Teilhaben.

    Ich habe unzählige Shopsysteme getestet weil ich meinen Kunden ein System liefern möchte das wirklich Zukunftsorientiert ist und das funktioniert
    Das ganze nennt sich TomatoCart basiert auf XTcommerce allerdings wurde es gerade für die Entwickler an ein MVC Angebunden das Backend des Shops ist wohl einzigartig und ermöglicht das entwickeln eines eigenen Workflows.

    Einfach mal anschauen.

    Ein Komplett angeppasstes Layout liegt hier :
    webd-sign.at/tomatocart1/

    Und hier die Webseite mit dem Admin Panel und dem original Frontend:
    externer Link

    Viel spass beim ausprobieren
    0 Mitglieder finden den Beitrag gut.
  9. #9
    User Pic
    oink Mitglied
    Registriert seit
    23.09.2010
    Beiträge
    740
    Beitragswertungen
    227 Beitragspunkte
    *kopfschüttel & grins*

    ich bezweifle das Botox spaghetticode erkennt wenn er welchen sieht.
    mal "define spaghetticode" in google eingegeben junge?
    die kurzdefinition ist einfach "Spaghetticode ist ein abwertender Begriff für Software-Quellcode, der komplexe und verworrene Kontrollstrukturen aufweist.".

    alles was du markiert hast ist inline css, toll du hast 3 mal ein clear und einmal ein float angekreidet, das kostet dich 2 min das in der css nachzutragen. Ich sehe den spaghetticode eher in den zeilen #11 bis #14, eine loginbox sollte ein seperates template sein um haupt templates übersichtlicher zu halten, also anstatt des ganzen codes von #11 - #14 gehört dort ein {include='bla/bla/blubb/lonbox.tpl'} rein.

    in der loginbox sollte smarty um gottes willen kein {php} verwenden müssen.
    die ganze zeile 12 sollte schon an der korrekten stelle (der customer codeteile z.b. login.php) gesetzt werden.

    so das aus diesem rotzcode einfach nur ein {if $account && $loggedin} wird.

    so da ich dir nun mal aufgeführt habe das du an den falschen stellen suchst, suche lieber nur nach html output und inline css. das solltest du hinbekommen.

    und ob du mit einem OOP basiertem shopsystem auskommst bezweifle ich noch stärker. denn alles was du an solch einem system im code verändern würdest wäre spaghetticode.

    nichts für ungut, aber die wahrheit
    oink oink
    before creation there must be destruction
    0 Mitglieder finden den Beitrag gut.
  10. #10
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Vielen Dank ihr beiden zwinker
    habt natürlich Recht^^ Bezogen auf dem falschen Shop-cms und dem Spaghetti code natürlich auch^^Das Shop cms werde ich nicht mehr ändern, dafür hab ich schon zu viele Artikel eingetragen, Design fertig usw. Wäre mir zu viel Arbeit wieder von vorne anfangen zu müssen

    Den Spaghetti code konnte ich teileweise entfernen, eben nur die Dinge, die eigentlich inline.css sind habe ich halt auch in die css verlegt, allerdings mit id^^Der rest ist noch so^^

    Was oink beschreibt, hat sicherlich Hand und Fuß, doch ist das alles zu komplex für mich und ich sehe nicht mehr durch^^

    Vielen Dank trotzdem^^

    Liebe Grüße

    Botox84
    0 Mitglieder finden den Beitrag gut.
  11. #11
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Um nochmal zurück auf die loginbox zu kommen, wie genau setz ich das durch?

    Ich habe jetzt in der index statt zeile 11-14 folgendes ersetzt:

    <div>{$box_longbox}</div>


    und in der box_longbox.htm habe ich das:

    div id="account">
            {if $account}{php} if (isset($_SESSION['customer_id'])) { {/php}
        <a href="{$logoff}">{#link_logoff#}</a> | {php} } {/php}{/if}{if $account}<a href="{$account}">{#link_account#}</a> | {/if}<a href="{$cart}">{#link_cart#}</a> | <a href="{$checkout}">{#link_checkout#}</a>
    </div>


    Allerdings haut das nicht hin, Design verschiebt sich usw.

    Jop, ich weiss, hab mal wieder nen Lacher gebracht weil wohl oder übel alles falsch ist....Aber so hab ichs verstanden^^

    Wer kanns mir genauer erklären?

    Liebe Grüße
    0 Mitglieder finden den Beitrag gut.
  12. #12
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Halli hallo, ich nochmal mit ner dummen Frage zwinker

    Ich habe jetzt z.B. diesen Code in der Index stehen

    <li class="topmenu"><a style="Spaghetti-Markupcolor:#FE3CC9" href="http://www.lr4you.de/checkout_shipping.php?XTCsid=do5noh4acltb1tn3ipgikoonvmb746n0"title="Kasse">Cash box</a></li>
    <li class="topmenu">|</li>
    <li class="topmenu"><a style="Spaghetti-Markupcolor:#FE3CC9" href="http://www.lr4you.de/shopping_cart.php?XTCsid=do5noh4acltb1tn3ipgikoonvmb746n0"title="Warenkorb">Shopping Cart</a></li>
    <li class="topmenu">|</li>
    <li class="topmenu"><a style="Spaghetti-Markupcolor:#FE3CC9" href="http://www.lr4you.de/account.php?XTCsid=do5noh4acltb1tn3ipgikoonvmb746n0"title="Ihr Konto">Your account</a></li>
    <li class="topmenu">|</li>
    <li class="topmenu"><a style="Spaghetti-Markupcolor:#FE3CC9" href="index.php"title="Startseite">Home</a></li>


    Wo die Spaghetti Fehler sind, steht direkt mit drin.
    Nun meine Noobfrage, wie genau bzw. was genau muss ich in der Index angeben und was genau muss in der css angeben werden, damit die Fehler weg sind und es trotzdem noch genau so ausschaut?

    Bin grad leicht am verzweifeln^^

    Liebe Grüße
    0 Mitglieder finden den Beitrag gut.
  13. #13
    User Pic
    oink Mitglied
    Registriert seit
    23.09.2010
    Beiträge
    740
    Beitragswertungen
    227 Beitragspunkte
    CSS:
    li.topmenu a { Spaghetti-Markupcolor:#FE3CC9;


    dann kann das zeug aus den a's raus
    before creation there must be destruction
    0 Mitglieder finden den Beitrag gut.
  14. #14
    User Pic
    Botox84 Mitglied
    Registriert seit
    09.08.2006
    Beiträge
    1.190
    Beitragswertungen
    21 Beitragspunkte
    Das versteh ich gerade nicht so richtig, was du damit meinst. weinen
    0 Mitglieder finden den Beitrag gut.
Geschlossen

Zurück zu Plauder Ecke

Optionen: Bei einer Antwort zu diesem Thema eine eMail erhalten