Vinnaren i pepparkakshustävlingen!
2018-10-07, 12:22
  #1
Medlem
Just nu har jag en SendOrder.php som innehåller nedanstående. Borde detta separas i olika filer eftersom det är olika "delar" men allt tillhör ändå själva ordern.

SendOrder.php
Kod:
//skapa en order från ett POST form (ca 10-20 rader)

//Ladda upp filer till FTP (10 rader)

//Skicka bekräftelsemail till utvalda personer (ca 30 rader, en del html formatering)

//Skriva till en textfil på servern med lite enkel statistik (ca 15 rader) 

Ingen databas används.

Tycker ni det fungerar att ha allt detta i en fil SendOrder.php eller borde det delas upp? Som ni ser är det rätt små delar som till viss del hör ihop. Allt ska körs när personen klickar på "skicka order" på webbsidan.

Antalet rader är en väldigt grov uppskattning och kan säkert i vissa fall vara några rader mindre och i vissa 10 rader fler.
__________________
Senast redigerad av _x3m_ 2018-10-07 kl. 12:25.
Citera
2018-10-07, 14:20
  #2
Moderator
Protons avatar
Citat:
Ursprungligen postat av _x3m_
Just nu har jag en SendOrder.php som innehåller nedanstående. Borde detta separas i olika filer eftersom det är olika "delar" men allt tillhör ändå själva ordern.

SendOrder.php
Kod:
//skapa en order från ett POST form (ca 10-20 rader)

//Ladda upp filer till FTP (10 rader)

//Skicka bekräftelsemail till utvalda personer (ca 30 rader, en del html formatering)

//Skriva till en textfil på servern med lite enkel statistik (ca 15 rader) 

Ingen databas används.

Tycker ni det fungerar att ha allt detta i en fil SendOrder.php eller borde det delas upp? Som ni ser är det rätt små delar som till viss del hör ihop. Allt ska körs när personen klickar på "skicka order" på webbsidan.

Antalet rader är en väldigt grov uppskattning och kan säkert i vissa fall vara några rader mindre och i vissa 10 rader fler.
Gör man det OO-style är ju svaret givet att din Order får representeras av en klass som får hamna i sin egen fil, din FTP-klient får vara en klass som bor i sin egen fil, din FileWriter (eller vad du nu vill kalla den) får vara en egen klass i sin egen fil etc.

Gör man det inte OO kanske man ändå kan fundera på att lägga FTP-klienten i sin egen fil, ifall man nu skulle vilja göra det från mer än ett ställe framöver. Det kan ju kanske även vara bra för läsbarheten av koden så varje fil inte innehåller såna enorma mängder kod att läsa igenom den dan det strular?
Citera
2018-11-02, 10:26
  #3
Medlem
Jimmits avatar
Som föregående talare skriver håller jag med om att de borde representeras av egna klasser.

Hade dessutom separerat ut de 3 senare delarna och placerat de i någon form av kösystem, så användaren inte behöver vänta på att detta ska bli klart innan sidan har laddats klart. Kan ju ta lite tid att ladda upp filer, kan även ta tid att skicka e-post, onödigt störmoment.
Citera

Stöd Flashback

Flashback finansieras genom donationer från våra medlemmar och besökare. Det är med hjälp av dig vi kan fortsätta erbjuda en fri samhällsdebatt. Tack för ditt stöd!

Stöd Flashback