Javascript-форум - Сокротить код

Javascript-форум (https://javascript.ru/forum/)

- Общие вопросы Javascript (https://javascript.ru/forum/misc/)

- - Сокротить код (https://javascript.ru/forum/misc/25701-sokrotit-kod.html)

Сокротить код

<!DOCTYPE html>
<html dir="ltr">
    <head>
        <meta charset="utf-8" />
        <style type="text/css">
            body { font: 14px Tahoma; }                     
        </style>
        <link rel="stylesheet" type="text/css" href="wysiwyg.css" />
        <script type="text/javascript">
            function WYSIWYG(conf) {
                this.mode = "rich";
                this.textarea = document.createElement("textarea");
                this.textarea.name = conf["target"];
                this.url = (conf["url"] === undefined) ? "." : conf["url"];
            }
                                  
            WYSIWYG.prototype.execute = function(command, value) { 
                this.iframeWindow.focus();
                this.iframeWindow.document.execCommand(command, null, value);
            }
            
            WYSIWYG.prototype.setBold = function() {
                this.execute("bold", "");
            }    
            
            WYSIWYG.prototype.setItalic = function() {
                this.execute("italic", "");
            } 
            
            WYSIWYG.prototype.setUnderline = function() {
                this.execute("underline", "");
            } 
            
            WYSIWYG.prototype.setStrike = function() {
                this.execute("strikeThrough", "");
            }
            
            WYSIWYG.prototype.update = function() {             
                var self = this;
                var interval = setInterval(
                    function() {
                        self.textarea.value = self.iframeDocument.body.innerHTML;
                        if (self.mode == "plain") {
                            clearInterval(interval);
                        }
                    },
                    40
                );
            }
            
            WYSIWYG.prototype.replace = function() {                        
                var self = this;                          
                
                // [*вырезано*]
                
                /* кнопки */
                // Bold
                var button = document.createElement("div");
                button.className = "button";
                button.innerHTML = '<span style="font-weight: bold;">B</span>';
                button.title = "Bold";
                button.onclick = function() {
                    self.setBold();
                }
                iframeBar.appendChild(button); 
                // Italic
                button = document.createElement("div");
                button.className = "button";
                button.innerHTML = '<span style="font-style: italic;">I</span>';
                button.title = "Italic";
                button.onclick = function() {
                    self.setItalic();
                }
                iframeBar.appendChild(button); 
                // Underline
                button = document.createElement("div");
                button.className = "button";
                button.innerHTML = '<span style="text-decoration: underline;">U</span>';
                button.title = "Underline";
                button.onclick = function() {
                    self.setUnderline();
                }
                iframeBar.appendChild(button);
                // Striked
                button = document.createElement("div");
                button.className = "button";
                button.innerHTML = '<span style="text-decoration: line-through;">S</span>';
                button.title = "Strike";
                button.onclick = function() {
                    self.setStrike();
                }
                iframeBar.appendChild(button);
                
                // [*вырезано*]
            }          
        </script>        
    </head>
    <body>
        
        <form action="#" method="post">
            <textarea name="content" cols="48" rows="4"></textarea>
        </form>
        
        <script type="text/javascript">
            wysiwyg = new WYSIWYG({"target" : "content"});
            wysiwyg.replace();
        </script>
    </body>
</html>

Меня не устраивают повторяющиеся фрагменты кода, пробовал в массив загонять, но там eval'ом приходиться выполнять, а ещё баг получается что у всех элементов при onclick один и тот же последний метод. Довольно тупой язык...

Цитата:

Сообщение от bugmenot

Довольно тупой язык...

может дело в тебе?

блин этот чувак скрипты в шапку подключает, расходимся пасоны...

Цитата:

Сообщение от Mахmахmахimus

блин этот чувак скрипты в шапку подключает, расходимся пасоны...

Для отладки можно.

Цитата:

Сообщение от bugmenot

Довольно тупой язык...

Просто не нужно немножко по-дебильному кодить.
Думаете после этого вам ответят?

Цитата:

Сообщение от bugmenot

Довольно тупой язык...

Правила организации кода везде одинаковые.

Цитата:

Сообщение от bugmenot (Сообщение 157080)

Меня не устраивают повторяющиеся фрагменты кода

Чувааак, для этого придумали аргументы в функции передавать!

Цитата:

Сообщение от bugmenot (Сообщение 157080)

а ещё баг получается что у всех элементов при onclick один и тот же последний метод.

А ещё придумали замыкания, чтобы метод последний не был один и тот же у всех onclick. Я уж молчу про addEventListener, хотя тебе наверное уже пора узнать что это такое. Хотя у тебя проблема что ты не в курсе что функции умеют принимать аргументы.

Цитата:

Сообщение от bugmenot (Сообщение 157080)

Довольно тупой язык...

Он таким и будет, пока ты его не изучишь.

Ок, почему не работает так?

// Bold
                var button = document.createElement("div");
                button.className = "button";
                button.innerHTML = '<span style="font-weight: bold;">B</span>';
                button.title = "Bold";
                // button.onclick = function() {
                //     self.setBold();
                // }
                button.onclick = this.setBold;
                iframeBar.appendChild(button);

Цитата:

Event thread: click
Uncaught exception: TypeError: 'this.execute' is not a function
Error thrown at line 15, column 16 in <anonymous function: WYSIWYG.prototype.setBold>() in http://localhost/html/editor/blank.htm:
this.execute("bold", "");

var buttons = [
                    {
                        "label" : '<span style="font-weight: bold;">B</span>',
                        "title" : "Bold",
                        "method" : "setBold"
                    },
                    
                    {
                        "label" : '<span style="text-decoration: underline;">U</span>',
                        "title" : "Underline",
                        "method" : "setUnderline"
                    }
                    
                ];
                
                for (var i = 0; i < buttons.length; ++i) {
                    var button = document.createElement("div"); 
                    button.className = "button";
                    button.innerHTML = buttons[i]["label"];
                    button.title =  buttons[i]["title"];
                    var method = buttons[i]["method"];
                    button.onclick = function() {
                        var code = "self." + method + "();";
                        alert(code);
                        eval(code);
                    }
                    iframeBar.appendChild(button);                
                }

А так какую бы "кнопку" не нажал выскакивает "self.setUnderline();"

Цитата:

Сообщение от bugmenot (Сообщение 157137)

Ок, почему не работает так?
...
Event thread: click
Uncaught exception: TypeError: 'this.execute' is not a function ...

Я что-то не пойму, ты меня что переводчиком хочешь нанять или учителем для себя?

http://phpclub.ru/faq/WhyForum