[SOLUCIONADO] Problema con funcion redefinida en php

[pablomanosdetijera]

Hola, tengo este problema. Tengo un archivo con funciones php que al llamarlo me da un error por una función redeclarada, pero no veo que haya ninguna funcion redeclarada. Este el texto del error que da el navegador:
"Fatal error: Cannot redeclare obtenerproducto() (previously declared in C:\xampplite\htdocs\misitio2\funciones\funciones.p hp:4) in C:\xampplite\htdocs\misitio2\funciones\funciones.p hp on line 10"


Y este es el código del archivo con la funciones php:

Código PHP:

  <?php
 
function obtenerProducto($id){
    include("../incluir/conexion.php");
    $sql="select*from productos where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['nombre'];
}
function obtenerDescripcion($id){
    include("../incluir/conexion.php");
    $sql="select*from productos where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['descripcion'];
}
function obtenerEstado($id){
    include("../incluir/conexion.php");
    $sql="select*from estado where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['estado actual'];
}
function obtenerSemana($id){
    include("../incluir/conexion.php");
    $sql="select*from semana where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['novedad'];
}
function obtenerCategoria($id){
    include("../incluir/conexion.php");
    $resultado=mysql_query("select*from categorias where id=$id",$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['nombre'];
}
 
?>   

Si alguien logra encontrar el error se lo agradeceré infinitamente.

[RabidFish]

Creo que deberías llamar una sola vez a conexion.php dentro de funciones.php, y después llamar a las funciones que utilizas de ese archivo en cada función de obtenerX(). Incluso creo que podrías llamar una sola vez a conectar() fuera de las funciones y serviría igual (siempre que necesites conectarte cada vez que incluyes el archivo funciones.php).

Código PHP:

  <?php
include("../incluir/conexion.php");
 
function obtenerProducto($id){
    conectar();
    $sql="select*from productos where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['nombre'];
}
function obtenerDescripcion($id){
    conectar();
    $sql="select*from productos where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['descripcion'];
}
function obtenerEstado($id){
    conectar();
    $sql="select*from estado where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['estado actual'];
}
function obtenerSemana($id){
    conectar();
    $sql="select*from semana where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['novedad'];
}
function obtenerCategoria($id){
    conectar();
    $resultado=mysql_query("select*from categorias where id=$id",$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['nombre'];
}
 
?>   

Fíjate también de no estar llamando muchas veces a funciones.php en tu script. Recuerda que mientras menos código generes y menos código repitas, tu software será menos propenso a errores y más fácil de modificar en el futuro.
Saludos.

[Italico76]

Es como te dicen y ampliando:

1) Los include o requiere van idealmente (salvo casos particulares como templates) solo al comienzo del archivo
2) No debes hacer include dentro de funciones !
3) conexion.php especialmente debes llamarlo con requiere_once o include_once

[pablomanosdetijera]

Iniciado por RabidFish

Creo que deberías llamar una sola vez a conexion.php dentro de funciones.php, y después llamar a las funciones que utilizas de ese archivo en cada función de obtenerX(). Incluso creo que podrías llamar una sola vez a conectar() fuera de las funciones y serviría igual (siempre que necesites conectarte cada vez que incluyes el archivo funciones.php).

Código PHP:

  <?php
include("../incluir/conexion.php");
 
function obtenerProducto($id){
    conectar();
    $sql="select*from productos where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['nombre'];
}
function obtenerDescripcion($id){
    conectar();
    $sql="select*from productos where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['descripcion'];
}
function obtenerEstado($id){
    conectar();
    $sql="select*from estado where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['estado actual'];
}
function obtenerSemana($id){
    conectar();
    $sql="select*from semana where id=$id";
    $resultado=mysql_query($sql,$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['novedad'];
}
function obtenerCategoria($id){
    conectar();
    $resultado=mysql_query("select*from categorias where id=$id",$conexion);
    $row=mysql_fetch_assoc($resultado);
    return $row['nombre'];
}
 
?>   

Fíjate también de no estar llamando muchas veces a funciones.php en tu script. Recuerda que mientras menos código generes y menos código repitas, tu software será menos propenso a errores y más fácil de modificar en el futuro.
Saludos.

Muchas gracias, fue muy útil tu comentario.