Как осознать ошибки в указанном коде?

Question

[keslo]

Добрый день.
Я тут на выходных начал осваивать PHP и запилил такой хард-код:

<?
	header('Content-Type: application/x-javascript; charset=utf8');
	
	$file = "base.json"; 			
	$fp = fopen($file, "a+");       
	
	$content = fread($fp, filesize($file));
	$obj = json_decode($content, true);
	
	$n = array("Min" => 1, "Sec" => 1);
	
	array_push($obj, $n);
	
	echo json_encode(json_encode($obj));
	
	fwrite($fp, "");
	fclose($fp);
	
?>

Идея кода такая:

- получаем запрос по AJAX
- проверяем присутствие файла base.json и считываем из него данные
- парсим прочитанные данные из JSON в массив
- дозаписываем в массив данные
- переводим в JSON и отправляем обратно

Код рабочий. Но получилось как-то ужасно. Например это:
echo json_encode(json_encode($obj));
Методом научного тыка использовал двойной json_decode и код заработал...
Почему так?

Буду благодарен вашим замечаниям и советам по улучшению кода.

Answer 1

[Иван]

1) У json корректный content-type application/json
2) Причем здесь ajax?
3) Зачем двойной json_decode?
4) ?> в конце php-файла ставить не нужно

<?php
    $file = file_get_contents('./file.json');
    $content = json_decode($file, true);
    $n = array("Min" => 1, "Sec" => 1);
    array_push($content, $n);
    
    header('Content-Type: application/json; charset=utf8');
    echo json_encode($obj);

Comments

[Иван]

@mytmid кстати говоря да, забыл про это

[Arris]

> 4) ?> в конце php-файла ставить не нужно

Кстати, а почему? Есть этому какое-нибудь серьезное обоснование?

[keslo]

@Arris Так надо, видимо)

[Иван]

@Arris в этом просто нет необходимости, можно и так и так. На работе скрипта не скажется, но, согласитесь, удобнее

[keslo]

@0neS А как в данном случае буде выглядеть вариант с использованием file_get_contents? Я попробовал использовать его, но мне выдается error в AJAX.

$content = file_get_contents('./base.json', true);
$arr = json_decode($content);

[Arris]

@0neS не соглашусь :) Мне неудобно и это требование для меня выглядит дико (особенно когда его необходимость постулируют) :) Открыл скобку - закрой :) Нажимать на 4 кнопки меньше разве что :)

[Arris]

@keslo идите что ли почитайте учебник Дмитрия Котерова "PHP". Пока не прочитаете хотя бы половину - не приходите сюда, добрый вам совет.

Еще идите и почитайте статью о том, что такое AJAX.

[Иван]

@Arris ну если вы в html хотите вывести переменную - тогда да, конечно нужно закрыть.

[keslo]

@Arris Прочитал и получебника, и статью про AJAX, но без практики это все ничто. А когда дело до практики дошло, то "читал-не читал" уже не важно. Код либо работает, либо нет. И это первое.

Мой вариант кода выше рабочий... но он страшно смотрится. Поэтому и обратился сюда.

В любом случае, спасибо. Учту ваши советы.