Hi :)
I'm really happy that we started so interesting discussion :-)
Mariusz is right !
So I'm putting here the right version of example, that we make
together
/including the problem with commas/:
$query_str = "select ";
if ($_POST["id_chbox"] == 1)
$query_str .= " id,";
if ($_POST["name_chbox"] == 1)
$query_str .= " name,";
.......and at the and cut the last 1 sign of this string - which /if
at least one checkbox is checked/ surely will be a comma
dealing with "no checkbox checked" that another subject ;-)
And - this is for Angelo - It would be great if you share code, that
you're writting!
Happy To Help
Asia Sledzik
------ Wiadomość oryginalna ------
Od: Mariusz Pekala <skoot@xxxxx>
Data: 2003-12-22 18:02
Temat: Re: [PHP] Interactive querys
:-> -----BEGIN PGP SIGNED MESSAGE-----
:-> Hash: SHA1
:->
:-> The idea is nice. I am using similiar technique in my work, but
you commited
:-> some mistakes in the code. It may cause some anger/frustration
amongst newbie
:-> ones. :-):->
:-> ripley@xxxxxxxxx (pon 22. grudzień 2003 11:46):
:-> > ------ Wiadomość oryginalna ------
:-> > Od: Ângelo Marcos Rigo <angelo_rigo@xxxxxxxxxxxx>
:-> > Data: 2003-12-15 14:23:-> >
:-> > Temat: Re: [PHP] Interactive querys
:-> [...]
:-> > 2. Second - you have only to build your query string - using
:-> > conditions "if..." :
:-> > start with:
:-> > $query_str = "select ";
:-> > if ($_POST["id_chbox"] = 1)
:-> It sets the value of possibly non-existing $_POST["id_chbox"] to
1, that
:-> expression is evaluated as TRUE, thus the following is always
executed.
:-> > $query_str += $_POST["id_chbox"];
:->
:-> You, probably, wanted to say: $query_str .= ' id';
:-> > if ($_POST["name_chbox"] = 1)
:-> > $query_str += ", ".$_POST["id_name"];
:-> Same here... == instead of =, and .= instead of += and ' name'
instead of
:-> $_POST['id_name'];
:-> Also, real code should check whether any field has been already
inserted in
:-> the $query_str before the comma will be used as separator.
:-> > ..............
:-> > and so on
:-> > You can also try to make this script "universal" by using
iterations
:-> > through $_POST variables - its possible to write something
like
:-> >
:-> > for ($n=0; n < ...;$n++)
:-> > {
:-> > $_POST[$cols[$n]."_chbox"]............
:-> > }
:-> >
:-> > The only minus of this method is that it can take some time :)
:->
:-> A few dozens of iterations should be pretty fast. And this
method is more
:-> 'elegant' than linear - if( x==1)... else if( x==2) ... else if(
x==3).. way.
:->
:-> >
:-> > I hope it helps
:-> > Asia Sledzik
:->
:-> Me too... ;-)
:-> Mariusz
:-> - --
:-> [http://skoot.qi.pl for GPG keys]
:-> "A computer programmer is someone who, when told to "Go to
Hell", sees
:-> the "Go to", rather than the destination, as harmful."
------------ R E K L A M A ------------
Gotowy do drogi? Swiateczny Kalendarz SAS zaprasza!
Przez 24 dni oferujemy 24 oferty specjalne do Europy i USA.
Tylko na stronie http://www.scandinavian.net/ (wybierz Polske)
