[С++] facepalm

бессмысленно и беспощадно

хотя, что хотел сделать автор вроде понятно

В толкс же.

Кусочек-то - модельный. Разбавь на 4 экрана кодом одной функции. И понятность станет примерно как сейчас у меня в проекте. :)

Не холивара ради.

цикл по целочисленной переменной зло. спасут мир итераторы и foreach (в нормальных языках).

кодеры мелкософта рвутся в опенсурц? нет пути!

n контрольных выстрелов

ах эти индусы

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

> Поясни мне

возможно у него разбитая на токены строка, которые он хотел одним махом привести к нижнему регистру, хотя хз конечно

Код рабочий. Области видимости разные.
А стиль - даже не просто вырвиглазный...

Интересно, а какое i будет использоваться внутри вложенного цикла? А вообще, я так понял, строка str n раз будет преобразована в нижний регистр? Кстати, интересный способ реализации usleep() со случайным параметром :)

> Интересно, а какое i будет использоваться внутри вложенного цикла?

уже написали, в стандарте все четко расписано про области видимости

Т.е. внутри под-цикла будет i под-цикла? А внешнего - i внешнего?
Просто проверяю.
Вообще, когда видишь быдлокод, появляется масса действительно детских вопросов, на которых можешь запороться на собеседованиях...

нормальный код. Главное — это простые и понятные интерфейсы, а не такие мелочи.

Только объясните мне, зачем n раз переводить строку в нижний регистр? Или это действительно вместо usleep?

рабочий код. Главное — это простые и понятные интерфейсы, а не такие мелочи.

fixed

Разница между рабочим и нормальным бывает существенной.
Как пример - в процессе рефакторинга.

> Разница между рабочим и нормальным бывает существенной.
> Как пример - в процессе рефакторинга.

√-1

Хм.. не знаток C++: но разве tolower(str[i]) применяется не ко всей строке i? если да, то малость непонятно нахрена выравнивать регистр n + str[i].lenght раз ? ето вместо sleep()(или как там в C++ задержки делаются)? Чот хз.. странный код на самом деле..

> Хм.. не знаток C++: но разве tolower(str[i]) применяется не ко всей строке i? если да, то малость непонятно нахрена выравнивать регистр n + str[i].lenght раз ? ето вместо sleep()(или как там в C++ задержки делаются)? Чот хз.. странный код на самом деле..

tolower применяется к i-тому символу строки, str - это массив char str[]

str[i].lenght

жаба детектед

lenght

граммар-наци негодуэ

> Ваше мнение?

нет поддержки unicode, соответственно код гавно.

Может, это вероятностный tolower. Может перевести символ в нижний регистр, а может и не перевести. Выполняя его n раз, мы повышаем вероятность правильного результата.

Вы неправильно код читаете! Гораздо интереснее читать его, пытаясь придумать его полезное использование! Я уже полчаса себе мозги ломаю, фантазии не хватает....

аналогично :(

У меня этот, с позволенья сказать, код находился в функции-убивалке процессов...
ps ...мышления, наверное :)

Как это говно потом сопровождать? Быть может, я слишком эстет? :) Ваше мнение?

такие шедевры надо срочно в биореактор

да, работка будет вполне себе негритянская :)

Может, это вероятностный tolower. Может перевести символ в нижний регистр, а может и не перевести.

:)

объясните нерадивому шазанах этот код?

for(int i=0; i<n; i++) { 
  for(int i=0; str[i]; i++) { 
    str[i] = tolower(str[i]); 
  } 
} 

области видимости переменных i не пересекаются во внешнем и вложенном цикле. значение n не детерминировано во внешнем цикле. значение i во внутреннем цикле =0. это получается n раз преобразовать первый символ str tolower? это такой аналог nop из assembler'а или костыльная реализация sleep() ?

> начение i во внутреннем цикле =0. это получается n раз преобразовать первый символ str tolower?

а вот и вероятный автор

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

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

И меня поразил не сам код (он банален), но его стиль - использование одного имени переменной в разных смыслах (страх наделить вложенный индекс другим именем? желание кинуть понты перед возможным будущим мейнтейнером? хз), проверка конца (вложенного) цикла на соплях (хорошо, что сейчас там asciiz-строка)

Давайте разовьем тему. Может там вместо конца строки ловится исключение и в обработчике подсовывается новая строка?

а вот и вероятный автор

вероятностный

Вы хоть откройте тайну: зачем n раз преобразовывать строку в нижний регистр?

str=0 означает конец строки. str[0]=0 значит, что строка пустая (точнее, её будут считать таковой большинство функций, заточенных под работу со строками).

Очень удобно, при n=0 строка не преобразуется, при n>=1 - преобразуется.

а чем if неудобен?

:) а что, нельзя сделать

if(n)
  for(int i=0; str[i]; i++)
    str[i] = tolower(str[i]); 

Видимо, нельзя :)

проверил, с первого раза не все символы преобразуются. вот и нужны для уверенности n проходов.

а вот и вероятный автор

вероятностный

урл

проверил, с первого раза не все символы преобразуются. вот и нужны для уверенности n проходов.

Этот код явно не для этой цели.

это понятно, пытаюсь понять логику и где это можно нормально применить

> Быть может, я слишком эстет? :)

Да.

Феерично. Как оно работает то ?

пытаюсь понять логику и где это можно нормально применить

не пытайся, спаси свой моск

Оригинально все выглядит примерно так:

n = daite_kolvo_strok();
for(int i=0; i<n; i++) {
  kakoy_to_kod(i);
  str = imja_sestra(i);
  for(int i=0; str[i]; i++) { 
    str[i] = tolower(str[i]); 
  }
  kakoy_to_kod_2(i);
}

Выжимка, соответственно:

for(int i=0; i<n; i++) {
  for(int i=0; str[i]; i++) { 
    str[i] = tolower(str[i]); 
  }
}

for(int i=0; i<n; i++) {
  for(int i=0; str[i]; i++) {
  }
}

PS Однако, доставляет то, как народ чешет тему насчет того а что же этот код делает :)

Почитал обсуждение, согласился с тобой :)

> Почитал обсуждение, согласился с тобой :)

Единственная претензия по делу - не поддерживается Unicode. Ноэто тоже может быть неважно...

проверил, с первого раза не все символы преобразуются. вот и нужны для уверенности n проходов.

У меня с первого раза работает:

[09.04 15:35 ~/tmp]
cat 2.c 
#include <stdio.h>
#include <ctype.h>

main(){
	char str[] = "ПРОВЕРКА НА ПрАвИлЬнОсТь\nTEST RIGHT ToLoWeR WoRk\n\n";
	char *ptr = str;
	do{
		*ptr = tolower(*ptr);
		ptr++;
	}while(*ptr);
	printf("было: %s", str);
	printf("стало: %s", str);
}[09.04 15:35 ~/tmp]
cat 2.c 
#include <stdio.h>
#include <ctype.h>

main(){
	char str[] = "ПРОВЕРКА НА ПрАвИлЬнОсТь\nTEST RIGHT ToLoWeR WoRk\n\n";
	char *ptr = str;
	do{
		*ptr = tolower(*ptr);
		ptr++;
	}while(*ptr);
	printf("было: %s", str);
	printf("стало: %s", str);
}
[09.04 15:35 ~/tmp]
gcc 2.c 
[09.04 15:35 ~/tmp]
./a.out 
было: ПРОВЕРКА НА ПрАвИлЬнОсТь
test right tolower work

стало: ПРОВЕРКА НА ПрАвИлЬнОсТь
test right tolower work