Solve undifind id , foreach and undefined name age and email in datafield

Question

1.00/5 (2 votes)

See more:

index.php

<?php

include_once("../class/Crud.php");

$crud = new Crud();

$query = "SELECT * FROM users ORDER BY id DESC ";
$result = $crud->getData($query);
//echo "

"; print_r($result); exit; 

?>

<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Homepage



<a href="add.php">Add New Data</a><br><br>
	
    	        <?php
		
		foreach($result as $key => $res) {
			echo "";
			echo "";
			echo "";
			echo "";
			echo " ";
		}
		
		?>
    <table width="80%" border="0"><tbody><tr>        <td>Name</td>        <td>Age</td>        <td>Email</td>        <td>Update</td>        </tr><tr><td>".$res['name']."</td><td>".$res['age']."</td><td>".$res['email']."</td><td><a href=\"edit.php?id=$res[id]\">Edit |
				  <a href=\"delete.php?id=$res[id]\" onclick=\"return confirm('Are you sure you want to delete?')\">Delete</td></tr></tbody></table>





Crud.php



<?php

include_once '../class/DbConfig.php';

class Crud extends DbConfig {
	public function __construct() {
		parent::__construct();
	}
	
	public function getData($query) {
		$result = $this->connection->query($query);
		
		if($result == false) {
			return false;
		}
		
		$rows = array();
		
		while($row = $result->fetch_assoc()) {
			$rows[] = $row;
		}
		return $rows;
	}
	
	public function execute($query) {
		$result = $this->connection->query($query);
		
		if($result == false) {
			echo "can't execute the command";
			return false;
		} else {
			return true;
		}
	}
	
	public function escape_string($value) {
		return $this->connection->real_escape_string($value);
	}
}

?>


add.php


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Add data



<?php

include_once("../class/Crud.php");
include_once("../class/Validation.php");

$crud = new Crud();
$validation = new Validation();

if(isset($_POST['Submit'])) {
	$name = $crud->escape_string($_POST['name']);
	$age = $crud->escape_string($_POST['age']);
	$email = $crud->escape_string($_POST['email']);
	
	$msg = $validation->check_empty($_POST, array('name', 'age', 'email'));
	$check_age = $validation->is_age_valid($_POST['age']);
	$check_email = $validation->is_email_valid($_POST['email']);
	
	if($msg != null) {
		echo $msg;
		echo "<br><a>Go back</a>";
	} else if(!$check_age) {
		echo "Please provide proper age.";
	} else if(!$check_email) {
		echo "Please provide proper email.";
	}
	else {
		$result = $crud->execute("INSERT INTO users(name, age, email)VALUES('$name', '$age', '$email')");
		
		echo "<font color="green">Data added successfully.";
		echo "<br><a href="index.php">View result</a>";
	}
}

?>




Validation.php


<?php

class Validation {
	public function check_empty($data, $fields) {
		$msg = null;
		foreach($fields as $value) {
			if(empty($data[$value])) {
				$msg .= "$value field empty <br>";
			}
		}
		return $msg;
	}
	
	public function is_age_valid($age) {
		if(preg_match("/^[0-9]+$/", $age)) {
			return true;
		}
		return false;
	}
	
	public function is_email_valid($email) {
		if(filter_var($email, FILTER_VALIDATE_EMAIL)) {
			return true;
		}
		return false;
	}
}

?>


DbCongig.php


<?php

class DbConfig {
	private $_host = 'localhost';
	private $_username = 'root';
	private $_password = '';
	private $_database = 'test1';
	
	protected $connection;
	
	public function __construct() {
		if(!isset($this->connection)) {
			$this->connection = new mysqli($this->_host, $this->_username, $this->_password, $this->_database);
			
			if(! $this->connection) {
				echo "can't connect to database";
				exit;
			}
		}
		return $this->connection;
	}
}

?>


add.html


<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Add data



<a href="add.php">Home</a>
<br><br>


<table width="25%" border="0"><tbody><tr><td>Name</td><td></td></tr><tr><td>Age</td><td></td></tr><tr><td>Email</td><td></td></tr><tr><td></td><td></td></tr></tbody></table>





edit.php


<?php

include_once("../class/Crud.php");

$crud = new Crud();

$id = $crud->escape_string($_GET['id']);

$result = $crud->getData("SELECT * FROM users WHERE id=$id");

foreach($result as $res) {
	$name = $res['name'];
	$age = $res['age'];
	$email = $res['email'];
}

?>
<!DOCTYPE html PUBLIC "-//W3C//DTD XHTML 1.0 Transitional//EN" "http://www.w3.org/TR/xhtml1/DTD/xhtml1-transitional.dtd">


<meta http-equiv="Content-Type" content="text/html; charset=utf-8" />
<title>Edit data



<a href="index.php">Home</a>
<br><br>


	
    	        
                
                
        <table border="0"><tbody><tr>        <td>Name</td>        <td></td>        </tr><tr>        <td>Age</td>        <td></td>        </tr><tr>        <td>Email</td>        <td></td>        </tr><tr>        <td></td>        <td></td>        </tr>	</tbody></table>





editaction.php


<?php

include_once("../class/Crud.php");
include_once("../class/Validation.php");

$crud = new Crud();
$validation = new Validation();

if(isset($_POST['update'])) {
	$id = $crud->escape_string($_POST['id']);
	
	$name = $crud->escape_string($_POST['name']);
	$age = $crud->escape_string($_POST['age']);
	$email = $crud->escape_string($_POST['email']);
	
	$msg = $validation->check_empty($_POST, array('name', 'age', 'email'));
	$check_age = $validation->is_age_valid($_POST['age']);
	$check_email = $validation->is_email_valid($_POST['email']);
	
	if($msg) {
		echo $msg;
		echo "<br><a>Go Back</a>";
	} elseif(! $check_age) {
		echo "Please provide proper age.";
	} elseif(! $check_email) {
		echo "Please provide proper email.";
	} else {
		$result = $crud->execute("UPDATE users SET name='$name', age='$age', email='$email' WHERE id=$id ");
		header("Location:index.php");
	}
}

?>


error:

Notice: Undefined index: id in C:\xampp\htdocs\crud_php\class\edit.php on line 7

Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\crud_php\class\edit.php on line 11

inside name field:   <br>Notice:  Undefined variable: name in C:\xampp\htdocs\crud_php\class\edit.php on line 33<br>


inside age field:   <br>Notice:  Undefined variable: age in C:\xampp\htdocs\crud_php\class\edit.php on line 38<br>



inside email field:   <br>Notice:  Undefined variable: email in C:\xampp\htdocs\crud_php\class\edit.php on line 43<br>

What I have tried:

<big>i try every solution but its shows error like this</big>


Notice: Undefined index: id in C:\xampp\htdocs\crud_php\class\edit.php on line 7

Warning: Invalid argument supplied for foreach() in C:\xampp\htdocs\crud_php\class\edit.php on line 11

inside name field:   <br>Notice:  Undefined variable: name in C:\xampp\htdocs\crud_php\class\edit.php on line 33<br>


inside age field:   <br>Notice:  Undefined variable: age in C:\xampp\htdocs\crud_php\class\edit.php on line 38<br>



inside email field:   <br>Notice:  Undefined variable: email in C:\xampp\htdocs\crud_php\class\edit.php on line 43<br></font>

Posted 9-Apr-18 7:21am

Member 13761429

Updated 9-Apr-18 15:50pm

Add a Solution

Comments

Kornfeld Eliyahu Peter 9-Apr-18 15:16pm

The only way to solve these problems is by learning the basics of PHP...

1 solution

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

ThilinaMD · Answer 1 · 2018-04-09T15:52:00

Solution 1

So error is clearly stated in line 7. You should use table row values inside $res[] by using inverted commas. $res[id] should change as $res['id'] or $res["id"]. Make sure table column names and $res[] values are same.

Posted 9-Apr-18 15:52pm

ThilinaMD

Comments

Member 13761429 10-Apr-18 4:51am

not solved

ThilinaMD 10-Apr-18 7:58am

It will be usefull if you learn basic curd in php through any online tutorial like tutorialspoint

Member 13761429 10-Apr-18 5:39am

<?php

include_once("../class/Crud.php");

$crud = new Crud();

$id = $crud->escape_string($_GET['id']);

$result = $crud->delete($id, 'users');

if($result) {
header("Location: index.php");
}

?>

also error when run delete.php its work but when particularly run delete,php its shows error:

Notice: Undefined index: id in C:\xampp\htdocs\crud_php\class\delete.php on line 7

ThilinaMD 10-Apr-18 7:54am

You have to pass a value for the parameter id from a form field(input tag) or as a url parameter like delete.php?id=1

Member 13761429 10-Apr-18 23:03pm

i already did it but its not work