16.03.2018, 10:15
|
|
Интересующийся
|
|
Регистрация: 16.03.2018
Сообщений: 13
|
|
Как сделать правильно и красиво ?
В общем есть код:
$.getJSON("https://spreadsheets.google.com/feeds/list/" + spreadsheetID + "/od6/public/values?alt=json", function(data) {
$(data.feed.entry).filter(function(){
if(this.gsx$slag.$t===location.pathname) return $('section').prepend('<header><h2>'+this.gsx$title.$t+'</h2></header><img src="'+this.gsx$image.$t+'"/><p>'+this.gsx$content.$t+'</p>').exit();
});
$(data.feed.entry).each(function(){
$('section').prepend('<article class="'+this.gsx$type.$t + '"><header><h2>' + this.gsx$title.$t + '</h2></header><img src="'+ this.gsx$image.$t +'"/></article>');
});
});
Он конечно работает, но выглядит стремно ( Подскажет кто как сделать покрасивей?
Последний раз редактировалось WqR128, 16.03.2018 в 11:33.
Причина: fix
|
|
16.03.2018, 10:33
|
|
CacheVar
|
|
Регистрация: 19.08.2010
Сообщений: 14,217
|
|
Сообщение от WqR128
|
Подскажет кто как сделать покрасивей?
|
Какие у тебя критерии красивости?
Напихать в имена свойств помимо $ еще чего-то.
|
|
16.03.2018, 10:43
|
Профессор
|
|
Регистрация: 04.12.2012
Сообщений: 3,791
|
|
jq.filter как-то криво используется, по-моему.
Что делает функция "text" - загадка такая же как и тип объекта "data.feed.entry".
|
|
16.03.2018, 11:05
|
|
Интересующийся
|
|
Регистрация: 16.03.2018
Сообщений: 13
|
|
Про text() забудем, а data.feed.entry массив массивов object
Цитата:
|
(16) [{…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}, {…}]
0
:
category
:
[{…}]
content
:
{type: "text", $t: "title: title1, content: <h3>rest</h3>555, image: https://unsplash.it/800/600?image=1, slag: /1"}
gsx$content
:
{$t: "<h3>rest</h3>555"}
gsx$id
:
{$t: "1"}
gsx$image
:
{$t: "https://unsplash.it/800/600?image=1"}
gsx$slag
:
{$t: "/1"}
gsx$title
:
{$t: "title1"}
gsx$type
:
{$t: ""}
gsx$video
:
{$t: ""}
id
:
{$t: "https://spreadsheets.google.com/feeds/list/1wcExie…hPzpYkA9mLBISZaPKzjfStq9c/od6/public/values/cokwr"}
link
:
[{…}]
title
:
{type: "text", $t: "1"}
updated
:
{$t: "2018-03-16T07:05:17.026Z"}
__proto__
:
Object
1
:
{id: {…}, updated: {…}, category: Array(1), title: {…}, content: {…}, …}
2
:
{id: {…}, updated: {…}, category: Array(1), title: {…}, content: {…}, …}
3
:
...
|
Последний раз редактировалось WqR128, 16.03.2018 в 11:32.
Причина: Более читабельный вид
|
|
16.03.2018, 11:16
|
Профессор
|
|
Регистрация: 04.12.2012
Сообщений: 3,791
|
|
WqR128, а что-за метод "delete" вы используете в строке 3?
|
|
16.03.2018, 11:30
|
|
Интересующийся
|
|
Регистрация: 16.03.2018
Сообщений: 13
|
|
Nexus, согласен exit() выглядит уместнее
|
|
16.03.2018, 11:34
|
|
CacheVar
|
|
Регистрация: 19.08.2010
Сообщений: 14,217
|
|
WqR128, зачем ты $ в имена свойств наставил?
|
|
16.03.2018, 11:41
|
|
Интересующийся
|
|
Регистрация: 16.03.2018
Сообщений: 13
|
|
ksa, ну мне $ не нужен, но гугла такие переменные
|
|
16.03.2018, 11:44
|
Профессор
|
|
Регистрация: 04.12.2012
Сообщений: 3,791
|
|
Сообщение от WqR128
|
Nexus, согласен exit() выглядит уместнее
|
Вы jQuery используете?
|
|
16.03.2018, 11:50
|
|
Интересующийся
|
|
Регистрация: 16.03.2018
Сообщений: 13
|
|
Nexus, да
|
|
|
|