php - codeigniter, opret bruger / tjek bruger - Udvikleren.dk

Tags: php

Bruger #12118 @ 23.10.10 01:22

Hej, jeg er igang med codeigniter, men jeg har et lille problem og det er at jeg godt kan oprette den samme bruger 2 gange, og jeg kan ikke få den til at gå over på en side som hedder "oprettet.php", altså en view fil.

her er min kode, håber i kan se hva jeg gør galt

PHP kode

 <?php
 
 class Signup extends Controller {
 
     public function __construct() {
         parent::Controller();
         $this->load->library('form_validation');
     }
 
     public function index() {
         $data['content'] = 'create_user_view';
          $this->load->view('includes/template', $data);
     }
 
     public function submit() {
 
         if ($this->_submit_validate() === FALSE) {
             $this->index();
             return;
         }
         
         $this->db->where('username', $this->input->post('username'));
         $this->db->where('password', md5($this->input->post('password')));
         $query = $this->db->get('users');
         
         if($query->num_rows == 0)
         {
             $new_member_insert_data = array(
                 'fornavn' => $this->input->post('fornavn'),
                 'efternavn' => $this->input->post('efternavn'),
                 'email' => $this->input->post('email'),            
                 'hood' => $this->input->post('hood'),
                 'city' => $this->input->post('city'),
                 'username' => $this->input->post('username'),
                 'password' => md5($this->input->post('password'))                        
             );
 
             $insert = $this->db->insert('users', $new_member_insert_data);
             return $insert;
             redirect('oprettet_view');
             //$data['content'] = 'oprettet_view';
              //$this->load->view('includes/template', $data);
         } else {
             return FALSE;
             
         }
 
     }
 
     private function _submit_validate() {
 
         // validation rules
         $this->form_validation->set_rules('fornavn', 'Fornavn',
             'required|alpha_numeric|min_length[2]|max_length[30]');
             
         $this->form_validation->set_rules('efternavn', 'Efternavn',
             'required|alpha_numeric|min_length[2]|max_length[70]');
             
         $this->form_validation->set_rules('hood', 'Hood',
             'required|numeric|min_length[4]|max_length[4]');
             
         $this->form_validation->set_rules('city', 'City',
             'required|min_length[4]|max_length[90]');
             
         $this->form_validation->set_rules('username', 'Username',
             'required|alpha_numeric|min_length[6]|max_length[12]');            
             
         $this->form_validation->set_rules('username', 'Username',
             'required|alpha_numeric|min_length[6]|max_length[12]');
 
         $this->form_validation->set_rules('password', 'Password',
             'required|min_length[6]|max_length[12]');
 
 
         $this->form_validation->set_rules('email', 'E-mail',
             'required|valid_email');
 
         return $this->form_validation->run();
 
     }
 }

Bruger #8223 @ 23.10.10 01:31

1.065

To ting:

#1: Hvis du siger where() og derefter where() igen overskriver du den første where(). Dermed tjekker den reelt kun på om passwordet findes. Du behøver slet ikke tjekke passwordet, eftersom der ikke skal være to brugere med samme brugernavn uanset om deres passwords er ens eller ej.

#2: Du laver "return" i din submit() action inden du kan lave dit redirect. Efter dit return stopper kørslen af funktionen, og du kan dermed ikke nå til redirect(). Du skal slet ikke returne data i din Controller, så bare fjern return helt.

Derudover redirecter du til noget der hedder oprettet_view. Det betyder at du henviser brugeren til ditdomæne.dk/oprettet_view/

Medmindre du har en controller der hedder oprettet_view, så er det forkert. Hvis du vil vise oprettet_view-view filen skal du istedet sige $this->load->view('opretttet_view');

Indlæg senest redigeret d. 23.10.2010 01:32 af Bruger #8223

Bruger #12118 @ 23.10.10 02:00

wow

gode guldkorn dér

tusind tak det virker nu, ved bare ikke lige helt hvis nu brugernavnet er optaget hvordan får jeg sendt en besked til brugeren så han kan se det er optaget, er der noget med det der validation helper hvor man kan skrive en besked selv ?

tak ihvertfald

Bruger #8223 @ 23.10.10 13:17

1.065

du tjekker allerede om brugernavnet findes. I din ovenstående kode skriver du return false. Istedet for at sige return false kan du lave $data['fejl'] = 'brugernavnet er optaget' og så sende det med i din $this->load->view - og så i viewfilen skrive <?php if(isset($fejl)){ echo $fejl; }?> der hvor du vil have den.

Når du har styr på det vil jeg anbefale dig at lægge database-koden ind i en model og at lave brugernavns-checket som en callback-funktion i en validation rule, men den slags kan du godt vente med til du er lidt mere verseret i viewfiler og controllerfunktioner.

Hvis du har flere spørgsmål skyder du bare løs

Indlæg senest redigeret d. 23.10.2010 13:17 af Bruger #8223

Bruger #12118 @ 23.10.10 15:23

tror jeg vil prøve med call-back det virker mere "normalt" at gøre

, men det med at smide database filerne ind i en model fil er det ikke dumt at oprette en model fil bare for at smide 2 linier kode ind, og i en model fil kan man ikke hente input felternes værdi ud, altså $this->input->post('username')

Bruger #8223 @ 23.10.10 16:01

1.065

Jo, alle libraries, inklusiv input-librariet, er tilgængelige i modeller, sålænge parent::Model() får lov at køre(dette gør den automatisk hvis du ikke laver din egen constructor), så du kan sagtens lave både validation, input->post() og andre ting i modeller.

Det er aldrig dumt at smide database-håndtering ind i modeller, uanset hvor lidt/meget det drejer sig om - modeller er beregnet til at være datalaget.

Det vil betyde at du har noget der minder om dette:

controller:
submit(){
//en masse validation-rules, og hvis alt er korrekt:
if($this->validation->run(){
$this->users_model->create_user();
}
}

Bruger #12118 @ 23.10.10 19:15

okay

, jeg prøvede nemlig bare på et tidspunkt at ville gøre som du skriver men så kunne jeg ikke få fat i det information som brugeren skrev i input felterne, men må lige prøve det igen

.

når jeg bruger de der inbyggede form helpere ved du så hvordan jeg kan få sat en class ind lige her

<input class="farve" name="brugernavn" />

kan nemlig ikke lige finde noget på codeigniters hjemmeside omkring det, ved heller ikke helt hvad jeg skal søge efter..

men det er det her jeg mener

form_input('brugernavn', set_value('brugernavn'));

Bruger #8223 @ 23.10.10 19:29

1.065

Du siger bare:

Kode

form_input('brugernavn', set_value('brugernavn'), 'class="farve"');

det tredje argument er "alt muligt andet" du vil have med i tagget.

Alternativt kan du bruge 1 argument, et array, fx:

Kode

$input = array(
  'name' => 'brugernavn',
  'id' => 'brugernavn',
  'value' => set_value('brugernavn'),
  'class' => 'farve'
);
form_input($input);

Bruger #12118 @ 23.10.10 19:50

ahh nice tusind tak

lære fanme meget lige nu

, jeg har os det typiske problem med æøå at det bliver mærkelige tegn, i min config fil har jeg sat den til

$config['charset'] = "iso-8859-1";

og i min header.php fil hedder den os iso-8859-1

min database er bare standard: latin1_swedish_ci

Bruger #8223 @ 24.10.10 11:33

1.065

har du et <meta http-equiv="content-type" content="text/html;ISO-8859-1"/> tag i dit <head>?

Hvad charset kører din editor med?

Alternativt kan du prøve at ændre alting til ISO-8859-15 istedet for ...59-1.

Selvom det allerbedste er at bruge UTF-8, men så skal du til at konvertere din database.

Bruger #12118 @ 24.10.10 13:27

rettede det til UTF-8 og nu virker det

Tusind mange tak

codeigniter, opret bruger / tjek bruger

Karma barometer (30 dage)

Modtaget

Givet

Favorit hos
Forum tråde
Artikler