Problema con permisos...

[MasterXNicoX]

Hola gente, les comento que en mi sitio tengo un contador que muestra, en forma de texto, las visitas que tuvo el sitio...

El problema es que en donde me tendría que mostrar las visitas aparece esto:

Warning: fopen(c.txt) [function.fopen]: failed to open stream: Permission denied in /www/misitio.com/htdocs/web/v.php on line 1

Warning: fwrite(): supplied argument is not a valid stream resource in /www/misitio.com/htdocs/web/v.php on line 1

Warning: fclose(): supplied argument is not a valid stream resource in /www/misitio.com/htdocs/web/v.php on line 1
Ya se Mostraron: 3851371 Páginas de MiSitio!

-------------------------

Estoy usando este script de php...

<? $archivo = "c.txt";$contador = 0;$fp = fopen($archivo,"r");$contador = fgets($fp, 26);fclose($fp);++$contador;$fp = fopen($archivo,"w+");fwrite($fp, $contador, 26);fclose($fp);echo "Ya se Mostraron: $contador Páginas de Misitio!";?>

Lo que este último hace es:

Abrir el archivo "c.txt"... Este contiene solamente un número que es el que se muestra en el site y a medida que hay una visita nueva se le agrega 1 al numero que contenía este archivo...

---

No se que puede estar fallando en el código... Alguien me puede dar una mano??

Lo que no entiendo es porque aparece esto... ya que dias anteriores ese error no aparecia y no he modificado el código del script....

Aclaro que el archivo "c.txt" tiene permisos "777".

Saludos y espero que puedan ayudarme

Que tengan un lindo día!!

[ZequeZ]

Sería mas fácil localizar el error si dejas una linea después de cada ; así el log te marca la linea que provoca el error :P.

[MasterXNicoX]

Ok, gracias, ahora lo intento...

es que en este momento no me aparece el error :

Pasa de a ratos solamente

Saludos!!

[jonatanc]

Hola , por lo que comentas pueden ser o un error de php , seria util que pongas lo siguiente en tu php debajo de:

Código php:

<?php
error_reporting(-1);
?>
 

Y por otro lado podria ser que si ese fichero esta trabajando en un web hosting , hayan instalado algun patch para securizar y esten bloqueando algunas funciones.

Saludos!

[albert1980]

mira haber si /www/misitio.com/htdocs/web/c.txt existe, de lo contrario igual se encuentra en /www/misitio.com/htdocs/c.txt y no funciona

[MasterXNicoX]

Hola y gracias x las respuestas...

Jona, eso es para que no se muestre el error?? Lo tengo que aplicar dentro de mi archivo php???

---

Albert, el fichero si existe, todo funciona perfectamente la mayor parte del tiempo, sólo a veces aparecen esos errores...

[jonatanc]

Lo tenes que incluir dentro de v.php o el script que te genere ese permission denied , y al contrario es el maximo nivel de error , es para que te muestre sugerencias de optimización y todos los errores.

Saludos!

[albert1980]

Pues ahora mismo lo único que se me ocurre es que solo aparece ese error cuando dos usuarios acceden al fichero al mismo tiempo.

usa algo como esto:

Código php:

<?
$archivo = "../numero.dat";
$abre = fopen($archivo, "r");
$total = fread($abre, filesize($archivo));
fclose($abre);
$abre = fopen($archivo, "w");
$total = $total + 1;
if ( flock( $abre, LOCK_EX ) ) {
   $grabar = fwrite($abre, $total);
   flock( $abre, LOCK_UN ); 
}
fclose($abre);
?>
 

[gjx2]

El problema del script esta en la mala utilizacion de la funcion

Edito:
error en fopen no en fwrite


Este es el codigo limpio.

Código PHP:

$archivo = "c.txt";
 
$contador = 0;
 
$fp = fopen($archivo,"r");
 
$contador = fgets($fp, 26);
 
fclose($fp);
$contador++;
$fp = fopen($archivo,"w+");
fwrite($fp, $contador,26);
fclose($fp);
echo "Ya se Mostraron: $contador Páginas de Misitio!";
 

Edito:
Seria bueno agregarle esta linea al codigo antes de $contador++

Código PHP:

if(empty($contador)){
$contador = 1;
}
 

[MasterXNicoX]

Ok, gracias a los tres

Ahora pruebo todas las soluciones y les comento como me fué...

-----

Albert, para que sirve el flock?

[MasterXNicoX]

gjx2 qué problema habia con el fwrite??

Podrias explicarme un poco???

Y me podrían explicar para que sirve el numerito 26???

Saludos!!

Les dejo karma a los que me ayudaron

[gjx2]

Otro correccion le hago a mi codigo

Código PHP:

$archivo = "c.txt";
 
$contador = 0;
 
 if(!file_exists($archivo)){
 fopen($archivo,"w+"); 
 }
 
$fp = fopen($archivo,"r+");
 
$contador = fgets($fp, 26);
 
fclose($fp);
 
if(empty($contador)){
$contador = 0;
}
 
$contador++;
$fp = fopen($archivo,"w");
fwrite($fp, $contador);
fclose($fp);
echo "Ya se Mostraron: $contador Páginas de Misitio!";
 

Le agrege esta linea

Código PHP:

if(!file_exists($archivo)){
 fopen($archivo,"w+"); 
 }
 

Para que no genere error si no existe el archivo.

[albert1980]

Iniciado por MasterXNicoX

Ok, gracias a los tres

Ahora pruebo todas las soluciones y les comento como me fué...

-----

Albert, para que sirve el flock?

para bloquear el archivo

[gjx2]

Iniciado por MasterXNicoX

gjx2 que problema habia con el fwrite??

Podrias explicarme un poco???

Y me podrían explicar para que sirve el numerito 26???

Saludos!!

Les dejo karma a los que me ayudaron

No perdon quise decir en la fopen
Estavas leyendo un archivo que no existia
Cuando expecificas el 26 le estas diciendo que el maximo de caracteres que debe de escribir se limite a 26 :-s

[MasterXNicoX]

Pero ese archivo si existia xD osea.. es más, casi siempre funciona...

Lo del numerito 26 para qué sirve??

[gjx2]

@fopen
@fgets
@fwrite
para que no te salgan los warn..

te decia que el 26 o 27 .... o lo qu tu quieras , es para indicarle la cantidad de char que debe de escribir en el archivo
al igual que cuando haces fgets el numerico le indica la cantidad de char que debe leer


Si quieres saber mas de fwrite
http://us3.php.net/manual/en/function.fwrite.php

[albert1980]

Si casi siempre funciona te aconsejo poner error_reporting(0); antes del fopen y ya verás como no vuelven a salir esos errores :D

@gjx2 Te me has adelantado :p

[gjx2]

No importa albert1980
Eres de los mejores
Se me olvido mencionar
@fclose

Lo estrano es que este sr dice que antes le funcionaba :-s
y realmente aparenta que si
Ya se Mostraron: 3851371 Páginas de MiSitio!

quisas todo se hubiece evitado si la primera respuesta fuera
error_reporting(0)
o
@fopen
@fgets
@fclose

[jonatanc]

Hola , te presento otra solución quizas no quieres utilizar base de datos pero seria lo mas logico:

1) Creamos una tabla en la base de datos MYSQL que se llame contador , dicha tabla contendra un campon unico que sera del tipo integer , dicho campo se llamara valor.

2)

cuenta.php

Código php:

<?php
$link=mysql_connect('localhost','username','password') or die('Error');
if($link){
mysql_select_db('tablename') or die('Error seleccionado tabla');
}
$sql="UPDATE contador SET valor=valor+1";
$sql=mysql_query($sql,$link) or die('error contando visita');
}
$query="SELECT valor FROM contador";
$query=mysql_query($sql,$link) or die("Error contando visita");
$dato=mysql_fetch_row($query);
 
echo "Esta pagina contiene ".$dato['0']." impresiones ";
?>
 

Como veras quizas es algo mas complicado pero te solucionara todo tipo de problema con ficheros y es muchisimo mas escalable , por otro lado utilize el termino impresiones ya que al no evaluar la ip lo que realmente estas contando no son visitas reales sino impresiones , la diferencia entre visita real (visitas unicas) e impresiones radica en que visitas unica cuenta una ip como una visita y si dicha ip existe no la vuelve a contar , impresiones cuenta siempre por mas que sea la misma ip.

Las soluciones que incluyen poner a 0 el error_reporting sinceramente son parches y no soluciones , es verdad que si el script se encuentra en un ambito de produccion que lo ve gente real y no solo los programadores es esencial poner dicho valor a 0 para evitar que muestre errores que afecten la seguridad de tu aplicacion web , pero estamos engañandonos xq si realmente hay un error no saldra y no obtendremos el resultado esperado , lo mas util es analizar el codigo y buscar el motiv del error , solventarlo y luego si aplicar error_reporting.

Como todos decimos en argentina , el argentino con un rollo de alambra soluciona todo , el tema cuando se acabe el alambre como solucionamos los problemas?.

Saludos!

[MasterXNicoX]

Claro xD jajaja...

gjx2.. lo de @fopen, @fgets, etc.. tengo que ponerlo asi por ejemplo??..

$contador++;
$fp = fopen($archivo,"w");
@fwrite($fp, $contador);
@fclose($fp);
echo "Ya se Mostraron: $contador Páginas de Misitio!";

------------

Albert.. lo de "error_reporting(0);" va antes del primer fopen??

[gjx2]

Si senior.
En las lineas donde te dice que se genera el wrnig.

[MasterXNicoX]

Jonathan muchas gracias

Con el tema este que te cito a continuación...

Como veras quizas es algo mas complicado pero te solucionara todo tipo de problema con ficheros y es muchisimo mas escalable , por otro lado utilize el termino impresiones ya que al no evaluar la ip lo que realmente estas contando no son visitas reales sino impresiones , la diferencia entre visita real (visitas unicas) e impresiones radica en que visitas unica cuenta una ip como una visita y si dicha ip existe no la vuelve a contar , impresiones cuenta siempre por mas que sea la misma ip.

Ya lo sé xD por eso puse "ya se mostraron N páginas de mi sitio" y no "mi sitio tuvo N visitas"

Tendré en cuenta tu solución si es que me genera otro tipo de errores las soluciones que me dieron gjx2 y albert.

[gjx2]

El unico problema que veo en un futuro
es con la funcion

fgets ya que le especificas un numero predeterminado de lectura

algo mas amplio

fgets($fp,100);

y con fwrite no expecificarle la cantidad de chr que escriba

solo dejarlo

fwrite($fp,$archivo);

Haci si el archivo sobre pasa los 26 chr no tendrias que volver a editar el script.

Suerte.

[MasterXNicoX]

Ok.. y por último....

Es lo mismo que ponga lo de "error_reporting(0);" o las arrobas antes de los fwrite,fopen... o tengo que poner ambas cosas para que los errores no se muestren??

-----

jonatan, el código funciona bién??

[MasterXNicoX]

El unico problema que veo en un futuro
es con la funcion

fgets ya que le especificas un numero predeterminado de lectura

algo mas amplio

fgets($fp,100);

y con fwrite no expecificarle la cantidad de chr que escriba

solo dejarlo

fwrite($fp,$archivo);

Haci si el archivo sobre pasa los 26 chr no tendrias que volver a editar el script.

Suerte.

Ah ok, ahora lo arreglo

Igual para que las impresiones sobrepasen los 26 caracteres tengo que tener billones de impresiones y no lo veo muy probable xD pero igual lo edito

[jonatanc]

Hola , el arroba " @" suprime los errores en ejecución , con lo cual es más lento que utilizar error_reporting(0); , sinceramente no es aconsejable ni uno ni otro en el momento de desarrollo como te he dicho pero si quieres usar uno , usa error_reporting(0);.

Saludos!

[gjx2]

Aprovecho el hilo para decir
Las @ te impiden salir los wrn de las lineas donde las especificaste.

El error_reporting(0) te impide cualquier error del archivo

En realidad para un script tan pequeno
es demaciada teoria vale lo mismo usar @ o error_reporting(0) .

Por otro lado



la funciona fgets no funciona con filezise () que seria la manera en la que tendrias algo mas dinamico.

en ves de fgets utiliza fread para que puedas usar el filezise

$contador = fread($fp,filezise($archivo));

[MasterXNicoX]

Iniciado por jona

Hola , el arroba " @" suprime los errores en ejecución , con lo cual es más lento que utilizar error_reporting(0); , sinceramente no es aconsejable ni uno ni otro en el momento de desarrollo como te he dicho pero si quieres usar uno , usa error_reporting(0);.

Ok muchas gracias, y no es que menosprecie tu código, es más, ya lo guardé por cualquier inconveniente jejeje y te lo agradezco mucho

-------------------------

gjx2, me conformo con el fgets, sinceramente no busco algo más dinámico pero gracias igual por el consejo

-------------------------

Gracias nuevamente a todos chicos

Que estén bien!!!

[jonatanc]

Jaja esta bien no dije nada , solo recomiendo lo que creo mejor y más eficiente y escalable.

Saludos .

[MasterXNicoX]

okisss!!! todo claro

Un abrazo!