Поругайте код, мне будет полезно

1. Код должен быть читаемым и понятным, пусть и длиннее.

$p=$parent;
$g as $g1
$where as $nowhere
$result => $res => $r

Не надо так делать.

И вообще, если у вас есть переменная $result, то должно быть return $result, что очень логично)

2. В сниппетах не нужно делать echo, только return.

Про названия переменных уже сказали, это ужасно. Никогда так не делайте, называйте переменные правильно и не придется писать тонны комментариев к коду.
Второ момент, зачем тут такие странные манипуляции?

$g=$modx->getCollection('modResource',array('parent'=>$p));
foreach ($g as $g1){
    $where[]=$g1->get('id');
}

Лучше использовать getIterator, на большом количестве ресурсов памянть не закончится, это раз. Два раза вы делаете проход по массиву, абсолютно бесполезно. Вы же и так перебираете ресурсы, почему не сделать вот так?

foreach ($modx->getIterator('modResource', array('parent' => $parent)) as $resource) {

    $result = $modx->getObject('modTemplateVarResource', array('contentid' => $resource->get('id'), 'tmplvarid' => 1));

    ...
}

Большая вложенность кода в {} ухудшает читабельность кода, поэтому идеальный код — это прямой без ветвлений или стараться их избегать где возможно. Например, вместо проверки, есть ли значение в переменной и выполнения кода внутри if, можно проверить, что если значение пустое, то пропустить остальной код и перейти на следующую итерацию.

if (!$result) {
    continue;
}
$res = $result->get('value');

Про return из сниппета вам уже сказали, лучше собирать весь вывод в буфер в виде массива или строки и потом уже выводить одним return. Это даст возможность в дальнейшем, когда условия задачи изменятся, еще раз пройтись по массиву или строку и применить дополнительные действия над результатом.

Простой пример, у вас сниппет в зависимости от параметра должен выдавать или строку на экран или json в случае ajax-запроса. В таком случае можно сделать так:

$output = [];
foreach ($r as $r1) { 
    $output[] = $r1['text'];
}

return $ajax ? 
    json_encode($output)
    join("", $output);

И последнее, несмотря на то, что параметры в снипете уже присутствуют в виде переменных, лучше получать их значения через API MODX. Это в дальнейшем даст возможность использовать Property Sets (предустановленные параметры) и значения по умолчанию.

$parent = $modx->getOption('parent', $scriptProperties, null);

Для получения значений ТВ-параметров у ресурсов MODX есть метод getTVValue. Он делает то же самое, но код становится чище, опрятнее и гораздо более читабельным:

$res = $g1->getTVValue('tv_name');

Ну и в дальнейшем нужно углублять знания SQL и xPDO.

В этом коде делается много запросов к базе данных: сначала получаем все нужные ресурсы (1 запрос), и потом на каждый ресурс делаем дополнительный запрос в базу, чтобы получить значение TV-параметра. При большом количестве ресурсов сниппет будет создавать нехилую нагрузку.

Чтобы такого избежать, используем LEFT JOIN:

// Выборка ресурсов сразу с ТВ-шками
$q = $modx->newQuery('modResource', array('parent' => $parent));
$q->leftJoin('modTemplateVarResource', 'MIGX', 'modResource.id = MIGX.contentid AND MIGX.tmplvarid = 1');
$q->select('`MIGX`.`value` AS `text`');
$g = $modx->getIterator('modResource', $q);

// Получение нужных полей из JSON
foreach ($g as $g1){
    $res = $g1->get('text');
    if (!$res) {
        continue;
    }
    $r = $modx->fromJSON($res);
    foreach ($r as $r1){
        $output[] = $r1['text'];
    }
}