Voisiko joku katsoa oheista koodia ja kertoa mitä kaikkea siinä on vialla..
Lomake..
Palautelomake
WEB lomakkeeseen neuvoa...
3
792
Vastaukset 3
- teuro
Toki tuosta voin kertoa heti ettei se näytä kovin hyvältä selaimessa, mutta sitä tuskin haluat kuulla. form action=http... tästä puuttuu aloittava lainausmerkki. Roskapostin esto tuskin toimii, mutta häiritsee normaalia käyttäjää takuulla. Tarkistuskohdassa sulla $_POST['nimi'], mutta edellä olet käyttänyt $_POST['etunimi']?
Samassa kohdassa mikset käytä suoraan muuttujaa $etunimi? Jossain kohdassa voisi vähän tarkistella mitä noissa muuttujissa onkaan. Jokuhan voisi vaikka kirjoitella hieman haittakoodia ja voitaisiin vaikkapa suoritella palvelimella...
Puhelinnumeron tarkistus on ihan huuhaata, koska siinä saa olla ihna mitä tahansa paitsi pieniä kirjaimia. Vai tarkoittaako eregi, että mitään kirjaimia ei saa esiintyä, mutta kaikki muu on sallittua. Muutoin samat ongelmat, kuin etunimen kohdalla. Lopuksi pyydät kamalan kasan tietoa lähettäjästä, muttet lähetä niitä itsellesi voitko kertoa miksi näin?- PHP-viallinen
Kiitos vastauksesta, koodi on varmasti puutteellinen...olen eka kertaa mokomaa tekemässä. Lainailin koodinpätkiä sieltä täältä joten epäjohdonmukaisuuksia on kuten kyseltyjen tietojen välitys omaan sähköpostiin. Ajattelin liittää lomakkeeseen jonkun kuvan numerosarjasta jota spämmikone ei osaa lukea, en keksinyt helpompaa keinoa. Jep, puh nro tarkastus on varmaan turha.
- Jaska Perusjaska
Tuo ei todennäköisesti toimi, koska käytät palaute-muuttujaa, sekä lomakkeen palautteen muuttujana, että mail()-funktion palauttamaan dataan.
Ja tosiaan teen aivan turhaan kaiken maailman puhelinnumeroiden tarkistukset, kun et lähetä lomakedatasta muuta kuin palautteen sähköpostiisi.
Jokaisen lomakekentän sisällön tarkastaminen erikseen on ehkä hieman työlästä, kun koko roskan voisi hoitaa esim. foreachilla. Kaikki lomakedatahan on $_POST-taulukossa. Mielummin tyhjät kentät tarkastettaisiin javascriptilla ennen lomakkeen lähetystä. Muuten kaikki tiedot häviää, jos käyttäjä tekee virheen. Virheilmoituksetkaan ei tulisi näkyviin, koska käyttäjä ohjataan joka tapauksessa pois scriptistä. Joten ne pitäisi muuttaa vaikkapa die("Et antanut x tietoa!");
Eikä itseasiassa tuosta tarkistuskesta ole mitään hyötyä, koska scripti suoritetaan joka tapauksessa loppuun, oli kaikki kentät tyhjiä tai ei.
Ja keräät muutenkin turhaan kaikki tiedot omiin muuttujiin, kun et käytä niitä missään.
Kysyt lomakkeessa tietoja, joihin ei edes viitata scriptissä. Esim. vastaus-kenttä.
Tuossa eregissäkin on jotain pielessä. Nyt se taitaa hyväksyä kaikki muut merkit paitsi sulut ja kirjaimet a-z. Eregissä ei taida olla merkistystä kirjainten koolla, joten myös A-Z ovat pannassa. Käännä koko lause mielummin toisinpäin, että kaikki muut merkit paitsi 0-9, () ja - hylätään. Helpompi niin.
Tuo spam-esto saattaa olla toimiva. Ainakin hetken. Olet poistanut osia koodista, joten ei pysty sanomaan. Kyllä kuvastakin pystyy lukemaan tekstiä ja jos joka kerta on sama tarkistuskoodi&kuva, niin kyllä joskus joku robotti sen saa selville. Ja tosiaan käyttäjiä tuollainen vain kiusaa.
Ei tuo ihan valmis koodi vielä ole, mutta kyllä se siitä :)
Ketjusta on poistettu 0 sääntöjenvastaista viestiä.
Luetuimmat keskustelut
- 53887
Muistatko Mikkelin panttivankidraaman?
Uusi draamasarja järkyttävästä tapauksesta on tulossa. Tositapahtumiin perustuva sarja ammentaa vuoden 1986 Mikkelin pan63643- 55640
Iäkäs Jämsäläinen mies kuoli poliisiautoon matkalla Jyväskylän putkaan
Iäkäs vanhus humalassa niin huonossa kunnossa, ettei pystynyt huolehtimaan itsestään niin ainoa apu sillä hetkellä oli43562Olen luovuttanut
Välimme menivät niin pahasti solmuun, ettei niitä voi enää korjata. On aika jatkaa elämässä eteenpäin. Toivon sulle kaik31532Ernest Lawson täräytti erikoisen heiton TTK-lehdistötilaisuudessa: " Onko tässä tarkoituksena...?"
Ernest Lawson esitteli uudet TTK-tähtioppilaat ja opettajat torstaina 6.8. lehdistölle. Tulevalla kaudella on yksi hausk4502En välitä sinusta yhtään
Olet pelkkä itsestään liikoja luuleva ämmä. Kierrän sinut kaukaa nyt ja aina. Olit mulle pelkkä lelu vaan.32498- 73446
- 33425
- 37416