[MySQL][PHP]Dodawanie recordu Opcje

[topik53]

Witam mam taki kodzik

[PHP] pobierz, plaintext 
<form action="add.php" method="get">
 
<strong>Nazwa konta:</strong><input name="tytul" type="text" value="" /><br>
 
<strong>Haslo:</strong><input name="tresc" type="password" value="" /><br>
 
 
<input type="submit" value="Zarejestruj" />
 
</form>
<?php
require('conn.php');
session_start();
 
if (isset($_SESSION['user_id']) and isset($_SESSION['login']))
{
echo $_SESSION['login'];
if (isset($_GET['tytul']) and isset($_GET['tresc']))
{
$zapytanie="INSERT INTO tresc_not (tytul,tresc) VALUES('$tytul','$tresc')";
mysql_query($zapytanie) or die("Wystąpił błąd" );
echo('Konto '.$tytul.' zostalo utworzone');
}
}
?>
[PHP] pobierz, plaintext 

Chodzi o to zeby sprawdzilo czy jest zalogowane , i jezeli jest dodalo notke+tytul, jezeli nie jest "nie jestes zalogowany"
dodaje mi rekordy do bazy ale jak np wpisze asdgadfgadfg to daje mi 0

[bastard13]

Ogólnie to tak:

[PHP] pobierz, plaintext 
if(isset($_SESSION['user']))
{ //zaloowany
}
else
{ //nie zalogowany
   if(isset($_GET['tytul']))
   {//przesłany get
     //walidacja i inne działania, które mają być w tym momencie wykonane
   }
   else
   {
   //wyświetl formularz
    }
}
[PHP] pobierz, plaintext 

Poza tym do logowania lepiej użyć posta, a nie geta. Zauważ, że jeżeli przesyłasz getem to absolutnie bez sensu jest ustawienie inputa z hasłem na password, ponieważ całe hasło i tak zostanie przesłane w linku.

[topik53]

Zaraz sprawdze, co do geta przy logowaniu... to tobie to postem (IMG:style_emoticons/default/smile.gif)

[meehow18]

z tego Co wiem, to sesje powinieneś mieć wystartowaną, na samym początku pliku.

[topik53]

Ogólnie to tak:

[PHP] pobierz, plaintext 
if(isset($_SESSION['user']))
{ //zaloowany
}
else
{ //nie zalogowany
   if(isset($_GET['tytul']))
   {//przesłany get
     //walidacja i inne działania, które mają być w tym momencie wykonane
   }
   else
   {
   //wyświetl formularz
    }
}
[PHP] pobierz, plaintext 

Poza tym do logowania lepiej użyć posta, a nie geta. Zauważ, że jeżeli przesyłasz getem to absolutnie bez sensu jest ustawienie inputa z hasłem na password, ponieważ całe hasło i tak zostanie przesłane w linku.

O ile mi sie nie pomieszalo to sie pomyliles.
Chodzilo mi o to że Jak będzie ZALOGOWANY to wtedy może dodać rekord!
nawet po wykorzystaniu twojej wersji dodaje mi 0

[lord2105]

proszę o zrzut bazy danych

[topik53]

[SQL] pobierz, plaintext 
-- phpMyAdmin SQL Dump
-- version CBA.PL
-- http://www.phpmyadmin.net
--
-- Host: mysql.cba.pl
-- Czas wygenerowania: 13 Wrz 2010, 20:00
-- Wersja serwera: 5.0.77
-- Wersja PHP: 5.2.14
 
SET SQL_MODE="NO_AUTO_VALUE_ON_ZERO";
SET time_zone = "+00:00";
 
 
/*!40101 SET @OLD_CHARACTER_SET_CLIENT=@@CHARACTER_SET_CLIENT */;
/*!40101 SET @OLD_CHARACTER_SET_RESULTS=@@CHARACTER_SET_RESULTS */;
/*!40101 SET @OLD_COLLATION_CONNECTION=@@COLLATION_CONNECTION */;
/*!40101 SET NAMES utf8 */;
 
--
-- Baza danych: `lototest_cba_pl`
--
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla  `notki`
--
 
CREATE TABLE IF NOT EXISTS `notki` (
  `id_not` int(11) NOT NULL AUTO_INCREMENT,
  `id_us` int(11) NOT NULL,
  PRIMARY KEY  (`id_not`)
) ENGINE=MyISAM DEFAULT CHARSET=latin1 AUTO_INCREMENT=1 ;
 
--
-- Zrzut danych tabeli `notki`
--
 
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla  `tresc_not`
--
 
CREATE TABLE IF NOT EXISTS `tresc_not` (
  `id_not` int(11) NOT NULL,
  `tytul` int(50) NOT NULL,
  `tresc` longtext character SET utf8 collate utf8_polish_ci NOT NULL
) ENGINE=MyISAM DEFAULT CHARSET=latin1;
 
--
-- Zrzut danych tabeli `tresc_not`
--
 
INSERT INTO `tresc_not` (`id_not`, `tytul`, `tresc`) VALUES
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, ''),
(0, 0, 'asd');
 
-- --------------------------------------------------------
 
--
-- Struktura tabeli dla  `user`
--
 
CREATE TABLE IF NOT EXISTS `user` (
  `id` int(11) NOT NULL AUTO_INCREMENT,
  `login` varchar(50) NOT NULL,
  `password` varchar(40) NOT NULL,
  PRIMARY KEY  (`id`)
) ENGINE=MyISAM  DEFAULT CHARSET=latin1 AUTO_INCREMENT=2 ;
 
--
-- Zrzut danych tabeli `user`
--
 
INSERT INTO `user` (`id`, `login`, `password`) VALUES
(1, 'asd', 'ceedf12f8fe3dc63d35b2567a59b93bd62ff729a');
[SQL] pobierz, plaintext 

wiem ze baza jest zle zaprojektowana, postaram sie jjutro poprawic

Ten post edytował topik53 13.09.2010, 19:05:40

[Mostrom]

Nie wiem czy masz już taką sesję jak np $_SESSION["zalogowany"], to jest taka sesja, która będzie miała informacje czy czy jesteś zalogowany. Skrypt dodanie rekordu wstaw poprostu w takim warunku

[PHP] pobierz, plaintext 
if($_SESSION["zalogowany"]=1){
   mysql_query($zapytanie) or die("Wystąpił błąd" );
}
else {
   echo "Nie jesteś zalogowany. Spadaj";
}
[PHP] pobierz, plaintext 

Ten post edytował Mostrom 13.09.2010, 19:40:32

[lord2105]

[SQL] pobierz, plaintext 
 `tytul` int(50) NOT NULL,
[SQL] pobierz, plaintext 

tytuł może być intem chyba nie (IMG:style_emoticons/default/tongue.gif) masz odp dlaczego dodaje Ci zero pozdrawiam

[topik53]

Dziekuje (IMG:style_emoticons/default/tongue.gif)