Говнокод #19863 — JavaScript

Говнокод: по колено в коде.

Нашли или выдавили из себя код, который нельзя назвать нормальным, на который без улыбки не взглянешь? Не торопитесь его удалять или рефакторить, — запостите его на говнокод.ру, посмеёмся вместе!

JavaScript / Говнокод #19863

<!DOCTYPE html>
<html>
<head>
  <meta charset="utf-8">
  <title>Task 8_1</title>
  <script type="text/javascript">
  function charRemover(item) {
  var charsMoreThanTwoTimes = "";
  var charsMoreThanTwoTimes2 = "";
  var tmpString="";
  var result = "";
  var almostResult = "";
  var nextWordBeginning = 0;
  for (var a = 0; a < item.length; a++) {
           
   if ((a == nextWordBeginning+1)&&(nextWordBeginning > 0)) {tmpString="";}
   if ((a!=item.length-1)&&((item[a]!=' ') && (item[a]!=',') && (item[a]!='.') && (item[a]!=';') && (item[a]!=':') && (item[a]!='!') && (item[a]!='?')))
   {
     tmpString = tmpString + item[a];
   }        
     
    else if ((a == item.length -1 ) && ((item[a]!=' ') && (item[a]!=',') && (item[a]!='.') && (item[a]!=';') && (item[a]!=':') && (item[a]!='!') && (item[a]!='?')))
     {
     tmpString = tmpString + item[a];

    for (var j = 0; j < tmpString.length; j++)
      {
          var counter = 0;
          for (var k = 0; k < tmpString.length; k++)
          {
              if (tmpString[k] == tmpString[j]) {counter++; }
          }
          if (counter == 1) {almostResult = almostResult + tmpString[j];}
          if (counter > 1) { charsMoreThanTwoTimes = charsMoreThanTwoTimes + tmpString[j];}
      }
    } 
     
  else if ((a != item.length-1)&&((item[a]==' ') || (item[a]==',') || (item[a]=='.') || (item[a]==';') || (item[a]==':') || (item[a]=='!'))){     
     for (var j = 0; j < tmpString.length; j++)
      {
          var counter = 0;
          for (var k = 0; k < tmpString.length; k++)
          {
              if (tmpString[k] == tmpString[j]) {counter++;}
          }
          if (counter == 1) {almostResult = almostResult + tmpString[j];}
          if (counter > 1) { charsMoreThanTwoTimes = charsMoreThanTwoTimes + tmpString[j];}          
      }
      almostResult = almostResult + item[a];
     nextWordBeginning = a;
    }
   }
  counter = 0;

  //пройдемся еще раз и удалим символы, которые встречаются более одного раза
  for (var i = 0; i < almostResult.length; i++)
  {
    var shoundCharToBeRemoved = false;      
     for (var j = 0 ; j < charsMoreThanTwoTimes.length; j++)
     {
         if (almostResult[i] == charsMoreThanTwoTimes[j]) {shoundCharToBeRemoved = true;}
     }
     if (shoundCharToBeRemoved == true) {}
     else {result = result + almostResult[i];}
  }
    return result;
  }
        
  var sourceString = "У попа была собака";   
  var destString = charRemover(sourceString); 
  </script>
</head>
<body>
    <script type="text/javascript">
        document.write(destString);
    </script>
</body>
</html>

Только что начал изучать JavaScript. Знаю, что говнокод, так что не судите строго !

Сама задача: Нanucaть функцuю, нa вxoд кoтoрoй nрuxoдuт cтрoкa, cocтoящaя uз неcкoлькux cлoв. cлoвa
рaзделены nрoбельнымu cuмвoлaмu (nрoбел, тaбуляцuя) u знaкaмu nреnuнaнuя (?!:;,.). Нужнo
вернуть cтрoку, в кoтoрoй будут удaлены вcе cuмвoлы, noвтoряющuеcя xoть в oднoм uз cлoв бoлее
oднoro рaзa.

Запостил:

ololenok, 22 Апреля 2016

Комментарии (19) RSS

guest 22.04.2016 22:03 # 0

JavaScript не нужен
Впрочем, как и HTML
Ответить
kerman 22.04.2016 22:27 # 0

Во-первых, открой для себя массивы и итерацию по ним. Во-вторых, разбивай на вменяемые куски или даже выделяй их в отдельные методы. Читать этот бойлерплейт нереально...
Ответить
- guest 22.04.2016 22:41 # −3
  
  массивы не нужен
  Ответить

guest 22.04.2016 23:17 # −3

Яваскрипт не нужен.

var sourceString = Console.ReadLine();
var charsToDelete = sourceString.Split("?!:;,. \t".ToCharArray())
	.SelectMany(x => x.GroupBy(y => y)
		.Where(y => y.Count() > 1)
		.Select(y => y.Key))
	.Distinct()
	.ToArray();
Console.WriteLine(new string(sourceString.Where(x => !charsToDelete.Contains(x)).ToArray()));
Console.ReadKey();

Ответить

guest 22.04.2016 23:30 # 0

Верно, хотя linq тоже не нужен
Ответить

bakagaijin 23.04.2016 10:32 # +6

Как интересно задача написана. Чтобы студенты не смогли прогуглить?
Ответить
- inkanus-gray 23.04.2016 11:13 # +4
  
  Да, придётся не гуглить, а яндексить, потому что Яндекс уже в курсе «оптического» транслита:
  https://yandex.ru/search/?text=n%D1%80%D0%B5nu%D0%BDa%D0%BDu%D1%8F
  Ответить
  - bormand 23.04.2016 11:17 # +1
    
    Зато гугл по этому слову показывает ГК ;)
    Ответить
    - inkanus-gray 23.04.2016 12:13 # +3
      
      Bopeцuu, reнepeцuu, кoбeнaцuu, нamaлuзaцuu, pacnoлoжeнue uнфopмaцuu.
      Ответить
    - Vasiliy 23.04.2016 13:09 # +3
      
      ждем когда гугл по запросу "гоатсе" будет показывать ГК
      Ответить
      - bormand 23.04.2016 13:12 # 0
        
        Не гоатсе, а roaтce.
        Ответить
        
        inkanus-gray 23.04.2016 16:15 # +1
        
        Не roaтce, а roamce.
        
        P.S. А нет, не получится:
        http://www.deviantart.com/tag/roamce
        Ответить
        
        inkanus-gray 23.04.2016 16:20 # +1
        
        Кстати: Return on Average Tangible Common Equity (ROATCE).
        Ответить
        
        inkanus-gray 23.04.2016 16:23 # +2
        
        И контрольный в голову: Γοατσε. Хотя, может быть, правильнее так: Γκοατσε.
        Ответить
        
        inkanus-gray 23.04.2016 16:32 # +2
        
        А ещё можно наоборот, латинские буквы подменять кириллицей: gоаtsе.
        
        P.S. А на этом слове Гугл почему-то догадался о транслите.
        Ответить
- bot 25.04.2016 07:55 # 0
  
  Не студенты, а преподаватели, которые лабы проверяют.
  Ответить
  - 1024-- 25.04.2016 19:01 # 0
    
    Действительно, что ли? А зачем?
    
    Тех, кто делает не сам, видно и так. А те, кто честно решают задачи, порою разбираются в теме ещё меньше тех, кто списал.
    Единственное здравое объяснение - что преподавателю настолько скучно, что он начинает флудить и троллить.
    Ответить

kegdan 23.04.2016 12:39 # +3

do =:(((2&(=/\)#}.)@/:~)@>@;:@(#~(=>./)@(',.?:!'&i.))) 
      do2 =: (#~ (= >./)@(i.~ (#~-.@=&' ')@,/@do)) 
      do2 'worlds on worlds sfddas asf a,asf e af...,'
worl on worl f f ,f e f...,
      do2 'aa dasd fgf hds.,&'
 s g hs.,&

Ответить

bot 25.04.2016 07:52 # +3

ITS Λ VЭЯY IИTЭЯЭSTIИG TΛSK!
Ответить

Говнокод: по колено в коде.

JavaScript / Говнокод #19863

Комментарии (19) RSS

Добавить комментарий