473,403 Members | 2,323 Online
Bytes | Software Development & Data Engineering Community
Post Job

Home Posts Topics Members FAQ

Join Bytes to post your question to a community of 473,403 software developers and data experts.

Quick question

I am trying to do a SQL table insert. OK, no big deal. Well I am having
a problem with this code. I build a data structure $fields_values and
pass it to a function insertIntoDB() to issue the insert.

The problem is in the actual INSERT Statement. The values for the column
data do not have "" around the Col values. So SQL flags the fields as
wrong. Example:

INSERT INTO `Log` (ipAddress,action,groupPageName,namePageName,syste m)
VALUES (127.0.0.1,browse,Main,HomePage,Testing new code)

Can some one point me in the right direction on wrapping " around the
data values so it will insert into the table.

$table = 'myTable';

$fields_values = array
(
'ipAddress' =$_SERVER['REMOTE_ADDR'],
'action' =$action,
'groupPageName' =FmtPageName('$Group', $pagename),
'namePageName' =FmtPageName('$Name', $pagename);,
'system' =$system
);

insertIntoDB($strTableName, $fields_values);

function insertIntoDB($table, $fields_values)
{
$fields = implode(array_keys($fields_values), ',');
$values = implode(array_values($fields_values), ',');
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
('.$values.')';
$res = mysql_query($sqlStatement)OR die(mysql_error());
return true;
}

--
Thanks in Advance...
IchBin, Pocono Lake, Pa, USA http://weconsultants.phpnet.us
'If there is one, Knowledge is the "Fountain of Youth"'
-William E. Taylor, Regular Guy (1952-)
Sep 7 '06 #1
2 1338
In your insertIntoDB function, change:
$values = implode(array_values($fields_values), ',');
to this:
$values = implode(array_values($fields_values), '","');
and this:
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
('.$values.')';
to this:
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
("'.$values.'")';
This should quote everything. I don't remember if quoting non-string
values matters, if so you would need to loop through $values and append
quotes if is_string($value[n]) instead.

IchBin wrote:
I am trying to do a SQL table insert. OK, no big deal. Well I am having
a problem with this code. I build a data structure $fields_values and
pass it to a function insertIntoDB() to issue the insert.

The problem is in the actual INSERT Statement. The values for the column
data do not have "" around the Col values. So SQL flags the fields as
wrong. Example:

INSERT INTO `Log` (ipAddress,action,groupPageName,namePageName,syste m)
VALUES (127.0.0.1,browse,Main,HomePage,Testing new code)

Can some one point me in the right direction on wrapping " around the
data values so it will insert into the table.

$table = 'myTable';

$fields_values = array
(
'ipAddress' =$_SERVER['REMOTE_ADDR'],
'action' =$action,
'groupPageName' =FmtPageName('$Group', $pagename),
'namePageName' =FmtPageName('$Name', $pagename);,
'system' =$system
);

insertIntoDB($strTableName, $fields_values);

function insertIntoDB($table, $fields_values)
{
$fields = implode(array_keys($fields_values), ',');
$values = implode(array_values($fields_values), ',');
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
('.$values.')';
$res = mysql_query($sqlStatement)OR die(mysql_error());
return true;
}

--
Thanks in Advance...
IchBin, Pocono Lake, Pa, USA http://weconsultants.phpnet.us
'If there is one, Knowledge is the "Fountain of Youth"'
-William E. Taylor, Regular Guy (1952-)
Sep 9 '06 #2
IchBin wrote:
>I am trying to do a SQL table insert. OK, no big deal. Well I am having
a problem with this code. I build a data structure $fields_values and
pass it to a function insertIntoDB() to issue the insert.

The problem is in the actual INSERT Statement. The values for the column
data do not have "" around the Col values. So SQL flags the fields as
wrong. Example:

INSERT INTO `Log` (ipAddress,action,groupPageName,namePageName,syste m)
VALUES (127.0.0.1,browse,Main,HomePage,Testing new code)

Can some one point me in the right direction on wrapping " around the
data values so it will insert into the table.

$table = 'myTable';

$fields_values = array
(
'ipAddress' =$_SERVER['REMOTE_ADDR'],
'action' =$action,
'groupPageName' =FmtPageName('$Group', $pagename),
'namePageName' =FmtPageName('$Name', $pagename);,
'system' =$system
);

insertIntoDB($strTableName, $fields_values);

function insertIntoDB($table, $fields_values)
{
$fields = implode(array_keys($fields_values), ',');
$values = implode(array_values($fields_values), ',');
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
('.$values.')';
$res = mysql_query($sqlStatement)OR die(mysql_error());
return true;
}

--
Thanks in Advance...
IchBin, Pocono Lake, Pa, USA http://weconsultants.phpnet.us
'If there is one, Knowledge is the "Fountain of Youth"'
-William E. Taylor, Regular Guy (1952-)
Joshie Surber wrote:
In your insertIntoDB function, change:
$values = implode(array_values($fields_values), ',');
to this:
$values = implode(array_values($fields_values), '","');
and this:
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
('.$values.')';
to this:
$sqlStatement = 'INSERT INTO `'.$table.'` ('.$fields.') VALUES
("'.$values.'")';
This should quote everything. I don't remember if quoting non-string
values matters, if so you would need to loop through $values and append
quotes if is_string($value[n]) instead.
[Corrected message order in this thread]

Thanks Joshie the following resolved the problem:

$values="'".implode(array_values($fields_values)," ','")."'";

--
Thanks in Advance...
IchBin, Pocono Lake, Pa, USA http://weconsultants.phpnet.us
'If there is one, Knowledge is the "Fountain of Youth"'
-William E. Taylor, Regular Guy (1952-)
Sep 10 '06 #3

This thread has been closed and replies have been disabled. Please start a new discussion.

Similar topics

0
by: emmanuelkatto | last post by:
Hi All, I am Emmanuel katto from Uganda. I want to ask what challenges you've faced while migrating a website to cloud. Please let me know. Thanks! Emmanuel
1
by: nemocccc | last post by:
hello, everyone, I want to develop a software for my android phone for daily needs, any suggestions?
0
marktang
by: marktang | last post by:
ONU (Optical Network Unit) is one of the key components for providing high-speed Internet services. Its primary function is to act as an endpoint device located at the user's premises. However,...
0
by: Hystou | last post by:
Most computers default to English, but sometimes we require a different language, especially when relocating. Forgot to request a specific language before your computer shipped? No problem! You can...
0
Oralloy
by: Oralloy | last post by:
Hello folks, I am unable to find appropriate documentation on the type promotion of bit-fields when using the generalised comparison operator "<=>". The problem is that using the GNU compilers,...
0
jinu1996
by: jinu1996 | last post by:
In today's digital age, having a compelling online presence is paramount for businesses aiming to thrive in a competitive landscape. At the heart of this digital strategy lies an intricately woven...
0
tracyyun
by: tracyyun | last post by:
Dear forum friends, With the development of smart home technology, a variety of wireless communication protocols have appeared on the market, such as Zigbee, Z-Wave, Wi-Fi, Bluetooth, etc. Each...
0
agi2029
by: agi2029 | last post by:
Let's talk about the concept of autonomous AI software engineers and no-code agents. These AIs are designed to manage the entire lifecycle of a software development project—planning, coding, testing,...
0
isladogs
by: isladogs | last post by:
The next Access Europe User Group meeting will be on Wednesday 1 May 2024 starting at 18:00 UK time (6PM UTC+1) and finishing by 19:30 (7.30PM). In this session, we are pleased to welcome a new...

By using Bytes.com and it's services, you agree to our Privacy Policy and Terms of Use.

To disable or enable advertisements and analytics tracking please visit the manage ads & tracking page.