Re: Interactive querys - final version

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(at)qi(dot)pl>

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(at)gazeta(dot)pl (pon 22. grudzień 2003 11:46):

:-> > ------ Wiadomość oryginalna ------

:-> > Od: Ângelo Marcos Rigo <angelo_rigo(at)yahoo(dot)com(dot)br>

:-> > 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)