De PHPhulp kroeg
Waardeloze scripts
Laatste tijd zijn er steeds meer scripts die echt totaal niets toevoegen aan de library. Zeker, gelukkig is de meerderheid goed, maar zoals vandaag waren er weer 2 die niets toevoegden. Nieuwelingen die dachten dat het zo werkte, wannabe's die aandacht nodig hadden.
Vraagje; hoe gaat dat met de volgende versie worden? Mag weer iedereen een script toevoegen zonder dat deze is goedgekeurd of op z'n minst is nagekeken?
Denk dat het namelijk een noodzaak is dat er een beveiliging in komt dat je minstens 100 posts nodig hebt om een script in de library toe te voegen zonder dat deze eerst moet worden goedgekeurd. En dat is niet om lullig te zijn want er zullen zeer zeker ook mensen zijn die zich 2 dagen geleden hebben aangemeld maar wellicht wel een ZEND 5 diploma hebben. Ik bedoel niet dat zij geen script mogen toevoegen (mensen met minder dan 100 posts) maar dat hun scripts/tutorials/boeken eerst moet worden goedgekeurd.
Geen idee hoe dat in de volgende versie wordt geregeld, dat wordt ons helaas niet verteld. Maar het is toch een feature wat ik en ik denk andere mensen ook, graag zouden zien..
Groetjes,
Chris H.
Nog een mogelijkheid: wel meteen weergeven maar als er meer dan bijvoorbeeld 10 meldingen zijn die aangeven dat het script nutteloos is komt het in een tijdelijke map waar het aan de admin is om te beoordelen of het script verwijderd moet worden. Onjuiste meldingen kunnen natuurlijk ook gestraft worden. Voordeel van dit systeem is in mijn oog dat het minder werk is voor de Admins en je geeft daarnaast iedereen de mogelijkheid om mee te bepalen.
Dat van Scrptr klinkt ook wel goed, van 10 meldingen. Daar ben ik ook wel voor.
Persoonlijk denk ik dat er ook meer een soort van ratings systeem zou moeten zijn. Als een script dan niet vrij snel wordt weggehaalt, valt het wel op dat het neit goed is.
Ik begin een beetje last te krijgen van deja vu's.
Voor de rest,
Ben ik het eigenlijk wel stiekem een heel klein beetje heel erg eens met scrptr
Ik meende te hebben gelezen dat in de toekomstige versie van PHPhulp scripts eerst gecontroleerd moeten worden - stel hier niet te veel strengs van voor, het is vooral bedoelt om de niet nuttige toevoegingen en dingen die eigenlijk in het forum horen ertussen uit te halen. Afkraken van het script als het idee wel origineel en nuttig (in de zin van herbruikbaar of leerzaam neem ik aan) is is aan ons, diegenen die mogen reageren voorbehouden >:)
Momenteel worden de slechtere scripts er ook wel tussenuit gehaald, maar het probleem is dat Bas & Arend de enige "active" admins zijn die dat kunnen doen. En zo actief zijn Bas & Arend nu ook weer niet. Daarom zal het een paar dagen duren voordat ze verdwijnen. Gewoon negeren is een goeie optie, melden is op zich voldoende want de mods & admins zien zelf ook wel in wat er mis is met een script. En volgens mij PM'en ze de auteur ook even netjes met een reden, althans dat was het idee..
edit: Arend is er natuurlijk ook nog :P
Gewijzigd op 01/01/1970 01:00:00 door Jelmer -
Ontopic; kan er niet zo mooi stem systeem in komen, en als een script dan -10 heeft dat die niet meer zichtbaar is zeg maar?
Op PHPFreakz hebben ze een beoordeling bij de scripts, maar volgens mij verhelpt dat niet de stomme scripts van geplaatst worden. Het geeft je enkel de mogelijkheid om te sorteren op geweldigheid. Ook vind ik de beoordelingen zelf een beetje twijfelachtig. Stel dat iemand een goed script plaatst, maar dat script is gespecialiseerd in MySQL en gebruikt enkel wat strings om foutmeldingen weer te geven, maar maakt gebruik van backsticks en variabelen binnen quotes. Wordt het dan afgekeurd omdat menigeen keihard is aangeleerd dat backsticks vies zijn, en variabelen hoe dan ook buiten quotes horen (en dat die() of exit() nooit of te nimmer nodig zijn, eval evil is, en het uit den boze is om ob_* te gebruiken, ongeacht waarvoor) Dat zie je nu ook al zo nu en dan. Mensen begrijpen de helft van het script niet, maar merken wel dat er variabelen binnen quotes staan, en beoordelen het op basis daarvan als slecht.
'Chris:
Hallo,
.... Nieuwelingen die dachten dat het zo werkte, wannabe's die aandacht nodig hadden.
Groetjes,
Chris H.
.... Nieuwelingen die dachten dat het zo werkte, wannabe's die aandacht nodig hadden.
Groetjes,
Chris H.
Wat je zegt is wel waar , alleen een puntje over dat aandacht ,
Waarom zouden ze aandacht willen hebben ?
Als hun trots zijn over hun script die natuurlijk in een newbie z,n ogen hardstikke goed zijn , Waarom zou hij dan aandacht trekken ?.
En verder vind ik dat de mods/admins gwn even een klein beetje bij moeten scripten @ phphulp , dat ze voortaan maar eerst zelf de scripts moeten goedkeuren ipv dat ze geljk gepubliceert worden .
Gewijzigd op 01/01/1970 01:00:00 door Kevichill
Maar hij heeft het over aandachtstrekkers, en een aandachtstrekker is een aandachtstrekker, niet iets anders.
Een banaan is een banaan, niet een appel.
Get it?
Bijscripten van die dingen is geen handige oplossing, wat we wel kunnen doen (en de poster van het script heeft daar ook wat aan) is aangeven wat er nog verbeterd kan worden aan het script bijvoorbeeld, en het script na aanpassing op actief zetten.
Er zijn altijd mensen die een "handiger" oplossing kennen en die denken dat een script niet goed is omdat het ook gemakkelijker/ korter kan. Maar een keuringssysteem is zeker een idee, ik geloof dat dat ook wel de bedoeling is.
- Veiligheid (mysql_real_escape_string, checken op nummeriek bij id's, etc...)
- Foutafhandeling (geen or die etc..)
- Variabelen buiten quotes.
- Genoeg commentaar.
Dit zijn een aantal punten waar elk script denk ik aan moet voldoen, of dit nu een pro PHP$ script is met OOP of een stukje PHP code wat door een beginner gemaakt is. Misschien is het handig om eens een FAQ oid op te stellen over de beveiliging en foutafhandeling van scripts, met eventueel linkjes naar tutorials hier over. Ik wordt er zelf ook af een toe een beetje moe van om elke keer maar weer diezelfde vragen te beantwoorden (ik weet dat ik niet hoef te reageren, maar je snapt me vast).
Voor beginnende programmeurs is het nog vrij lastig om zelf zo'n foutenafhandeling te doen. Je kan beter eisen stellen aan de functionaliteit van het script. Later kunnen gevorderde programmeurs hun mening/opmerking achterlaten hoe het script verbetert kan worden. Want als je mij nu een script geeft dan zal ik echt niet heel het script zonder fouten kunnen herschrijven. Maar ik ben het wel eens met je andere punten ;)
'Scrptr:
Helemaal voor,
Nog een mogelijkheid: wel meteen weergeven maar als er meer dan bijvoorbeeld 10 meldingen zijn die aangeven dat het script nutteloos is komt het in een tijdelijke map waar het aan de admin is om te beoordelen of het script verwijderd moet worden. Onjuiste meldingen kunnen natuurlijk ook gestraft worden. Voordeel van dit systeem is in mijn oog dat het minder werk is voor de Admins en je geeft daarnaast iedereen de mogelijkheid om mee te bepalen.
Nog een mogelijkheid: wel meteen weergeven maar als er meer dan bijvoorbeeld 10 meldingen zijn die aangeven dat het script nutteloos is komt het in een tijdelijke map waar het aan de admin is om te beoordelen of het script verwijderd moet worden. Onjuiste meldingen kunnen natuurlijk ook gestraft worden. Voordeel van dit systeem is in mijn oog dat het minder werk is voor de Admins en je geeft daarnaast iedereen de mogelijkheid om mee te bepalen.
Zet er dan wel een beveiliging op van maximaal 1 stem per script per gebruiker. Anders krijgen we dat weer...
Het is duidelijk dat er iets aan heel dat systeem moet gebeuren. Ook de duidelijkheid wáár men een script kan posten moet verbeterd worden. In eerste instantie kon ik het niet vinden, dat menuutje bovenin valt nogal weg.
Dan is het nog maar afwachten wanneer de nieuwe versie komt. Er is al een aantal keer een deadline gegeven, maar nog nooit is aan deze deadline voldaan. Ik vind het ook een beetje irritant dat wij niks meer over de nieuwe versie gehoord hebben sinds de januari-deadline. We moeten het zo maar even opvangen als er over gesproken wordt in topics... Geen goede zaak ;-).
(ook nog nooit gedaan hoor :-p)
'erwin:
het gaat natuurlijk niet alleen om de scripts, er zijn ook nog tutorials.
Quote:
Had ik al in m'n beginpost staan :)maar dat hun scripts/tutorials/boeken eerst moet worden goedgekeurd.
Afijn, die nieuwe versie wordt ook maand na maand na maand uitgesteld. Denk eerder dat wij de RC1 van FireFox 4 zullen hebben, dan een nieuwe, bugfree PHPHulp.nl :P